Skip to content
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

Commit 2a68ddc

Browse files
committedOct 17, 2024·
testsuite: Disable flaky T2 spectests and update helper script
1 parent 7393d58 commit 2a68ddc

File tree

5 files changed

+25
-13
lines changed

5 files changed

+25
-13
lines changed
 

‎test/testsuite/T2_FPDelete.c

+3
Original file line numberDiff line numberDiff line change
@@ -292,6 +292,8 @@ DSI *dsi = &Conn->dsi;
292292
/* ----------- */
293293
void FPDelete_test()
294294
{
295+
// FIXME: all flaky with Netatalk 4.0; mix of fails and false passes
296+
#if 0
295297
fprintf(stdout,"===================\n");
296298
fprintf(stdout,"FPDelete page 143\n");
297299
test146();
@@ -300,5 +302,6 @@ void FPDelete_test()
300302
test363();
301303
#endif
302304
test364();
305+
#endif
303306
}
304307

‎test/testsuite/T2_FPOpenFork.c

+5
Original file line numberDiff line numberDiff line change
@@ -1542,5 +1542,10 @@ void FPOpenFork_test()
15421542
test236();
15431543
test237();
15441544
test238();
1545+
// FIXME: test seems to require manually moving data file to
1546+
// data/resource_fork_conversion_test
1547+
// Disabling for now.
1548+
#if 0
15451549
test431();
1550+
#endif
15461551
}

‎test/testsuite/T2_FPSetFileParms.c

+3
Original file line numberDiff line numberDiff line change
@@ -220,6 +220,9 @@ void FPSetFileParms_test()
220220
fprintf(stdout,"FPSetFileParms page 262\n");
221221
test89();
222222
test120();
223+
// FIXME: test stalls indefinitely
224+
#if 0
223225
test426();
226+
#endif
224227
}
225228

‎test/testsuite/meson.build

+2-1
Original file line numberDiff line numberDiff line change
@@ -223,7 +223,8 @@ T2_spectest = executable(
223223
spectest_sh = find_program('spectest.sh')
224224

225225
test(
226-
'spectest',
226+
'AFP specification test suite',
227227
spectest_sh,
228228
is_parallel: true,
229+
timeout: 90,
229230
)

‎test/testsuite/spectest.sh

+12-12
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,8 @@ check_return() {
1414

1515
echo =====================================
1616

17-
if [ ! -f spectest.conf ] ; then
18-
cat > spectest.conf <<EOF
17+
if [ ! -f ./test/testsuite/spectest.conf ] ; then
18+
cat > ./test/testsuite/spectest.conf <<EOF
1919
# AFPSERVER=127.0.0.1
2020
# AFPPORT=548
2121
# USER1=
@@ -36,7 +36,7 @@ EOF
3636
exit 0
3737
fi
3838

39-
. ./spectest.conf
39+
. ./test/testsuite/spectest.conf
4040

4141
# cleanup
4242
if test ! -z "$LOCALVOL1PATH" ; then
@@ -52,28 +52,28 @@ rm -f spectest.log
5252

5353
##
5454
printf "Running spectest with two users..."
55-
./spectest -"$AFPVERSION" -h "$AFPSERVER" -p "$AFPPORT" -u "$USER1" -d "$USER2" -w "$PASSWD" -s "$VOLUME1" -S "$VOLUME2" >> spectest.log 2>&1
55+
./test/testsuite/spectest -"$AFPVERSION" -h "$AFPSERVER" -p "$AFPPORT" -u "$USER1" -d "$USER2" -w "$PASSWD" -s "$VOLUME1" -S "$VOLUME2" >> ./test/testsuite/spectest.log 2>&1
5656
check_return
5757

5858
##
5959
printf "Running spectest with local filesystem modifications..."
60-
./T2_spectest -"$AFPVERSION" -h "$AFPSERVER" -p "$AFPPORT" -u "$USER1" -d "$USER2" -w "$PASSWD" -s "$VOLUME1" -S "$VOLUME2" -c "$LOCALVOL1PATH" >> spectest.log 2>&1
60+
./test/testsuite/T2_spectest -"$AFPVERSION" -h "$AFPSERVER" -p "$AFPPORT" -u "$USER1" -d "$USER2" -w "$PASSWD" -s "$VOLUME1" -S "$VOLUME2" -c "$LOCALVOL1PATH" >> ./test/testsuite/spectest.log 2>&1
6161
check_return
6262

6363
echo =====================================
6464
echo Failed tests
65-
echo ––––––––––––
66-
grep "summary.*FAIL" spectest.log | sed s/test//g | sort -n | uniq
65+
echo ------------
66+
grep "summary.*FAIL" ./test/testsuite/spectest.log | sed s/test//g | sort -n | uniq
6767
echo =====================================
6868

6969
echo Skipped tests
70-
echo –––––––––––––
71-
egrep 'summary.*NOT TESTED|summary.*SKIPPED' spectest.log | sed s/test//g | sort -n | uniq
70+
echo -------------
71+
egrep 'summary.*NOT TESTED|summary.*SKIPPED' ./test/testsuite/spectest.log | sed s/test//g | sort -n | uniq
7272
echo =====================================
7373

74-
echo Successfull tests
75-
echo –––––––––––––––––
76-
grep "summary.*PASSED" spectest.log | sed s/test//g | sort -n | uniq
74+
echo Successful tests
75+
echo ----------------
76+
grep "summary.*PASSED" ./test/testsuite/spectest.log | sed s/test//g | sort -n | uniq
7777
echo =====================================
7878

7979
# cleanup

0 commit comments

Comments
 (0)
Please sign in to comment.