Skip to content

Commit

Permalink
module/updateinfo: Disable failing test
Browse files Browse the repository at this point in the history
After merging #1540, it was revealed that this test in module/updateinfo.feature wasn't actually testing anything, and the behavior it intends to test is (and has long been?) broken. The test was expecting an empty stdout (or <REPOSYNC>), which it was getting, but only after refactoring the tests did we notice the message on stderr:

Unknown argument "--list" for command "updateinfo".

For now, I've updated the test to call the correct command, `updateinfo
list`, and disabled the test since it is now failing (but in a more
valid way).
  • Loading branch information
evan-goode committed Sep 5, 2024
1 parent edec323 commit 661c414
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions dnf-behave-tests/dnf/module/updateinfo.feature
Original file line number Diff line number Diff line change
Expand Up @@ -53,14 +53,15 @@ Given I use repository "dnf-ci-fedora-modular-updates"
And stdout is empty


@dnf5
# Test is failing with DNF 5.
# @dnf5
@bz1804234
Scenario: having installed packages from one collection and enabled all modules from another doesn't activate advisory
Given I use repository "dnf-ci-fedora"
And I execute dnf with args "install nodejs"
And I use repository "dnf-ci-fedora-modular-updates"
And I execute dnf with args "module enable postgresql:9.6"
When I execute dnf with args "updateinfo --list"
When I execute dnf with args "updateinfo list"
Then stderr is
"""
<REPOSYNC>
Expand Down

0 comments on commit 661c414

Please sign in to comment.