-
Notifications
You must be signed in to change notification settings - Fork 10
Open
Labels
type: enhancementNew feature or requestNew feature or request
Description
Context
Various review tools (CodeRabbit, Sourcery, Snyk, etc.) have flagged security improvements and refactoring opportunities in this project. Before implementing these changes, we need proper acceptance/E2E tests to ensure we don't break working functionality.
The Problem
Currently, there's no way to test the webapp's integration with Switcher devices without real hardware. This makes it risky to:
- Apply security fixes (e.g., running container as non-root user)
- Refactor code based on review tool recommendations
- Upgrade dependencies that might affect device communication
Blocker
We're adding a testing module to aioswitcher that will provide mock device infrastructure:
Discussion: TomerFi/aioswitcher#851
Plan
- Wait for / implement
aioswitcher.testingmodule - Add acceptance tests to this project using the mock infrastructure
- Then proceed with security fixes and refactoring
Acceptance Criteria
-
aioswitcher.testingmodule is available - Acceptance tests cover core API endpoints
- Tests run in CI without real devices
- Safe to proceed with security/refactoring changes
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
type: enhancementNew feature or requestNew feature or request