-
Notifications
You must be signed in to change notification settings - Fork 165
ci: add npm run conformance #374
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.
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
Conversation
Signed-off-by: Grant Timmerman <[email protected]>
Signed-off-by: Grant Timmerman <[email protected]>
set -e | ||
|
||
# Navigate to the conformance tests and install the local FF. | ||
cd test/conformance; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[nit] May want to put the rest of this in a subshell by adding parenthesis around the whole thing. If you run this from repo root, this would put you in this directory after, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When running this from root (or even any directory within the npm package), I don't see the directory changing.
I believe npm scripts spawn a shell process under the hood, so npm run X
will start a new shell, run, then exit.
Signed-off-by: Grant Timmerman <[email protected]>
CONTRIBUTING.md
Outdated
@@ -34,6 +34,14 @@ To run an individual test, you can run a command such as the following: | |||
npm run test -- -g 'loading function' | |||
``` | |||
|
|||
### Conformance Tests | |||
|
|||
To run the conformance tests, run: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we add a note about needing to install Go 1.16 first?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds good. I added a note there.
Signed-off-by: Grant Timmerman <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Adds command to run conformance tests:
npm run conformance
Very similar to the Go setup, with a few minor modifications.
Conformance
package-lock.json
is ignored, because it's just pointing tofile:../../
.FULL LOGS