-
Notifications
You must be signed in to change notification settings - Fork 244
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
Fixing e2e vs integration test naming #1878
Fixing e2e vs integration test naming #1878
Conversation
0cc0058
to
52e7a22
Compare
1fbdc22
to
4762ac3
Compare
b26beb7
to
5dfbec8
Compare
ping @amitkrout |
Makefile
Outdated
# Run core beta flow e2e tests | ||
.PHONY: test-e2e-beta | ||
test-e2e-beta: | ||
go test -v github.com/openshift/odo/tests/e2escenarios --ginkgo.focus="odo core beta flow" -ginkgo.slowSpecThreshold=$(SLOW_SPEC_THRESHOLD) -ginkgo.v -timeout $(TIMEOUT) |
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.
why go test
here but other commands use ginkgo
?
It would be nice to unify it all
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.
Yup, will fix those.
@@ -12,5 +12,7 @@ make bin | |||
export PATH="$PATH:$(pwd)" | |||
export CUSTOM_HOMEDIR="/tmp/artifacts" | |||
|
|||
make test-e2e-scenarios | |||
make test-e2e-beta |
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.
wouldn't it be better to use make test-e2e-all
to make sure that new e2e scenarios are executed once added?
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.
+1
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.
shouldn't this use test-e2e-all
?
5dfbec8
to
82120e8
Compare
82120e8
to
1efefee
Compare
1efefee
to
2828e5d
Compare
2828e5d
to
1b7a3e6
Compare
75b8104
to
edcbb56
Compare
92a8829
to
04035b3
Compare
Could be a potential flake |
/hold cancel |
/lgtm |
After this gets merged you should open PR against release repo to stop using the old scripts
|
What is the purpose of this change? What does it change?
Was the change discussed in an issue?
related to #1780
How to test changes?
Review the change properly