Skip to content

Add test case for aggregation $addFields stage - #94

Merged
noisersup merged 1 commit into
OpenDocDB:mainfrom
noisersup:test-aggregation-stage-addFields
Jun 6, 2025
Merged

noisersup merged 1 commit into
OpenDocDB:mainfrom
noisersup:test-aggregation-stage-addFields

Conversation

@noisersup

@noisersup noisersup commented Jun 5, 2025 •

Copy link
Copy Markdown
Collaborator

No description provided.

@noisersup noisersup self-assigned this Jun 5, 2025
@noisersup
noisersup requested a review from chilagrow June 5, 2025 13:38
@noisersup
noisersup marked this pull request as ready for review June 5, 2025 13:38
Copilot AI review requested due to automatic review settings June 5, 2025 13:38
@noisersup
noisersup requested a review from AlekSi as a code owner June 5, 2025 13:38
@noisersup
noisersup enabled auto-merge (squash) June 5, 2025 13:38

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces a comprehensive test case for the aggregation $addFields stage, ensuring that various data types and edge cases are properly handled.

  • Added a JSON file test case (cts/aggregation/stages/addFields.json) covering a range of supported data types.
  • Added corresponding mongosh request and response tests to validate aggregation behavior.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
cts/aggregation/stages/addFields.json New test cases in JSON to verify $addFields stage functionality.
cts-mongosh/aggregation/stages/addFields/simple.response.js Response test cases covering various data type edge cases.
cts-mongosh/aggregation/stages/addFields/simple.request.js Request test case to trigger the $addFields stage.

@noisersup
noisersup merged commit 596c48c into OpenDocDB:main Jun 6, 2025
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.

4 participants