https://progress.opensuse.org/https://progress.opensuse.org/themes/openSUSE/favicon/favicon.ico?15829177842020-09-25T08:40:44ZopenSUSE Project Management ToolopenQA Project - action #71536: Unhandled perl warnings in t/ui/15-admin-workers.t, not failing tests as expected (possibly other test modules as well)https://progress.opensuse.org/issues/71536?journal_id=3271842020-09-25T08:40:44Zmkittlermarius.kittler@suse.com
<ul><li><strong>Assignee</strong> set to <i>mkittler</i></li></ul> openQA Project - action #71536: Unhandled perl warnings in t/ui/15-admin-workers.t, not failing tests as expected (possibly other test modules as well)https://progress.opensuse.org/issues/71536?journal_id=3272142020-09-25T09:28:19Zmkittlermarius.kittler@suse.com
<ul><li><strong>Status</strong> changed from <i>Workable</i> to <i>In Progress</i></li></ul><p>PR for fixing the warning itself: <a href="https://github.com/os-autoinst/openQA/pull/3422" class="external">https://github.com/os-autoinst/openQA/pull/3422</a></p>
<blockquote>
<p>Try to narrow down the source of the problem and identify from which process the warning comes</p>
</blockquote>
<p>Here the warning comes indeed from a sub process. In this case it is the web UI but we generally wouldn't catch warnings in any sub processes (which are usually spawned by the test helpers <code>create_webapi</code>, <code>create_websocket_server</code>, <code>create_scheduler</code> and <code>create_live_view_handler</code>).</p>
<blockquote>
<p>Extend the existing approach to handle warnings …</p>
</blockquote>
<p>That would be possible, see <a href="https://github.com/os-autoinst/openQA/pull/3423" class="external">https://github.com/os-autoinst/openQA/pull/3423</a> for a draft. However, it isn't a clear improvement so far (see the PR description) so I'm still trying to improve it.</p>
openQA Project - action #71536: Unhandled perl warnings in t/ui/15-admin-workers.t, not failing tests as expected (possibly other test modules as well)https://progress.opensuse.org/issues/71536?journal_id=3274962020-09-28T13:08:35Zmkittlermarius.kittler@suse.com
<ul><li><strong>Status</strong> changed from <i>In Progress</i> to <i>Resolved</i></li></ul><p>I've improved the PR. It still isn't optimal (see <a href="https://github.com/os-autoinst/openQA/pull/3423#issuecomment-699896044" class="external">https://github.com/os-autoinst/openQA/pull/3423#issuecomment-699896044</a>) but at least warnings will be catched now in sub processes and the main test will fail and stop if that's the case.</p>