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
Add make integration-actions command #1948
Conversation
Signed-off-by: Javier López Barba <javier@okteto.com>
Codecov Report
@@ Coverage Diff @@
## master #1948 +/- ##
=======================================
Coverage 33.42% 33.42%
=======================================
Files 122 122
Lines 13841 13841
=======================================
Hits 4626 4626
Misses 8715 8715
Partials 500 500 Continue to review full report at Codecov.
|
@@ -40,7 +40,11 @@ test: | |||
|
|||
.PHONY: integration | |||
integration: | |||
go test github.com/okteto/okteto/integration -tags=integration --count=1 -v -timeout 45m | |||
go test github.com/okteto/okteto/integration -tags="common integration actions" --count=1 -v -timeout 45m |
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.
I think it should be fine with 15m timeout
@@ -1,5 +1,5 @@ | |||
//go:build integration | |||
// +build integration | |||
//go:build actions |
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.
could you add an envvar to skip the execution of the context
action?
that why we can automate the execution of that tests only if the CLI version under test support the context
command
Signed-off-by: Javier López Barba <javier@okteto.com>
Signed-off-by: Javier López Barba <javier@okteto.com>
integration/actions_test.go
Outdated
@@ -205,6 +205,10 @@ func TestContextAction(t *testing.T) { | |||
t.Skip("this test is not required for windows e2e tests") | |||
return | |||
} | |||
if os.Getenv("OKTETO_SKIP_CTX_TEST") != "" { |
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.
Can we call it OKTETO_SKIP_CONTEXT_TEST
to make it more verbouse?
Signed-off-by: Javier López Barba <javier@okteto.com>
Signed-off-by: Javier López Barba <javier@okteto.com>
Signed-off-by: Javier López Barba <javier@okteto.com>
Signed-off-by: Javier López Barba javier@okteto.com
Adds a make command to run only actions integration tests
Proposed changes