[master 5/5] dammit

vathpela installerbot-noreply at redhat.com
Tue Oct 13 22:06:54 UTC 2015


From: Peter Jones <pjones at redhat.com>

---
 blivet/devicelibs/edd.py                           | 21 ++----
 .../host6/target6:0:0/6:0:0:0/block/sr0/device     |  1 +
 .../ata7/host6/target6:0:0/6:0:0:0/block/sr0/foo   |  0
 .../host6/target6:0:1/6:0:1:0/block/sdb/device     |  1 +
 .../ata7/host6/target6:0:1/6:0:1:0/block/sdb/foo   |  0
 .../0000:00:01.1/ata7/link7/ata_link/foo           |  0
 .../0000:00:01.1/ata7/link7/ata_link/link7/device  |  1 +
 .../ata7/link7/dev7.0/ata_device/dev7.0/device     |  1 +
 .../pci0000:00/0000:00:01.1/ata7/link7/dev7.0/foo  |  0
 .../ata7/link7/dev7.1/ata_device/dev7.1/device     |  1 +
 .../pci0000:00/0000:00:01.1/ata7/link7/dev7.1/foo  |  0
 .../host0/target0:0:0/0:0:0:0/block/sda/device     |  1 +
 .../ata1/host0/target0:0:0/0:0:0:0/block/sda/foo   |  0
 .../host9/target9:0:0/9:0:0:0/block/sdd/device     |  1 +
 .../host9/target9:0:0/9:0:0:0/block/sdd/foo        |  0
 .../0000:00:07.0/virtio1/block/vda/device          |  1 +
 .../pci0000:00/0000:00:07.0/virtio1/block/vda/foo  |  0
 .../host10/target10:0:1/10:0:1:0/block/sde/device  |  1 +
 .../host10/target10:0:1/10:0:1:0/block/sde/foo     |  0
 .../host8/target8:0:0/8:0:0:0/block/sdc/device     |  1 +
 .../host8/target8:0:0/8:0:0:0/block/sdc/foo        |  0
 tests/devicelibs_test/edd_test.py                  | 81 ++++++++++++++++------
 22 files changed, 77 insertions(+), 35 deletions(-)
 create mode 120000 tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:01.1/ata7/host6/target6:0:0/6:0:0:0/block/sr0/device
 delete mode 100644 tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:01.1/ata7/host6/target6:0:0/6:0:0:0/block/sr0/foo
 create mode 120000 tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:01.1/ata7/host6/target6:0:1/6:0:1:0/block/sdb/device
 delete mode 100644 tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:01.1/ata7/host6/target6:0:1/6:0:1:0/block/sdb/foo
 delete mode 100644 tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:01.1/ata7/link7/ata_link/foo
 create mode 120000 tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:01.1/ata7/link7/ata_link/link7/device
 create mode 120000 tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:01.1/ata7/link7/dev7.0/ata_device/dev7.0/device
 delete mode 100644 tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:01.1/ata7/link7/dev7.0/foo
 create mode 120000 tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:01.1/ata7/link7/dev7.1/ata_device/dev7.1/device
 delete mode 100644 tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:01.1/ata7/link7/dev7.1/foo
 create mode 120000 tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:03.0/ata1/host0/target0:0:0/0:0:0:0/block/sda/device
 delete mode 100644 tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:03.0/ata1/host0/target0:0:0/0:0:0:0/block/sda/foo
 create mode 120000 tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:06.7/usb1/1-1/1-1:1.0/host9/target9:0:0/9:0:0:0/block/sdd/device
 delete mode 100644 tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:06.7/usb1/1-1/1-1:1.0/host9/target9:0:0/9:0:0:0/block/sdd/foo
 create mode 120000 tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:07.0/virtio1/block/vda/device
 delete mode 100644 tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:07.0/virtio1/block/vda/foo
 create mode 120000 tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:0a.0/host10/target10:0:1/10:0:1:0/block/sde/device
 delete mode 100644 tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:0a.0/host10/target10:0:1/10:0:1:0/block/sde/foo
 create mode 120000 tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:0b.0/virtio3/host8/target8:0:0/8:0:0:0/block/sdc/device
 delete mode 100644 tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:0b.0/virtio3/host8/target8:0:0/8:0:0:0/block/sdc/foo

