fix(test): remove expected_rc bypass mechanism from common E2E resource #909

Merged
freemo merged 3 commits from fix/e2e-remove-rc-bypass into master 2026-03-14 03:50:52 +00:00

3 Commits

Author SHA1 Message Date
freemo 8c9d9c8c33 fix(providers): add missing module docstring to providers package
CI / benchmark-publish (pull_request) Has been skipped
CI / lint (pull_request) Successful in 15s
CI / quality (pull_request) Successful in 16s
CI / build (pull_request) Successful in 23s
CI / e2e_tests (pull_request) Successful in 26s
CI / security (pull_request) Successful in 32s
CI / typecheck (pull_request) Successful in 33s
CI / unit_tests (pull_request) Successful in 2m35s
CI / docker (pull_request) Successful in 36s
CI / integration_tests (pull_request) Successful in 3m27s
CI / coverage (pull_request) Successful in 4m56s
CI / lint (push) Successful in 11s
CI / build (push) Successful in 15s
CI / quality (push) Successful in 17s
CI / e2e_tests (push) Successful in 25s
CI / typecheck (push) Successful in 31s
CI / benchmark-regression (push) Has been skipped
CI / security (push) Successful in 35s
CI / integration_tests (push) Successful in 2m39s
CI / unit_tests (push) Successful in 3m9s
CI / docker (push) Successful in 35s
CI / coverage (push) Successful in 4m58s
CI / benchmark-publish (push) Has been cancelled
CI / benchmark-regression (pull_request) Successful in 35m21s
2026-03-14 03:45:18 +00:00
freemo a69af285a8 fix(test): remove retry masking and output filtering in database integration test
- Remove silent retry logic in Run Python Script that masked intermittent
  failures by retrying on empty stdout + non-zero RC
- Change output filter from 'in' to 'startswith' so only lines beginning
  with log prefixes are filtered, not lines containing them mid-string
- Remove [error from the filter list so error lines are never silently
  discarded from test output

Fixes #898
2026-03-14 03:45:18 +00:00
freemo 00c46c12fd 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:45:18 +00:00