(SP: 1) [SHOP] fix stale TEST_LEGAL_CONSENT import in test helper#443
(SP: 1) [SHOP] fix stale TEST_LEGAL_CONSENT import in test helper#443liudmylasovetovs merged 1 commit intodevelopfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe test helper Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Description
Fixes a build-breaking TypeScript error caused by a stale
TEST_LEGAL_CONSENTimport after migrating to thecreateTestLegalConsent()factory.The helper
makeCheckoutReqwas still importing a removed constant, which caused Netlify build to fail during type checking. The fix updates the helper to use the runtime factory instead of a cached import-time value, aligning test helpers with the current legal-consent contract.Related Issue
Issue: #<issue_number>
Changes
TEST_LEGAL_CONSENTimport withcreateTestLegalConsent()in test helperDatabase Changes (if applicable)
How Has This Been Tested?
Screenshots (if applicable)
N/A
Checklist
Before submitting
Reviewers
Summary by CodeRabbit