fix(test): convert soft warnings to hard failures in architecture test #908

Merged
freemo merged 2 commits from fix/integration-architecture-warn-to-fail into master 2026-03-14 03:39:28 +00:00

2 Commits

Author SHA1 Message Date
freemo 9148590542 fix(providers): add missing module docstring to providers package
CI / benchmark-publish (pull_request) Has been skipped
CI / lint (pull_request) Successful in 14s
CI / build (pull_request) Successful in 14s
CI / quality (pull_request) Successful in 16s
CI / e2e_tests (pull_request) Successful in 24s
CI / security (pull_request) Successful in 31s
CI / typecheck (pull_request) Successful in 38s
CI / integration_tests (pull_request) Successful in 2m41s
CI / unit_tests (pull_request) Successful in 5m4s
CI / coverage (pull_request) Successful in 4m59s
CI / docker (pull_request) Successful in 43s
CI / lint (push) Successful in 12s
CI / build (push) Successful in 14s
CI / quality (push) Successful in 15s
CI / e2e_tests (push) Successful in 26s
CI / security (push) Successful in 29s
CI / typecheck (push) Successful in 35s
CI / benchmark-regression (push) Has been skipped
CI / unit_tests (push) Successful in 2m5s
CI / integration_tests (push) Successful in 2m36s
CI / docker (push) Successful in 42s
CI / coverage (push) Successful in 4m59s
CI / benchmark-publish (push) Has been cancelled
CI / benchmark-regression (pull_request) Successful in 34m34s
2026-03-14 03:33:37 +00:00
freemo 64e606099c fix(test): convert soft warnings to hard failures in architecture test
Replace Run Keyword And Return Status soft checks with direct assertions
in three test cases:

- Docstring check now uses Should Contain directly instead of logging
  WARN, so missing docstrings cause a hard test failure.
- Settings file (config/settings.py) existence is now a hard File Should
  Exist assertion instead of a conditional that silently skips all
  checks when the file is absent.
- Exceptions file (core/exceptions.py) existence is now a hard File
  Should Exist assertion instead of a conditional that silently skips
  the exception hierarchy checks when the file is absent.
2026-03-14 03:33:37 +00:00