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 6 test scenarios for deplist #183
Conversation
j-mracek
commented
Sep 23, 2016
- Enhance of ci-dnf-stack
|
||
Scenario: Deplist with --latest-limit and verbose | ||
When I execute "dnf" command "repoquery --deplist --latest-limit 1 --verbose TestA" with "success" | ||
Then line from "stdout" should "start" with "package: TestA-1.0.0-2.noarch" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why don't you use same comparison as above?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Impossible with verbose mode
Feature: Deplist as commmand and option | ||
|
||
Scenario: Feature Setup | ||
Given I use the repository "upgrade_1" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not to use generation on the fly? this would help visibility.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Less work and now idea how to setup more complicate repo, with versions, and requires. Pity that you pushed the changes, because they are not mature.
Actually I removed 2 lines + added About 2 lines - it's bug somewhere in ur PR for DNF. After last package it should not print extra newline. |
@ignatenkobrain Fixed @xfail now you can merge it. |
@ignatenkobrain Please can you merge it, it was refactored. |
split this PR and that bit from #182 |
@ignatenkobrain Done |
@dnf-bot r+ |