|
 |
aebde2 |
From c21ad308b1313a35cafa1664e5eb4772925bc005 Mon Sep 17 00:00:00 2001
|
|
|
aebde2 |
From: Tomas Bzatek <tbzatek@redhat.com>
|
|
|
aebde2 |
Date: Thu, 22 Apr 2021 18:05:29 +0200
|
|
|
aebde2 |
Subject: [PATCH 1/2] tests: Mark Drive.ATA tests as unstable
|
|
|
aebde2 |
|
|
|
aebde2 |
Some of the tests operate on physical ATA drives, comparing values between
|
|
|
aebde2 |
smartctl output and udisks. Different libraries used, different approach
|
|
|
aebde2 |
to retrieve some ATA features and values. Turned out this is not working
|
|
|
aebde2 |
correctly on some SATA disks with each approach giving slightly different
|
|
|
aebde2 |
results, presumably for the quirks in place.
|
|
|
aebde2 |
---
|
|
|
aebde2 |
src/tests/dbus-tests/test_drive_ata.py | 8 +++++---
|
|
|
aebde2 |
1 file changed, 5 insertions(+), 3 deletions(-)
|
|
|
aebde2 |
|
|
|
aebde2 |
diff --git a/src/tests/dbus-tests/test_drive_ata.py b/src/tests/dbus-tests/test_drive_ata.py
|
|
|
aebde2 |
index 3187367e..e91bd02f 100644
|
|
|
aebde2 |
--- a/src/tests/dbus-tests/test_drive_ata.py
|
|
|
aebde2 |
+++ b/src/tests/dbus-tests/test_drive_ata.py
|
|
|
aebde2 |
@@ -4,7 +4,7 @@ import re
|
|
|
aebde2 |
import unittest
|
|
|
aebde2 |
import time
|
|
|
aebde2 |
|
|
|
aebde2 |
-from udiskstestcase import UdisksTestCase
|
|
|
aebde2 |
+import udiskstestcase
|
|
|
aebde2 |
|
|
|
aebde2 |
SMART_CMDLINE_FAIL = 1 << 0
|
|
|
aebde2 |
SMART_OPEN_READ_FAIL = 1 << 1
|
|
|
aebde2 |
@@ -32,7 +32,7 @@ def _get_sata_disks():
|
|
|
aebde2 |
|
|
|
aebde2 |
|
|
|
aebde2 |
for disk in _get_sata_disks():
|
|
|
aebde2 |
- ret, out = UdisksTestCase.run_command("smartctl -a /dev/%s" % disk)
|
|
|
aebde2 |
+ ret, out = udiskstestcase.UdisksTestCase.run_command("smartctl -a /dev/%s" % disk)
|
|
|
aebde2 |
|
|
|
aebde2 |
# Only the following bits in the exit status mean the device failed to
|
|
|
aebde2 |
# provide valid SMART data, others may be set for different reasons (see
|
|
|
aebde2 |
@@ -46,7 +46,7 @@ for disk in _get_sata_disks():
|
|
|
aebde2 |
else:
|
|
|
aebde2 |
smart_unsupported.add(disk)
|
|
|
aebde2 |
|
|
|
aebde2 |
-class UdisksDriveAtaTest(UdisksTestCase):
|
|
|
aebde2 |
+class UdisksDriveAtaTest(udiskstestcase.UdisksTestCase):
|
|
|
aebde2 |
'''Noninvasive tests for the Drive.Ata interface'''
|
|
|
aebde2 |
|
|
|
aebde2 |
def get_smart_setting(self, disk, attr, out_prefix):
|
|
|
aebde2 |
@@ -102,6 +102,7 @@ class UdisksDriveAtaTest(UdisksTestCase):
|
|
|
aebde2 |
intro_data = drive_intro.Introspect()
|
|
|
aebde2 |
self.assertNotIn('interface name="org.freedesktop.UDisks2.Drive.Ata"', intro_data)
|
|
|
aebde2 |
|
|
|
aebde2 |
+ @udiskstestcase.tag_test(udiskstestcase.TestTags.UNSTABLE)
|
|
|
aebde2 |
@unittest.skipUnless(smart_supported, "No disks supporting S.M.A.R.T. available")
|
|
|
aebde2 |
def test_properties(self):
|
|
|
aebde2 |
for disk in smart_supported:
|
|
|
aebde2 |
@@ -148,6 +149,7 @@ class UdisksDriveAtaTest(UdisksTestCase):
|
|
|
aebde2 |
# ninth field is the raw value
|
|
|
aebde2 |
self.assertEqual(int(pwon_s.value / 3600), int(pwon_attr[8]))
|
|
|
aebde2 |
|
|
|
aebde2 |
+ @udiskstestcase.tag_test(udiskstestcase.TestTags.UNSTABLE)
|
|
|
aebde2 |
@unittest.skipUnless(smart_supported, "No disks supporting S.M.A.R.T. available")
|
|
|
aebde2 |
def test_smart_get_attributes(self):
|
|
|
aebde2 |
for disk in smart_supported:
|
|
|
aebde2 |
--
|
|
|
aebde2 |
2.30.2
|
|
|
aebde2 |
|