tests: spsc_pbuf: Check return code

finikorg · fabiobaltieri · commit 8f4d8c5ccecd · 2023-01-17T21:50:16.000Z
Make check_buffer() call useful.

Signed-off-by: Andrei Emeltchenko &lt;andrei.emeltchenko@intel.com&gt;
diff --git a/tests/lib/spsc_pbuf/src/main.c b/tests/lib/spsc_pbuf/src/main.c
@@ -436,7 +436,7 @@ bool stress_read(void *user_data, uint32_t cnt, bool last, int prio)
 			zassert_true(false, "Unexpected error: %d, cnt:%d", len, ctx->read_cnt);
 		}
 
-		check_buffer(buf, len, ctx->read_cnt);
+		zassert_ok(check_buffer(buf, len, ctx->read_cnt));
 		ctx->read_cnt++;
 	}
 
@@ -499,7 +499,7 @@ bool stress_claim_free(void *user_data, uint32_t cnt, bool last, int prio)
 			return true;
 		}
 
-		check_buffer(buf, len, ctx->read_cnt);
+		zassert_ok(check_buffer(buf, len, ctx->read_cnt));
 
 		spsc_pbuf_free(ctx->pbuf, len);
 

Original file line number	Diff line number	Diff line change
`@@ -436,7 +436,7 @@ bool stress_read(void *user_data, uint32_t cnt, bool last, int prio)`
`436`	`436`	`zassert_true(false, "Unexpected error: %d, cnt:%d", len, ctx->read_cnt);`
`437`	`437`	`}`
`438`	`438`
`439`		`- check_buffer(buf, len, ctx->read_cnt);`
	`439`	`+ zassert_ok(check_buffer(buf, len, ctx->read_cnt));`
`440`	`440`	`ctx->read_cnt++;`
`441`	`441`	`}`
`442`	`442`
`@@ -499,7 +499,7 @@ bool stress_claim_free(void *user_data, uint32_t cnt, bool last, int prio)`
`499`	`499`	`return true;`
`500`	`500`	`}`
`501`	`501`
`502`		`- check_buffer(buf, len, ctx->read_cnt);`
	`502`	`+ zassert_ok(check_buffer(buf, len, ctx->read_cnt));`
`503`	`503`
`504`	`504`	`spsc_pbuf_free(ctx->pbuf, len);`
`505`	`505`