test: added test for disabling checking of updates during development

Author: shubheksha

src/program.js

@@ -92,7 +92,7 @@ export class Program {
     {
       checkForUpdates = defaultUpdateChecker, systemProcess = process,
       logStream = defaultLogStream, getVersion = defaultVersionGetter,
-      shouldExitProgram = true, localEnv = 'development',
+      shouldExitProgram = true, localEnv = WEBEXT_BUILD_ENV,
     }: Object = {}
   ): Promise<void> {
 

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

@@ -22,7 +22,7 @@ describe('util/updates', () => {
       assert.equal(updateNotifierStub.firstCall.args[0].pkg.name, 'web-ext');
       assert.equal(updateNotifierStub.firstCall.args[0].pkg.version, '1.0.0');
       assert.equal(updateNotifierStub.firstCall.args[0].updateCheckInterval,
-                  1000 * 60 * 60 * 24 * 7);
+                   1000 * 60 * 60 * 24 * 7);
     });
   });
 });

tests/unit/test.program.js

@@ -263,6 +263,22 @@ describe('program.Program', () => {
                     'some-package-version');
       });
   });
+
+  it('does not check for updates during development', () => {
+    const handler = spy();
+    const getVersion = () => 'some-package-version';
+    const checkForUpdates = sinon.stub();
+    const program = new Program(['run'])
+      .command('run', 'some command', handler);
+    return execProgram(program, {
+      checkForUpdates,
+      getVersion,
+      localEnv: 'development',
+    })
+      .then(() => {
+        assert.equal(checkForUpdates.called, false);
+      });
+  });
 });