Updated, now removing the whole tmp directory.


On Mon, Jan 27, 2014 at 6:18 PM, Calvin Metcalf <notifications@github.com>wrote:

> In bin/test-node.js:
>
> > -    process.exit(1);
> > -    return;
> > -  } else if (result.failed) {
> > -    console.log('[node] failed ' + result.failed + ' out of ' + result.assertions + ' tests')
> > -    process.exit(2);
> > -    return;
> > -  }
> > +function cleanup() {
> > +  var dirs = ['testdb*', '_pouch__allDbs*'].map(function (dir) {
> > +    return path.join(testsDir, dir);
> > +  }).join(' ');
> > +
> > +  // Remove test databases and test allDbs database.
> > +  exec('rm -r ' + dirs, function () {
> > +    // Remove the tests directory if empty
> > +    exec('rm -d' + testsDir);
>
> this never runs, @daleharvey <https://github.com/daleharvey> can correct
> me if I'm wrong, but I believe tmp was in the .gitignore due to the old
> grunt build step that we no longer used and a rm -r ./tmp shouldn't cause
> any issues.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/daleharvey/pouchdb/pull/1245/files#r9187322>
> .
>



-- 
Thanks
Rohan Jain