-
Notifications
You must be signed in to change notification settings - Fork 7
test(cli): fix migrateStorage test failures #164
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
Update test fixtures to include workspace_slug field in fetchCurrentUser mock data, matching the actual API response structure. This fixes test failures where profile names were defaulting to "default" instead of expected workspace slugs.
|
!release cli patch |
|
🚀 @Leechael release command accepted: Target branch: |
|
🎉 Release completed:
📦 Package Info
📄 Files included |
|
!release js patch |
|
🚀 @Leechael release command accepted: Target branch: |
|
❌ @Leechael release failed: Branch: |
Add missing object_type field to node_info mock data to match NodeRefSchema requirements.
📋 Check Results✨ JS SDK - Code FormattingShow format check results🔍 JS SDK - TypeScript Type CheckShow type check output🧪 JS SDK - Test ResultsShow test output📝 JS SDK - Lint CheckShow lint results🌐 JS SDK - Browser CompatibilityShow browser test results🌐 Browser Compatibility ReportBrowser compatibility tests completed across:
The SDK has been verified to work in modern browser environments. Check run: https://github.com/Phala-Network/phala-cloud/actions/runs/21546973255 |
|
!release js patch |
|
🚀 @Leechael release command accepted: Target branch: |
|
🎉 Release completed:
📦 Package Info
📄 Files included |
Summary
migrateStoragetests by addingworkspace_slugfield to mock datafetchCurrentUserTest plan
bun test cli/src/core/migrate-storage.test.ts- all 4 tests pass