ts: remove fixtures left behind when quick mode skips a testfile

The skip_if_quick_mode procedure returns out of the current testfile, so
the cleanup block at its end is not reached. Testfiles creating a
fixture directory before this skip point and removing it in this block
leave it behind under the testsuite directory after a quick mode run.
This is the case of 180-modulepath-ignore.exp and
060-cachebuild-ignore.exp.

Accept fixture paths as arguments of skip_if_quick_mode to remove them
when tests are skipped, and pass the directory both testfiles need
after the skip point. This avoids duplicating their fixture creation
block around the skip.

Assisted-by: Claude:claude-fable-5-1
Signed-off-by: Xavier Delaruelle <xavier.delaruelle@cea.fr>

Author: xdelaruelle

testsuite/config/base-config.exp

@@ -1390,10 +1390,12 @@ proc is_quick_mode {} {
         $::env(QUICKTEST)]}]
 }
 
-# if quick test mode is running stop processing of current test file
-proc skip_if_quick_mode {} {
+# if quick test mode is running stop processing of current test file after
+# removing given fixture paths
+proc skip_if_quick_mode {args} {
     if {[is_quick_mode]} {
         send_user "\tSkipping tests as quick test mode is enabled\n"
+        file delete -force {*}$args
         reset_test_env
         return -code 0 -level 2 0
     }

testsuite/modules.20-locate/180-modulepath-ignore.exp

@@ -93,7 +93,7 @@ unsetenv_var MODULES_MODULEPATH_IGNORE
 # execution
 testouterr_cmd sh {--no-modulepath-ignore avail -a -t -o } OK $tserr
 
-skip_if_quick_mode
+skip_if_quick_mode $mp1
 
 # ignore all accept some idiom with negated patterns
 file mkdir $mp2/app

testsuite/modules.30-cache/060-cachebuild-ignore.exp

@@ -105,7 +105,7 @@ testouterr_cmd sh {--no-modulepath-ignore --ignore-cache is-avail bar/2.0}\
 
 file delete $mp1cache
 
-skip_if_quick_mode
+skip_if_quick_mode $mp1
 
 # every element is recorded in cache when modulepath_ignore is disabled
 setenv_var MODULES_MODULEPATH_IGNORE 0