)]}'
{"tests/test_actions.py":[{"author":{"_account_id":1054,"name":"Darragh Bailey","email":"daragh.bailey@gmail.com","username":"dbailey-k"},"change_message_id":"142b335bb4f07fd5ce97ff84d97d2beab73f7ab8","unresolved":false,"context_lines":[{"line_number":31,"context_line":"from freezer_api.api.v1 import actions as v1_actions"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"class TestActionsCollectionResource(unittest.TestCase):"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"    def setUp(self):"},{"line_number":37,"context_line":"        self.mock_db \u003d Mock()"}],"source_content_type":"text/x-python","patch_set":5,"id":"fa1b9901_8fbd5b6a","line":34,"updated":"2015-08-19 10:58:21.000000000","message":"I would suggest considering converting to testtools.TestCase in the future. as it extends and builds on unittest, is consistent across python 2 and 3, and provides some methods for interacting with fixtures.","commit_id":"f71c8c05b3178b3e01acb798f53c359803349628"},{"author":{"_account_id":1054,"name":"Darragh Bailey","email":"daragh.bailey@gmail.com","username":"dbailey-k"},"change_message_id":"142b335bb4f07fd5ce97ff84d97d2beab73f7ab8","unresolved":false,"context_lines":[{"line_number":35,"context_line":""},{"line_number":36,"context_line":"    def setUp(self):"},{"line_number":37,"context_line":"        self.mock_db \u003d Mock()"},{"line_number":38,"context_line":"        self.mock_req \u003d Mock()"},{"line_number":39,"context_line":"        self.mock_req.get_header.return_value \u003d fake_action_0[\u0027user_id\u0027]"},{"line_number":40,"context_line":"        self.mock_req.status \u003d falcon.HTTP_200"},{"line_number":41,"context_line":"        self.resource \u003d v1_actions.ActionsCollectionResource(self.mock_db)"}],"source_content_type":"text/x-python","patch_set":5,"id":"fa1b9901_cf2113fd","line":38,"updated":"2015-08-19 10:58:21.000000000","message":"You may wish to consider specifying a spec class for the Mock() to use so that it can ensure any calls being made are against the correct api.\n\n self.mock_db \u003d Mock(spec\u003d\u003cdb_class\u003e)\n\nAlternatively, I frequently use patch with autospec\u003dTrue.\n\n def setUp(self):\n     self.mock_db_patch \u003d patch(\u0027\u003cimport_location\u003e.\u003cpackage\u003e.\u003cmodule\u003e.\u003cclass\u003e\u0027, autospec\u003dTrue)\n     mock_db \u003d self.mock_db_patch.start()\n\n def tearDown(self):\n     self.mock_db_path.stop()\n\n\n\nHelps make it clear where the object that you need to mock is operating in your code.","commit_id":"f71c8c05b3178b3e01acb798f53c359803349628"}]}