chore: don't warn for unmocked data: URLs#1224
Conversation
I noticed this in test logs: ``` [WebServer] [test-fixtures] NO FIXTURE PATTERN [WebServer] ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ [WebServer] URL does not match any known fixture pattern [WebServer] URL: data:application/octet-stream;base64,AGFzbQEAAAABugM3YAF/AGACf38AYAF/AX9gA39/fwBgAn98AGACf38Bf2ADf39/AX9gBH9/[...snip...] ````
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
📝 WalkthroughWalkthroughThe change modifies the global fetch override in the cache module to exclude data: URLs from fixture interception. Previously, data: URLs were eligible to receive fixture-driven responses; with this alteration, such URLs now bypass the fixture system and are fetched through the original network path. The remaining fixture processing logic for other URL types, including esm.sh handling and fallback fixture-based responses, remains unchanged. 🚥 Pre-merge checks | ✅ 1✅ Passed checks (1 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
No actionable comments were generated in the recent review. 🎉 Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
I noticed this in test logs:
This isn't an actual network request, so we should ignore it.