ClearML/clearml-server
1
0
Fork 0
mirror of https://github.com/clearml/clearml-server synced 2025-03-03 02:33:02 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix internal error returned to clients

Browse Source
This commit is contained in:
allegroai 2022-07-08 18:03:38 +03:00
parent c6f1beafdd
commit 3695f25a5f
2 changed files with 13 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
apiserver/service_repo/service_repo.py
View File

@ -8,6 +8,7 @@ import jsonmodels.models
from apiserver.apierrors import APIError, errors
from apiserver.config_repo import config
from apiserver.database.errors import translate_errors_context
from apiserver.utilities.partial_version import PartialVersion
from .apicall import APICall
from .auth import Identity
@ -283,7 +284,8 @@ class ServiceRepo(object):
# In case call does not require authorization, parsing the identity.company might raise an exception
company = cls._get_company(call, endpoint)
ret = endpoint.func(call, company, call.data_model)
with translate_errors_context():
ret = endpoint.func(call, company, call.data_model)
# allow endpoints to return dict or model (instead of setting them explicitly on the call)
if ret is not None:

11
apiserver/tests/automated/test_projects_edit.py
View File

@ -1,4 +1,4 @@
from apiserver.apierrors.errors.bad_request import InvalidProjectId
from apiserver.apierrors.errors.bad_request import InvalidProjectId, ExpectedUniqueData
from apiserver.apierrors.errors.forbidden import NoWritePermission
from apiserver.config_repo import config
from apiserver.tests.automated import TestService
@ -32,3 +32,12 @@ class TestProjectsEdit(TestService):
res = self.api.projects.get_all(id=[p1])
self.assertEqual([p.id for p in res.projects], [p1])
self.api.projects.update(project=p1, name="Test public change 2")
def test_project_name_uniqueness(self):
name1 = "Test name1"
p1 = self.create_temp("projects", name=name1, description="test")
with self.api.raises(ExpectedUniqueData):
p2 = self.create_temp("projects", name=name1, description="test")
p2 = self.create_temp("projects", name="Test name2", description="test")
with self.api.raises(ExpectedUniqueData):
self.api.projects.update(project=p2, name=name1)