Claude/plan testing strategy orrt s#1033
Merged
Merged
Conversation
Add comprehensive test coverage for: - PerformanceMonitor: formatBytes, grades, collection toggle, snapshots, system timings, network metrics, and history methods - ConnectionCode: parse error handling, error class, input normalization - Network types: command ID generation, counter reset, lobby code generation These tests focus on modules with clean dependency chains that work with the CommonJS test compilation setup.
- Replace Node.js test runner with Vitest for ES module support - Convert all existing tests to Vitest syntax - Add Entity and World (ECS) tests with archetype query coverage - Add BehaviorTree tests (composite, decorator, action nodes) - Add DesyncDetection utility tests - Total: 185 tests across 14 test files
- Coverage summary with module breakdown - Test file inventory (185 tests across 14 files) - Prioritized checklist of 60+ modules to test - Testing guidelines and best practices - Coverage goals and CI integration notes
- SystemRegistry tests (23 tests, 88% coverage) - Dependency registration and validation - Topological sort for execution order - Circular dependency detection - Conditional system creation - Component tests: - Health (38 tests, 94% coverage): damage, armor, shields, regen - Transform (20 tests, 100% coverage): position, rotation, distance - Velocity (18 tests, 100% coverage): magnitude, normalize, scale Total: 284 tests across 18 test files
- Add 96 tests for Unit component covering movement, combat, commands, patrol, worker functionality, transform/cloak/submarine/transport mechanics, healing/repair, and buff systems - Add 16 tests for Selectable component covering selection state, control groups, and visual properties - Add 23 tests for pathfinding SpatialGrid covering insert/remove, query by radius/rect, walkability, and cell operations - Update TESTING.md with new test inventory (419 total tests) Total tests: 419 across 21 test files
- Add 99 tests for Building component covering construction lifecycle, production queues, addon mechanics, lift-off/landing, supply depot lowering, and turret attack capabilities - Add 24 tests for Projectile component covering all behavior types (homing, ballistic, linear), damage types, trail types, and splash mechanics - Update TESTING.md with new test inventory (542 total tests) Total tests: 542 across 23 test files
- Add 38 tests for Resource component covering mineral/vespene types, extractor requirements, gatherer management, saturation tracking, and worker optimization constants - Add 48 tests for Ability component covering energy management, cooldowns, ability usage, and all DOMINION_ABILITIES presets - Update TESTING.md with new test inventory (628 total tests) Total tests: 628 across 25 test files
- Add 83 tests for Wall component covering gate mechanics (open/close/ lock/auto), turret mounting, upgrades (shielded/repair_drone/weapon), shield regeneration and damage absorption, and connection management for wall mesh appearance - Update TESTING.md with new test inventory (711 total tests) All components in Priority 3 checklist now have test coverage. Total tests: 711 across 26 test files
- Add 51 tests for combat config (damage types, armor types, multipliers) - Add 42 tests for projectile types (behaviors, speeds, trail types) - Tests cover data-driven combat calculations used by CombatSystem
- Add 34 tests for formations (positions, sorting, configuration) - Add 55 tests for unit categories (assignments, subcategories, priorities) - Add 45 tests for resource types (gather rates, saturation, starting bags) - Add 28 tests for VectorPool utility (acquire/release, normalize, clamp) - Total test count: 966 tests
- Add 49 tests for walls (definitions, line calculation, connection types) - Add 38 tests for AI config (conditions, rules, utility scoring) - Total test count: 1053 tests
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.