Fix several DX and test errors
-
Please check this box if this contribution uses AI-generated content (including content generated by GitLab Duo features) as outlined in the GitLab DCO & CLA. As a benefit of being a GitLab Community Contributor, you can request access to GitLab Duo.
This MR fixes a couple issues with the codebase that adds quality of life for developers (DX), and hopefully makes one of the flakey tests a little less flakey:
- Flakey test "destroy" functions that were previously named the same as functions in the "acceptance" tag are now renamed. This lets the default vscode settings run any accepts/flakey/settings test without needing to change tags.
- Read in the
policyProjectId
from theID
during theRead
function in both project/group security policy attachment to prevent an issue where the policy project GraphQL ID was occasionally not populated. - Fix an issue with the
SecurityProjectUnassignResponse
struct where the error format wasn't proper
Edited by Patrick Rice