fix a bug when autotrace includes or excludes multiple folders, fixes #195

Author: duncanpo

auto-instrumentation/+opentelemetry/+autoinstrument/AutoTrace.m

@@ -1,7 +1,7 @@
 classdef AutoTrace < handle
     % Automatic instrumentation with OpenTelemetry tracing.
 
-    % Copyright 2024 The MathWorks, Inc.
+    % Copyright 2024-2025 The MathWorks, Inc.
 
     properties (SetAccess=private)
         StartFunction function_handle   % entry function
@@ -190,7 +190,7 @@ function handleError(obj, ME)
     mfiles = fullfile(string({mfileinfo.folder}), string({mfileinfo.name}));
     mlxfileinfo = dir(fullfile(f, "*.mlx"));
     mlxfiles = fullfile(string({mlxfileinfo.folder}), string({mlxfileinfo.name}));
-    f = [mfiles; mlxfiles];
+    f = [mfiles(:); mlxfiles(:)];
 else
     % file
     f = processFileInput(f);

test/autotrace_examples/two_subfolders_example/helper1/subfolder_helper1_1.m

@@ -0,0 +1,6 @@
+function x = subfolder_helper1_1(x)
+% example code for testing auto instrumentation, helper function
+
+% Copyright 2025 The MathWorks, Inc.
+
+x = x * 2;

test/autotrace_examples/two_subfolders_example/helper1/subfolder_helper1_2.m

@@ -0,0 +1,6 @@
+function x = subfolder_helper1_2(x)
+% example code for testing auto instrumentation, helper function
+
+% Copyright 2025 The MathWorks, Inc.
+
+x = x * 3;

test/autotrace_examples/two_subfolders_example/helper2/subfolder_helper2_1.m

@@ -0,0 +1,6 @@
+function x = subfolder_helper2_1(x)
+% example code for testing auto instrumentation, helper function
+
+% Copyright 2025 The MathWorks, Inc.
+
+x = x * 4;

test/autotrace_examples/two_subfolders_example/helper2/subfolder_helper2_2.m

@@ -0,0 +1,6 @@
+function x = subfolder_helper2_2(x)
+% example code for testing auto instrumentation, helper function
+
+% Copyright 2025 The MathWorks, Inc.
+
+x = x * 5;

test/autotrace_examples/two_subfolders_example/helper2/subfolder_helper2_3.m

@@ -0,0 +1,6 @@
+function x = subfolder_helper2_3(x)
+% example code for testing auto instrumentation, helper function
+
+% Copyright 2025 The MathWorks, Inc.
+
+x = x * 5;

test/autotrace_examples/two_subfolders_example/two_subfolders_example.m

@@ -0,0 +1,13 @@
+function x = two_subfolders_example
+% Example code for testing auto instrumentation, with helper functions
+% in a two subfolders
+
+% Copyright 2025 The MathWorks, Inc.
+
+x = 10;
+x = subfolder_helper1_1(x);
+x = subfolder_helper1_2(x);
+x = subfolder_helper2_1(x);
+x = subfolder_helper2_2(x);
+x = subfolder_helper2_3(x);
+

test/tautotrace.m

@@ -1,7 +1,7 @@
 classdef tautotrace < matlab.unittest.TestCase
     % tests for AutoTrace
 
-    % Copyright 2024 The MathWorks, Inc.
+    % Copyright 2024-2025 The MathWorks, Inc.
 
     properties
         OtelConfigFile
@@ -155,6 +155,35 @@ function testIncludeFolder(testCase)
             verifyEqual(testCase, results{2}.resourceSpans.scopeSpans.spans.parentSpanId, results{3}.resourceSpans.scopeSpans.spans.spanId);
         end
 
+        function testMultipleIncludeFolders(testCase)
+            % testMultipleIncludeFolders: specify multiple folders in AdditionalFiles
+           
+            % Add example folders to the path
+            examplefolder = fullfile(fileparts(mfilename('fullpath')), "autotrace_examples", "two_subfolders_example");
+            testCase.applyFixture(matlab.unittest.fixtures.PathFixture(examplefolder, ...
+                "IncludingSubfolders",true));
+
+            % set up AutoTrace, turn off automatic detection and specify 
+            % dependencies using their folder name
+            at = opentelemetry.autoinstrument.AutoTrace(@two_subfolders_example, ...
+                "AutoDetectFiles", false, "AdditionalFiles", fullfile(examplefolder, ["helper1" "helper2"]));
+
+            % run the example
+            [~] = beginTrace(at);
+
+            % perform test comparisons
+            results = readJsonResults(testCase);
+            verifyNumElements(testCase, results, 6);
+
+            % check span names
+            verifyEqual(testCase, string(results{1}.resourceSpans.scopeSpans.spans.name), "subfolder_helper1_1");
+            verifyEqual(testCase, string(results{2}.resourceSpans.scopeSpans.spans.name), "subfolder_helper1_2");
+            verifyEqual(testCase, string(results{3}.resourceSpans.scopeSpans.spans.name), "subfolder_helper2_1");
+            verifyEqual(testCase, string(results{4}.resourceSpans.scopeSpans.spans.name), "subfolder_helper2_2");
+            verifyEqual(testCase, string(results{5}.resourceSpans.scopeSpans.spans.name), "subfolder_helper2_3");
+            verifyEqual(testCase, string(results{6}.resourceSpans.scopeSpans.spans.name), "two_subfolders_example");
+        end
+
         function testExcludeFolder(testCase)
             % testExcludeFolder: specify a folder in ExcludeFiles