{"action":"created","issue":{"url":"https://api.github.com/repos/fedspendingtransparency/data-act-broker-backend/issues/286","repository_url":"https://api.github.com/repos/fedspendingtransparency/data-act-broker-backend","labels_url":"https://api.github.com/repos/fedspendingtransparency/data-act-broker-backend/issues/286/labels{/name}","comments_url":"https://api.github.com/repos/fedspendingtransparency/data-act-broker-backend/issues/286/comments","events_url":"https://api.github.com/repos/fedspendingtransparency/data-act-broker-backend/issues/286/events","html_url":"https://github.com/fedspendingtransparency/data-act-broker-backend/pull/286","id":177892261,"number":286,"title":"db connection refactor part 2: unit tests","user":{"login":"bsweger","id":540544,"avatar_url":"https://avatars.githubusercontent.com/u/540544?v=3","gravatar_id":"","url":"https://api.github.com/users/bsweger","html_url":"https://github.com/bsweger","followers_url":"https://api.github.com/users/bsweger/followers","following_url":"https://api.github.com/users/bsweger/following{/other_user}","gists_url":"https://api.github.com/users/bsweger/gists{/gist_id}","starred_url":"https://api.github.com/users/bsweger/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bsweger/subscriptions","organizations_url":"https://api.github.com/users/bsweger/orgs","repos_url":"https://api.github.com/users/bsweger/repos","events_url":"https://api.github.com/users/bsweger/events{/privacy}","received_events_url":"https://api.github.com/users/bsweger/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2016-09-19T20:42:44Z","updated_at":"2016-09-20T13:55:51Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/fedspendingtransparency/data-act-broker-backend/pulls/286","html_url":"https://github.com/fedspendingtransparency/data-act-broker-backend/pull/286","diff_url":"https://github.com/fedspendingtransparency/data-act-broker-backend/pull/286.diff","patch_url":"https://github.com/fedspendingtransparency/data-act-broker-backend/pull/286.patch"},"body":"This PR mainly updates the unit tests to get a database fixture using the functions in `dataactcore/interfaces/db.py` instead of baseInterface + interface holder.\r\n\r\nA few miscellaneous items included:\r\n* Update the FSRS loader (another one I forgot in PR #272)\r\n* Migrations now use the common URI function in db.py instead of repeating the code to create a URI string\r\n\r\n@mab6bf @jworcestBAH @nmonga91 @cmc333333: heads-up that once this is merged, the syntax for referencing the pytest database fixture will change slightly.\r\n\r\nOld: `database.stagingDb`\r\nNew: `database`"},"comment":{"url":"https://api.github.com/repos/fedspendingtransparency/data-act-broker-backend/issues/comments/248308298","html_url":"https://github.com/fedspendingtransparency/data-act-broker-backend/pull/286#issuecomment-248308298","issue_url":"https://api.github.com/repos/fedspendingtransparency/data-act-broker-backend/issues/286","id":248308298,"user":{"login":"cmc333333","id":326918,"avatar_url":"https://avatars.githubusercontent.com/u/326918?v=3","gravatar_id":"","url":"https://api.github.com/users/cmc333333","html_url":"https://github.com/cmc333333","followers_url":"https://api.github.com/users/cmc333333/followers","following_url":"https://api.github.com/users/cmc333333/following{/other_user}","gists_url":"https://api.github.com/users/cmc333333/gists{/gist_id}","starred_url":"https://api.github.com/users/cmc333333/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cmc333333/subscriptions","organizations_url":"https://api.github.com/users/cmc333333/orgs","repos_url":"https://api.github.com/users/cmc333333/repos","events_url":"https://api.github.com/users/cmc333333/events{/privacy}","received_events_url":"https://api.github.com/users/cmc333333/received_events","type":"User","site_admin":false},"created_at":"2016-09-20T13:55:51Z","updated_at":"2016-09-20T13:55:51Z","body":"Makes a ton of sense; the only issue I'm imagining is if we need access to other methods of `InterfaceHolder` or it's subfields. That said, if we're moving away from that style (yay!) this'll be a nice forcing function."}}
{"actor":{"display_login":"cmc333333"}}