action #42932
closedcoordination #40484: [qe-core][functional][epic] Move different checks to separate test suites not to affect other functional tests
[functional][y] Move keyboard_selection to welcome module
0%
Description
Motivation¶
Currently, a test for switching keyboard layout is executed as a separate module, though it runs on Welcome screen. This forces welcome and keyboard_selection modules to be dependent on each other.
e.g. welcome module is forced to decide if press 'Next' button, depending on whether keyboard_selection module exist in Test Suite or not
send_key $cmd{next} unless get_var('INSTALL_KEYBOARD_LAYOUT');
Move the switch_keyboard_layout test to welcome module to have all the tests related to that page in one place.
Suggesitons¶
sub switch_keyboard_layout
added to welcome module;
Acceptance Criteria¶
- keyboard_selection module does not exist;
send_key $cmd{next}
is called without dependence on 'INSTALL_KEYBOARD_LAYOUT' variable.
Updated by okurz over 5 years ago
- Category set to Enhancement to existing tests
- Priority changed from Normal to Low
- Target version set to future
Please try to always find a category. Wouldn't merging the modules make review of test results harder?
Updated by riafarov over 5 years ago
- Description updated (diff)
- Status changed from New to Workable
Updated by JERiveraMoya over 5 years ago
- Due date set to 2018-10-16
- Status changed from Workable to In Progress
- Assignee set to JERiveraMoya
- Start date changed from 2018-10-25 to 2018-10-03
Updated by JERiveraMoya over 5 years ago
- Due date changed from 2018-10-16 to 2019-01-15
- Start date changed from 2018-10-03 to 2018-10-24
Updated by JERiveraMoya over 5 years ago
- Status changed from In Progress to Feedback
- Estimated time set to 1.00 h
Updated by riafarov over 5 years ago
To speed up tickets resolution, simply restart any test which contained the module, we don't have to wait for the new build.
Updated by JERiveraMoya over 5 years ago
There was some mess with git and my code was reverted accidentally. Sending PR again:
PR: Move keyboard selection to welcome
Updated by JERiveraMoya over 5 years ago
- Target version changed from future to Milestone 23