)]}'
{"tempest/api/database/versions/test_versions.py":[{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"82fecc334eeaa2131152770817969b6476bf38ae","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":29,"context_line":"    def test_list_db_versions(self):"},{"line_number":30,"context_line":"        resp, versions \u003d self.client.list_db_versions()"},{"line_number":31,"context_line":"        self.assertTrue(len(versions) \u003e 0, \"No database versions found\")"},{"line_number":32,"context_line":"        # List of all versions should contain the current version, and there"},{"line_number":33,"context_line":"        # should only be one \u0027current\u0027 version"}],"source_content_type":"text/x-python","patch_set":5,"id":"bad4c104_5b2c0dac","line":30,"updated":"2014-04-29 15:51:53.000000000","message":"It would be good to add self.assertEqual check here.","commit_id":"cd2a6268b306bc575328fdd30399882b3bb97f49"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"8539760f36fd075abc91fbda0eee64e98c3ff0d4","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":29,"context_line":"    def test_list_db_versions(self):"},{"line_number":30,"context_line":"        resp, versions \u003d self.client.list_db_versions()"},{"line_number":31,"context_line":"        self.assertTrue(len(versions) \u003e 0, \"No database versions found\")"},{"line_number":32,"context_line":"        # List of all versions should contain the current version, and there"},{"line_number":33,"context_line":"        # should only be one \u0027current\u0027 version"}],"source_content_type":"text/x-python","patch_set":5,"id":"bad4c104_7c9c646a","line":30,"updated":"2014-04-29 18:34:10.000000000","message":"Sure, what would you like me to do an \u0027equal\u0027 check on?","commit_id":"cd2a6268b306bc575328fdd30399882b3bb97f49"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"57bb9cd4387eba8524add14fdb7aacfe749c3b06","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":29,"context_line":"    def test_list_db_versions(self):"},{"line_number":30,"context_line":"        resp, versions \u003d self.client.list_db_versions()"},{"line_number":31,"context_line":"        self.assertTrue(len(versions) \u003e 0, \"No database versions found\")"},{"line_number":32,"context_line":"        # List of all versions should contain the current version, and there"},{"line_number":33,"context_line":"        # should only be one \u0027current\u0027 version"}],"source_content_type":"text/x-python","patch_set":5,"id":"bad4c104_ed49a08c","line":30,"in_reply_to":"bad4c104_7c9c646a","updated":"2014-04-29 20:30:37.000000000","message":"I put in a check for the resp.status, is that what you were referring to?","commit_id":"cd2a6268b306bc575328fdd30399882b3bb97f49"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"48d839dd5143fae866b9a5c138786c4b8c68f28f","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":29,"context_line":"    def test_list_db_versions(self):"},{"line_number":30,"context_line":"        resp, versions \u003d self.client.list_db_versions()"},{"line_number":31,"context_line":"        self.assertTrue(len(versions) \u003e 0, \"No database versions found\")"},{"line_number":32,"context_line":"        # List of all versions should contain the current version, and there"},{"line_number":33,"context_line":"        # should only be one \u0027current\u0027 version"}],"source_content_type":"text/x-python","patch_set":5,"id":"9ad9bd40_2231f00b","line":30,"in_reply_to":"bad4c104_ed49a08c","updated":"2014-04-29 23:29:31.000000000","message":"Peter, yes, I meant this.","commit_id":"cd2a6268b306bc575328fdd30399882b3bb97f49"},{"author":{"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},"change_message_id":"67acf124b1c4e044a65882454ea62e3e62cd6820","unresolved":false,"context_lines":[{"line_number":36,"context_line":"        for version in versions:"},{"line_number":37,"context_line":"            if \u0027CURRENT\u0027 \u003d\u003d version[\u0027status\u0027]:"},{"line_number":38,"context_line":"                current_versions.append(version[\u0027id\u0027])"},{"line_number":39,"context_line":"        self.assertTrue(1 \u003d\u003d len(current_versions),"},{"line_number":40,"context_line":"                        \"{0:d} current database versions found - should be 1\""},{"line_number":41,"context_line":"                        .format(len(current_versions)))"},{"line_number":42,"context_line":"        self.assertIn(self.db_current_version, current_versions)"}],"source_content_type":"text/x-python","patch_set":6,"id":"9ad9bd40_2e7884c3","line":39,"updated":"2014-05-05 10:57:40.000000000","message":"Instead of assertTrue you could do a assertEqual. With that you woudn\u0027t need a dedicated error message.","commit_id":"50785eed17006875a0c9d4b6f4e99f31dbafd093"}]}