HIL(QOL): Add missing timeouts to various tests

[JurajSadel]

No description provided.

[SergioGasquez]

LGTM! It would be cool to have a default timeout that we could set. Thanks!

[bugadani]

I'd prefer investing into probe-rs/embedded-test#3 instead.

[jessebraham]

I'd prefer investing into probe-rs/embedded-test#3 instead.

Thanks for pointing that out, was going to suggest opening an issue upstream.

[bjoernQ]

I guess probe-rs/embedded-test#3 is the best way to go (is no timeout at all ever a good idea ....?)

So, what should happen with this PR? Does it solve a real problem we have currently? (Then we might want to get it in until we have a global timeout)

[bugadani]

(is no timeout at all ever a good idea ....?)

The default is 30 seconds right now, there is no such thing as no timeout in embedded-test.

[bjoernQ]

(is no timeout at all ever a good idea ....?)

The default is 30 seconds right now, there is no such thing as no timeout in embedded-test.

Oh good - I never had the patience to wait 30 secs .... I don't see where the code sets 30 secs if None but I didn't try hard to find it

[bugadani]

My bad it's 60

[bjoernQ]

My bad it's 60

Ah thanks .... the runner does it on its own .... no wonder I wasn't able to find it in embedded-test itself 😆

[bugadani]

PR pending: probe-rs/embedded-test#44

[JurajSadel]

PR pending: probe-rs/embedded-test#44

Thank you! I will mark this as a draft and when it gets merged, I can update our HIL.

[MabezDev]

Thank you! I will mark this as a draft and when it gets merged, I can update our HIL.

You can use Daniel's branch with git and rev in hil-test for now