attempt to fix tests

Author: VitorVieiraZ

mergin/test/test_client.py

@@ -78,12 +78,12 @@ def mcStorage(request):
             break
     client_workspace_id = client_workspace["id"]
     client_workspace_storage = client_workspace["storage"]
-
+    
     def teardown():
         # back to original values... (1 project, api allowed ...)
         client.patch(
             f"/v1/tests/workspaces/{client_workspace_id}",
-            {"limits_override": {"storage": client_workspace_storage, "projects": 1, "api_allowed": True}},
+            {"limits_override": {"storage": client_workspace_storage, "projects": 1, "monthly_contributors": 500, "api_allowed": True}},
             {"Content-Type": "application/json"},
         )
 
@@ -2714,19 +2714,27 @@ def test_error_projects_limit_hit(mcStorage: MerginClient):
 
 
 def test_error_monthly_contributors_limit_hit(mcStorage: MerginClient):
-    test_project = "test_another_project_above_monthly_contributors_limit_hit"
+    test_project = "test_monthly_contributors_limit"
     test_project_fullname = STORAGE_WORKSPACE + "/" + test_project
 
     client_workspace = None
     for workspace in mcStorage.workspaces_list():
         if workspace["name"] == STORAGE_WORKSPACE:
             client_workspace = workspace
             break
-
     client_workspace_id = client_workspace["id"]
+    client_workspace_storage = client_workspace["storage"]
+
     mcStorage.patch(
         f"/v1/tests/workspaces/{client_workspace_id}",
-        {"limits_override": {"monthly_contributors": 0, "api_allowed": True}},
+        {
+            "limits_override": {
+                "storage": client_workspace_storage,
+                "projects": 50,
+                "monthly_contributors": 0,
+                "api_allowed": True
+            }
+        },
         {"Content-Type": "application/json"},
     )
 
@@ -2737,10 +2745,9 @@ def test_error_monthly_contributors_limit_hit(mcStorage: MerginClient):
 
     assert e.value.server_code == ErrorCode.MonthlyContributorsLimitHit.value
     assert e.value.detail == (
-        "Maximum number of workspace contributors is reached. "
-        "Please upgrade your subscription to push changes or create projects."
+       "Maximum number of workspace contributors is reached. Please upgrade your subscription to push changes or create projects. (MonthlyContributorsLimitHit)"
     )
     assert e.value.http_error == 422
     assert e.value.http_method == "POST"
-    assert e.value.url == f"{mc.url}v1/project/{API_USER}"
     assert e.value.contributors_quota == 0
+    assert e.value.url == f"{mcStorage.url}v1/project/testpluginstorage"