Fix lint and integration errors #2033
Open
+62
−27
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Integration: API key creation role handling (
libs/labelbox/src/labelbox/schema/api_key.py)client.get_roles()(normalized names likePROJECT_ADMIN) and API key creation (server role names likeProject Admin).Roleobjects fromclient.get_roles()are mapped back to the correct server role name before calling the create-API-key mutation.Integration: UserGroup deletion invalid-id handling (
libs/labelbox/src/labelbox/schema/user_group.py)UserGroup.delete()so invalid IDs raiseMalformedQueryException("Invalid user group id")instead of surfacing backend UUID parsing errors.Integration: model-setup-complete message drift (test update)
libs/labelbox/tests/integration/test_project_set_model_setup_complete.pyto accept either backend error message for the “model setup complete” precondition when deleting aProjectModelConfig.Lint (
libs/labelbox/src/labelbox/client.pyandlibs/labelbox/src/labelbox/schema/project.py)Fixes # (issue)
Type of change
Please delete options that are not relevant.
All Submissions
New Feature Submissions
Changes to Core Features
Note
Roleobjects back to exact server role names; string roles must match case-sensitively. Useserver_role_namein thecreateApiKeymutation.UserGroup.delete()idas UUID and raiseMalformedQueryExceptionon invalid input.test_project_set_model_setup_complete.pyto accept either backend message; updatetest_user_group.pyto use UUID-formatted ids.client.pyandproject.py(e.g.,labels(...)signature wrapping, slice spacing).Written by Cursor Bugbot for commit a58da5b. This will update automatically on new commits. Configure here.