diff --git a/blivet/devicelibs/edd.py b/blivet/devicelibs/edd.py
index 7d43bac..2adb1ba 100644
--- a/blivet/devicelibs/edd.py
+++ b/blivet/devicelibs/edd.py
@@ -504,22 +504,15 @@ def match_via_mbrsigs(self, mbr_dict):
                 return name
         return None
 
-def biosdev_to_edd_dir(biosdev):
-    return "/sys/firmware/edd/int13_dev%(biosdev)x" % {
-        'biosdev' : biosdev
-        }
-
 def collect_edd_data():
     edd_data_dict = {}
-    # BIOS dev numbers are 0-127 , with 0x80 unset for floppies and set
-    # for hard drives.  In practice, the kernel has had EDDMAXNR (the limit
-    # on how many EDD structures it will load) set to 6 since before the
-    # import into git, so that's good enough.
-    for biosdev in range(0x80, 0x86):
-        sysfspath = biosdev_to_edd_dir(biosdev)
-        if not os.path.exists("%s/%s" % (fsroot, sysfspath[1:])):
-            break
-        edd_data_dict[biosdev] = EddEntry(sysfspath)
+    exp = re.compile(r'.*/int13_dev(\d+)$')
+    globstr = os.path.join(fsroot, "sys/firmware/edd/int13_dev*")
+    for path in glob.glob(globstr):
+        match = exp.match(path)
+        biosdev = int("0x%s" % (match.group(1),), base=16)
+        log.debug("edd: found device 0x%x at %s", biosdev, path[len(fsroot):])
+        edd_data_dict[biosdev] = EddEntry(path[len(fsroot):])
     return edd_data_dict
 
 def collect_mbrs(devices):
