Remove skip for GridFS download timeout test#1912
Remove skip for GridFS download timeout test#1912vbabanin wants to merge 4 commits intomongodb:mainfrom
Conversation
- Remove JAVA-5839 skip entry for "timeoutMS applied to entire download, not individual parts" test so it runs in CI JAVA-5839
There was a problem hiding this comment.
Pull request overview
Removes a Jira-based skip so the Unified CSOT GridFS download timeout test can run in CI again (JAVA-5839).
Changes:
- Unskips the Unified CSOT test case covering GridFS download timeout behavior (“timeoutMS applied to entire download, not individual parts”).
- Updates the
specificationsgit submodule URL.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| driver-sync/src/test/functional/com/mongodb/client/unified/UnifiedTestModifications.java | Removes the JAVA-5839 skip entry so the GridFS download timeout unified test runs. |
| .gitmodules | Changes the specifications submodule remote URL (currently to a fork). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| [submodule "specifications"] | ||
| path = testing/resources/specifications | ||
| url = https://github.com/mongodb/specifications | ||
| url = https://github.com/vbabanin/specifications No newline at end of file |
There was a problem hiding this comment.
The submodule URL was changed from the official mongodb/specifications repo to a personal fork (vbabanin/specifications). This is a significant behavioral/supply-chain change and does not align with the PR goal (unskipping a test). Please revert the URL back to the official upstream (or whatever canonical URL this repo expects) so CI and local checkouts use the intended specifications source.
Remove JAVA-5839 skip entry for "timeoutMS applied to entire download, not individual parts" test so it runs in CI
JAVA-5839