)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"5b85984b4f89e4b2c7714d285cd19482c3e3d656","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"164c9e17_0053be06","updated":"2022-08-06 15:36:57.000000000","message":"Tempest is not suitable place for the API negative tests when they can be added in unit or functional test at project side, https://docs.openstack.org/tempest/latest/HACKING.html#negative-tests","commit_id":"bfc9e600d95d563881a0fc7355f632106d172063"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"35cb840982b9f8679c4369d70f0b4a3770fdcb2f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"742e95a4_cdfe9e0f","in_reply_to":"0b9ae1e5_4ce92cb0","updated":"2022-08-06 16:20:33.000000000","message":"it can be done even in unit test. All the existing tests are because of interop need and unless there is a negative test request from interop we do not add more negative test.","commit_id":"bfc9e600d95d563881a0fc7355f632106d172063"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"9dd22ca1c1abe0f0987a1c4187b7f10cb8c7a26c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"0b9ae1e5_4ce92cb0","in_reply_to":"164c9e17_0053be06","updated":"2022-08-06 15:47:48.000000000","message":"Can this be done as functional test just because it doesn\u0027t touch internal functions? Isn\u0027t this test of the same type of the other tests around?","commit_id":"bfc9e600d95d563881a0fc7355f632106d172063"}],"tempest/api/volume/admin/test_volume_types_negative.py":[{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"8eaa258fbfa48af6f116e9395d6c1c770c5cda65","unresolved":true,"context_lines":[{"line_number":72,"context_line":""},{"line_number":73,"context_line":"    @decorators.attr(type\u003d[\u0027negative\u0027])"},{"line_number":74,"context_line":"    @decorators.idempotent_id(\u0027d2ab0d9c-a1d2-451e-8343-d7f3a1b5070f\u0027)"},{"line_number":75,"context_line":"    def test_create_volume_type_encryption_nonexistent_type_id(self):"},{"line_number":76,"context_line":"        \"\"\"\"Test use non-int value to test create encryption\"\"\""},{"line_number":77,"context_line":"        volume_type \u003d self.create_volume_type()"},{"line_number":78,"context_line":"        type_id \u003d volume_type[\u0027id\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"6234dc8b_5b4a3648","line":75,"in_reply_to":"40f31346_252397c5","updated":"2022-06-24 09:45:55.000000000","message":"\u003e pep8: F811 redefinition of unused \u0027test_create_volume_type_encryption_nonexistent_type_id\u0027 from line 60\n\nPlease define a unique test name.","commit_id":"20b01ab9263da7e371dfe952d3934043dffdc07e"}]}