Skip to content

test: separate manual e2e tests and add one-to-one unit test structure#474

Open
0xWelt wants to merge 1 commit into
build/ty-configfrom
refactor/test-structure
Open

test: separate manual e2e tests and add one-to-one unit test structure#474
0xWelt wants to merge 1 commit into
build/ty-configfrom
refactor/test-structure

Conversation

@0xWelt
Copy link
Copy Markdown
Member

@0xWelt 0xWelt commented May 10, 2026

Changes

Test Structure

  • Move all manual e2e tests from testing/ops/ and testing/ui/*/e2e.py to testing/manual/
  • Update imports and sys.path for manual tests
  • Add norecursedirs = ['manual'] and --import-mode=importlib to pytest config

One-to-One Unit Tests

  • Create 108 test files mirroring the autowsgr source layout
  • Add missing __init__.py for namespace packages

Real Unit Tests for Core Modules

  • context/: build, equipment, expedition, fleet, game_context, resources, ship
  • scheduler/: launcher, scheduler
  • server/: main, schemas, serializers
  • infra/logger: 8 cases with mock
  • types.py: enum and model tests

Bug Fix

  • Fix save_image() creating a literal None/ directory when img_dir is None

Verification

  • pytest: 539 passed
  • pre-commit: all clean

@0xWelt 0xWelt changed the base branch from main to build/ty-config May 10, 2026 05:12
@codecov
Copy link
Copy Markdown

codecov Bot commented May 10, 2026

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

ℹ️ You can also turn on project coverage checks and project coverage reporting on Pull Request comment

Thanks for integrating Codecov - We've got you covered ☂️

@0xWelt 0xWelt requested a review from huan-yp May 10, 2026 09:03
@0xWelt 0xWelt force-pushed the build/ty-config branch from bb17d99 to 39f725d Compare May 10, 2026 09:11
@0xWelt 0xWelt force-pushed the refactor/test-structure branch 4 times, most recently from ab38edc to ed89452 Compare May 10, 2026 09:23
@0xWelt 0xWelt force-pushed the build/ty-config branch from 1995a76 to 9bc5c5b Compare May 10, 2026 09:30
@0xWelt 0xWelt force-pushed the refactor/test-structure branch 2 times, most recently from f859a28 to 42290d4 Compare May 10, 2026 09:34
@0xWelt 0xWelt force-pushed the build/ty-config branch from 9bc5c5b to ba535ac Compare May 10, 2026 09:34
@0xWelt 0xWelt force-pushed the refactor/test-structure branch from 42290d4 to 39583c3 Compare May 10, 2026 09:35
@0xWelt 0xWelt force-pushed the build/ty-config branch from ba535ac to 742b234 Compare May 10, 2026 09:41
@0xWelt 0xWelt force-pushed the refactor/test-structure branch from 39583c3 to a55db74 Compare May 10, 2026 09:42
@0xWelt 0xWelt force-pushed the build/ty-config branch from 742b234 to e0bb39f Compare May 10, 2026 09:43
@0xWelt 0xWelt force-pushed the refactor/test-structure branch from a55db74 to 3d23371 Compare May 10, 2026 09:44
@0xWelt 0xWelt force-pushed the build/ty-config branch from e0bb39f to 6a1be3f Compare May 10, 2026 09:58
@0xWelt 0xWelt force-pushed the refactor/test-structure branch from 3d23371 to 28ffd13 Compare May 10, 2026 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant