refs #215 - attempting to fix `--fail-fast` again

@@ -48,16 +48,19 @@ exports.create = function create(casper, options) {
  */
  */
 var Tester = function Tester(casper, options) {
 var Tester = function Tester(casper, options) {
     "use strict";
     "use strict";
-    /*jshint maxstatements:20*/
+    /*jshint maxstatements:30*/
 
 
     if (!utils.isCasperObject(casper)) {
     if (!utils.isCasperObject(casper)) {
         throw new CasperError("Tester needs a Casper instance");
         throw new CasperError("Tester needs a Casper instance");
     }
     }
 
 
+    var self = this;
+
     this.casper = casper;
     this.casper = casper;
 
 
     this.SKIP_MESSAGE = '__termination__';
     this.SKIP_MESSAGE = '__termination__';
 
 
+    this.aborted = false;
     this.executed = 0;
     this.executed = 0;
     this.currentTestFile = null;
     this.currentTestFile = null;
     this.currentSuiteNum = 0;
     this.currentSuiteNum = 0;
@@ -141,10 +144,10 @@ var Tester = function Tester(casper, options) {
         if (!phantom.casperTest) {
         if (!phantom.casperTest) {
             return;
             return;
         }
         }
-        if (msg === this.test.SKIP_MESSAGE) {
-            this.warn(f('--fail-fast: aborted remaining tests in "%s"', this.test.currentTestFile));
-            this.test.currentSuiteNum++;
-            return this.test.done();
+        if (msg === self.SKIP_MESSAGE) {
+            this.warn(f('--fail-fast: aborted remaining tests in "%s"', self.currentTestFile));
+            self.aborted = true;
+            return self.done();
         }
         }
         var line = 0;
         var line = 0;
         if (!utils.isString(msg)) {
         if (!utils.isString(msg)) {
@@ -152,15 +155,15 @@ var Tester = function Tester(casper, options) {
                 line = backtrace[0].line;
                 line = backtrace[0].line;
             } catch (e) {}
             } catch (e) {}
         }
         }
-        this.test.uncaughtError(msg, this.test.currentTestFile, line);
-        this.test.done();
+        self.uncaughtError(msg, self.currentTestFile, line);
+        self.done();
     });
     });
 
 
     this.casper.on('step.error', function onStepError(e) {
     this.casper.on('step.error', function onStepError(e) {
-        if (e.message !== this.test.SKIP_MESSAGE) {
-            this.test.uncaughtError(e, this.test.currentTestFile);
+        if (e.message !== self.SKIP_MESSAGE) {
+            self.uncaughtError(e, self.currentTestFile);
         }
         }
-        this.test.done();
+        self.done();
     });
     });
 };
 };
 
 
@@ -1076,10 +1079,11 @@ Tester.prototype.runSuites = function runSuites() {
         if (self.running) {
         if (self.running) {
             return;
             return;
         }
         }
-        if (self.currentSuiteNum === testFiles.length) {
+        if (self.currentSuiteNum === testFiles.length || self.aborted) {
             self.emit('tests.complete');
             self.emit('tests.complete');
             clearInterval(interval);
             clearInterval(interval);
             self.exporter.setSuiteDuration(self.calculateSuiteDuration());
             self.exporter.setSuiteDuration(self.calculateSuiteDuration());
+            self.aborted = false;
         } else {
         } else {
             self.runTest(testFiles[self.currentSuiteNum]);
             self.runTest(testFiles[self.currentSuiteNum]);
             self.exporter.setSuiteDuration(self.calculateSuiteDuration());
             self.exporter.setSuiteDuration(self.calculateSuiteDuration());