Skip to content

Fix the module/updateinfo.feature tests #1548

@jan-kolarik

Description

@jan-kolarik

Quoted from #1547:

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):

Assertion Failed: Stdout is not empty, it contains:
Name                   Type        Severity                  Package              Issued
FEDORA-2019-0329090518 enhancement none     nodejs-1:8.14.0-1.x86_64 2019-03-29 00:00:00

Another option would be to fix the test to use the correct command, but leave it enabled and failing.

cc @pkratoch, do you have an idea of what the problem is and how long it might take to fix? If it's complicated/low priority, we can go ahead and disable the test and put an issue on the backlog for it, otherwise maybe it's better to go ahead and fix it instead of disabling then immediately re-enabling it.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    Status

    Backlog

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions