Mocha example in README calls fixture.restore() twice?

[klebba]

This is the example in the README:

<script>
describe('<some-element>', function () {
  var someElement;

  beforeEach(function () {
    someElement = fixture('SomeElementFixture');
  });
});
</script>

This ends up calling restore() twice on the fixture, and seemingly creates the fixture again and again.

Would this not be preferable?:

<script>
describe('<some-element>', function () {
  var someElement = fixture('SomeElementFixture');;

  it('do some test', function () {
     // fixture is automatically restored in afterEach
  });
});
</script>

[nallown]

I think that's the idea, that it recreates the fixture every time before a test runs so you'll have a fresh instance to work with.

[klebba]

Why bother to call restore on an instance you're about to ditch?

[nallown]

If you do it beforeEach() then you'll generate a new fixture before each test, you know the it() tests.

So that way you'll have a new fresh element to work with before each test unlike in your second example where you have the same element to use throughout all of the it() tests in that describe() group.

Every test it() group should have a fresh element to work with which your second example wouldn't achieve, it would have the same one to use throughout all of the describe() group.