From eaba2a7b0658f648bf32219aec0ce827c94333a0 Mon Sep 17 00:00:00 2001 From: David Phillips Date: Sat, 27 Feb 2021 22:26:22 +1300 Subject: Make timer mandatory for data manager The NULL check was a bit of a hack to get the test building. Probably a bad idea to leave it around, since there's no time it's sane to leave the timestamps on data manager contexts unset. --- test_data_manager.c | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) (limited to 'test_data_manager.c') diff --git a/test_data_manager.c b/test_data_manager.c index 00b5a2a..044282b 100644 --- a/test_data_manager.c +++ b/test_data_manager.c @@ -3,6 +3,7 @@ #include "data_manager.h" #include "test_runner.h" #include "barometer.h" +#include "timer.h" #include "unity.h" static int mock_baro_called; @@ -23,18 +24,19 @@ static struct barometer mock_barometer = { RUNNER_DECLARE_TEST(test_data_manager_init_setting) { struct data_ctx ctx; + struct timer timer; + get_system_timer(&timer); reset_mock_get_baro(); - data_manager_init(&ctx, NULL, &mock_barometer); + data_manager_init(&ctx, &timer, &mock_barometer); TEST_ASSERT_EQUAL(1, mock_baro_called); } RUNNER_DECLARE_TEST(test_data_manager_first_readings) { struct data_ctx ctx; - reset_mock_get_baro(); - data_manager_init(&ctx, NULL, &mock_barometer); - TEST_ASSERT_EQUAL(1, mock_baro_called); - + struct timer timer; + get_system_timer(&timer); + data_manager_init(&ctx, &timer, &mock_barometer); reset_mock_get_baro(); data_manager_tick(&ctx); -- cgit v1.1