Skip to content

Commit c3c57fb

Browse files
maxleonardclaude
andcommitted
fix: add missing save mock responses for parallel ACL fetch
The parallel ACL fetching now consumes mock responses that were previously unused (fallback fetch was skipped when resource ACL existed). Add a 4th mock response for the saveAclFor step in the 3 affected "ignores the fallback ACL" tests. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 9aaad7c commit c3c57fb

File tree

1 file changed

+30
-0
lines changed

1 file changed

+30
-0
lines changed

src/access/wac.test.ts

Lines changed: 30 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2294,6 +2294,16 @@ describe("setAgentAccess", () => {
22942294
},
22952295
"https://some.pod/.acl",
22962296
),
2297+
)
2298+
// Save the ACL
2299+
.mockResolvedValueOnce(
2300+
mockResponse(
2301+
"",
2302+
{
2303+
status: 201,
2304+
},
2305+
"https://some.pod/resource.acl",
2306+
),
22972307
);
22982308

22992309
const result = await setAgentResourceAccess(
@@ -3012,6 +3022,16 @@ describe("setGroupResourceAccess", () => {
30123022
},
30133023
"https://some.pod/.acl",
30143024
),
3025+
)
3026+
// Save the ACL
3027+
.mockResolvedValueOnce(
3028+
mockResponse(
3029+
"",
3030+
{
3031+
status: 201,
3032+
},
3033+
"https://some.pod/resource.acl",
3034+
),
30153035
);
30163036

30173037
const result = await setGroupResourceAccess(
@@ -3695,6 +3715,16 @@ describe("setPublicResourceAccess", () => {
36953715
},
36963716
"https://some.pod/.acl",
36973717
),
3718+
)
3719+
// Save the ACL
3720+
.mockResolvedValueOnce(
3721+
mockResponse(
3722+
"",
3723+
{
3724+
status: 201,
3725+
},
3726+
"https://some.pod/resource.acl",
3727+
),
36983728
);
36993729

37003730
const result = await setPublicResourceAccess(

0 commit comments

Comments
 (0)