-
Notifications
You must be signed in to change notification settings - Fork 119
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
Run unit tests #1287
base: main
Are you sure you want to change the base?
Run unit tests #1287
Conversation
- Update the node provisioner test to expect the empty string as the node ip - Update the node01 provisioner test to pass etcd sync as false Signed-off-by: aerosouund <aerosound161@gmail.com>
There is an import called container and a method named container, rename the method to containerFromNames. Signed-off-by: aerosouund <aerosound161@gmail.com>
Signed-off-by: aerosouund <aerosound161@gmail.com>
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
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.
Which CI job will these tests be run in?
@brianmcarey |
Ok maybe we should update the job name to match the more generic nature of what is being run. |
Good point - however a separate task from this PR, no? Suggestion: how about changing |
Yes, but where should this change take place ? |
it's inside |
@brianmcarey WDYT? ^^ |
Sounds good to me - the lane should also be set to always run and be required too. |
@brianmcarey @dhiller |
What this PR does / why we need it:
With moving bash opts to golang, unit tests were introduced for every opt. But they aren't being run as a part of any CI process
This PR fixes all broken tests and changes the
make test
command of the gocli to run all tests in the projectChecklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.
Release note: