Move loop into CommandHandler

[kalikiana]

See: https://progress.opensuse.org/issues/81899

Note: Requires #2196

[codecov]

Codecov Report

Merging #2200 (8290a8f) into master (7a0df62) will increase coverage by 0.04%.
The diff coverage is 92.06%.

❗ Current head 8290a8f differs from pull request most recent head 2b37551. Consider uploading reports for the commit 2b37551 to get more accurate results

@@            Coverage Diff             @@
##           master    #2200      +/-   ##
==========================================
+ Coverage   92.15%   92.20%   +0.04%     
==========================================
  Files         157      157              
  Lines       15201    15228      +27     
==========================================
+ Hits        14009    14041      +32     
+ Misses       1192     1187       -5     

Impacted Files	Coverage Δ
isotovideo	92.76% <50.00%> (+2.36%)	⬆️
OpenQA/Isotovideo/CommandHandler.pm	100.00% <100.00%> (ø)
t/19-isotovideo-command-processing.t	97.91% <100.00%> (+0.22%)	⬆️
testapi.pm	74.74% <0.00%> (-0.18%)	⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[Martchus]

The use of the events looks generally good (especially considering events are already used here anyways).

[kalikiana]

The use of the events looks generally good (especially considering events are already used here anyways).

Yeah, that was a good idea.

Now I'm only unsure about my test for the rpc. Dropping fileno gets rid of warnings from the log message. Maybe I need to mock all file handles afterall.

[mergify]

This pull request is now in conflicts. Could you fix it? 🙏

[okurz]

codecov/project/fully_covered failed

[okurz]

Now patch coverage :)

[kalikiana]

Now patch coverage :)

Well, if you consider it useful I'll mark it as uncoverable. I don't fully understand why it shows up even though isotovideo is already exempt...