Skip to content

Commit

Permalink
remove unused dependencies
Browse files Browse the repository at this point in the history
  • Loading branch information
natthan-pigoux committed Oct 20, 2023
1 parent db07052 commit 606cb61
Showing 1 changed file with 0 additions and 7 deletions.
7 changes: 0 additions & 7 deletions src/diracx/routers/job_manager/sandboxes.py
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,6 @@
s3_object_exists,
)
from diracx.core.settings import ServiceSettingsBase
from diracx.db.sql.jobs.db import JobDB

if TYPE_CHECKING:
from types_aiobotocore_s3.client import S3Client
Expand Down Expand Up @@ -206,10 +205,8 @@ async def get_sandbox_file(

@router.get("/jobs/{job_id}/sandbox")
async def get_job_sandboxes(
job_db: JobDB,
job_id: int,
sandbox_metadata_db: SandboxMetadataDB,
user_info: Annotated[AuthorizedUserInfo, Depends(verify_dirac_access_token)],
) -> dict[Any, Any]:
"""Get input and output sandboxes of given job id at the same time."""
# TODO: check that user as created the job or is admin
Expand All @@ -220,11 +217,9 @@ async def get_job_sandboxes(

@router.get("/jobs/{job_id}/sandbox/{sandbox_type}")
async def get_job_sandbox(
job_db: JobDB,
job_id: int,
sandbox_metadata_db: SandboxMetadataDB,
sandbox_type: str,
user_info: Annotated[AuthorizedUserInfo, Depends(verify_dirac_access_token)],
) -> dict[Any, Any]:
"""Get input or output sandbox from given job"""
# TODO: check that user as created the job or is admin
Expand All @@ -234,13 +229,11 @@ async def get_job_sandbox(

@router.patch("/jobs/{job_id}/sandbox/output")
async def assign_sandbox(
job_db: JobDB,
job_id: int,
pfn: str,
sandbox_metadata_db: SandboxMetadataDB,
sb_type: str,
se_name: str,
user_info: Annotated[AuthorizedUserInfo, Depends(verify_dirac_access_token)],
) -> None:
# TODO: check that user as created the job or is admin
assert (

Check warning on line 239 in src/diracx/routers/job_manager/sandboxes.py

View check run for this annotation

Codecov / codecov/patch

src/diracx/routers/job_manager/sandboxes.py#L239

Added line #L239 was not covered by tests
Expand Down

0 comments on commit 606cb61

Please sign in to comment.