fix: Make header required and use it in integration tests

peerigon · May 7, 2021 · a028311 · a028311
1 parent dce2b7e
commit a028311
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/src/internals/utilities/api.ts b/src/internals/utilities/api.ts
@@ -29,7 +29,7 @@ export type Authentication = {
 };
 
 export type Config = {
-    appIdentifier?: string;
+    appIdentifier: string;
     authentication?: Authentication;
     baseUrl?: string;
 };
@@ -41,8 +41,12 @@ export class Api {
         },
     });
 
-    constructor({ baseUrl = DEFAULT_BASE_URL, authentication }: Config) {
-        this.config({ authentication, baseUrl });
+    constructor({
+        baseUrl = DEFAULT_BASE_URL,
+        authentication,
+        appIdentifier,
+    }: Config) {
+        this.config({ appIdentifier, authentication, baseUrl });
     }
 
     config = (config: Config) => {

diff --git a/tests/integration/integration.test.ts b/tests/integration/integration.test.ts
@@ -27,6 +27,7 @@ const hasCredentials =
                 user: process.env.CLOCKODO_USER!,
                 apiKey: process.env.CLOCKODO_API_KEY!,
             },
+            appIdentifier: "Clockodo-SDK Integration Test"
         });
     });
-Original file line number
+Diff line change
@@ Expand Up / @@ -27,6 +27,7 @@ const hasCredentials = @@
                     user: process.env.CLOCKODO_USER!,
                     apiKey: process.env.CLOCKODO_API_KEY!,
                 },
+                appIdentifier: "Clockodo-SDK Integration Test"
             });
         });
@@ Expand Down @@