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

Add several tests for check-upgrade with advisories and minimal options #1511

Merged
merged 1 commit into from
Jun 6, 2024

Conversation

j-mracek
Copy link
Contributor

Some of the tests are related to new option --minimal for check-upgrade

@@ -55,6 +98,15 @@ Scenario: upgrade-minimal sec-severity
| Action | Package |
| upgrade | advisory_B-0:1.0-2.x86_64 |

Scenario: check-upgrade --minimal with sec-severity
When I execute dnf4 with args "upgrade-minimal --sec-severity Moderate"
Copy link
Contributor

Choose a reason for hiding this comment

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

I believe the I execute dnf4 step shouldn't be here, it is not being run, it is a wrong command and these tests are only for dnf5.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thank you - I've replaced dnf4 and dnf5 by simple dnf

@@ -64,6 +116,15 @@ Scenario: upgrade-minimal with pkgs specified sec-severity
| Action | Package |
| upgrade | advisory_B-0:1.0-2.x86_64 |

Scenario: check-upgrade --minimal with pkgs specified sec-severity
When I execute dnf4 with args "upgrade-minimal advisory_B --sec-severity Moderate"
Copy link
Contributor

Choose a reason for hiding this comment

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

I believe the I execute dnf4 step shouldn't be here, it is not being run, it is a wrong command and these tests are only for dnf5.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

@j-mracek j-mracek force-pushed the advisories-minimal branch from 2ff2341 to 47885c7 Compare June 6, 2024 08:27
Some of the tests are related to new option --minimal for check-upgrade
@j-mracek j-mracek force-pushed the advisories-minimal branch from 47885c7 to 7e73511 Compare June 6, 2024 08:43
Copy link
Contributor

@kontura kontura left a comment

Choose a reason for hiding this comment

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

Thank you

@kontura kontura merged commit 382ec64 into rpm-software-management:main Jun 6, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants