Skip to content

feat: integration tests for nodejs generator - #140

Merged
beeme1mr merged 9 commits into
open-feature:mainfrom
Adityasinghvats:fix
Jun 13, 2025
Merged

feat: integration tests for nodejs generator#140
beeme1mr merged 9 commits into
open-feature:mainfrom
Adityasinghvats:fix

Conversation

@Adityasinghvats

Copy link
Copy Markdown
Contributor

This PR

Fixes #117

Notes

Follow-up Tasks

How to test

  1. Run nodejs test
make test-integration-nodejs
  1. Run all integration test
make test-integration

Screenshot 2025-06-10 233609

Signed-off-by: Adityasinghvats <131326798+Adityasinghvats@users.noreply.github.com>
Signed-off-by: Adityasinghvats <131326798+Adityasinghvats@users.noreply.github.com>
Signed-off-by: Adityasinghvats <131326798+Adityasinghvats@users.noreply.github.com>
Signed-off-by: Adityasinghvats <131326798+Adityasinghvats@users.noreply.github.com>
Signed-off-by: Adityasinghvats <131326798+Adityasinghvats@users.noreply.github.com>
Signed-off-by: Adityasinghvats <131326798+Adityasinghvats@users.noreply.github.com>
Signed-off-by: Adityasinghvats <131326798+Adityasinghvats@users.noreply.github.com>
@Adityasinghvats

Copy link
Copy Markdown
Contributor Author

Hey once again sorry for any inconvenience, I was not able to get rid of signoff comments from history , so I cherry picked commits and created a new branch and a new PR

  • Now the test exits with code 1, in case the test method is not found as suggested by @beeme1mr

@beeme1mr beeme1mr left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good, thanks! Sorry about the DCO issue.. I know it's a pain.

@beeme1mr
beeme1mr added this pull request to the merge queue Jun 13, 2025
Merged via the queue into open-feature:main with commit b867485 Jun 13, 2025
This was referenced Feb 6, 2026
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.

feat: integration tests for nodejs generator

2 participants