chore(deps): update gradle/actions action to v6#1337
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1337 +/- ##
=========================================
Coverage 77.62% 77.62%
Complexity 1267 1267
=========================================
Files 106 106
Lines 5904 5904
Branches 612 612
=========================================
Hits 4583 4583
Misses 1025 1025
Partials 296 296 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
The failing job is caused by a 400 Bad Request error from the following endpoint: PUT /_plugins/_security/api/resource/share. The error message is: access_level must be one of: This suggests that the test is trying to share resources (such as a workflow or its state) with an invalid access_level value. In the code, the shareConfig and patchSharingInfo helper methods are used to construct the payload for resource sharing. The access_level is passed as a variable (such as WORKFLOW_READ_ONLY_AG, WORKFLOW_FULL_ACCESS_AG, etc.). If these constants do not match the accepted values expected by the API, you will get this error. Solution:
For example, if the allowed values per API are "read_only" and "full_access", change their definitions at the top of FlowFrameworkResourceSharingRestApiIT.java from: private static final String WORKFLOW_READ_ONLY_AG = "workflow_read_only"; to: private static final String WORKFLOW_READ_ONLY_AG = "read_only"; Match the values to exactly what the resource sharing API accepts. After correcting these values, your resource sharing operations should succeed and the failing integration tests should pass. |
|
@cwperks @DarshitChanpura Was there a recent change in resource sharing setup? I'm not sure I trust Copilot's change suggestion. |
|
Here's the full chain:
So the question is: is We need to find where flow-framework registers its resource types with the security plugin and whether it calls |
|
Seems the new validation code is more strict and caught an incorrect filename in the earlier version: |
491ad5f to
5d48913
Compare
Signed-off-by: mend-for-github-com[bot] <mend-for-github-com[bot]@users.noreply.github.com>
5d48913 to
19884fc
Compare
This PR contains the following updates:
v5→v6Release Notes
gradle/actions (gradle/actions)
v6Compare Source
Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.