Always delete result files after tests

Use rimraf for recursive delete

Author: mohlendo

package.json

@@ -24,7 +24,8 @@
     "expect.js": "0.3.1",
     "mocha": "2.4.5",
     "mocks": "0.0.15",
-    "proxy": "^0.2.4"
+    "proxy": "^0.2.4",
+    "rimraf": "^2.5.4"
   },
   "bugs": "https://github.com/browserstack/browserstack-local-nodejs/issues",
   "homepage": "https://github.com/browserstack/browserstack-local-nodejs",

test/local.js

@@ -2,6 +2,7 @@ var expect = require('expect.js'),
     mocks = require('mocks'),
     path = require('path'),
     fs = require('fs'),
+    rimraf = require('rimraf'),
     Proxy = require('proxy'),
     browserstack = require('../index'),
     LocalBinary = require('../lib/LocalBinary');
@@ -181,15 +182,14 @@ describe('LocalBinary', function () {
   });
 
   afterEach(function () {
-    fs.rmdirSync(tempDownloadPath);
+    rimraf.sync(tempDownloadPath);
   });
 
   it('should download binaries without proxy', function (done) {
     this.timeout(600000);
     var conf = {};
     binary.download(conf, tempDownloadPath, function (result) {
       expect(fs.existsSync(result)).to.equal(true);
-      fs.unlinkSync(result);
       done();
     });
   });
@@ -203,7 +203,6 @@ describe('LocalBinary', function () {
     binary.download(conf, tempDownloadPath, function (result) {
       // test for file existence
       expect(fs.existsSync(result)).to.equal(true);
-      fs.unlinkSync(result);
       done();
     });
   });