fix(api): add public 'fields' function to api.propagator (open-teleme…

…try#1813) * fix(api): add public 'fields' function to api.propagator * chore: remove redundant comment Co-authored-by: Daniel Dyla <[email protected]>
dyladan · Feb 18, 2021 · baad68f · baad68f
1 parent b365b94
commit baad68f
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 3 deletions.
diff --git a/api/src/api/propagation.ts b/api/src/api/propagation.ts
@@ -96,6 +96,13 @@ export class PropagationAPI {
     return this._getGlobalPropagator().extract(context, carrier, getter);
   }
 
+  /**
+   * Return a list of all fields which may be used by the propagator.
+   */
+  public fields(): string[] {
+    return this._getGlobalPropagator().fields();
+  }
+
   /** Remove the global propagator */
   public disable() {
     delete _global[GLOBAL_PROPAGATION_API_KEY];

diff --git a/api/src/context/propagation/TextMapPropagator.ts b/api/src/context/propagation/TextMapPropagator.ts
@@ -67,9 +67,6 @@ export interface TextMapPropagator<Carrier = any> {
 
   /**
    * Return a list of all fields which may be used by the propagator.
-   *
-   * This list should be used to clear fields before calling inject if a carrier is
-   * used more than once.
    */
   fields(): string[];
 }

diff --git a/api/test/api/api.test.ts b/api/test/api/api.test.ts
@@ -150,6 +150,13 @@ describe('API', () => {
         assert.strictEqual(data.carrier, carrier);
         assert.strictEqual(data.getter, getter);
       });
+
+      it('fields', () => {
+        api.propagation.setGlobalPropagator(new TestTextMapPropagation());
+
+        const fields = api.propagation.fields();
+        assert.deepStrictEqual(fields, ['TestField']);
+      });
     });
   });
 });