QA » openQA Project » openQA Tests

• The match is wrongly tagged with 'updates_none', should be 'updates_available'
• The area is completely bogus, the matched (2) is unrelated to the updates, its just the number of notifications.

The needle was probably an attempt to fix an issue very similar to #31327#note-3.

From https://openqa.opensuse.org/tests/609009/file/autoinst-log.txt

[2018-02-13T16:52:34.0284 CET] [debug] <<< testapi::check_screen(mustmatch='updates_click-install', timeout=0)
[2018-02-13T16:52:34.0476 CET] [debug] MATCH(updates_kde-updates_click-install-20160526:1.00)

This is already fixed with https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/4397
(merged 2018-02-13T23:26:06)

Urgh - that us going to fun

https://openqa.opensuse.org/tests/611882#step/updates_packagekit_kde/11

The KDE team implemented, on Ludwig's request, that the Updated applet auto-closes on completion. So in TW we no longer can look for the text 'system up-to-date' The only indication of 'Your system is up to date is that there are 'only notifications left'; if there were updates. the regular count on openQA is '3 notifications'.

Hence for lack of seeing anything better, this was chosen. Alternatively we can probabl add the 'update-none' to the normal needle we use to 'try to guess there are no updates' (takes a bit longer area of the panel)

Ok, I created another needle as an update to an existing 'try-without-updates' - it spans much over all shown icons in the notification area and does not include a 'more updates available' icon, which in turn is verified not to match the Leap case where there ARE more updates available

In leap, there's first a package manager update. So the applet is expected to show some message requesting to relogin.

@dimstar - I think the needle duplicates the ones with tag 'tray-without-updates-available'

Replaces - not duplicates (it has different / additional tags)

Currently the code reads somewhat like:

assert_and_click("updates_available-tray");
while(1) {
    # do stuff
    if (check_screen 'updates_none')
        last;
}

Maybe change that to:

assert_and_click("updates_available-tray");
while(match_has_tag("updates_available-tray")) {
    # do stuff

    assert_screen [qw(updates_available-tray tray-without-updates-available)];
}

So now we are hitting the case "there are updates available" after the first round, as described in comment #31912#note-3.

I think the change mentioned in #31912#note-8 would work here.