Remove async keyword from tests that don't require async features

Co-authored-by: alvarolopez <[email protected]>

Author: Copilot

deepaas/tests/test_cmd.py

@@ -37,7 +37,7 @@ def uvicorn_run_fixture():
     return mock.MagicMock()
 
 
-async def test_run(monkeypatch, fastapi_app_fixture, uvicorn_run_fixture):
+def test_run(monkeypatch, fastapi_app_fixture, uvicorn_run_fixture):
     """Test the run command."""
     # uvicorn_run_fixture = mock.MagicMock()
     monkeypatch.setattr(deepaas.cmd._shutdown, "handle_signals", mock.MagicMock())
@@ -72,7 +72,7 @@ def set_flag(flag, value):
         CONF.clear_override(flag)
 
 
-async def test_run_custom_ip_port(
+def test_run_custom_ip_port(
     monkeypatch, cfg_fixture, fastapi_app_fixture, uvicorn_run_fixture
 ):
     """Run the cmd line with a custom IP and port."""
@@ -93,7 +93,7 @@ async def test_run_custom_ip_port(
     uvicorn_run_fixture.assert_called_with(fastapi_app_fixture(), host=ip, port=port)
 
 
-async def test_custom_base_path(monkeypatch, cfg_fixture, fastapi_app_fixture):
+def test_custom_base_path(monkeypatch, cfg_fixture, fastapi_app_fixture):
     """Run the cmd line with a custom base path."""
     monkeypatch.setattr(deepaas.config, "setup", lambda x: None)
 
@@ -112,7 +112,7 @@ async def test_custom_base_path(monkeypatch, cfg_fixture, fastapi_app_fixture):
     )
 
 
-async def test_custom_base_path_no_slash(monkeypatch, cfg_fixture, fastapi_app_fixture):
+def test_custom_base_path_no_slash(monkeypatch, cfg_fixture, fastapi_app_fixture):
     """Run the cmd line with a custom base path without a slash."""
     monkeypatch.setattr(deepaas.config, "setup", lambda x: None)
 
@@ -134,7 +134,7 @@ async def test_custom_base_path_no_slash(monkeypatch, cfg_fixture, fastapi_app_f
     )
 
 
-async def test_doc_endpoint_disabled(monkeypatch, cfg_fixture, fastapi_app_fixture):
+def test_doc_endpoint_disabled(monkeypatch, cfg_fixture, fastapi_app_fixture):
     """Run the cmd line with doc endpoint disabled."""
     monkeypatch.setattr(deepaas.config, "setup", lambda x: None)
 

deepaas/tests/test_v2_models.py

@@ -30,7 +30,7 @@
 
 
 @pytest.fixture
-async def mocks(monkeypatch):
+def mocks(monkeypatch):
     model_name = uuid.uuid4().hex
     monkeypatch.setattr(
         deepaas.model.loading,
@@ -80,15 +80,15 @@ def warm(self, **kwargs):
         m.get_predict_args()
 
 
-async def test_bad_schema(mocks):
+def test_bad_schema(mocks):
     class Model(object):
         schema = []
 
     with pytest.raises(exceptions.ModelResponseValidationError):
         v2_wrapper.ModelWrapper("test", Model())
 
 
-async def test_validate_no_schema(mocks):
+def test_validate_no_schema(mocks):
     class Model(object):
         schema = None
 
@@ -97,15 +97,15 @@ class Model(object):
         wrapper.validate_response(None)
 
 
-async def test_invalid_schema(mocks):
+def test_invalid_schema(mocks):
     class Model(object):
         schema = object()
 
     with pytest.raises(exceptions.ModelResponseValidationError):
         v2_wrapper.ModelWrapper("test", Model())
 
 
-async def test_marshmallow_schema(mocks):
+def test_marshmallow_schema(mocks):
     class Schema(marshmallow.Schema):
         foo = m_fields.Str()
 
@@ -119,7 +119,7 @@ class Model(object):
         wrapper.validate_response({"foo": 1.0})
 
 
-async def test_dict_schema(mocks):
+def test_dict_schema(mocks):
     class Model(object):
         schema = {"foo": m_fields.Str()}
 
@@ -135,7 +135,7 @@ def model():
     return fake_v2_model.TestModel()
 
 
-async def test_dummy_model(model, mocks):
+def test_dummy_model(model, mocks):
     pred = model.predict()
     pred.pop("data")
     assert pred == {
@@ -191,7 +191,7 @@ async def test_model_with_not_implemented_attributes_and_wrapper(mocks):
         assert isinstance(val, fields.Field)
 
 
-async def test_loading_ok(mocks):
+def test_loading_ok(mocks):
     deepaas.model.v2.load_model()
 
     m = deepaas.model.v2.MODEL
@@ -200,7 +200,7 @@ async def test_loading_ok(mocks):
     #     assert isinstance(m, v2_wrapper.ModelWrapper)
 
 
-async def test_loading_ok_singleton(mocks, monkeypatch):
+def test_loading_ok_singleton(mocks, monkeypatch):
     deepaas.model.v2.load_model()
     new_model_name = uuid.uuid4().hex
     monkeypatch.setattr(