Commit 1fc9bd16 by Andrew Nesbitt

Merge pull request #491 from am11/master

Build: Uses different strategy to test bins status
parents 3afe08e5 49e68922
...@@ -34,16 +34,21 @@ if (!force && !process.env.SKIP_NODE_SASS_TESTS) { ...@@ -34,16 +34,21 @@ if (!force && !process.env.SKIP_NODE_SASS_TESTS) {
fs.statSync(path.join(__dirname, 'bin', modPath, 'binding.node')); fs.statSync(path.join(__dirname, 'bin', modPath, 'binding.node'));
console.log('`'+ modPath+ '` exists; testing'); console.log('`'+ modPath+ '` exists; testing');
var total, failures;
var mocha = new Mocha({ var mocha = new Mocha({
reporter: 'dot', reporter: function (stats) {
total = stats.total;
failures = stats.failures;
},
ui: 'bdd', ui: 'bdd',
timeout: 999999 timeout: 999999
}); });
mocha.addFile(path.resolve(__dirname, 'test', 'test.js')); mocha.addFile(path.resolve(__dirname, 'test', 'test.js'));
mocha.run(function (done) { mocha.run(function () {
if (done !== 0) { // at least 90% of tests should pass
if ((total - failures) * 100 / total < 90) {
console.log('Problem with the binary; manual build incoming'); console.log('Problem with the binary; manual build incoming');
console.log('Please consider contributing the release binary to https://github.com/sass/node-sass-binaries for npm distribution.'); console.log('Please consider contributing the release binary to https://github.com/sass/node-sass-binaries for npm distribution.');
build(); build();
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment