)]}'
{"neutron_tempest_plugin/api/test_security_groups.py":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"3534d850a9f5114d6075ccaa7ee8ac42b7a7cd6c","unresolved":false,"context_lines":[{"line_number":319,"context_line":"    def test_sg_rules_quota_decrease_less_than_created(self):"},{"line_number":320,"context_line":"        self._create_max_allowed_sg_rules_amount()"},{"line_number":321,"context_line":"        new_quota \u003d self._decrease_sg_rules_quota()"},{"line_number":322,"context_line":"        self.assertEqual(self._get_sg_rules_quota(), new_quota)"},{"line_number":323,"context_line":""},{"line_number":324,"context_line":""},{"line_number":325,"context_line":"class SecGroupProtocolTest(base.BaseNetworkTest):"}],"source_content_type":"text/x-python","patch_set":1,"id":"1f493fa4_7a3e10db","line":322,"updated":"2020-05-03 15:54:13.000000000","message":"maybe we should also check here if all already created groups are still there, what do You think about such additional assertion?","commit_id":"5f9c076261e402b3466e524433f5b4a044a1ab01"},{"author":{"_account_id":19118,"name":"Eran Kuris","email":"ekuris@redhat.com","username":"ekuris"},"change_message_id":"97026812fc875ad937bb68f709b3f2603a9eef48","unresolved":false,"context_lines":[{"line_number":319,"context_line":"    def test_sg_rules_quota_decrease_less_than_created(self):"},{"line_number":320,"context_line":"        self._create_max_allowed_sg_rules_amount()"},{"line_number":321,"context_line":"        new_quota \u003d self._decrease_sg_rules_quota()"},{"line_number":322,"context_line":"        self.assertEqual(self._get_sg_rules_quota(), new_quota)"},{"line_number":323,"context_line":""},{"line_number":324,"context_line":""},{"line_number":325,"context_line":"class SecGroupProtocolTest(base.BaseNetworkTest):"}],"source_content_type":"text/x-python","patch_set":1,"id":"1f493fa4_b6170c52","line":322,"in_reply_to":"1f493fa4_7a3e10db","updated":"2020-05-04 12:23:41.000000000","message":"to be honest I don\u0027t see any reason to do that because first of all I am not touching the group that already created and also its not the purpose of the test. If you see it as mandatory I can do that but I think we can leave the test as is. What do you think?","commit_id":"5f9c076261e402b3466e524433f5b4a044a1ab01"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"b096f438490c7725b5cc06b2cd0e5bec591d17bb","unresolved":false,"context_lines":[{"line_number":319,"context_line":"    def test_sg_rules_quota_decrease_less_than_created(self):"},{"line_number":320,"context_line":"        self._create_max_allowed_sg_rules_amount()"},{"line_number":321,"context_line":"        new_quota \u003d self._decrease_sg_rules_quota()"},{"line_number":322,"context_line":"        self.assertEqual(self._get_sg_rules_quota(), new_quota)"},{"line_number":323,"context_line":""},{"line_number":324,"context_line":""},{"line_number":325,"context_line":"class SecGroupProtocolTest(base.BaseNetworkTest):"}],"source_content_type":"text/x-python","patch_set":1,"id":"1f493fa4_8be3b4a4","line":322,"in_reply_to":"1f493fa4_b6170c52","updated":"2020-05-05 07:10:12.000000000","message":"it\u0027s not mandatory. Lets go with the test like it is now.","commit_id":"5f9c076261e402b3466e524433f5b4a044a1ab01"}]}