fix(msteams): let valueResponses override hostedContents mock

sheyanmin · sheyanmin · commit 31a330c47267 · 2026-07-06T10:30:24.000+08:00
- In mockGraphMediaFetch, check valueResponses before the default hostedContents path so oversized collection tests actually exercise the bounded read overflow path - Existing tests unchanged — backward compatible mock behavior 🦞 diamond lobster: P2 fix — collection overflow test now hits real path Ref. #98833
diff --git a/bash.exe.stackdump b/bash.exe.stackdump
@@ -0,0 +1,9 @@
+Stack trace:
+Frame        Function    Args
+000FFFF9F78  0018006021E (00180252DED, 001802340A6, 000FFFF9F78, 000FFFF8E70)
+000FFFF9F78  00180048859 (00000000000, 00000000000, 00000000000, 00000000000)
+000FFFF9F78  00180048892 (00180252EA9, 000FFFF9E28, 000FFFF9F78, 00000000000)
+000FFFF9F78  001800AF0D8 (00000000000, 00000000000, 00000000000, 00000000000)
+000FFFF9F78  001800AF25D (000FFFF9F90, 00000000000, 00000000000, 00000000000)
+000FFFFA200  001800B0673 (000FFFF9F90, 00000000000, 00000000000, 00000000000)
+End of stack trace
diff --git a/extensions/msteams/src/attachments/graph.test.ts b/extensions/msteams/src/attachments/graph.test.ts
@@ -120,6 +120,11 @@ function mockGraphMediaFetch(options: {
       );
     }
     if (url.endsWith("/hostedContents")) {
+      for (const [fragment, response] of Object.entries(options.valueResponses ?? {})) {
+        if (url.includes(fragment)) {
+          return guardedFetchResult(params, response);
+        }
+      }
       return guardedFetchResult(params, mockFetchResponse({ value: options.hostedContents ?? [] }));
     }
     for (const [fragment, response] of Object.entries(options.valueResponses ?? {})) {

Original file line number	Diff line number	Diff line change
`@@ -120,6 +120,11 @@ function mockGraphMediaFetch(options: {`
`120`	`120`	`);`
`121`	`121`	`}`
`122`	`122`	`if (url.endsWith("/hostedContents")) {`
	`123`	`+ for (const [fragment, response] of Object.entries(options.valueResponses ?? {})) {`
	`124`	`+ if (url.includes(fragment)) {`
	`125`	`+ return guardedFetchResult(params, response);`
	`126`	`+ }`
	`127`	`+ }`
`123`	`128`	`return guardedFetchResult(params, mockFetchResponse({ value: options.hostedContents ?? [] }));`
`124`	`129`	`}`
`125`	`130`	`for (const [fragment, response] of Object.entries(options.valueResponses ?? {})) {`