From 56f09057addb1f9457d2cf5bf50b5039394ea1b1 Mon Sep 17 00:00:00 2001 From: Guillaume Abrioux Date: Wed, 18 Feb 2026 13:41:37 +0100 Subject: [PATCH] ceph-volume: fix test_reject_readonly_device unit test this mocks "partitions" in device info so Device._parse() does not raise KeyError and both /dev/cdrom and /dev/sr0 so the test passes when cdrom is a symlink to sr0. Signed-off-by: Guillaume Abrioux --- src/ceph-volume/ceph_volume/tests/util/test_device.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/ceph-volume/ceph_volume/tests/util/test_device.py b/src/ceph-volume/ceph_volume/tests/util/test_device.py index e238cdfb667..1667979088f 100644 --- a/src/ceph-volume/ceph_volume/tests/util/test_device.py +++ b/src/ceph-volume/ceph_volume/tests/util/test_device.py @@ -306,9 +306,10 @@ class TestDevice(object): @patch("ceph_volume.util.disk.has_bluestore_label", lambda x: False) def test_reject_readonly_device(self, fake_call, device_info): - data = {"/dev/cdrom": {"ro": "1"}} + readonly_info = {"ro": "1", "partitions": {}} + data = {"/dev/cdrom": dict(readonly_info), "/dev/sr0": dict(readonly_info)} lsblk = {"TYPE": "disk", "NAME": "cdrom"} - device_info(devices=data,lsblk=lsblk) + device_info(devices=data, lsblk=lsblk) disk = device.Device("/dev/cdrom") assert not disk.available -- 2.47.3