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

Tests for changes of history rollback transactions merging #1461

Merged
merged 2 commits into from
Mar 12, 2024

Conversation

jan-kolarik
Copy link
Member

  • Add tests for history rollback between states with the same version of a package installed.
  • Adjust history view merging test to match the new behavior

Requires: rpm-software-management/libdnf#1644

Based on the changes of transaction merging behavior for packages with the same version, as introduced in: rpm-software-management/dnf#2031.
@jan-kolarik jan-kolarik changed the title Tests for history rollback transactions merging changes Tests for changes of history rollback transactions merging Feb 27, 2024
@pkratoch pkratoch self-assigned this Mar 4, 2024
@pkratoch
Copy link
Contributor

pkratoch commented Mar 7, 2024

The tests look good, but waiting for the required libdnf change. It would be nice to cover more scenarios of sequences to rollback, but these suffice to test the corresponding issues.

@jan-kolarik
Copy link
Member Author

The tests look good, but waiting for the required libdnf change. It would be nice to cover more scenarios of sequences to rollback, but these suffice to test the corresponding issues.

I've created an issue for that: #1466.

@j-mracek j-mracek merged commit 8234f42 into dnf-4-stack Mar 12, 2024
2 of 3 checks passed
@j-mracek j-mracek deleted the jkolarik/history-rollback-merge-fix-tests branch March 12, 2024 12:56
@j-mracek
Copy link
Contributor

LGTM

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.

3 participants