{"action":"created","issue":{"url":"https://api.github.com/repos/HDF-NI/hdf5.node/issues/24","repository_url":"https://api.github.com/repos/HDF-NI/hdf5.node","labels_url":"https://api.github.com/repos/HDF-NI/hdf5.node/issues/24/labels{/name}","comments_url":"https://api.github.com/repos/HDF-NI/hdf5.node/issues/24/comments","events_url":"https://api.github.com/repos/HDF-NI/hdf5.node/issues/24/events","html_url":"https://github.com/HDF-NI/hdf5.node/issues/24","id":144530997,"number":24,"title":"`npm test` and `mocha test` fail","user":{"login":"shumway","id":1060709,"avatar_url":"https://avatars.githubusercontent.com/u/1060709?v=3","gravatar_id":"","url":"https://api.github.com/users/shumway","html_url":"https://github.com/shumway","followers_url":"https://api.github.com/users/shumway/followers","following_url":"https://api.github.com/users/shumway/following{/other_user}","gists_url":"https://api.github.com/users/shumway/gists{/gist_id}","starred_url":"https://api.github.com/users/shumway/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/shumway/subscriptions","organizations_url":"https://api.github.com/users/shumway/orgs","repos_url":"https://api.github.com/users/shumway/repos","events_url":"https://api.github.com/users/shumway/events{/privacy}","received_events_url":"https://api.github.com/users/shumway/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2016-03-30T10:03:05Z","updated_at":"2016-03-30T12:27:46Z","closed_at":null,"body":"### Problem\r\n\r\n```\r\n$ mocha test\r\nmodule.js:328\r\n    throw err;\r\n    ^\r\n\r\nError: Cannot find module './build/Release/hdf5'\r\n```\r\n\r\n### Issues\r\n\r\n- I am probably doing something wrong, but I can't find documentation.\r\n- I know `hdf5` is installed correctly because I can use it directly from the node REPL.\r\n- If I only `sudo npm install -g` then `npm test` gives an error about misiing co-mocha, which is in the package.json devDependencies. Doing `npm install` local doesn't work, but it does install the co-mocha dependency.\r\n- The testing strategy and test status are not documented anywhere that I could find. I'd like to add that to the documentation pages and refer to it on the getting started / install landing page.\r\n- Do we want to add continuous integration on <https://travis-ci.org> and a test status badge on the readme.md?\r\n\r\nI can work on this, but some feedback from current developers on the state of the test suite would be helpful. Any suggestions or requests regarding testing are welcome."},"comment":{"url":"https://api.github.com/repos/HDF-NI/hdf5.node/issues/comments/203406900","html_url":"https://github.com/HDF-NI/hdf5.node/issues/24#issuecomment-203406900","issue_url":"https://api.github.com/repos/HDF-NI/hdf5.node/issues/24","id":203406900,"user":{"login":"shumway","id":1060709,"avatar_url":"https://avatars.githubusercontent.com/u/1060709?v=3","gravatar_id":"","url":"https://api.github.com/users/shumway","html_url":"https://github.com/shumway","followers_url":"https://api.github.com/users/shumway/followers","following_url":"https://api.github.com/users/shumway/following{/other_user}","gists_url":"https://api.github.com/users/shumway/gists{/gist_id}","starred_url":"https://api.github.com/users/shumway/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/shumway/subscriptions","organizations_url":"https://api.github.com/users/shumway/orgs","repos_url":"https://api.github.com/users/shumway/repos","events_url":"https://api.github.com/users/shumway/events{/privacy}","received_events_url":"https://api.github.com/users/shumway/received_events","type":"User","site_admin":false},"created_at":"2016-03-30T12:27:46Z","updated_at":"2016-03-30T12:27:46Z","body":"The advantages I've seen for continuous integration on travis-ci:\r\n\r\n- It's a backup check on the test status with multiple developers (not crucial right now)\r\n- It documents the build instructions for a new system. I've found this very useful for C++ packages with dependencies on linux distros, but it's not as crucial with npm modules. The HDF5 dependency and different build instructions for mac, windows, and linux may make this worthwhile for hdf5.node.\r\n- It may be useful to check for regressions if we support several node versions.\r\n- It can be useful for generating distributable binaries. Not sure how this complements or duplicates AWS.\r\n- New users may like the peace of mind seeing “build:passing” at the top of the README, with a link to the latest travis-ci build / test log. I find it's a nice sanity check if I'm having trouble installing a package on my machine.\r\n\r\nNo urgency, just offering to add travis-ci support if/when we want it.\r\n"}}