https://progress.opensuse.org/https://progress.opensuse.org/themes/openSUSE/favicon/favicon.ico?15829177842018-09-18T13:15:25ZopenSUSE Project Management ToolopenQA Tests - action #41201: [qam][desktop] test fails in pidgin_IRChttps://progress.opensuse.org/issues/41201?journal_id=1500682018-09-18T13:15:25Zbfilhobfilho@suse.com
<ul></ul><p>pidgin_IRC failing from 20180915-1 multiple times, seems to be missing the a keypress events: <br>
[2018-09-17T03:22:30.0365 CEST] [debug] /var/lib/openqa/cache/tests/sle/tests/x11/pidgin/pidgin_IRC.pm:46 called testapi::send_key<br>
[2018-09-17T03:22:30.0365 CEST] [debug] <<< testapi::send_key(key='alt-`', do_wait=0)<br>
...<br>
[2018-09-17T03:22:30.0579 CEST] [debug] /var/lib/openqa/cache/tests/sle/tests/x11/pidgin/pidgin_IRC.pm:49 called testapi::send_key<br>
[2018-09-17T03:22:30.0580 CEST] [debug] <<< testapi::send_key(key='ctrl-w', do_wait=0)<br>
...<br>
/var/lib/openqa/cache/tests/sle/tests/x11/pidgin/pidgin_IRC.pm:54 called testapi::assert_screen <br>
...<br>
<a href="https://openqa.suse.de/tests/2059231#downloads" class="external">https://openqa.suse.de/tests/2059231#downloads</a></p>
openQA Tests - action #41201: [qam][desktop] test fails in pidgin_IRChttps://progress.opensuse.org/issues/41201?journal_id=1501432018-09-19T03:25:38ZGraceWanggrace.wang@suse.com
<ul></ul><p>SLED12SP4 also met the same issue.<br>
See <a href="https://openqa.suse.de/tests/2069539#step/pidgin_IRC/21" class="external">https://openqa.suse.de/tests/2069539#step/pidgin_IRC/21</a> for details. </p>
<p>From the below log:</p>
<p>[2018-09-18T05:36:40.0247 CEST] [debug] >>> testapi::_check_backend_response: match=pidgin-ctcp-version,pidgin-irc-sledtesting timed out after 30 (assert_screen)<br>
[2018-09-18T05:36:40.0541 CEST] [debug] no candidate needle with tag(s) 'pidgin-ctcp-version, pidgin-irc-sledtesting' matched</p>
<p>It seems that it failed when executing the below code:<br>
assert_screen([qw(pidgin-ctcp-version pidgin-irc-sledtesting)]);</p>
openQA Tests - action #41201: [qam][desktop] test fails in pidgin_IRChttps://progress.opensuse.org/issues/41201?journal_id=1503862018-09-19T12:31:08Zmsmeissnmeissner@suse.com
<ul></ul><p>could it be possible it cannot connect to freenode anymore?</p>
<p>due to the recent spambot attack there?</p>
<p>I tried to look at the video but its not easily visible if the connect fails and how</p>
openQA Tests - action #41201: [qam][desktop] test fails in pidgin_IRChttps://progress.opensuse.org/issues/41201?journal_id=1504522018-09-19T14:05:07Zokurzokurz@suse.com
<ul><li><strong>Subject</strong> changed from <i>test fails in pidgin_IRC</i> to <i>[desktop] test fails in pidgin_IRC</i></li></ul><p>msmeissn wrote:</p>
<blockquote>
<p>I tried to look at the video but its not easily visible if the connect fails and how</p>
</blockquote>
<p>did you try to download the video locally and step through it frame by frame, e.g. with "mpv"? The video player within the browser skips over some frames and you will most likely not be able to see every single step which might be important for investigation.</p>
openQA Tests - action #41201: [qam][desktop] test fails in pidgin_IRChttps://progress.opensuse.org/issues/41201?journal_id=1505392018-09-19T14:31:46Zmsmeissnmeissner@suse.com
<ul></ul><p>hmm, it seems to turn the bubble green, so it seems connected.</p>
<p>but the ctcp check seems not to come in for some reason.<br>
also not when i try it on tumbleweed pidgin.</p>
openQA Tests - action #41201: [qam][desktop] test fails in pidgin_IRChttps://progress.opensuse.org/issues/41201?journal_id=1506082018-09-19T15:26:55Zmsmeissnmeissner@suse.com
<ul></ul><p>the control flow is weird in the test.</p>
<p>the ctcp handling has assert_screen([qw(pidgin-ctcp-version pidgin-irc-sledtesting)]);<br>
but at this time we have not yet joined the #sledtesting channel it matches on.</p>
<p>should this block move a bit lower?</p>
openQA Tests - action #41201: [qam][desktop] test fails in pidgin_IRChttps://progress.opensuse.org/issues/41201?journal_id=1506292018-09-19T16:00:50Zmsmeissnmeissner@suse.com
<ul></ul><p>proposal for a patch <a href="https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/5805" class="external">https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/5805</a></p>
<p>(untested)</p>
openQA Tests - action #41201: [qam][desktop] test fails in pidgin_IRChttps://progress.opensuse.org/issues/41201?journal_id=1512922018-09-24T11:07:38Zjbaier_czjbaier@suse.cz
<ul></ul><p>This one would be hard to debug as it works locally: <a href="http://panigale.suse.cz/tests/277" class="external">http://panigale.suse.cz/tests/277</a></p>
openQA Tests - action #41201: [qam][desktop] test fails in pidgin_IRChttps://progress.opensuse.org/issues/41201?journal_id=1513162018-09-24T12:16:42Zpcervinkapcervinka@suse.com
<ul></ul><p>I re-checked logic sequence of the test and I think it is not correct in assert_screen([qw(pidgin-spoofing-ip pidgin-irc-sledtesting)]). There is a chance of pidgin-spoofing-ip, which may occur (but don't have to). But why is there match for pidgin-irc-sledtesting? This shouldn't be there, because this match is supposed to be done little bit later. I think <a class="user active user-mention" href="https://progress.opensuse.org/users/83">@msmeissn</a> had good idea to remove it and there should be if (check_screen('pidgin-spoofing-ip')) { instead of assert_screen with match_has_tag. SImilar incorrect logic is also for CTCP check.<br>
I did a test run with check_screen and it looks fine <a href="http://10.100.12.105/tests/226#" class="external">http://10.100.12.105/tests/226#</a>.</p>
openQA Tests - action #41201: [qam][desktop] test fails in pidgin_IRChttps://progress.opensuse.org/issues/41201?journal_id=1513192018-09-24T12:21:23Zpcervinkapcervinka@suse.com
<ul></ul><p>I also tried to reproduced it manually in virtual machine and ip spoofing check or CTCP didn't occur. So those steps are really optional.</p>
openQA Tests - action #41201: [qam][desktop] test fails in pidgin_IRChttps://progress.opensuse.org/issues/41201?journal_id=1513222018-09-24T12:23:30Zpcervinkapcervinka@suse.com
<ul></ul><p>Suggested code change:</p>
<pre><code> # Warning of spoofing ip may appear
if (check_screen('pidgin-spoofing-ip')) {
wait_screen_change {
send_key is_sle('<15') ? "alt-tab" : "alt-`";
};
wait_screen_change {
send_key "ctrl-w"; # close it
};
}
# CTCP Version and warning about scan may appear
if (check_screen('pidgin-ctcp-version')) {
wait_screen_change { send_key "ctrl-w"; } # close it
}
</code></pre> openQA Tests - action #41201: [qam][desktop] test fails in pidgin_IRChttps://progress.opensuse.org/issues/41201?journal_id=1514572018-09-24T20:07:21Zokurzokurz@suse.com
<ul></ul><p>Personally I always prefer multi-tag <code>assert_screen</code> with <code>match_has_tag</code> because I think this is the most efficient way for tests without any unnecessary sleep time. So what is wrong with an assert_screen with multiple tags? E.g. instead of <code>check_screen('pidgin-ctcp-version')</code> why not check for both, this screen and the alternative, and only act upon the warning if it shows up.</p>
openQA Tests - action #41201: [qam][desktop] test fails in pidgin_IRChttps://progress.opensuse.org/issues/41201?journal_id=1521982018-09-26T07:25:19Zpcervinkapcervinka@suse.com
<ul><li><strong>Project</strong> changed from <i>openQA Tests</i> to <i>119</i></li><li><strong>Subject</strong> changed from <i>[desktop] test fails in pidgin_IRC</i> to <i>[qam][desktop] test fails in pidgin_IRC</i></li><li><strong>Category</strong> deleted (<del><i>Bugs in existing tests</i></del>)</li></ul> openQA Tests - action #41201: [qam][desktop] test fails in pidgin_IRChttps://progress.opensuse.org/issues/41201?journal_id=1524652018-09-27T09:06:38Zpcervinkapcervinka@suse.com
<ul><li><strong>Project</strong> changed from <i>119</i> to <i>openQA Tests</i></li><li><strong>Category</strong> set to <i>Bugs in existing tests</i></li><li><strong>Status</strong> changed from <i>New</i> to <i>In Progress</i></li><li><strong>Assignee</strong> set to <i>pcervinka</i></li></ul> openQA Tests - action #41201: [qam][desktop] test fails in pidgin_IRChttps://progress.opensuse.org/issues/41201?journal_id=1525582018-09-27T14:23:19Zpcervinkapcervinka@suse.com
<ul></ul><p><a class="user active user-mention" href="https://progress.opensuse.org/users/83">@msmeissn</a> I will further continue with this one. Could you please close your PR? Okurz had good point and I would<br>
like to explore the other way to fix.</p>
openQA Tests - action #41201: [qam][desktop] test fails in pidgin_IRChttps://progress.opensuse.org/issues/41201?journal_id=1531522018-10-01T10:57:06Zpcervinkapcervinka@suse.com
<ul></ul><p>New PR to properly detect pidgin status(and don't waste time by check_screen) was created:<br>
<a href="https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/5853" class="external">https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/5853</a></p>
openQA Tests - action #41201: [qam][desktop] test fails in pidgin_IRChttps://progress.opensuse.org/issues/41201?journal_id=1531822018-10-01T11:50:29Zpcervinkapcervinka@suse.com
<ul><li><strong>Status</strong> changed from <i>In Progress</i> to <i>Feedback</i></li></ul><p>PR merged, let's see few more runs on o.s.d.</p>
openQA Tests - action #41201: [qam][desktop] test fails in pidgin_IRChttps://progress.opensuse.org/issues/41201?journal_id=1539922018-10-05T08:03:02Zpcervinkapcervinka@suse.com
<ul><li><strong>Status</strong> changed from <i>Feedback</i> to <i>Resolved</i></li></ul>