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

acceptance: bazelify trc ceremony tests #4209

Merged
merged 1 commit into from
Jun 8, 2022

Conversation

matzf
Copy link
Contributor

@matzf matzf commented May 31, 2022

Bazelify TRC ceremony tests.
The motivation/goal is that developers can run all integration tests locally with a single command.

Change invocation of test in CI to explicitly run these tests with bazel; this will be streamlined by combining this with the acceptance test steps in a follow up PR.


This change is Reviewable

@matzf matzf force-pushed the bazelify-trc-ceremony-test branch 11 times, most recently from 69c1949 to 68f5d43 Compare June 1, 2022 07:31
@matzf matzf marked this pull request as ready for review June 1, 2022 07:31
@matzf matzf force-pushed the bazelify-trc-ceremony-test branch from 68f5d43 to 4e0919d Compare June 2, 2022 14:38
@matzf matzf requested a review from oncilla June 2, 2022 14:40
Copy link
Contributor

@oncilla oncilla left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewed 6 of 6 files at r1, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @matzf)


.bazelrc line 26 at r1 (raw file):


test:integration --test_tag_filters=integration,-lint
test:integration_go --config=integration -- //... -//acceptance/... -//demo/...

Suggestion:

test:integration_go --config=integration -- //... -//acceptance/... -//demo/... -//tools/cryptoplayground/...

@matzf matzf force-pushed the bazelify-trc-ceremony-test branch from 4e0919d to 6d493e2 Compare June 7, 2022 13:29
Copy link
Contributor Author

@matzf matzf left a comment

Choose a reason for hiding this comment

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

Reviewable status: 5 of 6 files reviewed, 1 unresolved discussion (waiting on @oncilla)


.bazelrc line 26 at r1 (raw file):


test:integration --test_tag_filters=integration,-lint
test:integration_go --config=integration -- //... -//acceptance/... -//demo/...

Done.

Copy link
Contributor

@oncilla oncilla left a comment

Choose a reason for hiding this comment

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

Reviewed 1 of 1 files at r2, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @matzf)

@oncilla oncilla merged commit dc32230 into scionproto:master Jun 8, 2022
@matzf matzf deleted the bazelify-trc-ceremony-test branch June 16, 2022 14:39
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.

2 participants