ci: test without / with MEX

Author: scivision

.github/workflows/composite-buildtool/action.yml

@@ -13,18 +13,32 @@ runs:
       startup-options: ${{ matrix.startup-options }}
       tasks: check
 
-  - name: Test task
-    if: ${{ matrix.release >= 'R2022b' || startsWith(matrix.release, 'latest') }}
+  - name: Legacy Mex Test
+    if: ${{ matrix.release >= 'R2022b' && matrix.release < 'R2024b' }}
     uses: matlab-actions/run-build@v2
     with:
       startup-options: ${{ matrix.startup-options }}
-      tasks: test
+      tasks: legacyMex test
 
+  - name: Non-Mex Test
+    if: ${{ matrix.release >= 'R2024b' || startsWith(matrix.release, 'latest') }}
+    uses: matlab-actions/run-build@v2
+    with:
+      startup-options: ${{ matrix.startup-options }}
+      tasks: test
 
-  - name: Run tests (manual)
-    if: ${{ matrix.release < 'R2022b' && !startsWith(matrix.release, 'latest') }}
-    uses: matlab-actions/run-tests@v2
+  - name: Mex Group Test
+    if: ${{ matrix.release >= 'R2024b' || startsWith(matrix.release, 'latest') }}
+    uses: matlab-actions/run-build@v2
     with:
-      source-folder: ${{ github.workspace }}
-      select-by-folder: test
-      strict: true
+      startup-options: ${{ matrix.startup-options }}
+      tasks: mex test
+
+
+  # - name: Run tests (manual)
+  #   if: ${{ matrix.release < 'R2022b' && !startsWith(matrix.release, 'latest') }}
+  #   uses: matlab-actions/run-tests@v2
+  #   with:
+  #     source-folder: ${{ github.workspace }}
+  #     select-by-folder: test
+  #     strict: true

buildfile.m

@@ -9,11 +9,8 @@
   plan("check") = matlab.buildtool.tasks.CodeIssuesTask(pkg_name, IncludeSubfolders=true);
 end
 
-if isMATLABReleaseOlderThan("R2024b")
-  plan("test").Dependencies = "legacyMex";
-else
+if ~isMATLABReleaseOlderThan("R2024b")
 
-  plan("test").Dependencies = "mex";
   plan("clean") = matlab.buildtool.tasks.CleanTask;
 
   [compiler_id, compiler_opt] = get_compiler_options();

private/get_mex_sources.m

@@ -3,30 +3,22 @@
   top (1,1) string  % package directory
 end
 
-%% source C++
+win = "";
+if ispc
+win = fullfile(top, "src/windows.cpp");
+end
+
+mac = fullfile(top, "src/macos.cpp");
 
-% Matlab and Java don't have these functions
 srcs = {fullfile(top, "src/is_char_device.cpp"), ...
   fullfile(top, "src/set_permissions.cpp"), ...
-  fullfile(top, "src/is_rosetta.cpp"), ...
+  [fullfile(top, "src/is_rosetta.cpp"), mac], ...
+  [fullfile(top, "src/windows_shortname.cpp"), win]
 };
 
-s = fullfile(top, "src/windows_shortname.cpp");
-if ispc
-  s(end+1) = fullfile(top, "src/windows.cpp");
-end
-srcs{end+1} = s;
-
-
-% isSymbolicLink() new in R2024b
+%% isSymbolicLink() new in R2024b
 if ~isMATLABReleaseOlderThan("R2024b")
-
-s = fullfile(top, "src/is_symlink.cpp");
-if ispc
-  s(end+1) = fullfile(top, "src/windows.cpp");
-end
-srcs{end+1} = s;
-
+srcs{end+1} = [fullfile(top, "src/is_symlink.cpp"), win];
 end
 
 end

test/TestMex.m

@@ -3,8 +3,14 @@
 methods (Test)
 
 function test_is_char_device(tc)
-% /dev/stdin may not be available on CI systems
+import matlab.unittest.fixtures.CurrentFolderFixture
+
+tc.applyFixture(CurrentFolderFixture(".."))
+% matlab exist() doesn't work for MEX detection with ".." leading path
 
+tc.assumeEqual(exist("+stdlib/is_char_device", "file"), 3)
+
+% /dev/stdin may not be available on CI systems
 if ispc
   n = "NUL";
 else