Thanks for submitting this. Let's add one unittest for this:
https://paste.ubuntu.com/p/vQfFfn57Xg/ diff --git a/tests/unittests/test_udev.py b/tests/unittests/test_udev.py index 919c7c0b..39c50e25 100644 --- a/tests/unittests/test_udev.py +++ b/tests/unittests/test_udev.py @@ -104,3 +104,15 @@ class TestUdevInfo(CiTestCase): m_subp.side_effect = util.ProcessExecutionError() with self.assertRaises(util.ProcessExecutionError): udevadm_info(mypath) + + @mock.patch('curtin.util.subp') + def test_udevadm_info_multiple_equals(self, m_subp): + """ udevadm_info handles parsing values with multiple '=' chars. """ + mypath = '/dev/nvme0n1' + m_subp.return_value = ( + "SCSI_IDENT_TARGET_VENDOR='clusterid=92901'", "") + info = udevadm_info(mypath) + m_subp.assert_called_with( + ['udevadm', 'info', '--query=property', '--export', mypath], + capture=True) + self.assertEqual({'SCSI_IDENT_TARGET_VENDOR': 'clusterid=92901'}, info) -- https://code.launchpad.net/~sbykov/curtin/+git/curtin/+merge/390527 Your team curtin developers is requested to review the proposed merge of ~sbykov/curtin:LP1895021 into curtin:master. -- Mailing list: https://launchpad.net/~curtin-dev Post to : [email protected] Unsubscribe : https://launchpad.net/~curtin-dev More help : https://help.launchpad.net/ListHelp
