[INFRA] randomize tests order (#566)

* randomize tests * fix doc generation and links * skip dead links * skip dead links * skip dead links * skip dead links * use temp_dir
bids-standard · May 11, 2023 · 3b2d45f · 3b2d45f
1 parent 287109a
commit 3b2d45f
Show file tree

Hide file tree

Showing 5 changed files with 12 additions and 14 deletions.
diff --git a/.github/workflows/mlc_config.json b/.github/workflows/mlc_config.json
@@ -1,10 +1,7 @@
 {
     "ignorePatterns": [
         {
-            "pattern": "^https://www.fil.ion.ucl.ac.uk/spm"
-        },
-        {
-            "pattern": "^https://.*github.com/.*"
+            "pattern": "fil.ion.ucl.ac.uk|mybinder.org|bids-matlab"
         }
     ]
 }
diff --git a/docs/requirements.txt b/docs/requirements.txt
@@ -5,3 +5,4 @@ sphinx_rtd_theme
 sphinx-copybutton
 myst-parser
 rich
+urllib3==1.26.15
diff --git a/run_tests.m b/run_tests.m
@@ -21,6 +21,7 @@
   if with_coverage
     success = moxunit_runtests(testFolder, ...
                                '-verbose', ...
+                               '-randomize_order', ...
                                '-recursive', ...
                                '-with_coverage', ...
                                '-cover', folderToCover, ...
@@ -29,6 +30,7 @@
   else
     success = moxunit_runtests(testFolder, ...
                                '-verbose', ...
+                               '-randomize_order', ...
                                '-recursive');
 
   end

diff --git a/tests/test_download_ds.m b/tests/test_download_ds.m
@@ -10,42 +10,36 @@
 
 function test_download_ds_moae()
 
-  rmdir(fullfile(bids.internal.root_dir(), 'tmp'), 's');
-
   output_dir = bids.util.download_ds('source', 'spm', ...
                                      'demo', 'moae', ...
-                                     'out_path', fullfile(bids.internal.root_dir(), 'tmp'), ...
+                                     'out_path', temp_dir(), ...
                                      'force', false, ...
                                      'verbose', true, ...
                                      'delete_previous', false);
 
   output_dir = bids.util.download_ds('source', 'spm', ...
                                      'demo', 'moae', ...
-                                     'out_path', fullfile(bids.internal.root_dir(), 'tmp'), ...
+                                     'out_path', temp_dir(), ...
                                      'force', true, ...
                                      'verbose', true, ...
                                      'delete_previous', false);
 
   output_dir = bids.util.download_ds('source', 'spm', ...
                                      'demo', 'moae', ...
-                                     'out_path', fullfile(bids.internal.root_dir(), 'tmp'), ...
+                                     'out_path', temp_dir(), ...
                                      'force', true, ...
                                      'verbose', true, ...
                                      'delete_previous', true);
 
-  rmdir(output_dir, 's');
-
 end
 
 function test_download_ds_facerep()
 
   output_dir = bids.util.download_ds('source', 'spm', ...
                                      'demo', 'facerep', ...
-                                     'out_path', fullfile(bids.internal.root_dir(), 'tmp'), ...
+                                     'out_path', temp_dir(), ...
                                      'force', true, ...
                                      'verbose', true, ...
                                      'delete_previous', true);
 
-  rmdir(output_dir, 's');
-
 end
diff --git a/tests/utils/temp_dir.m b/tests/utils/temp_dir.m
@@ -0,0 +1,4 @@
+function value = temp_dir()
+  value = tempname();
+  mkdir(value);
+end