{"action":"closed","issue":{"url":"https://api.github.com/repos/pughlab/cbioportal/issues/10","labels_url":"https://api.github.com/repos/pughlab/cbioportal/issues/10/labels{/name}","comments_url":"https://api.github.com/repos/pughlab/cbioportal/issues/10/comments","events_url":"https://api.github.com/repos/pughlab/cbioportal/issues/10/events","html_url":"https://github.com/pughlab/cbioportal/issues/10","id":73356478,"number":10,"title":"Fix cross-project reference to applicationContext-business.xml","user":{"login":"morungos","id":442090,"avatar_url":"https://avatars.githubusercontent.com/u/442090?v=3","gravatar_id":"","url":"https://api.github.com/users/morungos","html_url":"https://github.com/morungos","followers_url":"https://api.github.com/users/morungos/followers","following_url":"https://api.github.com/users/morungos/following{/other_user}","gists_url":"https://api.github.com/users/morungos/gists{/gist_id}","starred_url":"https://api.github.com/users/morungos/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/morungos/subscriptions","organizations_url":"https://api.github.com/users/morungos/orgs","repos_url":"https://api.github.com/users/morungos/repos","events_url":"https://api.github.com/users/morungos/events{/privacy}","received_events_url":"https://api.github.com/users/morungos/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":{"login":"morungos","id":442090,"avatar_url":"https://avatars.githubusercontent.com/u/442090?v=3","gravatar_id":"","url":"https://api.github.com/users/morungos","html_url":"https://github.com/morungos","followers_url":"https://api.github.com/users/morungos/followers","following_url":"https://api.github.com/users/morungos/following{/other_user}","gists_url":"https://api.github.com/users/morungos/gists{/gist_id}","starred_url":"https://api.github.com/users/morungos/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/morungos/subscriptions","organizations_url":"https://api.github.com/users/morungos/orgs","repos_url":"https://api.github.com/users/morungos/repos","events_url":"https://api.github.com/users/morungos/events{/privacy}","received_events_url":"https://api.github.com/users/morungos/received_events","type":"User","site_admin":false},"milestone":{"url":"https://api.github.com/repos/pughlab/cbioportal/milestones/1","html_url":"https://github.com/pughlab/cbioportal/milestones/M1","labels_url":"https://api.github.com/repos/pughlab/cbioportal/milestones/1/labels","id":1098610,"number":1,"title":"M1","description":"Stable build process, zero environment/configuration needed to spin up a portal (as far as possible)","creator":{"login":"morungos","id":442090,"avatar_url":"https://avatars.githubusercontent.com/u/442090?v=3","gravatar_id":"","url":"https://api.github.com/users/morungos","html_url":"https://github.com/morungos","followers_url":"https://api.github.com/users/morungos/followers","following_url":"https://api.github.com/users/morungos/following{/other_user}","gists_url":"https://api.github.com/users/morungos/gists{/gist_id}","starred_url":"https://api.github.com/users/morungos/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/morungos/subscriptions","organizations_url":"https://api.github.com/users/morungos/orgs","repos_url":"https://api.github.com/users/morungos/repos","events_url":"https://api.github.com/users/morungos/events{/privacy}","received_events_url":"https://api.github.com/users/morungos/received_events","type":"User","site_admin":false},"open_issues":2,"closed_issues":5,"state":"open","created_at":"2015-05-06T14:15:11Z","updated_at":"2015-05-15T19:07:53Z","due_on":"2015-05-15T04:00:00Z","closed_at":null},"comments":2,"created_at":"2015-05-05T16:08:13Z","updated_at":"2015-05-15T19:07:53Z","closed_at":"2015-05-15T19:07:53Z","body":"Currently, the applicationContext-business.xml is referenced from core, specifically via JdbcUtil.java:125, which accesses it through the classpath. Tests should switch to use a simpler Spring-based injection, which can probably be done without the full static restructuring, with a singleton of some kind. This will significantly smooth testability. In particular, this hard-wired classpath reference from a static method makes this context handling a significant issue, as it can never be switched for testing, and using alternate context files is a good testing technique. There's also no need for this to be accessed through the classpath in any event. "}}