chore: enable metadata tests for flagd #1290 (#1431)

Signed-off-by: Indermohan Singh <indermohansinghk7@gmail.com>
Signed-off-by: Todd Baert <todd.baert@dynatrace.com>
Co-authored-by: Todd Baert <todd.baert@dynatrace.com>

Author: imsingh

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

@@ -73,10 +73,13 @@ export const flagSteps: Steps =
     });
 
     then('the resolved metadata should contain', (table) => {
-      // TODO: implement metadata tests, https://github.com/open-feature/js-sdk-contrib/issues/1290
+      table.forEach((row: { key: string; metadata_type: string; value: string }) => {
+        const expectedValue = mapValueToType(row.value, row.metadata_type);
+        expect(state.details?.flagMetadata).toHaveProperty(row.key, expectedValue);
+      });
     });
 
     then('the resolved metadata is empty', () => {
-      // TODO: implement metadata tests, https://github.com/open-feature/js-sdk-contrib/issues/1290
+      expect(state.details?.flagMetadata).toEqual({});
     });
   };

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

@@ -73,6 +73,10 @@ export const providerSteps: Steps =
           flagdOptions['port'] = container.getPort(state.resolverType);
           type = 'sync-payload';
           break;
+        case 'metadata':
+          flagdOptions['port'] = container.getPort(state.resolverType);
+          type = 'metadata';
+          break;
         default:
           throw new Error('unknown provider type: ' + providerType);
       }

libs/providers/flagd/src/e2e/tests/in-process.spec.ts

@@ -23,7 +23,7 @@ describe('in-process', () => {
     loadFeatures(GHERKIN_FLAGD, {
       // remove filters as we add support for features
       // see: https://github.com/open-feature/js-sdk-contrib/issues/1096 and child issues
-      tagFilter: '@in-process and not @targetURI and not @sync and not @metadata and not @unixsocket',
+      tagFilter: '@in-process and not @targetURI and not @sync and not @unixsocket',
       scenarioNameTemplate: (vars) => {
         return `${vars.scenarioTitle} (${vars.scenarioTags.join(',')} ${vars.featureTags.join(',')})`;
       },

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

@@ -24,7 +24,7 @@ describe('rpc', () => {
       tagFilter:
         // remove filters as we add support for features
         // see: https://github.com/open-feature/js-sdk-contrib/issues/1096 and child issues
-        '@rpc and not @targetURI and not @metadata and not @caching and not @unixsocket',
+        '@rpc and not @targetURI and not @caching and not @unixsocket',
       scenarioNameTemplate: (vars) => {
         return `${vars.scenarioTitle} (${vars.scenarioTags.join(',')} ${vars.featureTags.join(',')})`;
       },