-
Notifications
You must be signed in to change notification settings - Fork 289
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
Fix Cloud Slack Dev E2E test after recent changes #1277
Conversation
d703b76
to
29bec08
Compare
c6f7218
to
d05de95
Compare
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.
LGTM
@@ -88,6 +88,8 @@ func TestCloudSlackE2E(t *testing.T) { | |||
err := envconfig.Init(&cfg) | |||
require.NoError(t, err) | |||
|
|||
cfg.Slack.Tester.CloudBasedTestEnabled = false // override property used only in the Cloud Slack E2E tests |
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.
it would be good to differentiate a bit more it as we now have two things named cloud slack e2e tests, maybe sth like:
- cloud Slack enablement tests
- cloud Slack conversation tests
but let's follow up on this in a separate PR once you will be back @pkosiec
Co-authored-by: Pawel Kosiec <[email protected]>
Description
Changes proposed in this pull request:
Fix integration tests on mainTesting
CI proves it: