Return error on timeout (bugfix)

[Hook25]

Description

If checkbox times out connecting to the agent, it prints an error but exists with return code 0. This is an issue because we can't rely on $? in this situation (like our canary pipeline does) and it clearly a bug. This fixes said bug.

Resolved issues

Fixes: CHECKBOX-1468

Documentation

N/A

Tests

This adds a test for the "new" situation

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 44.17%. Comparing base (b29ac00) to head (cb0a671).
Report is 113 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1302      +/-   ##
==========================================
+ Coverage   44.16%   44.17%   +0.01%     
==========================================
  Files         359      359              
  Lines       38812    38812              
  Branches     6581     6581              
==========================================
+ Hits        17141    17147       +6     
+ Misses      21007    21003       -4     
+ Partials      664      662       -2     

Flag	Coverage Δ
checkbox-ng	67.59% <100.00%> (+0.04%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

[p-gentili]

LGTM