Fix unexpected error in MinimalGeneratingSet for solvable non-pc gr…

…oups

It could end up returning nothing and not invoking TryNextMethod().
This caused a failure in the hap test suite.

This is a recent regression in 4.12.0.

lib/grp.gi

@@ -112,16 +112,16 @@ InstallMethod(MinimalGeneratingSet,"test solvable and 2-generator noncyclic",
 function(G)
 local i;
   if not HasIsSolvableGroup(G) and IsSolvableGroup(G) and
-  CanEasilyComputePcgs(G)  then
+     CanEasilyComputePcgs(G) then
     # discovered solvable -- redo
     return MinimalGeneratingSet(G);
   elif not IsSolvableGroup(G) then
     if IsGroup(G) and (not IsCyclic(G)) and HasGeneratorsOfGroup(G)
         and Length(GeneratorsOfGroup(G)) = 2 then
       return GeneratorsOfGroup(G);
     fi;
-    TryNextMethod();
   fi;
+  TryNextMethod();
 end);
 
 #############################################################################

tst/testbugfix/2022-09-09-MinimalGeneratingSet.tst

@@ -0,0 +1,5 @@
+gap> G:=Group((1,2),(2,3),(3,4));;
+gap> H:=Image(IsomorphismFpGroup(G));;
+gap> MinimalGeneratingSet(H);
+Error, no method found! For debugging hints type ?Recovery from NoMethodFound
+Error, no 4th choice method found for `MinimalGeneratingSet' on 1 arguments