teemtee / tmt

Test Management Tool
MIT License
79 stars 117 forks source link

`data-path` not set for results from `result: custom` #2770

Open lukaszachy opened 4 months ago

lukaszachy commented 4 months ago

This path exists as the TMT_TEST_DATA is created each time. However test like this

cat <<EOF > main.fmf
test: cp results.yaml \$TMT_TEST_DATA/
result: custom
EOF

cat <<EOF > results.yaml
- name: /
  result: pass
EOF

after tmt run has /var/tmp/tmt/run-087/default/plan/execute/results.yaml with data-path: even though data-path /var/tmp/tmt/run-087/default/plan/execute/data/guest/default-0/default-1/data/ exists.

happz commented 4 months ago

Right, this seems to be missing. https://github.com/teemtee/tmt/blob/main/tmt/steps/execute/__init__.py#L582 sets some interesting properties of the "main" result, to present consistent timestamps and stuff, but data path is not set anywhere in this area.