test: modify tests on 'hermione.ctx' method

Author: eGavr

test/lib/runner/index.js

@@ -46,6 +46,9 @@ describe('Runner', () => {
         sandbox.stub(MochaRunner.prototype, 'run');
         MochaRunner.prototype.run.returns(q());
 
+        sandbox.stub(MochaRunner, 'init');
+        sandbox.stub(MochaRunner, 'clean');
+
         sandbox.stub(RetryManager.prototype);
     });
 
@@ -68,6 +71,22 @@ describe('Runner', () => {
             assert.calledOnce(RetryManager.prototype.__constructor);
             assert.calledWith(RetryManager.prototype.__constructor, config);
         });
+
+        it('should init mocha runner on RUNNER_START event', () => {
+            const runner = new Runner(makeConfigStub());
+
+            runner.emit(RunnerEvents.RUNNER_START);
+
+            assert.calledOnce(MochaRunner.init);
+        });
+
+        it('should clean mocha runner on RUNNER_END event', () => {
+            const runner = new Runner(makeConfigStub());
+
+            runner.emit(RunnerEvents.RUNNER_END);
+
+            assert.calledOnce(MochaRunner.clean);
+        });
     });
 
     describe('run', () => {

test/lib/runner/mocha-runner/index.js

@@ -26,6 +26,8 @@ describe('mocha-runner', () => {
     const mkMochaAdapterStub_ = () => Object.create(MochaAdapter.prototype);
 
     beforeEach(() => {
+        sandbox.stub(MochaAdapter, 'init');
+        sandbox.stub(MochaAdapter, 'clean');
         sandbox.stub(MochaAdapter.prototype, 'addFiles').returnsThis();
         sandbox.stub(MochaAdapter.prototype, 'attachTestFilter').returnsThis();
         sandbox.stub(MochaAdapter.prototype, 'attachTitleValidator').returnsThis();
@@ -36,6 +38,22 @@ describe('mocha-runner', () => {
 
     afterEach(() => sandbox.restore());
 
+    describe('init', () => {
+        it('should init mocha adapter', () => {
+            MochaRunner.init();
+
+            assert.calledOnce(MochaAdapter.init);
+        });
+    });
+
+    describe('clean', () => {
+        it('should clean mocha adapter', () => {
+            MochaRunner.clean();
+
+            assert.calledOnce(MochaAdapter.clean);
+        });
+    });
+
     describe('run', () => {
         beforeEach(() => sandbox.stub(MochaAdapter, 'create', () => mkMochaAdapterStub_()));
 

test/lib/runner/mocha-runner/mocha-adapter.js

@@ -84,6 +84,26 @@ describe('mocha-runner/mocha-adapter', () => {
 
     afterEach(() => sandbox.restore());
 
+    describe('init', () => {
+        it('should add an empty hermione object to global', () => {
+            MochaAdapter.init();
+
+            assert.deepEqual(global.hermione, {});
+
+            delete global.hermione;
+        });
+    });
+
+    describe('clean', () => {
+        it('should delete hermione from global', () => {
+            global.hermione = 'some-global-value';
+
+            MochaAdapter.clean();
+
+            assert.isUndefined(global.hermione);
+        });
+    });
+
     describe('constructor', () => {
         it('should pass shared opts to mocha instance', () => {
             mkMochaAdapter_({grep: 'foo'});
@@ -138,54 +158,27 @@ describe('mocha-runner/mocha-adapter', () => {
             assert.deepEqual(mocha.files, []);
         });
 
-        it('should add global "hermione" object on "pre-require" event', () => {
-            const mochaAdapter = mkMochaAdapter_();
-
-            mochaAdapter.addFiles(['path/to/file']);
-            MochaStub.prototype.suite.emit('pre-require');
-
-            assert.isDefined(global.hermione);
-        });
-
         describe('hermione global', () => {
-            afterEach(() => delete global.hermione);
+            beforeEach(() => MochaAdapter.init());
+            afterEach(() => MochaAdapter.clean());
 
             it('hermione.skip should return SkipBuilder instance', () => {
-                const mochaAdapter = mkMochaAdapter_();
-
-                mochaAdapter.addFiles(['path/to/file']);
-                MochaStub.prototype.suite.emit('pre-require');
+                mkMochaAdapter_();
 
                 assert.instanceOf(global.hermione.skip, SkipBuilder);
             });
 
             it('hermione.only should return OnlyBuilder instance', () => {
-                const mochaAdapter = mkMochaAdapter_();
-
-                mochaAdapter.addFiles(['path/to/file']);
-                MochaStub.prototype.suite.emit('pre-require');
+                mkMochaAdapter_();
 
                 assert.instanceOf(global.hermione.only, OnlyBuilder);
             });
 
             it('hermione.ctx should return passed ctx', () => {
-                const mochaAdapter = mkMochaAdapter_({}, {some: 'ctx'});
-
-                mochaAdapter.addFiles(['path/to/file.js']);
-                MochaStub.prototype.suite.emit('pre-require');
+                mkMochaAdapter_({}, {some: 'ctx'});
 
                 assert.deepEqual(global.hermione.ctx, {some: 'ctx'});
             });
-
-            it('should remove global "hermione" object on "post-require" event', () => {
-                const mochaAdapter = mkMochaAdapter_();
-
-                mochaAdapter.addFiles(['path/to/file']);
-                MochaStub.prototype.suite.emit('pre-require');
-                MochaStub.prototype.suite.emit('post-require');
-
-                assert.isUndefined(global.hermione);
-            });
         });
     });