search

ARMmbed / htrun

MOVED: https://github.com/ARMmbed/mbed-os-tools (Flash, reset and run host supervised tests on mbed platforms)
8 stars 37 forks source link

Feature: dynamic timeouts #228

Closed screamerbg closed 5 years ago

screamerbg commented 5 years ago

This minimalistic PR adds support to dynamically sett/adjust a test timeout via DUT messages {{__timeout_set,int}}, {{__timeout_adjust,int}}.

The is particularly useful when a device is making a download (say firmware update), where the speed might dynamically change based on various conditions. With this feature, the DUT might notify the host that it's alive and running, just slower than expected.

A typical usecase would be a unified IP connectivity test, where the time to complete over Ethernet and Wifi would be significantly faster than 2G/3G, let alone NBIoT/CAT-M1.

As discussed with @bridadan

coveralls commented 5 years ago

Coverage Status

Coverage remained the same at 42.424% when pulling 056b2fde7f432e253856762e61fbf9da9d4e4f01 on screamerbg:feature_dynamic_timeout into 1c570ec77d8c5673dae55dc790302d86d712c36b on ARMmbed:master.

coveralls commented 5 years ago

Coverage Status

Coverage remained the same at 42.424% when pulling 056b2fde7f432e253856762e61fbf9da9d4e4f01 on screamerbg:feature_dynamic_timeout into 1c570ec77d8c5673dae55dc790302d86d712c36b on ARMmbed:master.

coveralls commented 5 years ago

Coverage Status

Coverage remained the same at 42.424% when pulling 5090b13cea664b7953d5542dd167adcd142dd20b on screamerbg:feature_dynamic_timeout into 1c570ec77d8c5673dae55dc790302d86d712c36b on ARMmbed:master.

theotherjimmy commented 5 years ago

@screamerbg Thanks for the PR. Do you need this in a release of htrun soon?

screamerbg commented 5 years ago

@theotherjimmy Yes please (if possible).

Here's an example usage - https://github.com/ARMmbed/simple-mbed-cloud-client/pull/49/commits/6cb6566c550c590498b8e32b1d504629006e4b90#diff-82f388e30987a4563c237f17ccb185dbR66

screamerbg commented 5 years ago

Bump @cmonr @theotherjimmy