{"action":"created","issue":{"url":"https://api.github.com/repos/MarginallyClever/Makelangelo/issues/115","labels_url":"https://api.github.com/repos/MarginallyClever/Makelangelo/issues/115/labels{/name}","comments_url":"https://api.github.com/repos/MarginallyClever/Makelangelo/issues/115/comments","events_url":"https://api.github.com/repos/MarginallyClever/Makelangelo/issues/115/events","html_url":"https://github.com/MarginallyClever/Makelangelo/pull/115","id":89018359,"number":115,"title":"Brought back and refactored code that got lost via PR, and updated Unit tests.","user":{"login":"virtuoushub","id":4303638,"avatar_url":"https://avatars.githubusercontent.com/u/4303638?v=3","gravatar_id":"","url":"https://api.github.com/users/virtuoushub","html_url":"https://github.com/virtuoushub","followers_url":"https://api.github.com/users/virtuoushub/followers","following_url":"https://api.github.com/users/virtuoushub/following{/other_user}","gists_url":"https://api.github.com/users/virtuoushub/gists{/gist_id}","starred_url":"https://api.github.com/users/virtuoushub/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/virtuoushub/subscriptions","organizations_url":"https://api.github.com/users/virtuoushub/orgs","repos_url":"https://api.github.com/users/virtuoushub/repos","events_url":"https://api.github.com/users/virtuoushub/events{/privacy}","received_events_url":"https://api.github.com/users/virtuoushub/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-06-17T13:36:40Z","updated_at":"2015-06-17T20:41:49Z","closed_at":"2015-06-17T17:02:12Z","pull_request":{"url":"https://api.github.com/repos/MarginallyClever/Makelangelo/pulls/115","html_url":"https://github.com/MarginallyClever/Makelangelo/pull/115","diff_url":"https://github.com/MarginallyClever/Makelangelo/pull/115.diff","patch_url":"https://github.com/MarginallyClever/Makelangelo/pull/115.patch"},"body":"By removing the code I may have reopened #105, please test and verify this PR resolves any issues seen.\r\n\r\nSee PR: https://github.com/MarginallyClever/Makelangelo/pull/114\r\nSee Issue: https://github.com/MarginallyClever/Makelangelo/issues/105\r\n\r\nThere is a [unit test](https://github.com/MakeHartford/Makelangelo/blob/dev/java/src/test/java/com/marginallyclever/makelangelo/PreferencesHelperTest.java#L96-L109) that ~~should~~ will fail if #105 is still an issue.\r\n\r\n#### To run unit tests from CLI\r\n```shell\r\nmvn clean test\r\n```"},"comment":{"url":"https://api.github.com/repos/MarginallyClever/Makelangelo/issues/comments/112943967","html_url":"https://github.com/MarginallyClever/Makelangelo/pull/115#issuecomment-112943967","issue_url":"https://api.github.com/repos/MarginallyClever/Makelangelo/issues/115","id":112943967,"user":{"login":"virtuoushub","id":4303638,"avatar_url":"https://avatars.githubusercontent.com/u/4303638?v=3","gravatar_id":"","url":"https://api.github.com/users/virtuoushub","html_url":"https://github.com/virtuoushub","followers_url":"https://api.github.com/users/virtuoushub/followers","following_url":"https://api.github.com/users/virtuoushub/following{/other_user}","gists_url":"https://api.github.com/users/virtuoushub/gists{/gist_id}","starred_url":"https://api.github.com/users/virtuoushub/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/virtuoushub/subscriptions","organizations_url":"https://api.github.com/users/virtuoushub/orgs","repos_url":"https://api.github.com/users/virtuoushub/repos","events_url":"https://api.github.com/users/virtuoushub/events{/privacy}","received_events_url":"https://api.github.com/users/virtuoushub/received_events","type":"User","site_admin":false},"created_at":"2015-06-17T20:41:49Z","updated_at":"2015-06-17T20:41:49Z","body":"[JSON](https://github.com/MarginallyClever/Makelangelo/blob/dev/java/src/test/java/com/marginallyclever/makelangelo/PreferencesHelperTest.java#L38) is something I create and should be [removing](https://github.com/MarginallyClever/Makelangelo/blob/dev/java/src/test/java/com/marginallyclever/makelangelo/PreferencesHelperTest.java#L54-L61) during the unit test. -1 is a case I [do not test for](https://github.com/MarginallyClever/Makelangelo/blob/dev/java/src/test/java/com/marginallyclever/makelangelo/PreferencesHelperTest.java#L96-L109). Thank you for alerting me to this deficiency in my test."}}