Skip to content

[In Progress] Enforcement coverage + grant-authority lockdown #2334

Draft
ruizhang0519 wants to merge 4 commits into
DataJunction:mainfrom
ruizhang0519:enforcement-coverage
Draft

[In Progress] Enforcement coverage + grant-authority lockdown #2334
ruizhang0519 wants to merge 4 commits into
DataJunction:mainfrom
ruizhang0519:enforcement-coverage

Conversation

@ruizhang0519

Copy link
Copy Markdown

Summary

Test Plan

  • PR has an associated issue: #
  • make check passes
  • make test shows 100% unit test coverage

Deployment Plan

@netlify

netlify Bot commented Jul 16, 2026

Copy link
Copy Markdown

Deploy Preview for thriving-cassata-78ae72 canceled.

Name Link
🔨 Latest commit dbfa38e
🔍 Latest deploy log https://app.netlify.com/projects/thriving-cassata-78ae72/deploys/6a5a46d5c5488b0008cdb7b8

@ruizhang0519
ruizhang0519 force-pushed the enforcement-coverage branch from d2d4900 to ffc3e61 Compare July 17, 2026 01:39
@ruizhang0519
ruizhang0519 force-pushed the enforcement-coverage branch from ffc3e61 to 3c8787d Compare July 17, 2026 01:39
@@ -0,0 +1,230 @@
"""

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was this change meant to include more than just the test coverage changes?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, it is still in progress! I committed the tests first to let the CI run. I will mark it as Ready For Rerview when all the change are in place!

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah I see, sounds good!

@ruizhang0519 ruizhang0519 changed the title Add route-coverage guard for mutating endpoints [In Progress] Enforcement coverage + grant-authority lockdown Jul 17, 2026
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.

2 participants