Un-ignore test.js

[pwnall]

This seems to target lib/test.js, which is definitely necessary.
Not having it causes a require error in lib/interfaces/bdd.js

[tj]

λ mocha (master): git ls-files | grep test.js
lib/test.js

[pwnall]

The file is in the repository, and this setup worked well for me so far.

For some reason, this causes breakage on a co-worker's setup. OSX with node 0.10.17 and the bundled npm 1.3.8, installed from the .pkg on the site. (not via nvm or anything like that) I'm pointing to a git branch of mocha in "dependencies".

The npm docs claim that it uses .gitignore if no .npmignore exists. It seems like the fact that lib/test.js is packaged is a bug, and my co-worker stumbled upon a config where npm actually does what the docs claim it does. I thought it'd be a good idea to not rely on buggy behavior.
https://npmjs.org/doc/developers.html#Keeping-files-out-of-your-package

[tj]

hmm lame, it should have been broken for a longggggg time then :s, mocha needs that file haha

[pwnall]

Yeah, it didn't get broken on my setups. I'm not sure why it broke this one time, but it was a pretty vanilla setup. Want me to investigate further?

[tj]

no big deal anymore since we're using .files in package.json it should be ok if it ever was messed up before