clean up tests by force flushing

Author: duncanpo

api/trace/include/opentelemetry-matlab/trace/TracerProviderProxy.h

@@ -48,7 +48,8 @@ class TracerProviderProxy : public libmexclass::proxy::Proxy {
 
     void postShutdown(libmexclass::proxy::method::Context& context) {
 	// Replace tracer provider with a no-op instance. Subsequent tracers and spans won't be recorded
-        CppTracerProvider.swap(nostd::shared_ptr<trace_api::TracerProvider>(new trace_api::NoopTracerProvider));
+	nostd::shared_ptr<trace_api::TracerProvider> noop(new trace_api::NoopTracerProvider);
+        CppTracerProvider.swap(noop);
     }
 
   protected:

sdk/trace/src/TracerProviderProxy.cpp

@@ -81,7 +81,8 @@ void TracerProviderProxy::shutdown(libmexclass::proxy::method::Context& context)
     matlab::data::ArrayFactory factory;
     auto result_mda = factory.createScalar(static_cast<trace_sdk::TracerProvider&>(*CppTracerProvider).Shutdown());
     context.outputs[0] = result_mda;
-    CppTracerProvider.swap(nostd::shared_ptr<trace_api::TracerProvider>(new trace_api::NoopTracerProvider));
+    nostd::shared_ptr<trace_api::TracerProvider> noop(new trace_api::NoopTracerProvider);
+    CppTracerProvider.swap(noop);
 }
 
 void TracerProviderProxy::forceFlush(libmexclass::proxy::method::Context& context) {

test/commonTeardown.m

@@ -3,6 +3,10 @@ function commonTeardown(testCase)
 %
 % Copyright 2023 The MathWorks, Inc.
 
+% Flush all spans that have not yet been exported
+tp = opentelemetry.trace.Provider.getTracerProvider();
+opentelemetry.sdk.trace.Cleanup.forceFlush(tp);
+
 % Terminate Collector if it is still running
 terminateCollector(testCase);
 
@@ -19,4 +23,4 @@ function commonTeardown(testCase)
 end
 if exist(testCase.PidFile, "file")
     delete(testCase.PidFile);
-end
+end