)]}'
{"mistral/tests/unit/engine/test_yaql_functions.py":[{"author":{"_account_id":9712,"name":"Dougal Matthews","email":"dougal.matthews@canonical.com","username":"dougal"},"change_message_id":"ee9aee28f4aaaf3430964fd8752984a5517605e3","unresolved":false,"context_lines":[{"line_number":485,"context_line":"              publish:"},{"line_number":486,"context_line":"                val1: \u003c% $.my_list !\u003d [] %\u003e"},{"line_number":487,"context_line":"                val2: \u003c% $.my_list.len() \u003e 0 %\u003e"},{"line_number":488,"context_line":"                val3: \u003c% $.my_list.len() \u003d 0 %\u003e"},{"line_number":489,"context_line":"        \"\"\""},{"line_number":490,"context_line":""},{"line_number":491,"context_line":"        wf_service.create_workflows(wf_text)"}],"source_content_type":"text/x-python","patch_set":6,"id":"9fdfeff1_0f8145fc","line":488,"updated":"2019-02-18 15:06:35.000000000","message":"Sorry I confused things with this suggestion. I don\u0027t think we need to include the len() expressions. I would be interested in seeing other tests with non-empty lists if you want to expand coverage beyond the specific tripleo issue.\n\ni.e. add this expression:\n\n\u003c% $.my_list2 !\u003d [1] %\u003e","commit_id":"f870e2d41bcd185b42d95101c0e661f81a272a95"},{"author":{"_account_id":9712,"name":"Dougal Matthews","email":"dougal.matthews@canonical.com","username":"dougal"},"change_message_id":"ee9aee28f4aaaf3430964fd8752984a5517605e3","unresolved":false,"context_lines":[{"line_number":493,"context_line":"        wf_ex \u003d self.engine.start_workflow("},{"line_number":494,"context_line":"            \u0027wf\u0027,"},{"line_number":495,"context_line":"            wf_input\u003d{"},{"line_number":496,"context_line":"                \u0027my_list\u0027: []"},{"line_number":497,"context_line":"            }"},{"line_number":498,"context_line":"        )"},{"line_number":499,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"9fdfeff1_2f868915","line":496,"updated":"2019-02-18 15:06:35.000000000","message":"and add input here:\n\n\"my_list2\": [1, ]","commit_id":"f870e2d41bcd185b42d95101c0e661f81a272a95"}],"mistral/utils/expression_utils.py":[{"author":{"_account_id":9712,"name":"Dougal Matthews","email":"dougal.matthews@canonical.com","username":"dougal"},"change_message_id":"ee9aee28f4aaaf3430964fd8752984a5517605e3","unresolved":false,"context_lines":[{"line_number":55,"context_line":""},{"line_number":56,"context_line":"    if (isinstance(obj, yaql_utils.SequenceType) and"},{"line_number":57,"context_line":"            not isinstance(obj, six.string_types) and"},{"line_number":58,"context_line":"            not isinstance(obj, tuple) and len(obj) !\u003d 0):"},{"line_number":59,"context_line":"        return list(rec(t, rec) for t in obj)"},{"line_number":60,"context_line":"    else:"},{"line_number":61,"context_line":"        return yaql_utils.convert_input_data(obj, rec)"}],"source_content_type":"text/x-python","patch_set":6,"id":"9fdfeff1_cfe33d06","line":58,"updated":"2019-02-18 15:06:35.000000000","message":"This is another special case handling, which I don\u0027t think is wise. For example, this expression will fail:\n\n\u003c% $.my_list !\u003d [1] %\u003e\n\nWith input:\n\n{\u0027my_list\u0027: [1, ]}","commit_id":"f870e2d41bcd185b42d95101c0e661f81a272a95"},{"author":{"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},"change_message_id":"8fdd22ec33302c523bb1d42fd1d47b3d0b71573e","unresolved":false,"context_lines":[{"line_number":55,"context_line":""},{"line_number":56,"context_line":"    if (isinstance(obj, yaql_utils.SequenceType) and"},{"line_number":57,"context_line":"            not isinstance(obj, six.string_types) and"},{"line_number":58,"context_line":"            not isinstance(obj, tuple) and len(obj) !\u003d 0):"},{"line_number":59,"context_line":"        return list(rec(t, rec) for t in obj)"},{"line_number":60,"context_line":"    else:"},{"line_number":61,"context_line":"        return yaql_utils.convert_input_data(obj, rec)"}],"source_content_type":"text/x-python","patch_set":6,"id":"9fdfeff1_2f48c985","line":58,"in_reply_to":"9fdfeff1_cfe33d06","updated":"2019-02-18 15:27:12.000000000","message":"Yes, I agree. I\u0027d rather find a reason first.","commit_id":"f870e2d41bcd185b42d95101c0e661f81a272a95"}]}