)]}'
{"neutron_tempest_plugin/api/base.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"d8836edfaec8cbd34e61fb12bb3e45e4f000d86e","unresolved":true,"context_lines":[{"line_number":1544,"context_line":"                pagination_args[\u0027marker\u0027] \u003d resources[-1][\u0027id\u0027]"},{"line_number":1545,"context_line":"            body \u003d self.list_method(**pagination_args)"},{"line_number":1546,"context_line":"            resources_ \u003d self._extract_resources(body)"},{"line_number":1547,"context_line":"            self.assertGreaterEqual(1, len(resources_))"},{"line_number":1548,"context_line":"            resources.extend(resources_)"},{"line_number":1549,"context_line":"        return self._test_resources(resources)"},{"line_number":1550,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"32b6c35d_c7af0722","line":1547,"range":{"start_line":1547,"start_character":12,"end_line":1547,"end_character":55},"updated":"2024-08-26 11:28:42.000000000","message":"I think a short explanation in the code would be needed here. Just to know, if a future, why this \"assertGreaterEqual\" check instead of a straight \"assertEqual\"","commit_id":"84767524f2a3a59882579c5975b5c03084baa280"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"6cdbc1c64d6c731aa9bf51be6a0cd570db2bc5ea","unresolved":false,"context_lines":[{"line_number":1544,"context_line":"                pagination_args[\u0027marker\u0027] \u003d resources[-1][\u0027id\u0027]"},{"line_number":1545,"context_line":"            body \u003d self.list_method(**pagination_args)"},{"line_number":1546,"context_line":"            resources_ \u003d self._extract_resources(body)"},{"line_number":1547,"context_line":"            self.assertGreaterEqual(1, len(resources_))"},{"line_number":1548,"context_line":"            resources.extend(resources_)"},{"line_number":1549,"context_line":"        return self._test_resources(resources)"},{"line_number":1550,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"f9dac67b_e939e7ac","line":1547,"range":{"start_line":1547,"start_character":12,"end_line":1547,"end_character":55},"in_reply_to":"32b6c35d_c7af0722","updated":"2024-08-26 11:34:52.000000000","message":"Done","commit_id":"84767524f2a3a59882579c5975b5c03084baa280"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"0d9ed797b84752989c7ca1a615e0717f0f5cd43f","unresolved":true,"context_lines":[{"line_number":1546,"context_line":"            resources_ \u003d self._extract_resources(body)"},{"line_number":1547,"context_line":"            # Empty resource list can be returned when any concurrent"},{"line_number":1548,"context_line":"            # tests delete them"},{"line_number":1549,"context_line":"            self.assertGreaterEqual(1, len(resources_))"},{"line_number":1550,"context_line":"            resources.extend(resources_)"},{"line_number":1551,"context_line":"        return self._test_resources(resources)"},{"line_number":1552,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"d3fb7b18_91a1a2e5","line":1549,"updated":"2024-08-28 18:51:54.000000000","message":"I got confused since I think my brain saw this check as backwards with the first value being a constant, but it does work. This is what my brain expected:\n\nassertLessEqual(len(resources_), 1)","commit_id":"a8c221ca48d69d8df74713dcbf589559dab62496"}]}