Skip to content

Add temporary NavigationModule support in EnroTest#202

Closed
isaac-udy-ai wants to merge 2 commits intomainfrom
fix/enro-test-application-context
Closed

Add temporary NavigationModule support in EnroTest#202
isaac-udy-ai wants to merge 2 commits intomainfrom
fix/enro-test-application-context

Conversation

@isaac-udy-ai
Copy link
Copy Markdown
Collaborator

Summary

  • Resolves the TODO in EnroTest.kt requesting functionality to temporarily install a NavigationModule for a particular test
  • Adds installModule()/removeModule() methods to EnroTest for test-scoped module management
  • Adds remove methods to BindingRepository, PathRepository, DecoratorRepository, and InterceptorRepository to support module removal
  • Adds removeModule() to EnroController as the counterpart to addModule()
  • Installed modules are automatically cleaned up when uninstallNavigationController() is called, ensuring test isolation

Test plan

  • Verify compilation passes on all targets
  • Verify existing tests still pass
  • Test installing and removing a NavigationModule within a test using the new API

🤖 Generated with Claude Code

isaac-udy-ai and others added 2 commits April 2, 2026 03:13
Use expect/actual pattern to provide platform-specific application context
retrieval. On Android, uses InstrumentationRegistry to get the application
context. On desktop, returns null.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Add installModule/removeModule to EnroTest so that tests can temporarily
add navigation bindings, interceptors, plugins, etc. for a specific test
and have them cleaned up automatically when the controller is uninstalled.

This required adding remove methods to BindingRepository, PathRepository,
DecoratorRepository, and InterceptorRepository, as well as a removeModule
method on EnroController.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@isaac-udy
Copy link
Copy Markdown
Owner

This was a test PR for an AI agent; closing it because there's no intention to merge it

@isaac-udy isaac-udy closed this Apr 1, 2026
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.

2 participants