Skip to content

Commit

Permalink
fix: add job_run_as_user to integ test (#57)
Browse files Browse the repository at this point in the history
Signed-off-by: Jericho Tolentino <68654047+jericht@users.noreply.github.com>
  • Loading branch information
jericht authored Oct 13, 2023
1 parent d8ac9f4 commit 962808f
Showing 1 changed file with 8 additions and 14 deletions.
22 changes: 8 additions & 14 deletions test/integ/test_job_submissions.py
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,8 @@
Farm,
Fleet,
Job,
PosixUser,
JobRunAsUser,
PosixSessionUser,
Queue,
QueueFleetAssociation,
TaskStatus,
Expand All @@ -48,8 +49,8 @@ def fleet(deadline_resources: DeadlineResources) -> Fleet:


@pytest.fixture(scope="session")
def job_run_as_user() -> PosixUser:
return PosixUser(
def job_run_as_user() -> PosixSessionUser:
return PosixSessionUser(
user="job-run-as-user",
group="job-run-as-user-group",
)
Expand All @@ -58,7 +59,7 @@ def job_run_as_user() -> PosixUser:
@pytest.fixture(scope="session")
def worker_config(
worker_config: DeadlineWorkerConfiguration,
job_run_as_user: PosixUser,
job_run_as_user: PosixSessionUser,
) -> DeadlineWorkerConfiguration:
return dataclasses.replace(
worker_config,
Expand All @@ -74,20 +75,13 @@ def queue_with_job_run_as_user(
farm: Farm,
fleet: Fleet,
deadline_client: DeadlineClient,
job_run_as_user: PosixUser,
job_run_as_user: PosixSessionUser,
) -> Generator[Queue, None, None]:
queue = Queue.create(
client=deadline_client,
display_name=f"Queue with jobsRunAsUser {job_run_as_user.user}",
farm=farm,
raw_kwargs={
"jobRunAsUser": {
"posix": {
"user": job_run_as_user.user,
"group": job_run_as_user.group,
},
},
},
job_run_as_user=JobRunAsUser(posix=job_run_as_user),
)

qfa = QueueFleetAssociation.create(
Expand Down Expand Up @@ -144,7 +138,7 @@ def test_jobs_run_as_user(
deadline_client: DeadlineClient,
farm: Farm,
queue_with_job_run_as_user: Queue,
job_run_as_user: PosixUser,
job_run_as_user: PosixSessionUser,
) -> None:
# WHEN
job = Job.submit(
Expand Down

0 comments on commit 962808f

Please sign in to comment.