[MED] Split tests.zig - Create tests/ directory with module-specific test files

[MichaelFisher1997]

Problem

The src/tests.zig file has grown to 1400+ lines and contains tests for multiple unrelated modules:

• Math (Vec3, Mat4, AABB, Frustum)
• World (Chunk, BlockType, PackedLight)
• Worldgen (Noise, generators)
• Meshing (AO calculator, lighting sampler)
• Texture atlas
• Biome constraints

This monolithic test file is difficult to navigate and maintain.

Current State

One file (src/tests.zig) contains tests for 15+ different modules.

Proposed Solution

Create a src/tests/ directory with module-specific test files:

src/tests/
├── main.zig           # Test runner/imports
├── math.zig           # Vec3, Mat4, AABB tests
├── chunk.zig          # Chunk, BlockType, PackedLight tests
├── worldgen.zig       # Noise, generator tests
├── meshing.zig        # AO, lighting sampler tests
└── texture.zig        # TextureAtlas tests

Benefits

• Tests are co-located by module
• Easier to find and run specific test suites
• Reduced merge conflicts in test files
• Clearer organization

Acceptance Criteria

• Create src/tests/ directory
• Split tests into logical files by module
• Update build.zig to run all tests
• All existing tests preserved
• Tests still pass
• Document new test organization in README or AGENTS.md

Related Files

• src/tests.zig
• build.zig

Estimated Effort

Low-Medium (mostly moving code)

[opencode-agent]

Classification: Refactoring/Feature Request

This is a well-structured issue with clear scope and acceptance criteria. The src/tests.zig file has actually grown to 2301 lines (more than the 1400 mentioned), confirming the need for this reorganization.

Analysis:

The proposed directory structure looks good. A few implementation notes:

1. Zig's test discovery: The current tests.zig already uses _ = @import("...") pattern (lines 55-68) to include tests from other modules. The new src/tests/main.zig should follow this pattern to import all sub-modules.

2. Build.zig update needed: Line 70 in build.zig points to src/tests.zig. This will need to change to src/tests/main.zig after the reorganization.

3. Module imports: Each new test file (math.zig, chunk.zig, etc.) will need appropriate imports - the current tests.zig shows the pattern for each module at the top of the file.

4. Test isolation: Currently all tests share the same std_options (line 13-15). Ensure src/tests/main.zig preserves this configuration.

Recommendation:
This is a Low-Medium effort task as estimated. The scope is well-contained and doesn't need to be broken down further. The acceptance criteria are clear and measurable.

This refactoring will significantly improve maintainability - with 2301 lines across 15+ modules, the current file is definitely past the threshold where splitting provides value.

No additional information needed; this is ready for implementation when prioritized.

opencode session  |  github run