chore: disable apis after use in tests (#1800)

Author: dyladan

packages/opentelemetry-instrumentation-http/test/functionals/http-enable.test.ts

@@ -67,7 +67,6 @@ const provider = new NodeTracerProvider({
 });
 provider.addSpanProcessor(new SimpleSpanProcessor(memoryExporter));
 instrumentation.setTracerProvider(provider);
-propagation.setGlobalPropagator(new DummyPropagation());
 
 function doNock(
   hostname: string,
@@ -104,6 +103,14 @@ export const responseHookFunction = (
 describe('HttpInstrumentation', () => {
   let contextManager: ContextManager;
 
+  before(() => {
+    propagation.setGlobalPropagator(new DummyPropagation());
+  });
+
+  after(() => {
+    propagation.disable();
+  });
+
   beforeEach(() => {
     contextManager = new AsyncHooksContextManager().enable();
     context.setGlobalContextManager(contextManager);

packages/opentelemetry-instrumentation-http/test/functionals/http-package.test.ts

@@ -67,12 +67,12 @@ describe('Packages', () => {
     });
     provider.addSpanProcessor(new SimpleSpanProcessor(memoryExporter));
     instrumentation.setTracerProvider(provider);
-    propagation.setGlobalPropagator(new DummyPropagation());
     beforeEach(() => {
       memoryExporter.reset();
     });
 
     before(() => {
+      propagation.setGlobalPropagator(new DummyPropagation());
       instrumentation.setConfig({
         applyCustomAttributesOnSpan: customAttributeFunction,
       });
@@ -81,6 +81,7 @@ describe('Packages', () => {
 
     after(() => {
       // back to normal
+      propagation.disable();
       nock.cleanAll();
       nock.enableNetConnect();
     });

packages/opentelemetry-instrumentation-http/test/functionals/https-enable.test.ts

@@ -69,7 +69,6 @@ const provider = new BasicTracerProvider({
 instrumentation.setTracerProvider(provider);
 const tracer = provider.getTracer('test-https');
 provider.addSpanProcessor(new SimpleSpanProcessor(memoryExporter));
-propagation.setGlobalPropagator(new DummyPropagation());
 
 function doNock(
   hostname: string,
@@ -94,12 +93,14 @@ describe('HttpsInstrumentation', () => {
 
   beforeEach(() => {
     contextManager = new AsyncHooksContextManager().enable();
+    propagation.setGlobalPropagator(new DummyPropagation());
     context.setGlobalContextManager(contextManager);
   });
 
   afterEach(() => {
     contextManager.disable();
     context.disable();
+    propagation.disable();
   });
 
   describe('enable()', () => {

packages/opentelemetry-instrumentation-http/test/functionals/https-package.test.ts

@@ -68,12 +68,12 @@ describe('Packages', () => {
     });
     provider.addSpanProcessor(new SimpleSpanProcessor(memoryExporter));
     instrumentation.setTracerProvider(provider);
-    propagation.setGlobalPropagator(new DummyPropagation());
     beforeEach(() => {
       memoryExporter.reset();
     });
 
     before(() => {
+      propagation.setGlobalPropagator(new DummyPropagation());
       instrumentation.setConfig({
         applyCustomAttributesOnSpan: customAttributeFunction,
       });
@@ -84,6 +84,7 @@ describe('Packages', () => {
       // back to normal
       nock.cleanAll();
       nock.enableNetConnect();
+      propagation.disable();
     });
 
     let resHeaders: http.IncomingHttpHeaders;

packages/opentelemetry-instrumentation-http/test/integrations/http-enable.test.ts

@@ -100,11 +100,16 @@ describe('HttpInstrumentation Integration tests', () => {
 
   beforeEach(() => {
     memoryExporter.reset();
+  });
+
+  before(() => {
+    propagation.setGlobalPropagator(new DummyPropagation());
     context.setGlobalContextManager(new AsyncHooksContextManager().enable());
   });
 
-  afterEach(() => {
+  after(() => {
     context.disable();
+    propagation.disable();
   });
   describe('enable()', () => {
     before(function (done) {
@@ -127,7 +132,6 @@ describe('HttpInstrumentation Integration tests', () => {
     const provider = new NodeTracerProvider({
       logger,
     });
-    propagation.setGlobalPropagator(new DummyPropagation());
     provider.addSpanProcessor(new SimpleSpanProcessor(memoryExporter));
     instrumentation.setTracerProvider(provider);
     beforeEach(() => {

packages/opentelemetry-instrumentation-http/test/integrations/https-enable.test.ts

@@ -140,7 +140,6 @@ describe('HttpsInstrumentation Integration tests', () => {
     const provider = new NodeTracerProvider({
       logger,
     });
-    propagation.setGlobalPropagator(new DummyPropagation());
     provider.addSpanProcessor(new SimpleSpanProcessor(memoryExporter));
     instrumentation.setTracerProvider(provider);
     beforeEach(() => {
@@ -153,6 +152,7 @@ describe('HttpsInstrumentation Integration tests', () => {
         /\/ignored\/regexp$/i,
         (url: string) => url.endsWith('/ignored/function'),
       ];
+      propagation.setGlobalPropagator(new DummyPropagation());
       instrumentation.setConfig({
         ignoreIncomingPaths: ignoreConfig,
         ignoreOutgoingUrls: ignoreConfig,
@@ -163,6 +163,7 @@ describe('HttpsInstrumentation Integration tests', () => {
 
     after(() => {
       instrumentation.disable();
+      propagation.disable();
     });
 
     it('should create a rootSpan for GET requests and add propagation headers', async () => {

packages/opentelemetry-instrumentation-xml-http-request/test/xhr.test.ts

@@ -150,6 +150,10 @@ describe('xhr', () => {
         );
       });
 
+      after(() => {
+        api.propagation.disable();
+      });
+
       describe('when request is successful', () => {
         let webTracerWithZone: api.Tracer;
         let webTracerProviderWithZone: WebTracerProvider;