ahmido/TenantAtlas
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects 1 Releases Wiki Activity

fix: hydrate assignments even when empty

Browse Source
This commit is contained in:
Ahmed Darrazi 2026-01-10 10:48:24 +01:00
parent 9801ebae82
commit ebc003bf78
2 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
app/Services/Inventory/InventorySyncService.php
View File

@ -295,7 +295,7 @@ private function executeRun(InventorySyncRun $run, Tenant $tenant, array $normal
if ($includeDeps && $this->shouldHydrateAssignments($policyType)) {
$existingAssignments = $policyData['assignments'] ?? null;
if (! is_array($existingAssignments)) {
if (! is_array($existingAssignments) || count($existingAssignments) === 0) {
$hydratedAssignments = $this->fetchAssignmentsForPolicyType($policyType, $tenant, $externalId, $warnings);
if (is_array($hydratedAssignments)) {
$policyData['assignments'] = $hydratedAssignments;

3
tests/Feature/DependencyExtractionFeatureTest.php
View File

@ -286,7 +286,8 @@ public function listPolicies(string $policyType, array $options = []): GraphResp
'id' => 'sc-1',
'name' => 'Settings Catalog Policy',
'roleScopeTagIds' => ['scope-tag-1'],
// assignments omitted intentionally (must be hydrated via /assignments)
// assignments present but empty (must still be hydrated via /assignments)
'assignments' => [],
]], 200);
}