Show correct needle versions in test result step matches

[asdil12]

This is a limited part of #5175
taken out to go step by step.

The needle version is only read from vars.json and only supports hashrefs - no branches/tags.
Network interaction by git on showing a needle is limited to the absolute minimum.
Also the needle editor is not changed, yet.

Ticket: https://progress.opensuse.org/issues/162035

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.00%. Comparing base (44a6d49) to head (df5890d).
Report is 2 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff            @@
##           master    #6190    +/-   ##
========================================
  Coverage   99.00%   99.00%            
========================================
  Files         396      398     +2     
  Lines       39804    40012   +208     
========================================
+ Hits        39406    39614   +208     
  Misses        398      398            

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[asdil12]

Tested manually and seems to work.

[Martchus]

You could mention @sdclarke and me as co-authors in the commit message.

[okurz]

some minor style suggestions and there is uncovered code

[okurz]

https://github.com/os-autoinst/openQA/pull/6190/files#r1969929906 and squash commits

[Martchus]

See my previous in-line comment.

Maybe rebasing helps to avoid the failing cacheing.

[okurz]

I am ok to accept an incomplete patch coverage as exception but please ensure that all test code at least is covered, see the failed codecov/project/tests

[Martchus]

Yes, the uncovered mocking is likely a leftover and not actually used anymore.