Skip to content

Commit 69a7a96

Browse files
committed
Run --bare testinstall test in CI
1 parent 59ae437 commit 69a7a96

4 files changed

Lines changed: 18 additions & 3 deletions

File tree

.github/workflows/CI.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ jobs:
4747
test-suites:
4848
[
4949
# base test: fast first test
50-
"testinstall",
50+
"testinstall-bare",
5151

5252
"teststandard",
5353

dev/ci.sh

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -422,6 +422,10 @@ GAPInput
422422

423423
;;
424424

425+
testinstall-bare)
426+
$GAP $(gap_cover_arg) --bare $SRCDIR/tst/testinstall.g
427+
;;
428+
425429
*)
426430
if [[ ! -f $SRCDIR/tst/${TEST_SUITE}.g ]]
427431
then

tst/testinstall/maxsub.tst

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,12 +9,14 @@ gap> SortedList(List(msc, IndexInParent));
99
[ 2, 3, 4 ]
1010

1111
#
12+
#@if IsPackageMarkedForLoading( "primgrp", "" )
1213
gap> G := GL(2,4);;
1314
gap> msc:=MaximalSubgroupClassReps(G);;
1415
gap> ForAll(msc, H -> Parent(H) = G);
1516
true
1617
gap> SortedList(List(msc, IndexInParent));
1718
[ 3, 5, 6, 10 ]
19+
#@fi
1820

1921
#
2022
gap> G := GL(2,5);;
@@ -25,10 +27,12 @@ gap> SortedList(List(msc, IndexInParent));
2527
[ 2, 5, 6, 10 ]
2628

2729
#
30+
#@if IsPackageMarkedForLoading( "primgrp", "" )
2831
gap> G := AlternatingGroup(5);;
2932
gap> msc := MaximalSubgroupClassReps(G);;
3033
gap> SortedList(List(msc, H -> Index(G, H)));
3134
[ 5, 6, 10 ]
35+
#@fi
3236

3337
#
3438
gap> STOP_TEST("maxsub.tst");

tst/testinstall/package.tst

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -435,9 +435,11 @@ false
435435
gap> IsPackageLoaded("mockpkg", ">=2.0");
436436
false
437437

438+
#
438439
# instruct GAP to load the package, and record all its declarations
439440
# the help book of mockpkg might already have been loaded in other tests
440441
# -> we suppress a warning about this
442+
#@if IsPackageMarkedForLoading( "gapdoc", "" )
441443
gap> old_warning_level := InfoLevel( InfoWarning );;
442444
gap> SetInfoLevel( InfoWarning, 0 );
443445
gap> PackageVariablesInfo("mockpkg", "0.1");;
@@ -481,9 +483,14 @@ gap> IsDateFormatValid := function( datestring )
481483
> end;;
482484
gap> IsDateFormatValid( GAPInfo.Date );
483485
true
486+
#@else
487+
gap> LoadPackage("mockpkg", false);
488+
oops, should not print here
489+
true
490+
#@fi
484491

485492
# Test the Cite() command (output changed with GAPDoc 1.6.6 and again with 1.6.7)
486-
#@if CompareVersionNumbers(InstalledPackageVersion("gapdoc"), "1.6.7")
493+
#@if IsPackageMarkedForLoading( "gapdoc", "1.6.7" )
487494
gap> Cite("mockpkg");
488495
Please use one of the following samples
489496
to cite mockpkg version from this installation
@@ -538,7 +545,7 @@ rg/}}},
538545
}
539546

540547

541-
#@else
548+
#@elif IsPackageMarkedForLoading( "gapdoc", "" )
542549
gap> Cite("mockpkg");
543550
Please use one of the following samples
544551
to cite mockpkg version from this installation

0 commit comments

Comments
 (0)