From 362cf0864a46b8ae6a66093cceba6859c801b443 Mon Sep 17 00:00:00 2001 From: Bill Richardson Date: Tue, 4 Mar 2014 13:34:52 -0800 Subject: Remove time-dependent test from coverage measurements Compiling with coverage enabled screws up the time that things take, so don't test for a particular speedup in that case. It fails unreliably. BUG=chrome-os-partner:20881 BRANCH=ToT TEST=manual cd src/platform/ec make coverage Before, it failed about half the time. Now it doesn't. Change-Id: I535f0193bf450a922b486777b296fea1b2768a1a Signed-off-by: Bill Richardson Reviewed-on: https://chromium-review.googlesource.com/188790 Reviewed-by: Randall Spangler --- test/utils.c | 3 +++ 1 file changed, 3 insertions(+) (limited to 'test') diff --git a/test/utils.c b/test/utils.c index 5d4a88d0fe..8ab31d71c8 100644 --- a/test/utils.c +++ b/test/utils.c @@ -97,7 +97,10 @@ static int test_memmove(void) TEST_ASSERT_ARRAY_EQ(buf + 100, buf, len); /* Expected about 4x speed gain. Use 3x because it fluctuates */ +#ifndef TEST_COVERAGE + /* Measuring coverage makes it fluctuate even more, so skip it. */ TEST_ASSERT((t1.val-t0.val) > (t3.val-t2.val) * 3); +#endif /* Test small moves */ memmove(buf + 1, buf, 1); -- cgit v1.2.1