Merge pull request #31 from mozilla-b2g/bail-if-thing-exists

Bug 1177833 - Gaia shouldn't download b2g always r=aus

src/main.js

@@ -4,6 +4,7 @@ import detectOS from './detectos';
 import detectURL from './detecturl';
 import download from './download';
 import extract from './extract';
+import { readdir } from 'mz/fs';
 import * as buildinfo from './moz_build_info';
 
 debug = debug('mozilla-download/main');
@@ -47,6 +48,18 @@ parser.addArgument(['dest'], {
 
 export default async function main(args=parser.parseArgs()) {
   try {
+    // Bail if thing exists
+    try {
+      let contents = await readdir(args.dest);
+      if (contents && contents.length) {
+        // We have dest dir and it has contents
+        debug('Found b2g at dest', args.dest);
+        return;
+      }
+    } catch (error) {
+    }
+
+    debug('No b2g found. Will download to', args.dest);
     let url = await detectURL(args);
     debug('Artifact url', url);
     let path = await download(url, args);

test/fixtures/b2g/random

@@ -0,0 +1 @@
+wha

test/main_test.js

@@ -6,6 +6,18 @@ import { tempdir } from '../src/temp';
 
 suite('main', function() {
   let cases = [
+    {
+      name: 'already exists',
+      args: {
+        product: 'b2g-desktop',
+        os: 'linux-x86_64',
+        branch: 'mozilla-central',
+        dest: `${__dirname}/fixtures/b2g`
+      },
+      verify: function() {
+        assert.deepEqual(fs.readdirSync(`${__dirname}/fixtures/b2g`), ['random']);
+      }
+    },
     {
       name: 'mozilla-central linux-x86_64 b2g-desktop opt',
       args: {
@@ -97,7 +109,7 @@ suite('main', function() {
       }
 
       let dest = await tempdir();
-      testCase.args.dest = dest;
+      testCase.args.dest = testCase.args.dest || dest;
       await main(testCase.args);
       testCase.verify();
     });