ilkkao / co-mocha Goto Github PK

/node_modules/mocha-co/lib/runnable.js:9
  , co = require('co');
  ^
SyntaxError: Unexpected token ,

latest package from npm.js. version 1.17.1

There seems to be a conflict in coffee-script tests.

There's one issue not specific to mocha-co but worth being aware of because it complicates things, which is that Coffee will automatically compile functions containing yield into generators - that also breaks the block scope for tests, because it doesn't use a fat arrow. So the @ (this) which would otherwise provide a consistent scope between forEach and it functions is no longer available.

The main problem I've found is that if there's an error in the generator, or even a console log, neither will make it back up to mocha or output in test results. They just seem to be suppressed within the function, so I've found after hours of testing that some predicate tests were actually failing silently, even though they appeared to pass.

Hi,

The semver spec specifies that stuff after a hyphen is a prerelease version:

A pre-release version MAY be denoted by appending a hyphen and a series of dot separated identifiers immediately following the patch version
Precedence refers to how versions are compared to each other when ordered. [...] When major, minor, and patch are equal, a pre-release version has lower precedence than a normal version. Example: 1.0.0-alpha < 1.0.0

Because of this, when trying to install gulp-co-mocha with yarn, as the dependency to this module is defined as ^1.17.1-co.1, it ends up resolving to 1.17.1 which is a broken build, because it considers ^1.17.1-co.1 a pre-release due to the hyphen.

Is it possible to release the current build as 1.17.2 in order to ask the maintainer of gulp-co-mocha to fix it on their layer too?

PS: Before i came here i created an issue on yarn's repo yarnpkg/yarn#1545

Cheers!