diff --git a/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:01.1/ata7/host6/target6:0:0/6:0:0:0/block/sr0/device b/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:01.1/ata7/host6/target6:0:0/6:0:0:0/block/sr0/device
new file mode 120000
index 0000000..5be613c
--- /dev/null
+++ b/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:01.1/ata7/host6/target6:0:0/6:0:0:0/block/sr0/device
@@ -0,0 +1 @@
+../../../6:0:0:0
\ No newline at end of file
diff --git a/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:01.1/ata7/host6/target6:0:0/6:0:0:0/block/sr0/foo b/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:01.1/ata7/host6/target6:0:0/6:0:0:0/block/sr0/foo
deleted file mode 100644
index e69de29..0000000
diff --git a/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:01.1/ata7/host6/target6:0:1/6:0:1:0/block/sdb/device b/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:01.1/ata7/host6/target6:0:1/6:0:1:0/block/sdb/device
new file mode 120000
index 0000000..420088f
--- /dev/null
+++ b/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:01.1/ata7/host6/target6:0:1/6:0:1:0/block/sdb/device
@@ -0,0 +1 @@
+../../../6:0:1:0
\ No newline at end of file
diff --git a/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:01.1/ata7/host6/target6:0:1/6:0:1:0/block/sdb/foo b/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:01.1/ata7/host6/target6:0:1/6:0:1:0/block/sdb/foo
deleted file mode 100644
index e69de29..0000000
diff --git a/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:01.1/ata7/link7/ata_link/foo b/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:01.1/ata7/link7/ata_link/foo
deleted file mode 100644
index e69de29..0000000
diff --git a/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:01.1/ata7/link7/ata_link/link7/device b/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:01.1/ata7/link7/ata_link/link7/device
new file mode 120000
index 0000000..5927252
--- /dev/null
+++ b/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:01.1/ata7/link7/ata_link/link7/device
@@ -0,0 +1 @@
+../../../link7
\ No newline at end of file
diff --git a/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:01.1/ata7/link7/dev7.0/ata_device/dev7.0/device b/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:01.1/ata7/link7/dev7.0/ata_device/dev7.0/device
new file mode 120000
index 0000000..79f3f95
--- /dev/null
+++ b/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:01.1/ata7/link7/dev7.0/ata_device/dev7.0/device
@@ -0,0 +1 @@
+../../../dev7.0
\ No newline at end of file
diff --git a/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:01.1/ata7/link7/dev7.0/foo b/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:01.1/ata7/link7/dev7.0/foo
deleted file mode 100644
index e69de29..0000000
diff --git a/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:01.1/ata7/link7/dev7.1/ata_device/dev7.1/device b/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:01.1/ata7/link7/dev7.1/ata_device/dev7.1/device
new file mode 120000
index 0000000..8ef79ff
--- /dev/null
+++ b/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:01.1/ata7/link7/dev7.1/ata_device/dev7.1/device
@@ -0,0 +1 @@
+../../../dev7.1
\ No newline at end of file
diff --git a/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:01.1/ata7/link7/dev7.1/foo b/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:01.1/ata7/link7/dev7.1/foo
deleted file mode 100644
index e69de29..0000000
diff --git a/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:03.0/ata1/host0/target0:0:0/0:0:0:0/block/sda/device b/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:03.0/ata1/host0/target0:0:0/0:0:0:0/block/sda/device
new file mode 120000
index 0000000..79ba802
--- /dev/null
+++ b/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:03.0/ata1/host0/target0:0:0/0:0:0:0/block/sda/device
@@ -0,0 +1 @@
+../../../0:0:0:0
\ No newline at end of file
diff --git a/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:03.0/ata1/host0/target0:0:0/0:0:0:0/block/sda/foo b/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:03.0/ata1/host0/target0:0:0/0:0:0:0/block/sda/foo
deleted file mode 100644
index e69de29..0000000
diff --git a/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:06.7/usb1/1-1/1-1:1.0/host9/target9:0:0/9:0:0:0/block/sdd/device b/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:06.7/usb1/1-1/1-1:1.0/host9/target9:0:0/9:0:0:0/block/sdd/device
new file mode 120000
index 0000000..f083126
--- /dev/null
+++ b/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:06.7/usb1/1-1/1-1:1.0/host9/target9:0:0/9:0:0:0/block/sdd/device
@@ -0,0 +1 @@
+../../../9:0:0:0/
\ No newline at end of file
diff --git a/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:06.7/usb1/1-1/1-1:1.0/host9/target9:0:0/9:0:0:0/block/sdd/foo b/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:06.7/usb1/1-1/1-1:1.0/host9/target9:0:0/9:0:0:0/block/sdd/foo
deleted file mode 100644
index e69de29..0000000
diff --git a/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:07.0/virtio1/block/vda/device b/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:07.0/virtio1/block/vda/device
new file mode 120000
index 0000000..9cc4643
--- /dev/null
+++ b/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:07.0/virtio1/block/vda/device
@@ -0,0 +1 @@
+../../../virtio1
\ No newline at end of file
diff --git a/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:07.0/virtio1/block/vda/foo b/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:07.0/virtio1/block/vda/foo
deleted file mode 100644
index e69de29..0000000
diff --git a/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:0a.0/host10/target10:0:1/10:0:1:0/block/sde/device b/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:0a.0/host10/target10:0:1/10:0:1:0/block/sde/device
new file mode 120000
index 0000000..7613635
--- /dev/null
+++ b/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:0a.0/host10/target10:0:1/10:0:1:0/block/sde/device
@@ -0,0 +1 @@
+../../../10:0:1:0
\ No newline at end of file
diff --git a/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:0a.0/host10/target10:0:1/10:0:1:0/block/sde/foo b/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:0a.0/host10/target10:0:1/10:0:1:0/block/sde/foo
deleted file mode 100644
index e69de29..0000000
diff --git a/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:0b.0/virtio3/host8/target8:0:0/8:0:0:0/block/sdc/device b/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:0b.0/virtio3/host8/target8:0:0/8:0:0:0/block/sdc/device
new file mode 120000
index 0000000..04ec5f3
--- /dev/null
+++ b/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:0b.0/virtio3/host8/target8:0:0/8:0:0:0/block/sdc/device
@@ -0,0 +1 @@
+../../../8:0:0:0
\ No newline at end of file
diff --git a/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:0b.0/virtio3/host8/target8:0:0/8:0:0:0/block/sdc/foo b/tests/devicelibs_test/edd_data/absurd_virt/sys/devices/pci0000:00/0000:00:0b.0/virtio3/host8/target8:0:0/8:0:0:0/block/sdc/foo
deleted file mode 100644
index e69de29..0000000
diff --git a/tests/devicelibs_test/edd_test.py b/tests/devicelibs_test/edd_test.py
index dfb40f5..36629e8 100644
--- a/tests/devicelibs_test/edd_test.py
+++ b/tests/devicelibs_test/edd_test.py
@@ -52,8 +52,8 @@ def setUp(self):
         edd.log.debug = mock.MagicMock(name='debug')
 
     def tearDown(self):
