Skip to content

fix: pytest marker registration + config repair (split 4/6 of #125)#133

Merged
cryptoxdog merged 1 commit into
mainfrom
fix/pytest-config
Jul 23, 2026
Merged

fix: pytest marker registration + config repair (split 4/6 of #125)#133
cryptoxdog merged 1 commit into
mainfrom
fix/pytest-config

Conversation

@cryptoxdog

Copy link
Copy Markdown
Collaborator

Summary

Pytest configuration repair extracted from #125 (commit 42fdbd1 plus the pytest.ini hunk of 9b0d03c).

Changes

  • pytest.ini: register compliance, performance, slow, contract, finding markers so --strict-markers no longer rejects marked tests; drop global -x (stop-on-first-failure hid the true failure count in CI); add -ra
  • pyproject.toml: sync the [tool.pytest.ini_options] marker list and document that pytest.ini is the authoritative config

Local verification

  • pytest tests/ --collect-only completes with zero collection errors

See #130 for the full split-series map replacing #125.

@cursor

cursor Bot commented Jul 22, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@github-actions

github-actions Bot commented Jul 22, 2026

Copy link
Copy Markdown

Too Many Files Changed
Changed: 76 files
Limit: 50 files
Action Required: Split into multiple focused PRs

PR Too Large
Lines changed: 1390
Limit: 1000 lines
Action Required: Break into smaller, atomic PRs

📋 Best Practices for Large Changes

  1. Refactoring + Features: Separate into 2 PRs
  2. Multiple Features: One PR per feature
  3. Database + Code: Separate migration from logic
  4. Generated Code: Separate from manual changes

🚫 This PR is blocked from merging until size limits are met.

Extracted from #125 (commit 42fdbd1 + pytest.ini hunk of 9b0d03c):

- pytest.ini: register compliance/performance/slow/contract/finding
  markers so --strict-markers no longer rejects marked tests
- pytest.ini: drop global -x (stop-on-first-failure hid the true
  failure count in CI); add -ra for skip/xfail reporting
- pyproject.toml: sync the [tool.pytest.ini_options] marker list and
  document that pytest.ini is the authoritative config

Local verification: pytest --collect-only completes with zero
collection errors (previously interrupted by strict-marker rejections).
@cryptoxdog
cryptoxdog changed the base branch from main to fix/ruff-mypy-manual July 22, 2026 21:20
@sonarqubecloud

Copy link
Copy Markdown

@cryptoxdog
cryptoxdog changed the base branch from fix/ruff-mypy-manual to main July 23, 2026 00:52
@cryptoxdog
cryptoxdog merged commit 3a17aa9 into main Jul 23, 2026
11 of 21 checks passed
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.

1 participant