From b531e65f6abf0017949fadfd33424f672ee90991 Mon Sep 17 00:00:00 2001 From: Andreas Antener Date: Thu, 24 Nov 2016 11:04:11 +0100 Subject: [PATCH] Airspeed check: updated message and explained test parameters --- src/modules/commander/PreflightCheck.cpp | 2 +- .../commander/commander_tests/state_machine_helper_test.cpp | 5 ++++- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/src/modules/commander/PreflightCheck.cpp b/src/modules/commander/PreflightCheck.cpp index 930da0180a..03fcbdfc5d 100644 --- a/src/modules/commander/PreflightCheck.cpp +++ b/src/modules/commander/PreflightCheck.cpp @@ -402,7 +402,7 @@ static bool airspeedCheck(orb_advert_t *mavlink_log_pub, bool optional, bool rep if (fabsf(airspeed.differential_pressure_filtered_pa) > 15.0f && !prearm) { if (report_fail) { - mavlink_log_critical(mavlink_log_pub, "PREFLIGHT FAIL: AIRSPEED CALIBRATION ISSUE"); + mavlink_log_critical(mavlink_log_pub, "PREFLIGHT FAIL: HIGH AIRSPEED, CHECK CALIBRATION"); } success = false; goto out; diff --git a/src/modules/commander/commander_tests/state_machine_helper_test.cpp b/src/modules/commander/commander_tests/state_machine_helper_test.cpp index 19a5a96e21..ac576f4efc 100644 --- a/src/modules/commander/commander_tests/state_machine_helper_test.cpp +++ b/src/modules/commander/commander_tests/state_machine_helper_test.cpp @@ -296,7 +296,10 @@ bool StateMachineHelperTest::armingStateTransitionTest(void) false /* no pre-arm checks */, nullptr /* no mavlink_log_pub */, &status_flags, - 5.0f, check_gps, 2e6); + 5.0f, /* avionics rail voltage */ + check_gps, + 2e6 /* 2 seconds after boot, everything should be checked */ + ); // Validate result of transition ut_compare(test->assertMsg, test->expected_transition_result, result);