For those last two, did you really want: "if persistent_xml_param and destination_xml:" to match the conditional we have on master?
I haven't heavily reviewed the test patches, but that looks good to me.
For those last two, did you really want: "if persistent_ xml_param and destination_xml:" to match the conditional we have on master?
I haven't heavily reviewed the test patches, but that looks good to me.