tests: drivers: counter: Fix coverity issue (190952)

nordic-krch · MaureenHelm · commit 244aecf2d2d0 · 2019-03-05T12:37:18.000-06:00
Fix issue reported by coverity regarding using volatile
variables in zassert_equal macro.

Signed-off-by: Krzysztof Chruscinski &lt;krzysztof.chruscinski@nordicsemi.no&gt;
diff --git a/tests/drivers/counter/counter_basic_api/src/test_counter.c b/tests/drivers/counter/counter_basic_api/src/test_counter.c
@@ -141,9 +141,9 @@ void test_set_top_value_with_alarm_instance(const char *dev_name)
 
 	k_busy_wait(5.2*COUNTER_PERIOD_US);
 
-	zassert_true(top_cnt == 5,
+	zassert_true((u32_t)top_cnt == 5,
 			"Unexpected number of turnarounds (%d) (dev: %s).\n",
-			top_cnt, dev_name);
+			(u32_t)top_cnt, dev_name);
 }
 
 void test_set_top_value_with_alarm(void)

Original file line number	Diff line number	Diff line change
`@@ -141,9 +141,9 @@ void test_set_top_value_with_alarm_instance(const char *dev_name)`
`141`	`141`
`142`	`142`	`k_busy_wait(5.2*COUNTER_PERIOD_US);`
`143`	`143`
`144`		`- zassert_true(top_cnt == 5,`
	`144`	`+ zassert_true((u32_t)top_cnt == 5,`
`145`	`145`	`"Unexpected number of turnarounds (%d) (dev: %s).\n",`
`146`		`- top_cnt, dev_name);`
	`146`	`+ (u32_t)top_cnt, dev_name);`
`147`	`147`	`}`
`148`	`148`
`149`	`149`	`void test_set_top_value_with_alarm(void)`