-        super(EddTestCase, self).tearDown()
         edd.log = self._edd_logger
+        super(EddTestCase, self).tearDown()
 
     def _set_fs_root(self, edd_module, fsroot):
         if fsroot is None:
@@ -64,13 +64,18 @@ def _set_fs_root(self, edd_module, fsroot):
 
     def _respool_logs(self):
         log = edd.log
-        edd.log = mock.MagicMock(name='log')
         for logname in ["debug", "info", "warning", "error"]:
             logger = getattr(log, logname)
             newlogger = getattr(self._edd_logger, logname)
             for call in logger.call_args_list:
                 newlogger(*call[0])
-            setattr(edd.log, logname, mock.MagicMock(name=logname))
+
+    def _clear_logs(self):
+        edd.log = mock.MagicMock(name='log')
+        edd.log.info = mock.MagicMock(name='info')
+        edd.log.warning = mock.MagicMock(name='warning')
+        edd.log.error = mock.MagicMock(name='error')
+        edd.log.debug = mock.MagicMock(name='debug')
 
     def _check_logs(self, debugs=None, infos=None, warnings=None, errors=None):
         for (left, right) in ((debugs, edd.log.debug),
@@ -83,16 +88,6 @@ def _check_logs(self, debugs=None, infos=None, warnings=None, errors=None):
             right.sort()
             self.assertEqual(left, right)
 
-    def test_biosdev_to_edd_dir(self):
-        self._set_fs_root(edd, None)
-        path = edd.biosdev_to_edd_dir(138)
-        self.assertEqual("/sys/firmware/edd/int13_dev8a", path)
-        self.assertEqual(edd.log.debug.called, False)
-        self.assertEqual(edd.log.info.called, False)
-        self.assertEqual(edd.log.warning.called, False)
-        self.assertEqual(edd.log.error.called, False)
-        self._respool_logs()
-
     def test_collect_edd_data_sata_usb(self):
         # test with sata sda, usb sdb
         self._set_fs_root(edd, "sata_usb")
@@ -111,15 +106,21 @@ def test_collect_edd_data_sata_usb(self):
             }
 
         edd_dict = edd.collect_edd_data()
+        self._respool_logs()
         self.assertEqual(len(edd_dict), 2)
         self.assertEqual(fakeedd[0x80], edd_dict[0x80])
         self.assertEqual(fakeedd[0x81], edd_dict[0x81])
-        self._check_logs()
-        self.assertEqual(edd.log.debug.called, False)
+        debugs = [
+            ("edd: found device 0x%x at %s", 0x80,
+                    "/sys/firmware/edd/int13_dev80"),
+            ("edd: found device 0x%x at %s", 0x81,
+                    "/sys/firmware/edd/int13_dev81"),
+            ]
+        self._check_logs(debugs=debugs)
         self.assertEqual(edd.log.info.called, False)
         self.assertEqual(edd.log.warning.called, False)
         self.assertEqual(edd.log.error.called, False)
-        self._respool_logs()
+        self._clear_logs()
 
     def test_get_edd_dict_sata_usb(self):
         # test with sata sda, usb sdb
@@ -150,6 +151,10 @@ def test_get_edd_dict_sata_usb(self):
         debugs = [
             ("edd: data extracted from 0x%x:\n%s", 0x80, fakeedd[0x80]),
             ("edd: data extracted from 0x%x:\n%s", 0x81, fakeedd[0x81]),
+            ("edd: found device 0x%x at %s", 0x80,
+                    "/sys/firmware/edd/int13_dev80"),
+            ("edd: found device 0x%x at %s", 0x81,
+                    "/sys/firmware/edd/int13_dev81"),
             ]
         infos = [
             ("edd: MBR signature on %s is zero. new disk image?", "sda"),
@@ -162,9 +167,10 @@ def test_get_edd_dict_sata_usb(self):
                 "/sys/firmware/edd/int13_dev81"),
             ("edd: interface details: %s", "USB     \tserial_number: 30302e31"),
             ]
+        self._respool_logs()
         self._check_logs(debugs, infos, warnings)
         self.assertEqual(edd.log.error.called, False)
