)]}'
{"tests/unit/test_mock.py":[{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"4b8bd82ba2ab258ff39e13e40e051181990747c5","unresolved":false,"context_lines":[{"line_number":282,"context_line":"                    \"messages\": error_msgs"},{"line_number":283,"context_line":"                })"},{"line_number":284,"context_line":""},{"line_number":285,"context_line":""},{"line_number":286,"context_line":"class VariantsTestCase(test.TestCase):"},{"line_number":287,"context_line":"    def setUp(self):"},{"line_number":288,"context_line":"        self.variants \u003d Variants([\"test\", \"foo\", \"bar\"])"}],"source_content_type":"text/x-python","patch_set":4,"id":"ba3cc151_ac2653cc","line":285,"updated":"2015-06-30 11:00:53.000000000","message":"Can you add here comment ? Something like:\n\n    #################################\n    #\n    # The following unittests are written for code above.\n    #\n    #################################","commit_id":"01649d0703a0c84cdd108ace5fdc96f22ff20208"},{"author":{"_account_id":13609,"name":"Pavel Boldin","email":"pboldin@mirantis.com","username":"pboldin"},"change_message_id":"12953e9821b969e45d4fc6ae2189d1fa11ed2b51","unresolved":false,"context_lines":[{"line_number":282,"context_line":"                    \"messages\": error_msgs"},{"line_number":283,"context_line":"                })"},{"line_number":284,"context_line":""},{"line_number":285,"context_line":""},{"line_number":286,"context_line":"class VariantsTestCase(test.TestCase):"},{"line_number":287,"context_line":"    def setUp(self):"},{"line_number":288,"context_line":"        self.variants \u003d Variants([\"test\", \"foo\", \"bar\"])"}],"source_content_type":"text/x-python","patch_set":4,"id":"ba3cc151_478c4035","line":285,"in_reply_to":"ba3cc151_ac2653cc","updated":"2015-06-30 11:03:17.000000000","message":"Will do.\n\nPlease review further.","commit_id":"01649d0703a0c84cdd108ace5fdc96f22ff20208"},{"author":{"_account_id":10475,"name":"Alexander Maretskiy","email":"amaretskiy@mirantis.com","username":"maretskiy"},"change_message_id":"26404d1c4ac1a29cc5f3017e944b648005d0f271","unresolved":false,"context_lines":[{"line_number":291,"context_line":"    def test_print(self):"},{"line_number":292,"context_line":"        self.assertEqual("},{"line_number":293,"context_line":"            \"{\u0027mock_test\u0027, \u0027mock_foo\u0027, \u0027mock_bar\u0027}\","},{"line_number":294,"context_line":"            str(self.variants)"},{"line_number":295,"context_line":"        )"},{"line_number":296,"context_line":""},{"line_number":297,"context_line":"    def test_print_long(self):"}],"source_content_type":"text/x-python","patch_set":4,"id":"ba3cc151_c116c7f0","line":294,"updated":"2015-07-01 09:44:45.000000000","message":"At least repr(self.variants) should be also checked since we have __repr__() explicitly implemented","commit_id":"01649d0703a0c84cdd108ace5fdc96f22ff20208"},{"author":{"_account_id":10475,"name":"Alexander Maretskiy","email":"amaretskiy@mirantis.com","username":"maretskiy"},"change_message_id":"26404d1c4ac1a29cc5f3017e944b648005d0f271","unresolved":false,"context_lines":[{"line_number":298,"context_line":"        variants \u003d Variants([\"test\", \"foo\", \"bar\", \"buz\"])"},{"line_number":299,"context_line":"        self.assertEqual("},{"line_number":300,"context_line":"            \"{\u0027mock_test\u0027, \u0027mock_foo\u0027, \u0027mock_bar\u0027, ...}\","},{"line_number":301,"context_line":"            str(variants)"},{"line_number":302,"context_line":"        )"},{"line_number":303,"context_line":""},{"line_number":304,"context_line":"    def test_equal(self):"}],"source_content_type":"text/x-python","patch_set":4,"id":"ba3cc151_e1cf4b7d","line":301,"updated":"2015-07-01 09:44:45.000000000","message":"same here","commit_id":"01649d0703a0c84cdd108ace5fdc96f22ff20208"},{"author":{"_account_id":10475,"name":"Alexander Maretskiy","email":"amaretskiy@mirantis.com","username":"maretskiy"},"change_message_id":"26404d1c4ac1a29cc5f3017e944b648005d0f271","unresolved":false,"context_lines":[{"line_number":303,"context_line":""},{"line_number":304,"context_line":"    def test_equal(self):"},{"line_number":305,"context_line":"        self.assertEqual([\"test\", \"foo\", \"bar\"], self.variants)"},{"line_number":306,"context_line":"        self.assertEqual(self.variants, self.variants)"},{"line_number":307,"context_line":""},{"line_number":308,"context_line":"    def test_contains(self):"},{"line_number":309,"context_line":"        self.assertIn(\"test\", self.variants)"}],"source_content_type":"text/x-python","patch_set":4,"id":"ba3cc151_81675f24","line":306,"updated":"2015-07-01 09:44:45.000000000","message":"I would also check L38 by something like that:\n\n  mock_variants \u003d mock.Mock(variants\u003d[\"test\", \"foo\", \"bar\"])\n  self.assertEqual(mock_variants, self.variants)","commit_id":"01649d0703a0c84cdd108ace5fdc96f22ff20208"},{"author":{"_account_id":11748,"name":"Chris St. Pierre","email":"cstpierr@cisco.com","username":"stpierre"},"change_message_id":"f7198586f9045500de56fd19f3167d930efe8d11","unresolved":false,"context_lines":[{"line_number":307,"context_line":"        self.assertEqual(self.variants, self.variants)"},{"line_number":308,"context_line":""},{"line_number":309,"context_line":"        mock_variants \u003d mock.Mock(variants\u003d[\"test\", \"foo\", \"bar\"])"},{"line_number":310,"context_line":"        self.assertEqual(mock_variants, self.variants)"},{"line_number":311,"context_line":""},{"line_number":312,"context_line":"    def test_contains(self):"},{"line_number":313,"context_line":"        self.assertIn(\"test\", self.variants)"}],"source_content_type":"text/x-python","patch_set":5,"id":"ba3cc151_e86215ad","line":310,"updated":"2015-07-06 16:53:41.000000000","message":"This should also test comparing a Variants object with a list:\n\n    self.assertEqual(self.variants, [\"test\", \"foo\", \"bar\"])\n\nYou should also add negative tests.","commit_id":"ef6acaf408f2e1a507c16adfaeed171fe9b07d09"},{"author":{"_account_id":13609,"name":"Pavel Boldin","email":"pboldin@mirantis.com","username":"pboldin"},"change_message_id":"d3c8e5e1d40434919cf6da2ea78aa09ed53f52fc","unresolved":false,"context_lines":[{"line_number":307,"context_line":"        self.assertEqual(self.variants, self.variants)"},{"line_number":308,"context_line":""},{"line_number":309,"context_line":"        mock_variants \u003d mock.Mock(variants\u003d[\"test\", \"foo\", \"bar\"])"},{"line_number":310,"context_line":"        self.assertEqual(mock_variants, self.variants)"},{"line_number":311,"context_line":""},{"line_number":312,"context_line":"    def test_contains(self):"},{"line_number":313,"context_line":"        self.assertIn(\"test\", self.variants)"}],"source_content_type":"text/x-python","patch_set":5,"id":"9a41bdd9_e1afbd92","line":310,"in_reply_to":"ba3cc151_e86215ad","updated":"2015-07-11 15:40:10.000000000","message":"There is such a test above.\n\nOK about negative tests.","commit_id":"ef6acaf408f2e1a507c16adfaeed171fe9b07d09"}]}