Disable all tests with --watch

[saper]

Testing cli this way is completely
unreliable. A new method of testing
is needed.

[xzyfer]

👍
On 25 Oct 2015 8:20 am, "Marcin Cieślak" notifications@github.com wrote:

Testing cli this way is completely
unreliable. A new method of testing

is needed.

You can view, comment on, or merge this pull request online at:

#1216
Commit Summary

• Disable all tests with --watch

File Changes

• M test/cli.js
  https://github.com/sass/node-sass/pull/1216/files#diff-0 (11)

Patch Links:

• https://github.com/sass/node-sass/pull/1216.patch
• https://github.com/sass/node-sass/pull/1216.diff

—
Reply to this email directly or view it on GitHub
#1216.

[xzyfer]

Looks like you missed one?

 1) cli "before all" hook for "should return a correct version info":
     Error: timeout of 2000ms exceeded. Ensure the done() callback is being called in this test.
      at null.<anonymous> (C:\projects\node-sass\node_modules\mocha\lib\runnable.js:189:19)

[saper]

Looks like node 5.x has watching broken on my favourite OS nodejs/node#3541

[saper]

this before all hook checking libsass version must be rewritten in an async way. Will fix of course.

[xzyfer]

I believe this has been done.