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.

1 files changed, 7 insertions, 5 deletions

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);