mocha > 2.4.5 fails with reporter xunit about grunt-mocha HOT 10 CLOSED

@omatic-hacker Here's a patch code I wrote for my project, and how to use it.

from grunt-mocha.

I’m seeing the same issue with XUnit failing due to isPending(). I originally reported it with mochajs (see referenced issue above), but it appears to originate with grunt-mocha.

I have some sample code that recreates this issue.

Mocha XUnit Reporter Sample
https://github.com/matthewriley/mocha-xunit-reporter-sample

from grunt-mocha.

Furthermore, it's possible to run the exact same test with "grunt-mocha-test" or "grunt-simple-mocha" and have the XUnit reporter work fine, but with "grunt-mocha" the isPending function is mysteriously missing when the test is sent to the XUnit reporter.

from grunt-mocha.

I found the root suit (for example, this) is not instanceof "mocha/lib/suite".

so isPending is not on its prototype chain. it this the root cause?

from grunt-mocha.

That function is not inheriting Suite because it is not a suite constructor; in fact, it's not even creating the root suite: it takes an existing suite as a parameter.

(You can also add console.log calls to the XUnit reporter to see that the tests are missing the isPending function.)

Additionally, the problem does not occur with Mocha itself and not with some other integrations either, so what we really need to determine is what Grunt-Mocha is doing differently.

from grunt-mocha.

Sorry I didn't state it clear, I mean that the parameter itself is the root suite, not the function.

from grunt-mocha.

Is Grunt-Mocha creating a root suite of its own without it inheriting Mocha's Suite? As far as I can tell, Mocha is creating the root suite here: https://github.com/mochajs/mocha/blob/master/lib/mocha.js#L85 using the Suite object from here: https://github.com/mochajs/mocha/blob/master/lib/mocha.js#L41

from grunt-mocha.

Hi guys, is there any workarounds for this bug? I would like to use grunt-mocha on my client side and generate xUnit reports, but always got: Fatal error: test.isPending is not a function

@kmiyashiro is the project is still maintained?

I really appreciate any help you can provide.

from grunt-mocha.

@cades Thanks, I will try your patch!

from grunt-mocha.

@omatic-hacker I fixed this problem in my fork of grunt-mocha-blanket(https://github.com/eugenet8k/grunt-mocha-blanket) which could be used instead of grunt-mocha. It's basically the same thing but with built-in blanketjs coverage support. Idk, maybe it will save you some time.

from grunt-mocha.