Merge pull request #206 from mozilla/app-id-typo

Fix typo on Context.ts

Author: Beatriz Rizental

CHANGELOG.md

@@ -2,6 +2,9 @@
 
 [Full changelog](https://github.com/mozilla/glean.js/compare/v0.8.0...main)
 
+* [#206](https://github.com/mozilla/glean.js/pull/206): BUGFIX: Fix ping URL path.
+  * Application ID was being reporting as `undefined`.
+
 # v0.8.0 (2021-04-13)
 
 [Full changelog](https://github.com/mozilla/glean.js/compare/v0.7.0...v0.8.0)

glean/src/core/context.ts

@@ -79,7 +79,7 @@ export class Context {
     Context.instance._pingsDatabase = db;
   }
 
-  static get applicatinId(): string {
+  static get applicationId(): string {
     return Context.instance._applicationId;
   }
 

glean/tests/core/upload/index.spec.ts

@@ -207,10 +207,12 @@ describe("PingUploader", function() {
     await waitForGleanUploader();
 
     const url = postSpy.firstCall.args[0].split("/");
+    const appId = url[url.length - 4];
     const documentId = url[url.length - 1];
     const headers = postSpy.firstCall.args[2] || {};
 
     assert.strictEqual(documentId, expectedDocumentId);
+    assert.strictEqual(appId, Context.applicationId);
 
     assert.ok("Date" in headers);
     assert.ok("User-Agent" in headers);