Merge pull request #241 from mathworks/mbt-plugin-updates

Only ship mbt plugin if gRPC is installed

Author: sameagen-MW

CMakeLists.txt

@@ -627,13 +627,13 @@ install(DIRECTORY ${METRICS_SDK_MATLAB_SOURCES} DESTINATION .)
 install(DIRECTORY ${LOGS_SDK_MATLAB_SOURCES} DESTINATION .)
 install(DIRECTORY ${COMMON_SDK_MATLAB_SOURCES} DESTINATION .)
 install(DIRECTORY ${AUTO_INSTRUMENTATION_MATLAB_SOURCES} DESTINATION .)
-install(DIRECTORY ${INSTRUMENTATION_MBT_MATLAB_SOURCES} DESTINATION .)
 install(FILES ${EXPORTER_MATLAB_SOURCES} DESTINATION ${OTLP_EXPORTERS_DIR})
 if(WITH_OTLP_HTTP)
     install(FILES ${OTLP_HTTP_EXPORTER_MATLAB_SOURCES} DESTINATION ${OTLP_EXPORTERS_DIR})
 endif()
 if(WITH_OTLP_GRPC)
     install(FILES ${OTLP_GRPC_EXPORTER_MATLAB_SOURCES} DESTINATION ${OTLP_EXPORTERS_DIR})
+    install(DIRECTORY ${INSTRUMENTATION_MBT_MATLAB_SOURCES} DESTINATION .)
 endif()
 if(WITH_OTLP_FILE)
     install(FILES ${OTLP_FILE_EXPORTER_MATLAB_SOURCES} DESTINATION ${OTLP_EXPORTERS_DIR})

test/tbuildtoolplugin.m

@@ -29,11 +29,11 @@ function setup(testCase)
         end
 
         function onlyTestIfgRPCIsInstalled(testCase)
+            % Do not run if gRPC is not installed
             testCase.assumeTrue(logical(exist("opentelemetry.exporters.otlp.OtlpGrpcSpanExporter", "class")), ...
                 "Otlp gRPC exporter must be installed.");
-        end
-        
-        function createBuildRunner(testCase)
+
+            % Set up build runner
             plugin = matlab.buildtool.plugins.OpenTelemetryPlugin();
             testCase.BuildRunner = matlab.buildtool.BuildRunner.withNoPlugins();
             testCase.BuildRunner.addPlugin(plugin);
@@ -474,4 +474,4 @@ function buildToolDoesNotConfigureOTelWhenEnvVariableSet(testCase)
     end
 
     error("No span found");
-end
+end