-        self._respool_logs()
+        self._clear_logs()
 
     def test_collect_edd_data_absurd_virt(self):
         self._set_fs_root(edd, "absurd_virt")
@@ -198,6 +204,7 @@ def test_collect_edd_data_absurd_virt(self):
                                sysfspath="/sys/firmware/edd/int13_dev85"),
             }
         edd_dict = edd.collect_edd_data()
+        self._respool_logs()
         self.assertEqual(len(edd_dict), 6)
         self.assertEqual(fakeedd[0x80], edd_dict[0x80])
         self.assertEqual(fakeedd[0x81], edd_dict[0x81])
@@ -205,12 +212,25 @@ def test_collect_edd_data_absurd_virt(self):
         self.assertEqual(fakeedd[0x83], edd_dict[0x83])
         self.assertEqual(fakeedd[0x84], edd_dict[0x84])
         self.assertEqual(fakeedd[0x85], edd_dict[0x85])
-        self._check_logs()
-        self.assertEqual(edd.log.debug.called, False)
+        debugs = [
+            ("edd: found device 0x%x at %s", 0x80,
+                    "/sys/firmware/edd/int13_dev80"),
+            ("edd: found device 0x%x at %s", 0x81,
+                    "/sys/firmware/edd/int13_dev81"),
+            ("edd: found device 0x%x at %s", 0x82,
+                    "/sys/firmware/edd/int13_dev82"),
+            ("edd: found device 0x%x at %s", 0x83,
+                    "/sys/firmware/edd/int13_dev83"),
+            ("edd: found device 0x%x at %s", 0x84,
+                    "/sys/firmware/edd/int13_dev84"),
+            ("edd: found device 0x%x at %s", 0x85,
+                    "/sys/firmware/edd/int13_dev85"),
+            ]
+        self._check_logs(debugs=debugs)
         self.assertEqual(edd.log.info.called, False)
         self.assertEqual(edd.log.warning.called, False)
         self.assertEqual(edd.log.error.called, False)
-        self._respool_logs()
+        self._clear_logs()
 
     def test_get_edd_dict_absurd_virt(self):
         self._set_fs_root(edd, "absurd_virt")
@@ -276,6 +296,18 @@ def test_get_edd_dict_absurd_virt(self):
             ("edd: data extracted from 0x%x:\n%s", 0x83, fakeedd[0x83]),
             ("edd: data extracted from 0x%x:\n%s", 0x84, fakeedd[0x84]),
             ("edd: data extracted from 0x%x:\n%s", 0x85, fakeedd[0x85]),
+            ("edd: found device 0x%x at %s", 0x80,
+                    "/sys/firmware/edd/int13_dev80"),
+            ("edd: found device 0x%x at %s", 0x81,
+                    "/sys/firmware/edd/int13_dev81"),
+            ("edd: found device 0x%x at %s", 0x82,
+                    "/sys/firmware/edd/int13_dev82"),
+            ("edd: found device 0x%x at %s", 0x83,
+                    "/sys/firmware/edd/int13_dev83"),
+            ("edd: found device 0x%x at %s", 0x84,
+                    "/sys/firmware/edd/int13_dev84"),
+            ("edd: found device 0x%x at %s", 0x85,
+                    "/sys/firmware/edd/int13_dev85"),
             ]
         infos = [
             ("edd: collected mbr signatures: %s", { 'vda': '0x86531966',
@@ -292,7 +324,14 @@ def test_get_edd_dict_absurd_virt(self):
             ("edd: matched 0x%x to %s using PCI dev", 0x84, "sdc"),
             ("edd: matched 0x%x to %s using MBR sig", 0x85, "sdd"),
             ]
+        self._respool_logs()
         self._check_logs(debugs, infos)
         self.assertEqual(edd.log.warning.called, False)
         self.assertEqual(edd.log.error.called, False)
-        self._respool_logs()
+        self._clear_logs()
+
+try:
+    ws = os.environ['WORKSPACE']
+except KeyError:
+    ws = "/tmp"
+blivet.util.set_up_logging(log_file=os.path.join(ws, "blivet.log"))


-- 
To view this commit on github, visit https://github.com/rhinstaller/blivet/commit/ffe8c332f189bf235d751c7c40e3dabe458057d3


More information about the anaconda-patches mailing list