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

chore: use a license during e2e tests #172

Merged
merged 5 commits into from
Nov 8, 2023

Conversation

ricardomaraschini
Copy link
Contributor

in order to capture problems with the embedded license we need to use a version of the embedded cluster binary with an embedded kots license.

@ricardomaraschini ricardomaraschini force-pushed the use-binary-with-license-for-testing branch from f167eac to d47b65a Compare November 8, 2023 10:59
@ricardomaraschini ricardomaraschini marked this pull request as draft November 8, 2023 11:01
@ricardomaraschini ricardomaraschini force-pushed the use-binary-with-license-for-testing branch from d47b65a to 445f764 Compare November 8, 2023 11:02
in order to capture problems with the embedded license we need to use a
version of the embedded cluster binary with an embedded kots license.
@ricardomaraschini ricardomaraschini force-pushed the use-binary-with-license-for-testing branch from 445f764 to 21b415f Compare November 8, 2023 13:20
.gitignore Show resolved Hide resolved
Comment on lines 22 to 24
expect "Enter a new Admin Console password:" { send "password\r" }
expect "Confirm password:" { send "password\r" }
expect "Admin Console is ready!"
expect "Confirm password:" { send "password\r" }
Copy link
Member

Choose a reason for hiding this comment

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

should we really have to type the password three times?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

My mistake. Removed.

@ricardomaraschini ricardomaraschini marked this pull request as ready for review November 8, 2023 19:43
@ricardomaraschini ricardomaraschini merged commit 781f6b9 into main Nov 8, 2023
12 checks passed
@ricardomaraschini ricardomaraschini deleted the use-binary-with-license-for-testing branch November 8, 2023 23:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants