[FIX] run test create roi octave (#1285)

* run test create roi octave

* fix several tests in cpp roi

* reset submod

* update CPP_ROI

* fix default path output create roi

* fix typo

Author: Remi-Gau

lib/CPP_ROI

@@ -186,7 +186,7 @@ def create_roi(
 ) -> str:
     roi_name = "{ '" + "', '".join(roi_name) + "' }" if roi_name is not None else None  # type: ignore
     if roi_dir is None:
-        roi_dir = Path()
+        roi_dir = output_dir
 
     cmd = generate_cmd(
         bids_dir=bids_dir,

src/bidspm/bidspm.py

@@ -14,7 +14,7 @@ function bidsCreateROI(opt)
   %   opt.roi.atlas = 'wang';
   %   opt.roi.hemi = {'L', 'R'};
   %   opt.roi.name = {'V1v', 'V1d'};
-  %   opt.roi.space = {'IXI549Space''};
+  %   opt.roi.space = {'IXI549Space'};
   %
   %   bidsCreateROI(opt);
   %

src/workflows/roi/bidsCreateROI.m

@@ -13,10 +13,6 @@ function test_bidsCreateROI_boilerplate_only()
 
   markTestAs('slow');
 
-  if bids.internal.is_octave
-    moxunit_throw_test_skipped_exception('Need bug fix in CPP ROI extractRoiFromAtlas.');
-  end
-
   outputPath = tempName();
 
   bidspm(pwd, outputPath, ...
@@ -40,10 +36,6 @@ function test_bidsCreateROI_glasser()
 
   markTestAs('slow');
 
-  if bids.internal.is_octave
-    moxunit_throw_test_skipped_exception('Need bug fix in CPP ROI extractRoiFromAtlas.');
-  end
-
   outputPath = tempName();
 
   bidspm(pwd, outputPath, ...
@@ -54,6 +46,12 @@ function test_bidsCreateROI_glasser()
          'space', {'IXI549Space'}, ...
          'verbosity', 3);
 
+  assert(exist(fullfile(outputPath, ...
+                        'derivatives', ...
+                        'bidspm-roi', ...
+                        'reports', ...
+                        'create_roi_atlas-glasser_citation.md'), 'file') == 2);
+
   rois = spm_select('FPList', fullfile(outputPath, ...
                                        'derivatives', ...
                                        'bidspm-roi', 'group'), '.*glasser.*_mask\.nii');
@@ -65,10 +63,6 @@ function test_bidsCreateROI_wang()
 
   markTestAs('slow');
 
-  if bids.internal.is_octave
-    moxunit_throw_test_skipped_exception('Need bug fix in CPP ROI extractRoiFromAtlas.');
-  end
-
   outputPath = tempName();
 
   bidspm(pwd, outputPath, ...
@@ -90,10 +84,6 @@ function test_bidsCreateROI_neuromorphometrics()
 
   markTestAs('slow');
 
-  if bids.internal.is_octave
-    moxunit_throw_test_skipped_exception('Need bug fix in CPP ROI extractRoiFromAtlas.');
-  end
-
   outputPath = tempName();
 
   bidspm(pwd, outputPath, ...
@@ -133,10 +123,6 @@ function test_bidsCreateROI_one_hemisphere()
 
   markTestAs('slow');
 
-  if bids.internal.is_octave
-    moxunit_throw_test_skipped_exception('Need bug fix in CPP ROI extractRoiFromAtlas.');
-  end
-
   outputPath = tempName();
 
   bidspm(pwd, outputPath, 'subject', ...