ARMmbed / mbed-tools

⚠️ Beta Status: New command line tooling for Mbed OS
Apache License 2.0
45 stars 30 forks source link

config: Add `MBED_GREENTEA_TEST_RESET_TIMEOUT` #301

Closed LDong-Arm closed 3 years ago

LDong-Arm commented 3 years ago

Description

Targets that take longer to boot (e.g. TF-M targets which boot MCUboot and TF-M before application) have forced_reset_timeout in mbed-os/targets/targets.json to increase timeout after reset before starting a Greentea test. This is previously handled by mbedgt, but as we move to CTest and mbedhtrun, a target's reset timeout needs to be available in CMake and passed to mbedhtrun.

Test Coverage

codecov[bot] commented 3 years ago

Codecov Report

Merging #301 (98bc93c) into master (82a5802) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #301   +/-   ##
=======================================
  Coverage   97.08%   97.08%           
=======================================
  Files          92       92           
  Lines        2782     2782           
=======================================
  Hits         2701     2701           
  Misses         81       81           
LDong-Arm commented 3 years ago

We should probably have a test for this in tests/build/test_generate_config.py.

Done, tests added.