search

strongloop / strong-supervisor

Application supervisor that automatically adds cluster control and performance monitoring with StrongOps
Other
66 stars 20 forks source link

Cleanup tests #151

Closed rmg closed 9 years ago

rmg commented 9 years ago

The first commit fixes a bug I found while seeing what the tests do when you don't have a license. The third commit actually skips the tests that require a license if there isn't one defined.

Everything else is just cleanup that happened around those. I particularly like the last commit.

rmg commented 9 years ago

Turns out the coverage breaks a lot of things.. not sure how it managed to run for me locally the first few times, but it seems to be failing constantly for me now.

rmg commented 9 years ago

@slnode test please

rmg commented 9 years ago

@slnode test please

rmg commented 9 years ago

Much better. The full npm test runs in about 2 minutes for me and for CI, on a clean checkout.

@sam-github PTAL

sam-github commented 9 years ago

LGTM, except for the error message format.

rmg commented 9 years ago

@slnode test please

rmg commented 9 years ago

@slnode test please