Tests for pre-transaction-actions plugin, improve post-transaction-actions plugin test #1459
Conversation
|
It seems that CI il is not related to this PR. |
Yes that is an issue caused by libsolv update, here is a fix to update our CI: #1452. |
| | out | | | ||
|
|
||
|
|
||
| Scenario: Do not traceback when reason change is in transaction |
There was a problem hiding this comment.
The Scenario name mentions reason change but I don't see any changing reasons in the test.
| | key | value | | ||
| | installonlypkgs | installonlyA | | ||
| | installonly_limit | 2 | | ||
| When I execute dnf with args "-v install installonlyA-1.0" |
There was a problem hiding this comment.
Is the -v flag required in these executions?
There was a problem hiding this comment.
No. I will remove it.
| """ | ||
| When I execute dnf with args "-v install installonlyA-2.2" | ||
| Then the exit code is 0 | ||
| And Transaction is following |
There was a problem hiding this comment.
Reason change happen when install and remove kernel in one transaction. This is a workaround that fix logic of history DB in background of DNF.
There was a problem hiding this comment.
It looks like that reason change is added only in DNF5, and not in DNF4
Created from tests for post-transaction-actions.
- to store/check all lines use appending ">>" instead of ">" in output redirection - removed unnecessary dnf argument "-v" - added comment about reason change
jrohel
force-pushed
the
dnf4/tests_pre-transaction-actions_plugin
branch
from
c497dcc to
823563f
Compare
jrohel
changed the title
|
Thank you, the failing tests are not related. |
kontura
merged commit dfb129e
into
rpm-software-management:dnf-4-stack
Since rpm-software-management#1459 the tests require the pre-transaction plugin.
Since #1459 the tests require the pre-transaction plugin.
Created from tests for post-transaction-actions.
Requires rpm-software-management/dnf-plugins-core#523