Skip to content

Test: Add connector management epic readiness guard#245

Merged
smithaldon1 merged 1 commit into
mainfrom
test/issue-206-connector-epic-readiness
May 26, 2026
Merged

Test: Add connector management epic readiness guard#245
smithaldon1 merged 1 commit into
mainfrom
test/issue-206-connector-epic-readiness

Conversation

@smithaldon1

Copy link
Copy Markdown
Member

Summary

Closes #206.

  • Add an epic-level regression test that maps Epic: Protocol connection management and operator console #206 acceptance criteria onto the implemented connector-management surfaces.
  • Verify OPC-UA, MQTT, and BACnet connection profile fixtures validate through the shared contract.
  • Verify browser serialization redacts secret/certificate reference values and read-only test-connection failures stay structured/redacted.
  • Verify operator-console protocol pages, read-only adapters, normalized event emission, and safety docs remain present.
  • Add the required learning-log close-out entry for the connector-management epic.

Test Evidence

  • .venv/bin/python -m pytest services/simulator/tests/test_connection_management_epic_readiness.py -> 5 passed
  • make test -> 282 passed
  • make lint -> All checks passed!
  • make typecheck -> compileall packages services passed
  • cd apps/web && npm test -> 18 passed
  • cd apps/web && npm run lint -> passed
  • cd apps/web && npm run typecheck -> passed
  • make docs -> Docs are Markdown-only in the MVP skeleton; no docs checker is configured yet.
  • git diff --check -> passed

Docs Impact

Follow-up Work

  • Keep future connector changes aligned with the read-only safety model.
  • Future write paths, if any, should start with a separate ADR and governed-action design rather than extending this epic silently.

@smithaldon1 smithaldon1 left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed and approved for merge

@smithaldon1 smithaldon1 merged commit 29802bf into main May 26, 2026
3 checks passed
@smithaldon1 smithaldon1 deleted the test/issue-206-connector-epic-readiness branch May 26, 2026 23:48
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.

Epic: Protocol connection management and operator console

1 participant