Skip to content

Commit a440593

Browse files
committed
scheduler: set CONF status for timed out tests
When we stop the scheduler we always set command return code to 32 (skip), but we never set the test results status to CONF. This patch introduces a fix for it so report file is correctly generated. Signed-off-by: Andrea Cervesato <andrea.cervesato@suse.com>
1 parent b1ca38d commit a440593

2 files changed

Lines changed: 4 additions & 1 deletion

File tree

libkirk/scheduler.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@
2424
SchedulerError,
2525
)
2626
from libkirk.framework import Framework
27-
from libkirk.results import Results, SuiteResults, TestResults
27+
from libkirk.results import Results, ResultStatus, SuiteResults, TestResults
2828
from libkirk.sut import SUT, IOBuffer
2929

3030

@@ -537,6 +537,7 @@ async def _run_suite(self, suite: Suite) -> None:
537537
exec_time=0.0,
538538
retcode=32,
539539
stdout="",
540+
status=ResultStatus.CONF,
540541
)
541542
)
542543

libkirk/tests/test_scheduler.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99
import pytest
1010

1111
from libkirk.data import Suite, Test
12+
from libkirk.results import ResultStatus
1213
from libkirk.errors import KernelPanicError, KernelTaintedError, KernelTimeoutError
1314
from libkirk.sut_base import GenericSUT
1415
from libkirk.channels.shell import ShellComChannel
@@ -518,3 +519,4 @@ async def test_schedule_suite_timeout(self, workers, create_runner):
518519
assert 0 <= res.exec_time < 0.4
519520
assert res.return_code == 32
520521
assert res.stdout == ""
522+
assert res.status == ResultStatus.CONF

0 commit comments

Comments
 (0)