Feature/remilovoll/26/03/movepolicyfilemigrating

[lovoll]

Description

Related Issue(s)

• #{issue number}

Verification

• Your code builds clean without any errors or warnings
• Manual testing done (required)
• Relevant automated test added (if you find this hard, leave it and we'll help out)
• All tests run green

Documentation

• User documentation is updated with a separate linked PR in altinn-studio-docs. (if applicable)

[Copilot]

Pull request overview

This PR introduces feature-flagged support for a new instance-delegation policy file path format (intended for an EF-backed storage layout) and adds logic to move/copy existing policy blobs to the new location during instance assignment import.

Changes:

• Add AccessManagement.InstanceDelegation.EF feature flag branching to select “old” vs “new” instance delegation policy paths.
• Update instance assignment import flow to copy policy blobs to a new path using blob leases, and extend APIs/call sites with partyId.
• Update constructors/tests to inject IFeatureManager and add a new helper for the EF path format.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 10 comments.

Show a summary per file

File	Description
src/apps/Altinn.AccessManagement/test/AccessMgmt.Tests/PolicyAdministrationPointTest.cs	Updates test wiring for new PolicyAdministrationPoint constructor dependency (IFeatureManager).
src/apps/Altinn.AccessManagement/src/Altinn.AccessMgmt.Core/Utils/Helper/DelegationCheckHelper.cs	Treats a new “missing original policy file” error as eligible for error-queue handling.
src/apps/Altinn.AccessManagement/src/Altinn.AccessMgmt.Core/Services/Contracts/IAssignmentService.cs	Extends ImportInstanceAssignmentChange API with fromPartyId.
src/apps/Altinn.AccessManagement/src/Altinn.AccessMgmt.Core/Services/AssignmentService.cs	Adds blob-copy + lease logic for moving instance policy files to a new location.
src/apps/Altinn.AccessManagement/src/Altinn.AccessMgmt.Core/HostedServices/Services/SingleInstanceRightSyncService.cs	Passes partyId into ImportInstanceAssignmentChange.
src/apps/Altinn.AccessManagement/src/Altinn.AccessManagement.Core/Services/PolicyAdministrationPoint.cs	Adds feature-flag routing to choose old/new instance policy paths.
src/apps/Altinn.AccessManagement/src/Altinn.AccessManagement.Core/Services/AppsInstanceDelegationService.cs	Adds feature-flag handling and party lookups to construct an internal instance URN.
src/apps/Altinn.AccessManagement/src/Altinn.AccessManagement.Core/Helpers/DelegationHelper.cs	Adds TryGetNewDelegationPolicyPathFromInstanceRule for the new EF path format.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

[jonkjetiloye]

Litt småpirk fra copilot du kan vurdere.
Ellers ser det bra ut

[sonarqubecloud]

Quality Gate failed for 'Authorization Altinn.AccessManagement'

Failed conditions
10.8% Coverage on New Code (required ≥ 65%)
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE