Skip to content

Conversation

@f3l1x
Copy link
Member

@f3l1x f3l1x commented Jan 2, 2026

Document all test coverage gaps including:

  • 6 completely untested source classes
  • Missing test cases in existing tests
  • 5 new test category recommendations
  • ~105 total new test cases identified

Required for Doctrine ORM 3.x proxy factory compatibility.
New test files:
- ManagerProvider: EntityManagerProvider interface tests
- ContainerEntityListenerResolver: Entity listener resolution tests
- Binder: Closure binding utility tests
- SmartStatement: Statement conversion tests
- BuilderMan: DI helper tests
- OrmExtension.console: Console commands registration tests
- OrmExtension.customFunctions: Custom DQL functions tests
- OrmExtension.errors: Error handling and validation tests
- OrmExtension.multipleManagers: Multi-manager configuration tests

Extended existing tests:
- ContainerEventManager: hasListeners, getListeners, null args, multiple events
- ManagerRegistry: getConnectionNames, getManagerNames, reopen, getRepository
- MappingHelper: XML validation, error handling, fluent interface

New mock classes:
- DummyEntityListener, DummyHydrator, DummyStringFunction
@f3l1x f3l1x force-pushed the claude/analyze-missing-tests-hAxDu branch from b860a7c to 6a10db7 Compare January 2, 2026 19:07
- Fix class brace spacing in BuilderMan tests
- Remove unused variable in MappingHelper test
- Use sprintf instead of variable interpolation in console test
- Use single quotes for regex patterns in error tests
- Fix reference passing in ContainerEventManager test
- Add phpcs:ignoreFile to Binder test
@f3l1x f3l1x merged commit 2d8782f into master Jan 4, 2026
8 checks passed
@f3l1x f3l1x deleted the claude/analyze-missing-tests-hAxDu branch January 4, 2026 14:30
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.

3 participants