fix: make serial mode consistent with parallel

Author: shadowusr

lib/repeater.js

@@ -35,7 +35,8 @@ module.exports = class TestRepeater {
     handleTestEnd(test) {
         this._repeatCounter.testExecuted(test);
 
-        if (!this._parallelRepeats && this._repeatCounter.getRepeatsLeft(test) > 0) {
+        // In parallel mode, we always have 1 (initial run) + repeats test runs. We have + 1 here to account for that initial run
+        if (!this._parallelRepeats && this._repeatCounter.getRepeatsLeft(test) + 1 > 0) {
             this._testplane.addTestToRun(test, test.browserId);
         }
     }

test/lib/repeater.js

@@ -94,7 +94,8 @@ describe('Repeater', () => {
 
         it('should not add test to run when parallelRepeats is false and test has no repeats left', () => {
             const repeatCounter = _mkRepeatCounter();
-            repeatCounter.getRepeatsLeft = sinon.stub().returns(0);
+            // If repeats are set 0 and parallelRepeats is false, getRepeatsLeft would return -1 at runtime
+            repeatCounter.getRepeatsLeft = sinon.stub().returns(-1);
             const repeater = Repeater.create(testplane, repeatCounter, false);
             const test = _mkTest();