-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
[wip] [3006.x] Test fix #66777
base: 3006.x
Are you sure you want to change the base?
[wip] [3006.x] Test fix #66777
Conversation
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.
Wondering what version is being used and coming from, to be passed into the function
@@ -84,11 +84,10 @@ def test_salt_versions_report_minion(salt_cli, salt_call_cli, salt_minion): | |||
@pytest.mark.parametrize( | |||
"binary", ["master", "cloud", "syndic", "minion", "call", "api"] | |||
) | |||
def test_compare_versions(binary, install_salt): | |||
def test_compare_versions(binary, version, install_salt): |
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.
Wondering why have version passed in, since install_salt has several versions available in it.
Also just grepped and test_compare_versions is not called by anything, so wondering about version getting passed in and used on line 192-191
Presuming version is same as used in test_salt_version, that is, output of test.version, hence an optimization
What does this PR do?
What issues does this PR fix or reference?
Fixes
Previous Behavior
Remove this section if not relevant
New Behavior
Remove this section if not relevant
Merge requirements satisfied?
[NOTICE] Bug fixes or features added to Salt require tests.
Commits signed with GPG?
Yes/No
Please review Salt's Contributing Guide for best practices, including the
PR Guidelines.
See GitHub's page on GPG signing for more information about signing commits with GPG.