From e60f340a64044ba8f7ec42267726c3a26ef2b373 Mon Sep 17 00:00:00 2001
From: Max Horn <max@quendi.de>
Date: Thu, 9 Jul 2026 11:31:16 +0200
Subject: [PATCH] Fix TestRecogGL to always use a generating set

---
 tst/utils.g | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/tst/utils.g b/tst/utils.g
index c7e9623a3..1f37a602d 100644
--- a/tst/utils.g
+++ b/tst/utils.g
@@ -1,9 +1,8 @@
 TestRecogGL := function(d,q)
     local h, gens, g, ri, r, stamp;
     h := GL(d,q);
-    gens := List([1..10],x->PseudoRandom(h));
-    # FIXME: while this is a generating set with HIGH PROBABILITY, it is not always one.
-    # This could lead to spurious failures in the test suite...
+    # (ab)use product replacement algorithm to get randomized generators
+    gens := ProductReplacer(h)!.team;
     g := GroupWithGenerators(gens);
     ri := RECOG.TestGroup(g,false,Size(h));
     r := ri;