Project

General

Profile

Actions

action #120639

closed

coordination #121876: [epic] Handle openQA review failures in Yam squad - SLE 15 SP5

Fix check for network restart in yast2_lan_restart* test modules

Added by rainerkoenig over 1 year ago. Updated over 1 year ago.

Status:
Resolved
Priority:
High
Assignee:
Target version:
Start date:
2022-11-16
Due date:
% Done:

0%

Estimated time:

Description

Motivation

We observe the situation that in all yast2_lan_restart* tests we get a failure because in fact the network gets reloaded when it shouldn't be.

Because of this I filed bsc#1205402 for the current build.
The first comment states:

Does this test use the "Next" button to get out of the YaST network module?

In that case, it might happen that network settings are applied even though there was no real change. In some scenarios, this might even be desired; for example when a user wishes to restart the network (in case of unknown networking problems).

If you know there was no change, and you want to leave a YaST module without restarting anything, please use the "Cancel" button.

Looking at the code I must confirm this:

  • The test module issues a $network_settings->save_changes();
  • save_changes() performs $self->get_action_buttons()->press_ok(); in NetworkSettings/v4_3/NetworkSettingsController.pm
  • ActionButtons.pm assigns pres_ok to the $self->{btn_ok} = $self->{app}->button({id => 'next'});

ToDo to fix this

  • Add code to handle the cancel button in ActionButtons.pm and NetworkSettingsController.pm
  • Modify test modules so that they issue the cancel button instead of next

Acceptance criteria

AC1: Problem is fixed for all yast2_lan_restart* test modules

Actions

Also available in: Atom PDF