fix: web-ext should look for the extension id also in browser_specific_settings

Author: Lusito

src/util/manifest.js

@@ -15,7 +15,7 @@ const log = createLogger(__filename);
 // getValidatedManifest helper types and implementation
 
 export type ExtensionManifestApplications = {|
-  gecko: {|
+  gecko?: {|
     id?: string,
     strict_min_version?: string,
     strict_max_version?: string,
@@ -28,6 +28,7 @@ export type ExtensionManifest = {|
   version: string,
   default_locale?: string,
   applications?: ExtensionManifestApplications,
+  browser_specific_settings?: ExtensionManifestApplications,
   permissions?: Array<string>,
 |};
 
@@ -85,6 +86,15 @@ export default async function getValidatedManifest(
 
 
 export function getManifestId(manifestData: ExtensionManifest): string | void {
-  return manifestData.applications ?
-    manifestData.applications.gecko.id : undefined;
+  const manifestApps = [
+    manifestData.browser_specific_settings,
+    manifestData.applications,
+  ];
+  for (const apps of manifestApps) {
+    if (apps && apps.gecko && apps.gecko.id) {
+      return apps.gecko.id;
+    }
+  }
+
+  return undefined;
 }

tests/unit/test-cmd/test.sign.js

@@ -279,7 +279,7 @@ describe('sign', () => {
             apiSecret: stubs.signingConfig.apiSecret,
             apiUrlPrefix: stubs.signingConfig.apiUrlPrefix,
             downloadDir: artifactsDir,
-            id: applications.gecko.id,
+            id: applications.gecko && applications.gecko.id,
             timeout: stubs.signingConfig.timeout,
             version: stubs.preValidatedManifest.version,
             xpiPath: stubs.buildResult.extensionPath,

tests/unit/test-extension-runners/test.firefox-desktop.js

@@ -285,6 +285,7 @@ describe('util/extension-runners/firefox-desktop', () => {
     assert.equal(install.asProxy, true);
     assert.equal(install.manifestData.applications.gecko.id,
                  manifestData.applications &&
+                 manifestData.applications.gecko &&
                  manifestData.applications.gecko.id);
     assert.deepEqual(install.profile, fakeProfile);
     // This needs to be the source of the extension.

tests/unit/test-util/test.manifest.js

@@ -211,14 +211,39 @@ describe('util/manifest', () => {
   });
 
   describe('getManifestId', () => {
+    const id = 'basic-manifest@web-ext-test-suite';
+
+    ['applications', 'browser_specific_settings'].forEach((key) => {
+
+      describe(`with ${key}`, () => {
+
+        it('returns gecko.id if present', () => {
+          assert.equal(getManifestId({
+            ...manifestWithoutApps,
+            [key]: basicManifest.applications,
+          }), id);
+        });
+
+        it('returns undefined when gecko does not exist', () => {
+          assert.equal(
+            getManifestId({
+              ...manifestWithoutApps,
+              [key]: {},
+            }),
+            undefined
+          );
+        });
+
+      });
 
-    it('returns a gecko ID', () => {
-      assert.equal(getManifestId(basicManifest),
-                   'basic-manifest@web-ext-test-suite');
     });
 
-    it('returns undefined when ID is not specified', () => {
-      assert.strictEqual(getManifestId(manifestWithoutApps), undefined);
+    describe('without applications and browser_specific_settings', () => {
+
+      it('returns undefined when ID is not specified', () => {
+        assert.strictEqual(getManifestId(manifestWithoutApps), undefined);
+      });
+
     });
 
   });