Skip to content

npm run test command is not compatible with all env #394

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.

Sign up for GitHub

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

Jump to bottom
Open
gaurav-chaurasia opened this issue Mar 8, 2025 · 1 comment · May be fixed by #438
Open

npm run test command is not compatible with all env #394

gaurav-chaurasia opened this issue Mar 8, 2025 · 1 comment · May be fixed by #438
Labels
component:js sdk Issue/PR related to JavaScript SDK p3 status:triaged Issue/PR triaged to the corresponding sub-team type:feature request New feature request/enhancement

Comments

@gaurav-chaurasia
Copy link

gaurav-chaurasia commented Mar 8, 2025 •
edited
Loading

Description of the bug:

npm run test command in package.json is only compatible in linux/unix environment and fails when run on windows system

it runs npm run lint && npm run test:node:unit which fails on windows system

Actual vs expected behavior:

npm run test should run tests successfully on all platforms. but currently it only works on the Linux\Unix based systems and fails in the windows system
@gaurav-chaurasia gaurav-chaurasia mentioned this issue Mar 8, 2025
Open
@sevenhealer sevenhealer mentioned this issue Mar 8, 2025
Open
@Nikhilkumarr-dev Nikhilkumarr-dev mentioned this issue Mar 9, 2025
Open
@namrah77
Copy link

namrah77 commented Mar 9, 2025

A cross-platform solution is to use [npm-run-all] or [cross-env].
Install the dependencies and update your package.json file.
For npm-run-all

"scripts": {
  "test": "npm-run-all lint test:node:unit"
}

For cross-env

"scripts": {
"test": "cross-env npm run lint && npm run test:node:unit"
}

@mr-robot-404-cpu mr-robot-404-cpu mentioned this issue Mar 10, 2025
Open
@hkt74 hkt74 added type:feature request New feature request/enhancement component:js sdk Issue/PR related to JavaScript SDK labels Mar 12, 2025
@amirh amirh added status:triaged Issue/PR triaged to the corresponding sub-team p3 labels Mar 19, 2025
This was referenced Mar 19, 2025
Closed
Closed
@andyc2513 andyc2513 linked a pull request Mar 21, 2025 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
component:js sdk Issue/PR related to JavaScript SDK p3 status:triaged Issue/PR triaged to the corresponding sub-team type:feature request New feature request/enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

added cross-env to make npm run test on Windows andyc2513/generative-ai-js
4 participants
@amirh @hkt74 @gaurav-chaurasia @namrah77