fix(inject): allow user to update atomic testing with assessment capability (#5784)#6387
Conversation
There was a problem hiding this comment.
Pull request overview
This PR adjusts frontend permission gating so users who have the Assessment – Manage capability can edit existing Atomic Testings (inject update form is no longer forced into read-only when the user lacks resource-grant MANAGE on the inject itself).
Changes:
- Update the inject form
disabledcondition to allow editing whenability.can(MANAGE, ASSESSMENT)is true (even if the user cannot manage the specific inject resource by grant). - Keep existing protections intact (inject disabled, missing contract content, explicit
permissions.readOnly, and non-NONE inherited context behavior).
Codecov Report✅ All modified and coverable lines are covered by tests. ❌ Your project check has failed because the head coverage (2.95%) is below the target coverage (80.00%). You can increase the head coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## main #6387 +/- ##
============================================
- Coverage 43.68% 43.64% -0.04%
Complexity 6993 6993
============================================
Files 2252 2252
Lines 62094 62094
Branches 8179 8179
============================================
- Hits 27124 27104 -20
- Misses 33226 33246 +20
Partials 1744 1744
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Proposed changes
Testing Instructions
As an Admin:
As the user in the newly created group:
Related issues