Project

General

Profile

Actions

action #43031

closed

Fix failing t/ui/13-admin.t

Added by mkittler about 6 years ago. Updated about 6 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
-
Target version:
Start date:
2018-10-29
Due date:
% Done:

0%

Estimated time:

Description

Seems like t/ui/13-admin.t fails now quite often on Travis:

    #   Failed test 'advanced_kde not selectable twice'
    #   at ./t/ui/13-admin.t line 416.
    #     Structures begin differing at:
    #          $got->[1] = 'advanced_kde'
    #     $expected->[1] = 'client1'
    # Looks like you failed 1 test of 8.
#   Failed test 'edit mediums'
#   at ./t/ui/13-admin.t line 492.
Use of uninitialized value $expr in concatenation (.) or string at /usr/lib/perl5/5.18.2/B/Deparse.pm line 1370.
# Looks like you failed 1 test of 15.
Use of uninitialized value $expr in concatenation (.) or string at /usr/lib/perl5/5.18.2/B/Deparse.pm line 1370.
./t/ui/13-admin.t ....................
Dubious, test returned 1 (wstat 256, 0x100)
Failed 1/15 subtests

notes

Actions #1

Updated by mkittler about 6 years ago

  • Target version set to Current Sprint
Actions #2

Updated by EDiGiacinto about 6 years ago

Also https://github.com/os-autoinst/openQA/pull/1783 is affected by it (after rebasing)

Actions #3

Updated by mkittler about 6 years ago

  • Status changed from New to In Progress

There is no AJAX involved in the relevant part of the JavaScript code. Adding a wait_for_ajax nervelessness or even a regular sleep doesn't work. So I still have no clue why the JavaScript behaves wrong when the tests are executed within Travis. But it seems like it is now almost reproducible 100 % of the time (within Travis).

If I don't get a clue soon, we could also temporarily make that test non-fatal. Better than having red status on each PR.

Actions #4

Updated by mkittler about 6 years ago

  • Status changed from In Progress to Resolved
Actions #5

Updated by coolo about 6 years ago

  • Target version changed from Current Sprint to Done
Actions

Also available in: Atom PDF