Merge pull request #1436 from vojtechtrefny/main_coverage-open-fix

tests: Do not patch builtins.open directly

Author: vojtechtrefny

tests/unit_tests/devicelibs_test/lvm_test.py

@@ -9,32 +9,32 @@ class LVMTestCase(unittest.TestCase):
     def test_lvm_autoactivation(self):
         localconf = "global { event_activation = 0 }"
 
-        with patch("builtins.open", mock_open(read_data=localconf)):
+        with patch("blivet.devicelibs.lvm.open", mock_open(read_data=localconf)):
             # already disabled
             with self.assertRaises(RuntimeError):
                 lvm.disable_lvm_autoactivation()
 
         localconf = ""
-        with patch("builtins.open", mock_open(read_data=localconf)) as m:
+        with patch("blivet.devicelibs.lvm.open", mock_open(read_data=localconf)) as m:
             lvm.disable_lvm_autoactivation()
             m.assert_called_with("/etc/lvm/lvmlocal.conf", "a")
             handle = m()
             handle.write.assert_called_once_with("global { event_activation = 0 }")
 
         localconf = "test\ntest"
-        with patch("builtins.open", mock_open(read_data=localconf)) as m:
+        with patch("blivet.devicelibs.lvm.open", mock_open(read_data=localconf)) as m:
             # not disabled
             with self.assertRaises(RuntimeError):
                 lvm.reenable_lvm_autoactivation()
 
         localconf = "# global { event_activation = 0 }"
-        with patch("builtins.open", mock_open(read_data=localconf)) as m:
+        with patch("blivet.devicelibs.lvm.open", mock_open(read_data=localconf)) as m:
             # not disabled
             with self.assertRaises(RuntimeError):
                 lvm.reenable_lvm_autoactivation()
 
         localconf = "test\nglobal { event_activation = 0 }"
-        with patch("builtins.open", mock_open(read_data=localconf)) as m:
+        with patch("blivet.devicelibs.lvm.open", mock_open(read_data=localconf)) as m:
             lvm.reenable_lvm_autoactivation()
             m.assert_called_with("/etc/lvm/lvmlocal.conf", "w+")
             handle = m()