feat: add database provider abstraction#5
Conversation
- Create UserProvider entity for user-provider relationship - Replace database-specific array queries with standard JOINs - Add migration with data migration from providerIds array - Add addProviderToUser/removeProviderFromUser helper methods - Keep legacy providerIds for backwards compatibility 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
- Add user-provider.entity.spec.ts with 18 tests for parseProviderId, toProviderId, roundtrip conversion, and entity structure - Add users.service.provider.spec.ts with 12 tests for findByProviderId, addProviderToUser, and removeProviderFromUser - Fix Jest moduleNameMapper for src/* path resolution 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
|
Warning Rate limit exceeded@aabrius has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 28 minutes and 35 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (10)
✨ Finishing touches
🧪 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 |
Summary
Test plan
npm test -- --testPathPattern="user-provider.entity|users.service.provider"to verify all tests pass🤖 Generated with Claude Code