test(opentelemetry-configuration): ensure process.env is cleaned up after envVariableSubstitution() tests (#6026)

Author: cjihrig

experimental/CHANGELOG.md

@@ -33,6 +33,7 @@ For notes on migrating to 2.x / 0.200.x see [the upgrade guide](doc/upgrade-to-2
 * test(opentelemetry-configuration): preserve special process.env behavior [#6010](https://github.com/open-telemetry/opentelemetry-js/pull/6010) @cjihrig
 * test(sdk-logs): ensure process.env is cleaned up between tests [#6017](https://github.com/open-telemetry/opentelemetry-js/pull/6017) @cjihrig
 * test(otlp-grpc-exporter-base): remove duplicated delete statements [#6022](https://github.com/open-telemetry/opentelemetry-js/pull/6022) @cjihrig
+* test(opentelemetry-configuration): ensure process.env is cleaned up after envVariableSubstitution tests [#6026](https://github.com/open-telemetry/opentelemetry-js/pull/6026) @cjihrig
 
 ## 0.206.0
 

experimental/packages/opentelemetry-configuration/test/utils.test.ts

@@ -145,40 +145,47 @@ describe('config utils', function () {
     ]);
   });
 
-  it('should return correct values for envVariableSubstitution', function () {
-    process.env.TEST1 = 't1';
-    process.env.TEST2 = 't2';
-    process.env.TEST_LONG_NAME = '100';
-    process.env.TEST_ENDPOINT = 'http://test.com:4318/v1/traces';
-    assert.deepStrictEqual(envVariableSubstitution(null), undefined);
-    assert.deepStrictEqual(envVariableSubstitution(' '), ' ');
-    assert.deepStrictEqual(envVariableSubstitution('${TEST1}'), 't1');
-    assert.deepStrictEqual(
-      envVariableSubstitution('${TEST1},${TEST2}'),
-      't1,t2'
-    );
-    assert.deepStrictEqual(envVariableSubstitution('${TEST_LONG_NAME}'), '100');
-    assert.deepStrictEqual(envVariableSubstitution('${TEST3}'), '');
-    assert.deepStrictEqual(
-      envVariableSubstitution('${TEST3:-backup}'),
-      'backup'
-    );
-    assert.deepStrictEqual(
-      envVariableSubstitution(
-        '${TEST_ENDPOINT:-http://localhost:4318/v1/traces}'
-      ),
-      'http://test.com:4318/v1/traces'
-    );
-    assert.deepStrictEqual(
-      envVariableSubstitution(
-        '${TEST_NON_EXISTING:-http://localhost:4318/v1/traces}'
-      ),
-      'http://localhost:4318/v1/traces'
-    );
+  describe('envVariableSubstitution()', function () {
+    afterEach(function () {
+      delete process.env.TEST1;
+      delete process.env.TEST2;
+      delete process.env.TEST_LONG_NAME;
+      delete process.env.TEST_ENDPOINT;
+    });
 
-    delete process.env.TEST1;
-    delete process.env.TEST2;
-    delete process.env.TEST_LONG_NAME;
-    delete process.env.TEST_ENDPOINT;
+    it('should return correct values for envVariableSubstitution', function () {
+      process.env.TEST1 = 't1';
+      process.env.TEST2 = 't2';
+      process.env.TEST_LONG_NAME = '100';
+      process.env.TEST_ENDPOINT = 'http://test.com:4318/v1/traces';
+      assert.deepStrictEqual(envVariableSubstitution(null), undefined);
+      assert.deepStrictEqual(envVariableSubstitution(' '), ' ');
+      assert.deepStrictEqual(envVariableSubstitution('${TEST1}'), 't1');
+      assert.deepStrictEqual(
+        envVariableSubstitution('${TEST1},${TEST2}'),
+        't1,t2'
+      );
+      assert.deepStrictEqual(
+        envVariableSubstitution('${TEST_LONG_NAME}'),
+        '100'
+      );
+      assert.deepStrictEqual(envVariableSubstitution('${TEST3}'), '');
+      assert.deepStrictEqual(
+        envVariableSubstitution('${TEST3:-backup}'),
+        'backup'
+      );
+      assert.deepStrictEqual(
+        envVariableSubstitution(
+          '${TEST_ENDPOINT:-http://localhost:4318/v1/traces}'
+        ),
+        'http://test.com:4318/v1/traces'
+      );
+      assert.deepStrictEqual(
+        envVariableSubstitution(
+          '${TEST_NON_EXISTING:-http://localhost:4318/v1/traces}'
+        ),
+        'http://localhost:4318/v1/traces'
+      );
+    });
   });
 });