Skip to content

TOOLS-4148 Convert test/qa-tests/jstests/export/fields_csv.js to Go#931

Open
autarch wants to merge 1 commit into03-23-convert_test/qa-tests/jstests/export/field_file.js_to_gofrom
03-23-convert_test/qa-tests/jstests/export/fields_csv.js_to_go
Open

TOOLS-4148 Convert test/qa-tests/jstests/export/fields_csv.js to Go#931
autarch wants to merge 1 commit into03-23-convert_test/qa-tests/jstests/export/field_file.js_to_gofrom
03-23-convert_test/qa-tests/jstests/export/fields_csv.js_to_go

Conversation

@autarch
Copy link
Copy Markdown
Collaborator

@autarch autarch commented Mar 23, 2026

No description provided.

Copy link
Copy Markdown
Collaborator Author

autarch commented Mar 23, 2026

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

This was referenced Mar 23, 2026
@autarch autarch marked this pull request as ready for review March 24, 2026 15:02
@autarch autarch requested a review from a team as a code owner March 24, 2026 15:02
@autarch autarch requested review from tdq45gj and removed request for a team March 24, 2026 15:02
@autarch autarch changed the title Convert test/qa-tests/jstests/export/fields_csv.js to Go TOOLS-4148 Convert test/qa-tests/jstests/export/fields_csv.js to Go Mar 24, 2026
@autarch autarch force-pushed the 03-23-convert_test/qa-tests/jstests/export/fields_csv.js_to_go branch from f7c66f6 to bc4e6b2 Compare March 25, 2026 20:34
@autarch autarch force-pushed the 03-23-convert_test/qa-tests/jstests/export/field_file.js_to_go branch from c7d9f40 to 690b697 Compare March 25, 2026 20:34
@autarch autarch force-pushed the 03-23-convert_test/qa-tests/jstests/export/fields_csv.js_to_go branch from bc4e6b2 to cc2472e Compare March 25, 2026 20:47
@autarch autarch force-pushed the 03-23-convert_test/qa-tests/jstests/export/field_file.js_to_go branch from 9c9dbd5 to 7cb24bd Compare March 26, 2026 01:14
@autarch autarch force-pushed the 03-23-convert_test/qa-tests/jstests/export/fields_csv.js_to_go branch from cc2472e to 591774f Compare March 26, 2026 01:14
Copy link
Copy Markdown
Contributor

@tdq45gj tdq45gj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM


// TestExportNestedFieldsCSV verifies that mongoexport correctly handles nested
// field paths and $ projection in --fields with --csv output.
// Covers the nested field and projection scenarios of fields_csv.js.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's referencing a file we're removing. We should either clarify where to find the JS test or just remove the reference.

@autarch autarch force-pushed the 03-23-convert_test/qa-tests/jstests/export/field_file.js_to_go branch from 7cb24bd to c7402e5 Compare March 26, 2026 17:08
@autarch autarch force-pushed the 03-23-convert_test/qa-tests/jstests/export/fields_csv.js_to_go branch 2 times, most recently from 8479210 to f84c494 Compare March 26, 2026 21:53
@autarch autarch force-pushed the 03-23-convert_test/qa-tests/jstests/export/field_file.js_to_go branch from c7402e5 to bac7d87 Compare March 26, 2026 21:53
@autarch autarch force-pushed the 03-23-convert_test/qa-tests/jstests/export/fields_csv.js_to_go branch from f84c494 to 8a819a7 Compare March 27, 2026 18:27
@autarch autarch force-pushed the 03-23-convert_test/qa-tests/jstests/export/field_file.js_to_go branch from bac7d87 to 8435e34 Compare March 27, 2026 18:27
@autarch autarch force-pushed the 03-23-convert_test/qa-tests/jstests/export/field_file.js_to_go branch from 8435e34 to f562235 Compare March 27, 2026 20:27
@autarch autarch force-pushed the 03-23-convert_test/qa-tests/jstests/export/fields_csv.js_to_go branch from 8a819a7 to d3a3983 Compare March 27, 2026 20:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants