Move the config feature unit test logic to configuration.spec

* Removed the config.spec.ts from the e2e suite and moved the logic to configuration.spec.ts unit test
* Restored env vars safely by deleting only the added keys

Signed-off-by: Marko Mlakar <[email protected]>

Author: MarkoMlakar

libs/providers/flagd/src/e2e/step-definitions/configSteps.ts

@@ -20,7 +20,9 @@ export const configSteps: Steps = (state: State) => {
       state.options = {};
       state.config = undefined;
       state.events = [];
-      Object.keys(process.env).forEach((key) => delete process.env[key]);
+      Object.keys(process.env)
+        .filter((key) => !Object.prototype.hasOwnProperty.call(originalEnv, key))
+        .forEach((key) => delete process.env[key]);
       Object.assign(process.env, originalEnv);
     });
 

libs/providers/flagd/src/e2e/tests/config.spec.ts

@@ -2,6 +2,10 @@ import type { Config, FlagdProviderOptions } from './configuration';
 import { getConfig } from './configuration';
 import { DEFAULT_MAX_CACHE_SIZE } from './constants';
 import type { EvaluationContext } from '@openfeature/server-sdk';
+import { configSteps } from '../e2e/step-definitions/configSteps';
+import type { State } from '../e2e/step-definitions/state';
+import { autoBindSteps, loadFeatures } from 'jest-cucumber';
+import { CONFIG_FEATURE } from '../e2e';
 
 describe('Configuration', () => {
   const OLD_ENV = process.env;
@@ -163,4 +167,23 @@ describe('Configuration', () => {
       });
     });
   });
+
+  describe('config.feature', () => {
+    const state: State = {
+      resolverType: 'in-process',
+      options: {},
+      config: undefined,
+      events: [],
+    };
+
+    autoBindSteps(
+      loadFeatures(CONFIG_FEATURE, {
+        scenarioNameTemplate: (vars) => {
+          const tags = [...new Set([...vars.scenarioTags, ...vars.featureTags])];
+          return `${vars.scenarioTitle}${tags.length > 0 ? ` (${tags.join(', ')})` : ''}`;
+        },
+      }),
+      [configSteps(state)],
+    );
+  });
 });