{"action":"created","issue":{"url":"https://api.github.com/repos/nodejs/node/issues/7867","repository_url":"https://api.github.com/repos/nodejs/node","labels_url":"https://api.github.com/repos/nodejs/node/issues/7867/labels{/name}","comments_url":"https://api.github.com/repos/nodejs/node/issues/7867/comments","events_url":"https://api.github.com/repos/nodejs/node/issues/7867/events","html_url":"https://github.com/nodejs/node/pull/7867","id":167343836,"number":7867,"title":"[WIP] Update test-npm target","user":{"login":"gibfahn","id":15943089,"avatar_url":"https://avatars.githubusercontent.com/u/15943089?v=3","gravatar_id":"","url":"https://api.github.com/users/gibfahn","html_url":"https://github.com/gibfahn","followers_url":"https://api.github.com/users/gibfahn/followers","following_url":"https://api.github.com/users/gibfahn/following{/other_user}","gists_url":"https://api.github.com/users/gibfahn/gists{/gist_id}","starred_url":"https://api.github.com/users/gibfahn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gibfahn/subscriptions","organizations_url":"https://api.github.com/users/gibfahn/orgs","repos_url":"https://api.github.com/users/gibfahn/repos","events_url":"https://api.github.com/users/gibfahn/events{/privacy}","received_events_url":"https://api.github.com/users/gibfahn/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/nodejs/node/labels/in%20progress","name":"in progress","color":"ededed"},{"url":"https://api.github.com/repos/nodejs/node/labels/npm","name":"npm","color":"fad8c7"},{"url":"https://api.github.com/repos/nodejs/node/labels/test","name":"test","color":"990099"},{"url":"https://api.github.com/repos/nodejs/node/labels/tools","name":"tools","color":"d4c5f9"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":5,"created_at":"2016-07-25T11:23:52Z","updated_at":"2016-07-26T08:00:46Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/nodejs/node/pulls/7867","html_url":"https://github.com/nodejs/node/pull/7867","diff_url":"https://github.com/nodejs/node/pull/7867.diff","patch_url":"https://github.com/nodejs/node/pull/7867.patch"},"body":"##### Checklist\r\n\r\n- [ ] `make -j4 test` (UNIX), or `vcbuild test nosign` (Windows) passes\r\n- [x] tests and/or benchmarks are included\r\n- [ ] documentation is changed or added\r\n- [x] commit message follows commit guidelines\r\n\r\n##### Affected core subsystem(s)\r\ntest, npm\r\n\r\n\r\n##### Description of change\r\nUpdate the test-npm script to generate tap output, and add a windows version. \r\n\r\nRef: https://github.com/nodejs/build/issues/317\r\n\r\n#### Known Issues\r\n`test-npm` is currently failing due to https://github.com/npm/npm/issues/13457 which seems to be caused by https://github.com/nodejs/node/pull/7168 (see https://github.com/nodejs/node/pull/7168#issuecomment-234679659 for more info). I've added https://github.com/gibfahn/node/tree/rebased-test-npm-fix which is the rebased version of this branch. If you want to test this PR out, build from that branch.\r\n\r\nI've also seen failures due to https://github.com/npm/npm/issues/12220.\r\n\r\n#### Manual testing\r\n\r\nBuild [gibfahn:rebased-test-npm-fix](https://github.com/gibfahn/node/tree/rebased-test-npm-fix) to get the #7168 revert as well.\r\n\r\nOn windows, do `set debug=1` to print the TAP output to stdout, otherwise it gets redirected to test-npm.tap (I haven't found a `tee` equivalent for batch).\r\n\r\n\r\ncc @TheAlphaNerd "},"comment":{"url":"https://api.github.com/repos/nodejs/node/issues/comments/235190990","html_url":"https://github.com/nodejs/node/pull/7867#issuecomment-235190990","issue_url":"https://api.github.com/repos/nodejs/node/issues/7867","id":235190990,"user":{"login":"Fishrock123","id":1093990,"avatar_url":"https://avatars.githubusercontent.com/u/1093990?v=3","gravatar_id":"","url":"https://api.github.com/users/Fishrock123","html_url":"https://github.com/Fishrock123","followers_url":"https://api.github.com/users/Fishrock123/followers","following_url":"https://api.github.com/users/Fishrock123/following{/other_user}","gists_url":"https://api.github.com/users/Fishrock123/gists{/gist_id}","starred_url":"https://api.github.com/users/Fishrock123/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Fishrock123/subscriptions","organizations_url":"https://api.github.com/users/Fishrock123/orgs","repos_url":"https://api.github.com/users/Fishrock123/repos","events_url":"https://api.github.com/users/Fishrock123/events{/privacy}","received_events_url":"https://api.github.com/users/Fishrock123/received_events","type":"User","site_admin":false},"created_at":"2016-07-26T08:00:46Z","updated_at":"2016-07-26T08:00:46Z","body":"cc @nodejs/build, the unix script part looks _ok_ to me."}}
{"actor":{"display_login":"Fishrock123"}}