-
Notifications
You must be signed in to change notification settings - Fork 134
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
Split e2e tests from unit tests: #725
Conversation
Codecov Report
@@ Coverage Diff @@
## main #725 +/- ##
=======================================
Coverage 55.54% 55.54%
=======================================
Files 26 26
Lines 1100 1100
=======================================
Hits 611 611
Misses 474 474
Partials 15 15 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
This makes it clear that there are 2 different types of tests. Also remove nix in CI for tests. The update to the make build target is to improve the output. current: ❯ make build CGO_ENABLED=0 \ GOOS=linux \ GOARCH=amd64 \ go build \ \ -o ./bin/tink-server-linux-amd64 \ ./cmd/tink-server CGO_ENABLED=0 \ GOOS=linux \ GOARCH=amd64 \ go build \ \ -o ./bin/tink-worker-linux-amd64 \ ./cmd/tink-worker CGO_ENABLED=0 \ GOOS=linux \ GOARCH=amd64 \ go build \ \ -o ./bin/tink-controller-linux-amd64 \ ./cmd/tink-controller CGO_ENABLED=0 \ GOOS=linux \ GOARCH=amd64 \ go build \ \ -o ./bin/virtual-worker-linux-amd64 \ ./cmd/virtual-worker new: ❯ make build CGO_ENABLED=0 GOOS=linux GOARCH=amd64 go build -o ./bin/tink-server-linux-amd64 ./cmd/tink-server CGO_ENABLED=0 GOOS=linux GOARCH=amd64 go build -o ./bin/tink-worker-linux-amd64 ./cmd/tink-worker CGO_ENABLED=0 GOOS=linux GOARCH=amd64 go build -o ./bin/tink-controller-linux-amd64 ./cmd/tink-controller CGO_ENABLED=0 GOOS=linux GOARCH=amd64 go build -o ./bin/virtual-worker-linux-amd64 ./cmd/virtual-worker Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
We're getting: "warning: -s is deprecated as it is always enabled" The buf.lock was updated after a make verify. Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
@Mergifyio queue |
🛑 The pull request has been removed from the queuePull request #725 has been dequeued due to failing checks or checks timeout. You can take a look at In case of a failure due to a flaky test, you should first retrigger the CI. |
Description
This makes it clear that there are 2 different types of tests. Also remove nix in CI for tests. The update to the make build target is to improve the output.
current:
new:
Why is this needed
Fixes: #
How Has This Been Tested?
How are existing users impacted? What migration steps/scripts do we need?
Checklist:
I have: