Feature: detect variable assignment outside tests about eslint-plugin-mocha-cleanup HOT 4 CLOSED

Hi, @jasonkarns. It will very useful rule.
Looking forward fot PR. Thanks!

from eslint-plugin-mocha-cleanup.

@jasonkarns, am I right:

var someGlob = require(''); // valid
describe('my cool suite', function () {

  var abc = 1234; // invalid

  beforeEach(function () {
      var b = 124; // valid
      this.a = b; // valid
  });

  it ('my cool test', function () {
      var a = 123; // valid
  });

});

from eslint-plugin-mocha-cleanup.

Yeah, that's what I was intending. Ideally one would minimize the amount of variables set at the very top-level as well (limiting them to requires only), but I don't think that would be relevant for this rule.

from eslint-plugin-mocha-cleanup.

Awesome, thanks!

from eslint-plugin-mocha-cleanup.