From 3c63ab265786c2245e493140a81361e6ad1f7c27 Mon Sep 17 00:00:00 2001
From: def <dettlaff@riseup.net>
Date: Wed, 12 Oct 2022 18:07:39 +0400
Subject: [PATCH] fix tests

---
 tests/test_graphql/test_api_storage.py | 20 +++++++++-----------
 tests/test_graphql/test_job.py         |  4 +---
 2 files changed, 10 insertions(+), 14 deletions(-)

diff --git a/tests/test_graphql/test_api_storage.py b/tests/test_graphql/test_api_storage.py
index 94d0461..1fa2f78 100644
--- a/tests/test_graphql/test_api_storage.py
+++ b/tests/test_graphql/test_api_storage.py
@@ -1,13 +1,10 @@
 import pytest
-from selfprivacy_api.graphql.mutations.mutation_interface import (
-    GenericMutationReturn,
-)
 
 
 class BlockDeviceMockReturnNone:
     """Mock BlockDevices"""
 
-    def __init__(self, args, **kwargs):
+    def __init__(self, *args, **kwargs):
         self.args = args
         self.kwargs = kwargs
 
@@ -26,7 +23,7 @@ class BlockDeviceMockReturnNone:
 class BlockDeviceMockReturnTrue:
     """Mock BlockDevices"""
 
-    def __init__(self, args, **kwargs):
+    def __init__(self, *args, **kwargs):
         self.args = args
         self.kwargs = kwargs
 
@@ -45,7 +42,7 @@ class BlockDeviceMockReturnTrue:
 class BlockDeviceMockReturnFalse:
     """Mock BlockDevices"""
 
-    def __init__(self, args, **kwargs):
+    def __init__(self, *args, **kwargs):
         self.args = args
         self.kwargs = kwargs
 
@@ -62,8 +59,8 @@ class BlockDeviceMockReturnFalse:
 
 
 class BlockDevicesMockReturnTrue:
-    def get_block_device(self, name: str):
-        return True
+    def get_block_device(name: str):  # type: ignore
+        return BlockDeviceMockReturnTrue()
 
     def __new__(cls, *args, **kwargs):
         pass
@@ -73,7 +70,7 @@ class BlockDevicesMockReturnTrue:
 
 
 class BlockDevicesMockReturnNone:
-    def get_block_device(self, name: str):
+    def get_block_device(name: str):  # type: ignore
         return None
 
     def __new__(cls, *args, **kwargs):
@@ -86,7 +83,8 @@ class BlockDevicesMockReturnNone:
 @pytest.fixture
 def mock_block_devices_return_true(mocker):
     mock = mocker.patch(
-        "selfprivacy_api.utils.block_devices.BlockDevices",
+        "selfprivacy_api.graphql.mutations.storage_mutations.BlockDevices",
+        # "selfprivacy_api.utils.block_devices.BlockDevices",
         autospec=True,
         return_value=BlockDevicesMockReturnTrue,
     )
@@ -328,7 +326,7 @@ def test_graphql_unmount_volume_false(
     assert response.json()["data"]["unmountVolume"]["success"] is False
 
 
-def test_graphql_unmount_volume(authorized_client, mock_block_devices_return_none):
+def test_graphql_unmount_volume(authorized_client, mock_block_devices_return_true):
     response = authorized_client.post(
         "/graphql",
         json={
diff --git a/tests/test_graphql/test_job.py b/tests/test_graphql/test_job.py
index 0502406..dddb05c 100644
--- a/tests/test_graphql/test_job.py
+++ b/tests/test_graphql/test_job.py
@@ -94,11 +94,9 @@ def test_graphql_remove_job(
         json={
             "query": API_REMOVE_JOB_MUTATION,
             "variables": {
-                "Job": {
-                    "uid": "12345",
+                    "jobId": "12345",
                 },
             },
-        },
     )
     assert response.status_code == 200
     assert response.json().get("data") is None