From 538737ac89fb727501e14607e92ddf7aeb0f633b Mon Sep 17 00:00:00 2001
From: Andrew Joslin <andytjoslin@gmail.com>
Date: Wed, 4 Jun 2014 11:02:25 -0600
Subject: [PATCH] chore(ionic-snapshot): make it so screenshots are posting
 asynchronously

---
 config/lib/ionic-snapshot.js | 35 ++++++++++++++++++++++++++++-------
 config/protractor.conf.js    |  2 +-
 2 files changed, 29 insertions(+), 8 deletions(-)

diff --git a/config/lib/ionic-snapshot.js b/config/lib/ionic-snapshot.js
index 951d8e903d..49e2742d13 100755
--- a/config/lib/ionic-snapshot.js
+++ b/config/lib/ionic-snapshot.js
@@ -1,3 +1,4 @@
+var q = require('q');
 
 var IonicSnapshot = function(options) {
 
@@ -22,6 +23,7 @@ var IonicSnapshot = function(options) {
     self.width = browser.params.width || -1;
     self.height = browser.params.height || -1;
     self.highestMismatch = 0;
+    self.screenshotRequestPromises = [];
 
     self.flow = protractor.promise.controlFlow();
 
@@ -48,7 +50,6 @@ var IonicSnapshot = function(options) {
         };
       });
     });
-
     process.on('exit', function() {
       log(colors.green('Highest Mismatch:'), self.highestMismatch, '%');
     });
@@ -61,7 +62,7 @@ var IonicSnapshot = function(options) {
 
     if(!self.testData.total_specs) {
       self.testData.total_specs = 0;
-      var allSpecs = jasmine.getEnv().currentRunner().specs()
+      var allSpecs = jasmine.getEnv().currentRunner().specs();
       for(var sId in allSpecs) {
         self.testData.total_specs++;
       }
@@ -75,7 +76,8 @@ var IonicSnapshot = function(options) {
         browser.sleep(self.sleepBetweenSpecs).then(function(){
 
           browser.takeScreenshot().then(function(pngBase64){
-            log('spec:', spec.id + 1, 'of', self.testData.total_specs);
+            var specIdString = '[' + (spec.id+1) + '/' + self.testData.total_specs + ']';
+            log(specIdString, spec.getFullName());
 
             self.testData.spec_id = spec.id;
             self.testData.description = spec.getFullName();
@@ -83,22 +85,25 @@ var IonicSnapshot = function(options) {
             self.testData.png_base64 = pngBase64;
             pngBase64 = null;
 
+            var requestDeferred = q.defer();
+            self.screenshotRequestPromises.push(requestDeferred.promise);
+
             request.post(
               'http://' + self.domain + '/screenshot',
               { form: self.testData },
               function (error, response, body) {
-                log('reportSpecResults:', body);
+                log(specIdString, 'reportSpecResults:', body);
                 try {
                   var rspData = JSON.parse(body);
                   self.highestMismatch = Math.max(self.highestMismatch, rspData.Mismatch);
                 } catch(e) {
-                  log(colors.red('reportSpecResults error posting screenshot:'), e);
+                  log(specIdString, colors.red('reportSpecResults', 'error posting screenshot:'), e);
                 }
-                d.fulfill();
+                requestDeferred.resolve();
               }
             );
+            d.fulfill();
           });
-
         });
 
       });
@@ -107,6 +112,22 @@ var IonicSnapshot = function(options) {
     });
   };
 
+  IonicReporter.prototype.reportRunnerResults = function() {
+    var self = this;
+
+    self.flow.execute(function() {
+      var d = protractor.promise.defer();
+      log('Waiting for all screenshots to be posted...');
+      // allSettled waits until all the promises are done, whether they are rejected or resolved
+      q.allSettled(self.screenshotRequestPromises).then(function(all) {
+        d.fulfill();
+        log('Finished!');
+      });
+    });
+  };
+
+
+
   this.jasmine.getEnv().addReporter( new IonicReporter(options) );
 
 };
diff --git a/config/protractor.conf.js b/config/protractor.conf.js
index b66b215a23..548ab95495 100644
--- a/config/protractor.conf.js
+++ b/config/protractor.conf.js
@@ -10,7 +10,7 @@ exports.config = {
   jasmineNodeOpts: {
     showColors: true, // Use colors in the command line report.
     defaultTimeoutInterval: 120000,
-    isVerbose: true
+    // isVerbose: true
   },
 
   baseUrl: 'http://localhost:' + buildConfig.protractorPort,