Issue #365: RFE: basic role - Allow save extra logs to artifacts - standard-test-roles

standard-test-roles

#365 RFE: basic role - Allow save extra logs to artifacts

Closed: Fixed Jul 9, 2019 by astepano. Opened Jul 8, 2019 by bgoncalv.

It would be nice if a file created by a test case could also be saved to artifacts.

Currently the basic role only stores the test case execution log, but if the test creates some additional file in some cases it would be nice if this file could also be saved to artifacts.

Basically allow files created on test-subject to be saved on artifacts on test-runner

astepano commented Jul 8, 2019

@bgoncalv please add a case that requires this. Describe it why it would be useful, as well as please add words test-runner test-subject. Something something says me you want to pull from test-subject to test-runner, am I right ?

astepano commented Jul 8, 2019

@bgoncalv it would be necessary also add a note to STI.

Metadata Update from @astepano:
- Issue assigned to astepano

Jul 8, 2019

bgoncalv commented Jul 8, 2019

Example: https://src.fedoraproject.org/rpms/acpica-tools/blob/master/f/tests/badcode/runtest.sh#_20

It would be nice to have an option to save badcode.asl.expected to artifacts.

Metadata Update from @bgoncalv:
- Issue assigned to bgoncalv (was: astepano)

Jul 8, 2019

bgoncalv commented Jul 8, 2019

suggested implementation https://pagure.io/standard-test-roles/pull-request/367

ahs3 commented Jul 8, 2019

Example: https://src.fedoraproject.org/rpms/acpica-tools/blob/master/f/tests/badcode/runtest.sh#_20
It would be nice to have an option to save badcode.asl.expected to artifacts.

A file called badcode.asl.actual is also produced; it would be handy to be able to save these files so I can see what changed. In theory, the command iasl that produces these files should always work the same. In practice, it does not. If this test fails, these two files -- badcode.asl.actual and badcode.asl.expected are somehow different -- and being able to see the difference between these files tells me what needs to be fixed.

astepano commented Jul 9, 2019

Commit 3cdeea16 fixes this issue