javascript - Mocha Global Scoping Issues - Stack Overflow

I'm having a big problem with my mocha tests around a global object I'm using. I'm able to produce the following MRE which doesn't give the exact same error, but exemplifies the problematic (buggy?) behavior. Any insight would be much appreciated.

I have the following main.js file in /lib:

exports.exec = function(){
  console.log(test);
}

Then the following in /test/test.js:

var should = require('should');
var main = require('../lib/main');

global.test = {something: 1};

describe('normal test', function(){
  beforeEach(function(){
    global.test = {another: 2};
  }),

  afterEach(function(){
    delete global.test;
  });

  it ('might work with global', function(){
    main.exec();
  })
});

Finally, this is test/test2.js:

var should = require('should');
var main = require('../lib/main');

global.test = {third: 3};

describe('some test', function(){
  it ('messes up global', function(){
    main.exec();
  })
});

I expect that the first test would output {another:2} and the second would print {third: 3}. Indeed, this is the behavior I get when I run each test independently. e.g.

jeff@ubuntu:~/workspace/mocha-test$ mocha test/test2.js 

  { third: 3 }
․

  1 passing (6ms)

However, when running both test with npm packages should and mocha (1.16.1), I get the following output:

jeff@ubuntu:~/workspace/mocha-test$ mocha

  { another: 2 }
․․

  1 passing (6ms)
  1 failing

  1) some test messes up global:
     ReferenceError: test is not defined
      at Object.exports.exec (/home/jeff/workspace/mocha-test/lib/main.js:3:15)
      at Context.<anonymous> (/home/jeff/workspace/mocha-test/test/test2.js:8:10)
      at Test.Runnable.run (/usr/lib/node_modules/mocha/lib/runnable.js:211:32)
      at Runner.runTest (/usr/lib/node_modules/mocha/lib/runner.js:355:10)
      at /usr/lib/node_modules/mocha/lib/runner.js:401:12
      at next (/usr/lib/node_modules/mocha/lib/runner.js:281:14)
      at /usr/lib/node_modules/mocha/lib/runner.js:290:7
      at next (/usr/lib/node_modules/mocha/lib/runner.js:234:23)
      at Object._onImmediate (/usr/lib/node_modules/mocha/lib/runner.js:258:5)
      at processImmediate [as _immediateCallback] (timers.js:330:15)

I'm having a big problem with my mocha tests around a global object I'm using. I'm able to produce the following MRE which doesn't give the exact same error, but exemplifies the problematic (buggy?) behavior. Any insight would be much appreciated.

I have the following main.js file in /lib:

exports.exec = function(){
  console.log(test);
}

Then the following in /test/test.js:

var should = require('should');
var main = require('../lib/main');

global.test = {something: 1};

describe('normal test', function(){
  beforeEach(function(){
    global.test = {another: 2};
  }),

  afterEach(function(){
    delete global.test;
  });

  it ('might work with global', function(){
    main.exec();
  })
});

Finally, this is test/test2.js:

var should = require('should');
var main = require('../lib/main');

global.test = {third: 3};

describe('some test', function(){
  it ('messes up global', function(){
    main.exec();
  })
});

I expect that the first test would output {another:2} and the second would print {third: 3}. Indeed, this is the behavior I get when I run each test independently. e.g.

jeff@ubuntu:~/workspace/mocha-test$ mocha test/test2.js 

  { third: 3 }
․

  1 passing (6ms)

However, when running both test with npm packages should and mocha (1.16.1), I get the following output:

jeff@ubuntu:~/workspace/mocha-test$ mocha

  { another: 2 }
․․

  1 passing (6ms)
  1 failing

  1) some test messes up global:
     ReferenceError: test is not defined
      at Object.exports.exec (/home/jeff/workspace/mocha-test/lib/main.js:3:15)
      at Context.<anonymous> (/home/jeff/workspace/mocha-test/test/test2.js:8:10)
      at Test.Runnable.run (/usr/lib/node_modules/mocha/lib/runnable.js:211:32)
      at Runner.runTest (/usr/lib/node_modules/mocha/lib/runner.js:355:10)
      at /usr/lib/node_modules/mocha/lib/runner.js:401:12
      at next (/usr/lib/node_modules/mocha/lib/runner.js:281:14)
      at /usr/lib/node_modules/mocha/lib/runner.js:290:7
      at next (/usr/lib/node_modules/mocha/lib/runner.js:234:23)
      at Object._onImmediate (/usr/lib/node_modules/mocha/lib/runner.js:258:5)
      at processImmediate [as _immediateCallback] (timers.js:330:15)

• javascript
• node.js
• mocha.js

Share Improve this question Follow asked Dec 23, 2013 at 5:02 Jeff AllenJeff Allen 17.6k99 gold badges5252 silver badges7070 bronze badges 6

• 1 The more I look at it, the more confident I am that this is undesired behavior; posting an issue here: github./visionmedia/mocha/issues/1083 – Jeff Allen Commented Dec 23, 2013 at 5:09
• 1 I ran into this issue before too - I think it's best to make a quick little make file that runs each test independently with a simple for loop. – NG. Commented Dec 23, 2013 at 5:09
• Your issue will probably be closed - I think there was an issue that had something similar. The user wanted the require cache to be cleared. github./visionmedia/mocha/issues/536 – NG. Commented Dec 23, 2013 at 5:12
• 1 What a mess... Wish I had seen this before writing 100 tests in Mocha. – Jeff Allen Commented Dec 23, 2013 at 16:22
• 1 Sorry. Minimal, reproducible example. Maybe niche. – Jeff Allen Commented Jan 30, 2016 at 0:13

 |  Show 1 more ment

1 Answer 1

Sorted by: Reset to default 11

test2.js should be structured like this:

describe('some test', function(){
  before(function (){
    global.test = {third: 3};
  });

  it ('messes up global', function(){
    main.exec();
  })
});

travisjeffery on the GitHub issue mentioned in the ment explains:

mocha loads the files and then runs the suites, to reliably setup your tests the setup should be within the suite.

As @SB points out, this may not be amenable to sharing things like global variables across tests.