From 856a64377e912fcd088d8c9f62f090c228873436 Mon Sep 17 00:00:00 2001 From: Alex Date: Thu, 19 Oct 2023 12:33:25 +0200 Subject: [PATCH 1/2] Fixes tests rng --- test/random.c | 2 +- test/test_runner.c | 2 ++ 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/test/random.c b/test/random.c index 5ca4b814df80..80b86745f241 100644 --- a/test/random.c +++ b/test/random.c @@ -6,7 +6,7 @@ // Therefore the maximum error is 8*3584, or 28672. #define SHUFFLE_TEST_IMPL \ u32 i, j, error; \ - u16 indexSum[7]; \ + u16 indexSum[8]; \ memset(indexSum, 0, sizeof(indexSum)); \ for (i = 0; i < 1024; i++) \ { \ diff --git a/test/test_runner.c b/test/test_runner.c index 7ca66ec08185..ba6ae514b378 100644 --- a/test/test_runner.c +++ b/test/test_runner.c @@ -184,6 +184,8 @@ void CB2_TestRunner(void) else gTestRunnerState.timeoutSeconds = UINT_MAX; InitHeap(gHeap, HEAP_SIZE); + SeedRng(0); + SeedRng2(0); EnableInterrupts(INTR_FLAG_TIMER2); REG_TM2CNT_L = UINT16_MAX - (274 * 60); // Approx. 1 second. REG_TM2CNT_H = TIMER_ENABLE | TIMER_INTR_ENABLE | TIMER_1024CLK; From 6a5dfd699d6594aa0fcaef017894fe5c7f420308 Mon Sep 17 00:00:00 2001 From: Martin Griffin Date: Sun, 22 Oct 2023 09:09:59 +0100 Subject: [PATCH 2/2] Seed before setUp --- test/test_runner.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/test_runner.c b/test/test_runner.c index ba6ae514b378..0ae4f0a185e9 100644 --- a/test/test_runner.c +++ b/test/test_runner.c @@ -184,8 +184,6 @@ void CB2_TestRunner(void) else gTestRunnerState.timeoutSeconds = UINT_MAX; InitHeap(gHeap, HEAP_SIZE); - SeedRng(0); - SeedRng2(0); EnableInterrupts(INTR_FLAG_TIMER2); REG_TM2CNT_L = UINT16_MAX - (274 * 60); // Approx. 1 second. REG_TM2CNT_H = TIMER_ENABLE | TIMER_INTR_ENABLE | TIMER_1024CLK; @@ -203,6 +201,8 @@ void CB2_TestRunner(void) case STATE_RUN_TEST: gTestRunnerState.state = STATE_REPORT_RESULT; sCurrentTest.state = CURRENT_TEST_STATE_RUN; + SeedRng(0); + SeedRng2(0); if (gTestRunnerState.test->runner->setUp) gTestRunnerState.test->runner->setUp(gTestRunnerState.test->data); // NOTE: Assumes that the compiler interns __FILE__.