)]}'
{"mistral_tempest_tests/tests/api/v2/test_mistral_basic_v2.py":[{"author":{"_account_id":8592,"name":"Anastasia Kuznetsova","email":"akuznetsova@mirantis.com","username":"nastya-kuz"},"change_message_id":"20a3413dae4b5dbde1dd7c901c7208892d14dc24","unresolved":false,"context_lines":[{"line_number":167,"context_line":""},{"line_number":168,"context_line":"        for wf in body[\u0027workflows\u0027]:"},{"line_number":169,"context_line":"            for key in [\u0027id\u0027, \u0027name\u0027]:"},{"line_number":170,"context_line":"                self.assertIn(key, list(wf))"},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":173,"context_line":"    def test_get_list_workflows_with_pagination(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a5ae5dd_5ed2cb01","line":170,"range":{"start_line":170,"start_character":15,"end_line":170,"end_character":44},"updated":"2016-02-04 09:14:15.000000000","message":"Agreed with hardik, now it is another check, as far as I understand we need to check that wfs in returned list have only two fields \u0027id\u0027 and \u0027name\u0027. After your change wf with fields \u0027id\u0027, \u0027name\u0027, \u0027something_else\u0027 also will pass the test.","commit_id":"48b479867c2f75f0c256b77cebbdc5ec0b413ec8"},{"author":{"_account_id":18238,"name":"Hardik Parekh","email":"hardik.parekh047@gmail.com","username":"hparekh"},"change_message_id":"2f3ba2db668a8664aaccd61af739926d59812f7f","unresolved":false,"context_lines":[{"line_number":165,"context_line":""},{"line_number":166,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":167,"context_line":""},{"line_number":168,"context_line":"        for wf in body[\u0027workflows\u0027]:"},{"line_number":169,"context_line":"            for key in [\u0027id\u0027, \u0027name\u0027]:"},{"line_number":170,"context_line":"                self.assertIn(key, list(wf))"},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":173,"context_line":"    def test_get_list_workflows_with_pagination(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a5ae5dd_1b9d4659","line":170,"range":{"start_line":168,"start_character":4,"end_line":170,"end_character":44},"updated":"2016-02-04 08:37:44.000000000","message":"I think this test was fail due to order of the values.\nThis fix will check whether id and name field is exist in list or not. It is not checking whether two list is same or not. \nHow about using \u0027sorted\u0027 as following:\nself.assertListEqual(sorted([\u0027id\u0027, \u0027name\u0027]), sorted(wf.keys()))","commit_id":"48b479867c2f75f0c256b77cebbdc5ec0b413ec8"},{"author":{"_account_id":6732,"name":"Lingxian Kong","email":"anlin.kong@gmail.com","username":"kong"},"change_message_id":"47453c1c6bc9323836a72252cb77d77988ff066f","unresolved":false,"context_lines":[{"line_number":167,"context_line":""},{"line_number":168,"context_line":"        for wf in body[\u0027workflows\u0027]:"},{"line_number":169,"context_line":"            for key in [\u0027id\u0027, \u0027name\u0027]:"},{"line_number":170,"context_line":"                self.assertIn(key, list(wf))"},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":173,"context_line":"    def test_get_list_workflows_with_pagination(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a5ae5dd_b023e8a6","line":170,"range":{"start_line":170,"start_character":15,"end_line":170,"end_character":44},"in_reply_to":"5a5ae5dd_5ed2cb01","updated":"2016-02-04 10:45:03.000000000","message":"Yes, both of you are right. I\u0027ll fix that, thanks!","commit_id":"48b479867c2f75f0c256b77cebbdc5ec0b413ec8"}]}