)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":35676,"name":"Max","email":"max.lamprecht@digits.schwarz","username":"lamprechtm"},"change_message_id":"76944cfa6ab0e4e1550eec8ab2b060fd5aabbae5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"8dc00323_f6da7a5c","updated":"2026-06-08 09:34:30.000000000","message":"recheck - tempest - fixtures._fixtures.timeout.TimeoutException","commit_id":"9f880495a1cfc03655eb98b0f817b32cf3dfda41"}],"cinder/tests/unit/test_evaluator.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"6b86ff28f25cd4ff3d0482b552c4dd270868e12c","unresolved":true,"context_lines":[{"line_number":91,"context_line":"        self.assertTrue(evaluator.evaluate(\"(1 \u003d\u003d 1) AND NOT (5 \u003d\u003d 2)\"))"},{"line_number":92,"context_line":"        self.assertTrue(evaluator.evaluate(\"(1 \u003d\u003d 1) OR NOT (2 \u003d\u003d 2) \""},{"line_number":93,"context_line":"                                           \"AND (5 \u003d\u003d 5)\"))"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"    def test_ternary_conditional(self):"},{"line_number":96,"context_line":"        self.assertEqual(5, evaluator.evaluate(\"(1 \u003c 2) ? 5 : 10\"))"},{"line_number":97,"context_line":"        self.assertEqual(10, evaluator.evaluate(\"(1 \u003e 2) ? 5 : 10\"))"}],"source_content_type":"text/x-python","patch_set":1,"id":"8bd36f62_a7da3c6a","line":94,"updated":"2026-06-02 13:53:57.000000000","message":"It would be good to add another test method here that tests calls to evaluator.evaluate() with the same input string a couple of times (to exercise the cache).","commit_id":"15e7a51b69f5076c0444e79efd9fc4876fcc86ee"},{"author":{"_account_id":35676,"name":"Max","email":"max.lamprecht@digits.schwarz","username":"lamprechtm"},"change_message_id":"049806d643b4ec79933916e6bbae2a2219589c19","unresolved":true,"context_lines":[{"line_number":91,"context_line":"        self.assertTrue(evaluator.evaluate(\"(1 \u003d\u003d 1) AND NOT (5 \u003d\u003d 2)\"))"},{"line_number":92,"context_line":"        self.assertTrue(evaluator.evaluate(\"(1 \u003d\u003d 1) OR NOT (2 \u003d\u003d 2) \""},{"line_number":93,"context_line":"                                           \"AND (5 \u003d\u003d 5)\"))"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"    def test_ternary_conditional(self):"},{"line_number":96,"context_line":"        self.assertEqual(5, evaluator.evaluate(\"(1 \u003c 2) ? 5 : 10\"))"},{"line_number":97,"context_line":"        self.assertEqual(10, evaluator.evaluate(\"(1 \u003e 2) ? 5 : 10\"))"}],"source_content_type":"text/x-python","patch_set":1,"id":"1f3e62ac_be4463e8","line":94,"in_reply_to":"8bd36f62_a7da3c6a","updated":"2026-06-02 14:48:40.000000000","message":"I added a test that tests if the cache is working and also if the result works as expected\n\n\nAdditionally I replaced parseAll with parse_all as it is deprecated:\nhttps://pyparsing-docs.readthedocs.io/en/latest/pyparsing.html#pyparsing.ParserElement.parse_string","commit_id":"15e7a51b69f5076c0444e79efd9fc4876fcc86ee"}]}