-
Notifications
You must be signed in to change notification settings - Fork 146
feat(db): add MongoDB integration tests & CI support #1200
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
- Add comprehensive MongoDB integration tests (not mocked) - Add database comparison tests for file vs MongoDB parity - Add CI integration with MongoDB service - Add local Docker-based testing support - Add test scripts: test:mongo and test:mongo:docker - Improve MongoDB client error handling
✅ Deploy Preview for endearing-brigadeiros-63f9d0 canceled.
|
This comment was marked as off-topic.
This comment was marked as off-topic.
|
Thank you for checking, @coopernetes sir. This is not a duplicate of PR #1165. While #1165 focuses on Cypress E2E test setup and Docker Compose adjustments, this pull request introduces MongoDB integration test coverage, including:
These changes are related to issue #1168 and are separate from the Cypress work in #1165. Please let me know if anything needs to be adjusted or split further. I am happy to revise based on feedback. |
|
Conflicts to resolve and all 34 mongo tests are failing due to a lack of a conneciton string |
|
@sidshas03 could you take a look at the conflict and test failures (Error: MongoDB connection string is not provided) and we'll see about getting you a review. @coopernetes tells me that this is orthogonal to his PR and hence quite valid so many thanks for the contribution. |
|
Hey @kriswest, im really sorry i was unwell for weeks. thanks for the nudge and for checking with @coopernetes. Ack on both points:
|
pr-1168(#1168)