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

Add test for fate#317701 - Do not use perl-bootloader in yast2-bootloader #1571

Merged

Conversation

okurz
Copy link
Member

@okurz okurz commented Jul 26, 2016

On SLE 12-SP2 or greater check that no perl-Bootloader-YAML is present in the
installed system as described as a test case in fate#317701.

Locally verified on TW where it fails because perl-Bootloader-YAML is present.

Related progress issue: https://progress.opensuse.org/issues/11436

@okurz okurz force-pushed the feature/fate317701_no_perl_bootloader branch from ae8a503 to 8e1da0a Compare July 26, 2016 11:14
# (fate#317701)
sub run() {
select_console('user-console');
assert_script_run('rpm -qi perl-Bootloader-YAML ; [ $? = 1 ]');
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why not just assert_script_run ('! rpm -qi perl-Bootloader-YAML'); ? The bash construct looks pretty wicked

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

because I always forget where to put the exclamation mark ;-)

…ader

On SLE 12-SP2 or greater check that no perl-Bootloader-YAML is present in the
installed system as described as a test case in fate#317701.

Locally verified on TW where it fails because perl-Bootloader-YAML is present.

Verification run: http://lord.arch/tests/2057

Related progress issue: https://progress.opensuse.org/issues/11436
@okurz okurz force-pushed the feature/fate317701_no_perl_bootloader branch from 8e1da0a to 11fae39 Compare July 26, 2016 15:20
@okurz
Copy link
Member Author

okurz commented Jul 26, 2016

updated

@coolo coolo merged commit 28e3d10 into os-autoinst:master Jul 26, 2016
@okurz okurz deleted the feature/fate317701_no_perl_bootloader branch July 27, 2016 09:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants