Skip to content
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

Basic integration tests #11

Merged
merged 176 commits into from
Nov 10, 2023
Merged

Basic integration tests #11

merged 176 commits into from
Nov 10, 2023

Conversation

dherman
Copy link
Collaborator

@dherman dherman commented Sep 10, 2023

This PR implements basic integration test infrastructure using verdaccio to create a private proxy during CI, so that the packages being tested can be transparently published and then discarded.

dherman and others added 24 commits September 24, 2023 13:24
…e integration tests:

- use .npmrc to redirect between public registry and verdaccio
- only put scoped packages in verdaccio
- move integration tests into @neon-integration-tests scope
… since they're already in the shell scripts
done
exit 1
fi
run: ./test/lint/check-bundles.sh
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Moved the listing logic into a separate shell script file.

@@ -2,3 +2,6 @@
members = [
"pkgs/cargo-messages"
]
exclude = [
"test/integration/sniff-bytes"
Copy link
Collaborator Author

@dherman dherman Oct 30, 2023

Choose a reason for hiding this comment

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

This is needed so the root cargo workspace doesn't get confused by the Rust code inside the integration tests.

@dherman dherman marked this pull request as ready for review October 30, 2023 16:40
@dherman dherman merged commit c0fc11d into main Nov 10, 2023
2 checks passed
@dherman dherman deleted the integration-tests branch November 10, 2023 20:10
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.

1 participant