Merge pull request #797 from gemini-testing/fix/stats

fix: tests statistic should listen events on main runner

lib/runner/index.js

@@ -24,7 +24,9 @@ module.exports = class TestsRunner extends Runner {
         this.coverage = this.config.isCoverageEnabled() && Coverage.create(config);
 
         this._stateProcessor = stateProcessor;
+
         this._stats = RunnerStats.create();
+        this._stats.attachRunner(this);
 
         this._browserPool = pool.create(this.config, this);
 
@@ -87,7 +89,6 @@ module.exports = class TestsRunner extends Runner {
             Events.WARNING,
             Events.ERROR
         ]);
-        this._stats.attachRunner(runner);
 
         runner.on(Events.END_SUITE, (data) => this._suiteMonitor.suiteFinished(data.suite, data.browserId));
 

test/unit/runner/index.js

@@ -260,6 +260,18 @@ describe('runner', () => {
                     .then(() => assert.equal(testsStatistic.total, 2));
             });
 
+            it('should include tests state data emitted from main runner to the statistic', () => {
+                const runner = createRunner();
+
+                runner.emit(Events.ERROR, makeStateResult({name: 'some-name'}));
+
+                let testsStatistic;
+                runner.on(Events.END, (stat) => testsStatistic = stat);
+
+                return run(runner)
+                    .then(() => assert.equal(testsStatistic.total, 1));
+            });
+
             it('should not be immediately rejected if running of tests in some browser was rejected', () => {
                 const runner = createRunner();
                 const rejected = q.reject();