Differential D485

fixes for current code
ClosedPublic
Actions

Authored by garretraziel on Aug 6 2015, 8:38 AM.

Details

Reviewers
adamwill
jskladan
Commits
rOPENQATESTSa18b6f8f69f8: fixes for current code
Summary

There are some bugs that require immediate fixing - when renaming
base classes, I have forgotten to rename the packages. Another bug
fixed is badly named variable in tests (it worked though, because it
fallbacked on default value).

Test Plan

try to schledule all tests

Diff Detail

Repository
rOPENQATESTS os-autoinst-distri-fedora
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
garretraziel retitled this revision from to fixes for current code.Aug 6 2015, 8:38 AM
garretraziel updated this object.
garretraziel edited the test plan for this revision. (Show Details)
garretraziel added reviewers: jskladan, adamwill.
jskladan accepted this revision.Aug 6 2015, 8:53 AM

very ack!

This revision is now accepted and ready to land.Aug 6 2015, 8:53 AM
Closed by commit rOPENQATESTSa18b6f8f69f8: fixes for current code (authored by garretraziel). · Explain WhyAug 6 2015, 9:03 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents

Diff 1282

View Options

lib/anacondatest.pm

1package anacondalog; 1​package anacondatest;
2use base 'fedorabase'; 2​use base 'fedorabase';
3 3​
4# base class for all Anaconda (installation) tests 4​# base class for all Anaconda (installation) tests
5 5​
6# should be used in tests where Anaconda is running - when it makes sense 6​# should be used in tests where Anaconda is running - when it makes sense
7# to upload Anaconda logs when something fails 7​# to upload Anaconda logs when something fails
8 8​
9use testapi; 9​use testapi;
▲ Show 20 Lines • Show All 106 Lines • Show Last 20 Lines
View Options

lib/installedtest.pm

1package fedoralog; 1​package installedtest;
2use base 'fedorabase'; 2​use base 'fedorabase';
3 3​
4# base class for tests that run on installed system 4​# base class for tests that run on installed system
5 5​
6# should be used when with tests, where system is already installed, e. g all parts 6​# should be used when with tests, where system is already installed, e. g all parts
7# of upgrade tests, postinstall phases... 7​# of upgrade tests, postinstall phases...
8 8​
9use testapi; 9​use testapi;
Show All 31 Lines
View Options

tests/upgrade_postinstall_desktop.pm

1use base "installedtest"; 1​use base "installedtest";
2use strict; 2​use strict;
3use testapi; 3​use testapi;
4 4​
5 5​
6sub run { 6​sub run {
7 my $self = shift; 7​ my $self = shift;
8 my $password = get_var("PASSWORD", "weakpassword"); 8​ my $password = get_var("USER_PASSWORD", "weakpassword");
9 9​
10 # wait for GDM to appear 10​ # wait for GDM to appear
11 $self->boot_to_login_screen("graphical_login", 20); 11​ $self->boot_to_login_screen("graphical_login", 20);
12 12​
13 # login as normal user 13​ # login as normal user
14 send_key "ret"; 14​ send_key "ret";
15 assert_screen "graphical_login_input"; 15​ assert_screen "graphical_login_input";
16 type_string $password; 16​ type_string $password;
Show All 17 Lines