{"action":"created","issue":{"url":"https://api.github.com/repos/NuGet/NuGet2/issues/19","labels_url":"https://api.github.com/repos/NuGet/NuGet2/issues/19/labels{/name}","comments_url":"https://api.github.com/repos/NuGet/NuGet2/issues/19/comments","events_url":"https://api.github.com/repos/NuGet/NuGet2/issues/19/events","html_url":"https://github.com/NuGet/NuGet2/pull/19","id":111851420,"number":19,"title":"Add missing source files referenced by PDB.","user":{"login":"tom-englert","id":10229954,"avatar_url":"https://avatars.githubusercontent.com/u/10229954?v=3","gravatar_id":"","url":"https://api.github.com/users/tom-englert","html_url":"https://github.com/tom-englert","followers_url":"https://api.github.com/users/tom-englert/followers","following_url":"https://api.github.com/users/tom-englert/following{/other_user}","gists_url":"https://api.github.com/users/tom-englert/gists{/gist_id}","starred_url":"https://api.github.com/users/tom-englert/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tom-englert/subscriptions","organizations_url":"https://api.github.com/users/tom-englert/orgs","repos_url":"https://api.github.com/users/tom-englert/repos","events_url":"https://api.github.com/users/tom-englert/events{/privacy}","received_events_url":"https://api.github.com/users/tom-englert/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/NuGet/NuGet2/labels/cla-signed","name":"cla-signed","color":"207de5"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":6,"created_at":"2015-10-16T15:10:53Z","updated_at":"2015-10-17T15:00:22Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/NuGet/NuGet2/pulls/19","html_url":"https://github.com/NuGet/NuGet2/pull/19","diff_url":"https://github.com/NuGet/NuGet2/pull/19.diff","patch_url":"https://github.com/NuGet/NuGet2/pull/19.patch"},"body":"Fix for https://github.com/NuGet/Home/issues/304\r\n"},"comment":{"url":"https://api.github.com/repos/NuGet/NuGet2/issues/comments/148921813","html_url":"https://github.com/NuGet/NuGet2/pull/19#issuecomment-148921813","issue_url":"https://api.github.com/repos/NuGet/NuGet2/issues/19","id":148921813,"user":{"login":"yishaigalatzer","id":1238711,"avatar_url":"https://avatars.githubusercontent.com/u/1238711?v=3","gravatar_id":"","url":"https://api.github.com/users/yishaigalatzer","html_url":"https://github.com/yishaigalatzer","followers_url":"https://api.github.com/users/yishaigalatzer/followers","following_url":"https://api.github.com/users/yishaigalatzer/following{/other_user}","gists_url":"https://api.github.com/users/yishaigalatzer/gists{/gist_id}","starred_url":"https://api.github.com/users/yishaigalatzer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yishaigalatzer/subscriptions","organizations_url":"https://api.github.com/users/yishaigalatzer/orgs","repos_url":"https://api.github.com/users/yishaigalatzer/repos","events_url":"https://api.github.com/users/yishaigalatzer/events{/privacy}","received_events_url":"https://api.github.com/users/yishaigalatzer/received_events","type":"User","site_admin":false},"created_at":"2015-10-17T15:00:22Z","updated_at":"2015-10-17T15:00:22Z","body":"Completely agree. Command line tests in nuget.client repo are where the integration tests live.\n\nYishai\n________________________________\nFrom: Tom Englert<mailto:notifications@github.com>\nSent: ‎10/‎17/‎2015 1:13 AM\nTo: NuGet/NuGet2<mailto:NuGet2@noreply.github.com>\nCc: Yishai Galatzer<mailto:yigalatz@microsoft.com>\nSubject: Re: [NuGet2] Add missing source files referenced by PDB. (#19)\n\n\n@yishaigalatzer<https://na01.safelinks.protection.outlook.com/?url=https%3a%2f%2fgithub.com%2fyishaigalatzer&data=01%7c01%7cyigalatz%40microsoft.com%7c092e355146064ee4e6c408d2d6cae03c%7c72f988bf86f141af91ab2d7cd011db47%7c1&sdata=yq2umMiCT0X3XoBpH%2ft9H%2fKKMxjZNDrqB8DdLteJHjY%3d> this is just wrapping existing stuff that is already unit tested (it does not make sense unit-testing DisSymReader here). I'm not the friend of adding dummy unit tests just for the sake of having a unit test.\nThis would need some integration test, but I see no framework for this.\n\n—\nReply to this email directly or view it on GitHub<https://na01.safelinks.protection.outlook.com/?url=https%3a%2f%2fgithub.com%2fNuGet%2fNuGet2%2fpull%2f19%23issuecomment-148894887&data=01%7c01%7cyigalatz%40microsoft.com%7c092e355146064ee4e6c408d2d6cae03c%7c72f988bf86f141af91ab2d7cd011db47%7c1&sdata=DOp0JoPJw05tE3KLYxojQiPFTIVqZT57EvhUM4xpjmc%3d>.\n"}}