{"action":"created","issue":{"url":"https://api.github.com/repos/testing-cabal/testrepository/issues/4","labels_url":"https://api.github.com/repos/testing-cabal/testrepository/issues/4/labels{/name}","comments_url":"https://api.github.com/repos/testing-cabal/testrepository/issues/4/comments","events_url":"https://api.github.com/repos/testing-cabal/testrepository/issues/4/events","html_url":"https://github.com/testing-cabal/testrepository/pull/4","id":54247558,"number":4,"title":"Fix unhandled exception in get_test_run","user":{"login":"mtreinish","id":2447371,"avatar_url":"https://avatars.githubusercontent.com/u/2447371?v=3","gravatar_id":"","url":"https://api.github.com/users/mtreinish","html_url":"https://github.com/mtreinish","followers_url":"https://api.github.com/users/mtreinish/followers","following_url":"https://api.github.com/users/mtreinish/following{/other_user}","gists_url":"https://api.github.com/users/mtreinish/gists{/gist_id}","starred_url":"https://api.github.com/users/mtreinish/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mtreinish/subscriptions","organizations_url":"https://api.github.com/users/mtreinish/orgs","repos_url":"https://api.github.com/users/mtreinish/repos","events_url":"https://api.github.com/users/mtreinish/events{/privacy}","received_events_url":"https://api.github.com/users/mtreinish/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-01-13T21:00:28Z","updated_at":"2015-01-14T03:20:21Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/testing-cabal/testrepository/pulls/4","html_url":"https://github.com/testing-cabal/testrepository/pull/4","diff_url":"https://github.com/testing-cabal/testrepository/pull/4.diff","patch_url":"https://github.com/testing-cabal/testrepository/pull/4.patch"},"body":"This commit fixes the case when the open in get_test_run() raises an\r\nexception besides IOError.\r\n\r\nCloses-Bug: #1409936"},"comment":{"url":"https://api.github.com/repos/testing-cabal/testrepository/issues/comments/69863383","html_url":"https://github.com/testing-cabal/testrepository/pull/4#issuecomment-69863383","issue_url":"https://api.github.com/repos/testing-cabal/testrepository/issues/4","id":69863383,"user":{"login":"rbtcollins","id":499678,"avatar_url":"https://avatars.githubusercontent.com/u/499678?v=3","gravatar_id":"","url":"https://api.github.com/users/rbtcollins","html_url":"https://github.com/rbtcollins","followers_url":"https://api.github.com/users/rbtcollins/followers","following_url":"https://api.github.com/users/rbtcollins/following{/other_user}","gists_url":"https://api.github.com/users/rbtcollins/gists{/gist_id}","starred_url":"https://api.github.com/users/rbtcollins/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rbtcollins/subscriptions","organizations_url":"https://api.github.com/users/rbtcollins/orgs","repos_url":"https://api.github.com/users/rbtcollins/repos","events_url":"https://api.github.com/users/rbtcollins/events{/privacy}","received_events_url":"https://api.github.com/users/rbtcollins/received_events","type":"User","site_admin":false},"created_at":"2015-01-14T03:20:21Z","updated_at":"2015-01-14T03:20:21Z","body":"It will this way too I suspect, if the test fails while its patched...\r\n\r\nDon't patch open - patch repository,file.open and it should be fine.\r\n\r\n-Rob\r\n\r\nOn 14 January 2015 at 16:10, Matthew Treinish <notifications@github.com>\r\nwrote:\r\n\r\n> I attempted to use patch from fixtures and it didn't quite behave the way\r\n> I was expecting. (patching open() broke the test runner) So I think in the\r\n> short term just adding mock to the test requirements will be simpler.\r\n>\r\n> —\r\n> Reply to this email directly or view it on GitHub\r\n> <https://github.com/testing-cabal/testrepository/pull/4#issuecomment-69862740>\r\n> .\r\n>\r\n\r\n\r\n\r\n-- \r\nRobert Collins <rbtcollins@hp.com>\r\nDistinguished Technologist\r\nHP Converged Cloud"}}