Mocha Spec Reporter

[skbolton]

I feel like this might be a little OCD but I have noticed something weird when using testdouble and mocha. I am going to put an excerpt of one of my tests.

before(() => { td.replace(console, 'log') })
after(() => { td.reset() })

it('should console.log the state of Model', () => {
      Opportunity
        .fields('.', 'child.name')
        .where({
          id: 5
        })

      Opportunity.explain()

      const callCount = td.explain(console.log).callCount
      expect(callCount).to.equal(1)
    })

The test passes but the spec reporter doesn't show one of the check marks

I know things are technically working because if I change something in the test to make it break it does cause Mocha to fail.

PS. For what it is worth I love your explain functionality of testdouble and I have a feature in the library I am creating that shows off the influence. 😄

[searls]

I suspect mocking console.log is not going to end well for you, since lots of other libraries (including Mocha) will call console.log. It's likely that's why you get two instead of 1.

I recommend wrapping console.log to your own logger and mocking that. See this article. That's what we do in this lib here

[skbolton]

I am totally with you and I just want to point out that I took a photo of that failed test to prove I can make the test fail. In the first photo the tests are 'passing' but no check marks are showing up in the reporter.
Maybe I will do a test not mocking console.log and see if I can get the same behavior

[searls]

I'm sorry, I do not understand what your issue is. I don't know what the checkmarks you're looking for are.

[skbolton]

To my earlier comment I will mock something other than console.log and see if I can get the same behavior. In the explain test suite I have 3 tests that use testdouble. They are all passing but not showing up in the reporter.
They are still counting as a passed test just not showing up. Weird and nitpicky but I was just seeing if it was something you guys have encountered before.

[searls]

Could you please provide a minimal example project repo

[skbolton]

Yeah I will make one and follow up if needed. Thanks for your time!