)]}'
{"tests/sqlalchemy/test_models.py":[{"author":{"_account_id":7491,"name":"Viktor Serhieiev","email":"viktor.sergeyev@gmail.com","username":"vsergeyev"},"change_message_id":"a578c72172c8619c25434ca36820e24fd930a426","unresolved":false,"context_lines":[{"line_number":57,"context_line":"        h \u003d {\u0027a\u0027: \u00271\u0027, \u0027b\u0027: \u00272\u0027}"},{"line_number":58,"context_line":"        mb.update(h)"},{"line_number":59,"context_line":"        for key in h.keys():"},{"line_number":60,"context_line":"            self.assertTrue(key in mb)"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"    def test_modelbase_iteritems(self):"},{"line_number":63,"context_line":"        self.skipTest(\"Requires DB\")"}],"source_content_type":"text/x-python","patch_set":3,"id":"1ae5cdf2_2fade39b","line":60,"updated":"2014-06-25 16:00:45.000000000","message":"nit: \n\n self.assertIn()","commit_id":"ef46e2023541fda317a66b33aa57ee0ccde7d2d4"},{"author":{"_account_id":7491,"name":"Viktor Serhieiev","email":"viktor.sergeyev@gmail.com","username":"vsergeyev"},"change_message_id":"1f056ccd13ea43d0c834005fb0e31fd6378ef46e","unresolved":false,"context_lines":[{"line_number":57,"context_line":"        h \u003d {\u0027a\u0027: \u00271\u0027, \u0027b\u0027: \u00272\u0027}"},{"line_number":58,"context_line":"        mb.update(h)"},{"line_number":59,"context_line":"        for key in h.keys():"},{"line_number":60,"context_line":"            self.assertTrue(key in mb)"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"    def test_modelbase_iteritems(self):"},{"line_number":63,"context_line":"        self.skipTest(\"Requires DB\")"}],"source_content_type":"text/x-python","patch_set":3,"id":"1ae5cdf2_7eab1775","line":60,"in_reply_to":"1ae5cdf2_18ccebba","updated":"2014-06-25 18:09:35.000000000","message":"sounds reasonable, I agree with it.\nBut please add a single comment.","commit_id":"ef46e2023541fda317a66b33aa57ee0ccde7d2d4"},{"author":{"_account_id":8968,"name":"Max Lobur","email":"max_lobur@outlook.com","username":"max_lobur"},"change_message_id":"052486c617c84ee015e72b121075c23d63d35ace","unresolved":false,"context_lines":[{"line_number":57,"context_line":"        h \u003d {\u0027a\u0027: \u00271\u0027, \u0027b\u0027: \u00272\u0027}"},{"line_number":58,"context_line":"        mb.update(h)"},{"line_number":59,"context_line":"        for key in h.keys():"},{"line_number":60,"context_line":"            self.assertTrue(key in mb)"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"    def test_modelbase_iteritems(self):"},{"line_number":63,"context_line":"        self.skipTest(\"Requires DB\")"}],"source_content_type":"text/x-python","patch_set":3,"id":"1ae5cdf2_18ccebba","line":60,"in_reply_to":"1ae5cdf2_2fade39b","updated":"2014-06-25 17:05:05.000000000","message":"Well, I wanted to test exactly \"in\" syntax to make sure it works. Agree if I leave as is?","commit_id":"ef46e2023541fda317a66b33aa57ee0ccde7d2d4"},{"author":{"_account_id":8968,"name":"Max Lobur","email":"max_lobur@outlook.com","username":"max_lobur"},"change_message_id":"8bd5f4de38fc11a9f955a68a319b10a844166ab4","unresolved":false,"context_lines":[{"line_number":57,"context_line":"        h \u003d {\u0027a\u0027: \u00271\u0027, \u0027b\u0027: \u00272\u0027}"},{"line_number":58,"context_line":"        mb.update(h)"},{"line_number":59,"context_line":"        for key in h.keys():"},{"line_number":60,"context_line":"            self.assertTrue(key in mb)"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"    def test_modelbase_iteritems(self):"},{"line_number":63,"context_line":"        self.skipTest(\"Requires DB\")"}],"source_content_type":"text/x-python","patch_set":3,"id":"1ae5cdf2_4c121959","line":60,"in_reply_to":"1ae5cdf2_7eab1775","updated":"2014-06-26 10:46:03.000000000","message":"Thanks, I\u0027ll add comment","commit_id":"ef46e2023541fda317a66b33aa57ee0ccde7d2d4"},{"author":{"_account_id":7491,"name":"Viktor Serhieiev","email":"viktor.sergeyev@gmail.com","username":"vsergeyev"},"change_message_id":"a578c72172c8619c25434ca36820e24fd930a426","unresolved":false,"context_lines":[{"line_number":58,"context_line":"        mb.update(h)"},{"line_number":59,"context_line":"        for key in h.keys():"},{"line_number":60,"context_line":"            self.assertTrue(key in mb)"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"    def test_modelbase_iteritems(self):"},{"line_number":63,"context_line":"        self.skipTest(\"Requires DB\")"},{"line_number":64,"context_line":"        mb \u003d models.ModelBase()"}],"source_content_type":"text/x-python","patch_set":3,"id":"1ae5cdf2_0fe667e8","line":61,"updated":"2014-06-25 16:00:45.000000000","message":"Also, can you please add here one more check. Something like \n\n self.assertNotIn(\u0027fake_key\u0027, mb)","commit_id":"ef46e2023541fda317a66b33aa57ee0ccde7d2d4"},{"author":{"_account_id":8968,"name":"Max Lobur","email":"max_lobur@outlook.com","username":"max_lobur"},"change_message_id":"052486c617c84ee015e72b121075c23d63d35ace","unresolved":false,"context_lines":[{"line_number":58,"context_line":"        mb.update(h)"},{"line_number":59,"context_line":"        for key in h.keys():"},{"line_number":60,"context_line":"            self.assertTrue(key in mb)"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"    def test_modelbase_iteritems(self):"},{"line_number":63,"context_line":"        self.skipTest(\"Requires DB\")"},{"line_number":64,"context_line":"        mb \u003d models.ModelBase()"}],"source_content_type":"text/x-python","patch_set":3,"id":"1ae5cdf2_98b71b21","line":61,"in_reply_to":"1ae5cdf2_0fe667e8","updated":"2014-06-25 17:05:05.000000000","message":"yep, np!","commit_id":"ef46e2023541fda317a66b33aa57ee0ccde7d2d4"},{"author":{"_account_id":6849,"name":"Roman Podoliaka","email":"roman.podoliaka@gmail.com","username":"rpodolyaka"},"change_message_id":"46155a8cef209ee22bfa424e3cddfcc1c4e79495","unresolved":false,"context_lines":[{"line_number":57,"context_line":"        h \u003d {\u0027a\u0027: \u00271\u0027, \u0027b\u0027: \u00272\u0027}"},{"line_number":58,"context_line":"        mb.update(h)"},{"line_number":59,"context_line":"        for key in h.keys():"},{"line_number":60,"context_line":"            # Test \u0027in\u0027 syntax (instead of using .assertIn)"},{"line_number":61,"context_line":"            self.assertTrue(key in mb)"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"        self.assertFalse(\u0027non-existent-key\u0027 in mb)"}],"source_content_type":"text/x-python","patch_set":4,"id":"1ae5cdf2_e977299d","line":60,"updated":"2014-06-27 12:37:49.000000000","message":"Actually, assertIn() does the very same thing","commit_id":"167ffb66f40b86ab9f8880543d5346b52a51ac9e"},{"author":{"_account_id":8968,"name":"Max Lobur","email":"max_lobur@outlook.com","username":"max_lobur"},"change_message_id":"2e5135f3178896f030b9796a612341a68bfa9abf","unresolved":false,"context_lines":[{"line_number":57,"context_line":"        h \u003d {\u0027a\u0027: \u00271\u0027, \u0027b\u0027: \u00272\u0027}"},{"line_number":58,"context_line":"        mb.update(h)"},{"line_number":59,"context_line":"        for key in h.keys():"},{"line_number":60,"context_line":"            # Test \u0027in\u0027 syntax (instead of using .assertIn)"},{"line_number":61,"context_line":"            self.assertTrue(key in mb)"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"        self.assertFalse(\u0027non-existent-key\u0027 in mb)"}],"source_content_type":"text/x-python","patch_set":4,"id":"dab895d6_4de299d6","line":60,"in_reply_to":"1ae5cdf2_e977299d","updated":"2014-06-29 07:00:15.000000000","message":"but I don\u0027t want rely on that :)","commit_id":"167ffb66f40b86ab9f8880543d5346b52a51ac9e"}]}