Public Access
emilandDevin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
c9bf8dbcc6
devx tests were validating infra, grm, sso-bridge, and Mattermost OIDC workflow/skill files. This is an architecture violation — devx must not be aware of other repos. Those repos consume devx, not the other way round. Tests now only validate devx's own workflows and skills. Closes DEVX-160 Generated with [Devin](https://devin.ai) Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
32 lines
1.0 KiB
Markdown
32 lines
1.0 KiB
Markdown
# DEVX-160: Remove cross-repo contract tests from devx
|
|
|
|
## Problem
|
|
devx unit tests (`test_spec_driven_workflows.py`) were validating workflow
|
|
YAML and skill files in infra, grm, sso-bridge, and Mattermost OIDC repos.
|
|
This is an architecture violation — devx must not be aware of other repos.
|
|
Those repos consume devx; devx does not test them.
|
|
|
|
## Approach
|
|
Rewrite `test_spec_driven_workflows.py` to only test devx's own workflows
|
|
and skills. Remove all references to `_OBLACHNO_ROOT`, `_INFRA`, `_GRM`,
|
|
`_SSO_BRIDGE`, and parametrized repo lists.
|
|
|
|
REQ-1: No references to other repos in devx tests
|
|
REQ-2: All devx workflow/skill tests still pass
|
|
REQ-3: 100% coverage maintained
|
|
|
|
## Test Plan
|
|
- Run `pytest tests/unit/test_spec_driven_workflows.py` — all pass
|
|
- Run full test suite with coverage — 100%
|
|
|
|
## Deploy Plan
|
|
- Merge to master
|
|
|
|
## Rollback Plan
|
|
- Revert the merge commit
|
|
|
|
## Acceptance Criteria
|
|
- [x] REQ-1: No references to other repos in devx tests
|
|
- [x] REQ-2: All devx workflow/skill tests still pass
|
|
- [x] REQ-3: 100% coverage maintained
|