TestOsOpsCommon is updated

dmitry-lipetsk · dmitry-lipetsk · commit c2c2d4cb6c13 · 2025-04-02T10:22:56.000+03:00
New tests:
- test_read_binary__spec__negative_offset
- test_get_file_size
diff --git a/tests/test_local.py b/tests/test_local.py
@@ -114,29 +114,6 @@ def test_read_binary__spec__unk_file(self):
                 match=re.escape("[Errno 2] No such file or directory: '/dummy'")):
             self.operations.read_binary("/dummy", 0)
 
-    def test_read_binary__spec__negative_offset(self):
-        """
-        Test LocalOperations::read_binary with negative offset.
-        """
-
-        with pytest.raises(
-                ValueError,
-                match=re.escape("Negative 'offset' is not supported.")):
-            self.operations.read_binary(__file__, -1)
-
-    def test_get_file_size(self):
-        """
-        Test LocalOperations::get_file_size.
-        """
-        filename = __file__  # current file
-
-        sz0 = os.path.getsize(filename)
-        assert type(sz0) == int  # noqa: E721
-
-        sz1 = self.operations.get_file_size(filename)
-        assert type(sz1) == int  # noqa: E721
-        assert sz1 == sz0
-
     def test_get_file_size__unk_file(self):
         """
         Test LocalOperations::get_file_size.
diff --git a/tests/test_os_ops_common.py b/tests/test_os_ops_common.py
@@ -120,6 +120,32 @@ def test_read_binary__spec(self, os_ops: OsOperations):
         assert type(response5) == bytes  # noqa: E721
         assert len(response5) == 0
 
+    def test_read_binary__spec__negative_offset(self, os_ops: OsOperations):
+        """
+        Test RemoteOperations::read_binary with negative offset.
+        """
+        assert isinstance(os_ops, OsOperations)
+
+        with pytest.raises(
+                ValueError,
+                match=re.escape("Negative 'offset' is not supported.")):
+            os_ops.read_binary(__file__, -1)
+
+    def test_get_file_size(self, os_ops: OsOperations):
+        """
+        Test RemoteOperations::get_file_size.
+        """
+        assert isinstance(os_ops, OsOperations)
+
+        filename = __file__  # current file
+
+        sz0 = os.path.getsize(filename)
+        assert type(sz0) == int  # noqa: E721
+
+        sz1 = os_ops.get_file_size(filename)
+        assert type(sz1) == int  # noqa: E721
+        assert sz1 == sz0
+
     def test_isfile_true(self, os_ops: OsOperations):
         """
         Test isfile for an existing file.
diff --git a/tests/test_remote.py b/tests/test_remote.py
@@ -320,29 +320,6 @@ def test_read_binary__spec__unk_file(self):
         assert "No such file or directory" in str(x.value)
         assert "/dummy" in str(x.value)
 
-    def test_read_binary__spec__negative_offset(self):
-        """
-        Test RemoteOperations::read_binary with negative offset.
-        """
-
-        with pytest.raises(
-                ValueError,
-                match=re.escape("Negative 'offset' is not supported.")):
-            self.operations.read_binary(__file__, -1)
-
-    def test_get_file_size(self):
-        """
-        Test RemoteOperations::get_file_size.
-        """
-        filename = __file__  # current file
-
-        sz0 = os.path.getsize(filename)
-        assert type(sz0) == int  # noqa: E721
-
-        sz1 = self.operations.get_file_size(filename)
-        assert type(sz1) == int  # noqa: E721
-        assert sz1 == sz0
-
     def test_get_file_size__unk_file(self):
         """
         Test RemoteOperations::get_file_size.
