Skip to content

test: Add CSRF regression coverage and update endpoint classification - #144

Merged
mckaragoz merged 2 commits into
CodeBeamOrg:devfrom
mckaragoz:CSRFRegressionTests
Nov 6, 2026
Merged

mckaragoz merged 2 commits into
CodeBeamOrg:devfrom
mckaragoz:CSRFRegressionTests

Conversation

@mckaragoz

Copy link
Copy Markdown
Contributor

Summary

  • Add integration test coverage for login CSRF protection and session safety.
  • Update rate limiting classification tests to include the new antiforgery token endpoint.
  • Fix test expectations affected by the CSRF protection changes.

Notes

Test-only changes. No production behavior or public API changes.

@codecov

codecov Bot commented Nov 6, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.34%. Comparing base (abad249) to head (6a38b26).
⚠️ Report is 1 commits behind head on dev.

Additional details and impacted files
@@            Coverage Diff             @@
##              dev     #144      +/-   ##
==========================================
- Coverage   87.38%   87.34%   -0.04%     
==========================================
  Files         717      726       +9     
  Lines       20212    20339     +127     
  Branches     2464     2480      +16     
==========================================
+ Hits        17663    17766     +103     
- Misses       1769     1789      +20     
- Partials      780      784       +4     
Flag Coverage Δ
framework 87.34% <100.00%> (-0.04%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@mckaragoz
mckaragoz merged commit 7fdba58 into CodeBeamOrg:dev Nov 6, 2026
10 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