{"action":"created","issue":{"url":"https://api.github.com/repos/LongRunning/LongRunning/issues/9","labels_url":"https://api.github.com/repos/LongRunning/LongRunning/issues/9/labels{/name}","comments_url":"https://api.github.com/repos/LongRunning/LongRunning/issues/9/comments","events_url":"https://api.github.com/repos/LongRunning/LongRunning/issues/9/events","html_url":"https://github.com/LongRunning/LongRunning/pull/9","id":80282623,"number":9,"title":"Adding test suit","user":{"login":"cmodijk","id":771041,"avatar_url":"https://avatars.githubusercontent.com/u/771041?v=3","gravatar_id":"","url":"https://api.github.com/users/cmodijk","html_url":"https://github.com/cmodijk","followers_url":"https://api.github.com/users/cmodijk/followers","following_url":"https://api.github.com/users/cmodijk/following{/other_user}","gists_url":"https://api.github.com/users/cmodijk/gists{/gist_id}","starred_url":"https://api.github.com/users/cmodijk/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cmodijk/subscriptions","organizations_url":"https://api.github.com/users/cmodijk/orgs","repos_url":"https://api.github.com/users/cmodijk/repos","events_url":"https://api.github.com/users/cmodijk/events{/privacy}","received_events_url":"https://api.github.com/users/cmodijk/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":11,"created_at":"2015-05-24T21:56:22Z","updated_at":"2015-06-06T16:18:23Z","closed_at":"2015-06-06T16:14:33Z","pull_request":{"url":"https://api.github.com/repos/LongRunning/LongRunning/pulls/9","html_url":"https://github.com/LongRunning/LongRunning/pull/9","diff_url":"https://github.com/LongRunning/LongRunning/pull/9.diff","patch_url":"https://github.com/LongRunning/LongRunning/pull/9.patch"},"body":"I'm still working on this branch but i'm already creating the pull to let you know i'm working on it. I would also love to hear any feedback.\r\n\r\nOne side note i changed the `ResetClosedEntityManagers` from an exception to a continue because it is possible to have DoctrineORM and Doctrine Document mapper at te same time. I don't know it they would come from the same registry but the `ManagerRegistry` say's it's possible to get a `ObjectManager`. \r\n\r\nI also fixed a bug that is related to my other pull request #6"},"comment":{"url":"https://api.github.com/repos/LongRunning/LongRunning/issues/comments/109610158","html_url":"https://github.com/LongRunning/LongRunning/pull/9#issuecomment-109610158","issue_url":"https://api.github.com/repos/LongRunning/LongRunning/issues/9","id":109610158,"user":{"login":"matthiasnoback","id":1193078,"avatar_url":"https://avatars.githubusercontent.com/u/1193078?v=3","gravatar_id":"","url":"https://api.github.com/users/matthiasnoback","html_url":"https://github.com/matthiasnoback","followers_url":"https://api.github.com/users/matthiasnoback/followers","following_url":"https://api.github.com/users/matthiasnoback/following{/other_user}","gists_url":"https://api.github.com/users/matthiasnoback/gists{/gist_id}","starred_url":"https://api.github.com/users/matthiasnoback/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/matthiasnoback/subscriptions","organizations_url":"https://api.github.com/users/matthiasnoback/orgs","repos_url":"https://api.github.com/users/matthiasnoback/repos","events_url":"https://api.github.com/users/matthiasnoback/events{/privacy}","received_events_url":"https://api.github.com/users/matthiasnoback/received_events","type":"User","site_admin":false},"created_at":"2015-06-06T16:18:23Z","updated_at":"2015-06-06T16:18:23Z","body":"That's fantastic. Again: great work! And a lot of work as well. This is a very good baseline situation for any further development work. \r\n\r\nI realize now that before we can start splitting this one big package into several smaller ones we should reorganize the tests as well, since the subtree split should be done per directory, so all the tests for a plugin etc. should be in that directory as well. So whenever I'm going to split, I'll move everything around a bit until it fits with the tool (http://gitsplit.com/login/)."}}