svn commit: r1931798 - spamassassin/trunk/t

Author: gbechis
Date: Tue Feb 10 23:18:16 2026
New Revision: 1931798

Log:
don't run a test as root if it has a problem resulting from setuid nobody
bz #8364

Modified:
   spamassassin/trunk/t/SATest.pm
   spamassassin/trunk/t/spamc_optL.t
   spamassassin/trunk/t/spamd_client.t

Modified: spamassassin/trunk/t/SATest.pm
==============================================================================
--- spamassassin/trunk/t/SATest.pm	Tue Feb 10 20:21:24 2026	(r1931797)
+++ spamassassin/trunk/t/SATest.pm	Tue Feb 10 23:18:16 2026	(r1931798)
@@ -273,7 +273,7 @@ sub sa_t_init {
   if ($> == 0) {
     $tmp_dir_mode = 0777;
     umask 022;  # ensure correct permissions on files and dirs created here
-    # Bug 5529 initial fix: For now don't run a test as root if it has a problem resuting from setuid nobody
+    # Bug 5529 initial fix: For now don't run a test as root if it has a problem resulting from setuid nobody
     # FIXME: Eventually we can actually test setuid nobody and accessing ./log to make this test more fine grained
     #  and we can create an accessible temp dir that some of the tests can use. But for now just skip those tests.
     $SKIP_SETUID_NOBODY_TESTS = 1;

Modified: spamassassin/trunk/t/spamc_optL.t
==============================================================================
--- spamassassin/trunk/t/spamc_optL.t	Tue Feb 10 20:21:24 2026	(r1931797)
+++ spamassassin/trunk/t/spamc_optL.t	Tue Feb 10 23:18:16 2026	(r1931798)
@@ -7,6 +7,7 @@ use constant HAS_SDBM_FILE => eval { req
 
 use Test::More;
 plan skip_all => "No SPAMC exe" if $SKIP_SPAMC_TESTS;
+plan skip_all => "UID nobody tests" if $SKIP_SETUID_NOBODY_TESTS;
 plan skip_all => "No SDBM_File" unless HAS_SDBM_FILE;
 plan tests => 18;
 

Modified: spamassassin/trunk/t/spamd_client.t
==============================================================================
--- spamassassin/trunk/t/spamd_client.t	Tue Feb 10 20:21:24 2026	(r1931797)
+++ spamassassin/trunk/t/spamd_client.t	Tue Feb 10 23:18:16 2026	(r1931798)
@@ -7,6 +7,7 @@ use constant HAS_SDBM_FILE => eval { req
 
 use Test::More;
 plan skip_all => "Spamd tests disabled" if $SKIP_SPAMD_TESTS;
+plan skip_all => "UID nobody tests" if $SKIP_SETUID_NOBODY_TESTS;
 plan skip_all => "Long running tests disabled" unless conf_bool('run_long_tests');
 
 diag "\nIf test fails 'connection reset by peer', may be low memory on test machine";