Skip to content

Commit 9ffbee9

Browse files
committed
style: format
1 parent 22ec383 commit 9ffbee9

3 files changed

Lines changed: 23 additions & 22 deletions

File tree

tests/unit/aiplatform/test_pipeline_job_schedules.py

Lines changed: 11 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -438,7 +438,7 @@ def teardown_method(self):
438438
def test_block_until_complete_logs_symbolic_state_name(self):
439439
"""State log must use symbolic enum name, not a bare integer (regression for Python 3.11+)."""
440440
state_sequence = [
441-
gca_schedule.Schedule.State.ACTIVE, # first loop check
441+
gca_schedule.Schedule.State.ACTIVE, # first loop check
442442
gca_schedule.Schedule.State.COMPLETED, # second check exits loop
443443
]
444444
state_index = [0]
@@ -461,17 +461,18 @@ def get_state():
461461

462462
# time.time: first call sets previous_time=0; second gives 10 → triggers log (10 >= 5)
463463
time_vals = iter([0.0, 10.0, 20.0])
464-
with mock.patch("google.cloud.aiplatform.schedules.time.time", side_effect=time_vals), \
465-
mock.patch("google.cloud.aiplatform.schedules.time.sleep"), \
466-
mock.patch.object(
467-
aiplatform_schedules._LOGGER, "info",
468-
side_effect=lambda msg, *a, **kw: logged_messages.append(msg)
469-
):
464+
with mock.patch(
465+
"google.cloud.aiplatform.schedules.time.time", side_effect=time_vals
466+
), mock.patch(
467+
"google.cloud.aiplatform.schedules.time.sleep"
468+
), mock.patch.object(
469+
aiplatform_schedules._LOGGER,
470+
"info",
471+
side_effect=lambda msg, *a, **kw: logged_messages.append(msg),
472+
):
470473
aiplatform_schedules._Schedule._block_until_complete(mock_schedule)
471474

472-
state_log = next(
473-
(m for m in logged_messages if "current state" in m), None
474-
)
475+
state_log = next((m for m in logged_messages if "current state" in m), None)
475476
assert state_log is not None, "No 'current state' log message found"
476477
assert "ACTIVE" in state_log
477478
assert "current state:\n1" not in state_log

tests/unit/aiplatform/test_pipeline_jobs.py

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -726,11 +726,13 @@ def test_block_until_complete_logs_symbolic_state_name(
726726

727727
logged_messages = []
728728

729-
with patch.object(storage.Blob, "download_as_bytes") as mock_load, \
730-
mock.patch.object(
731-
pipeline_jobs._LOGGER, "info",
732-
side_effect=lambda msg, *a, **kw: logged_messages.append(msg)
733-
):
729+
with patch.object(
730+
storage.Blob, "download_as_bytes"
731+
) as mock_load, mock.patch.object(
732+
pipeline_jobs._LOGGER,
733+
"info",
734+
side_effect=lambda msg, *a, **kw: logged_messages.append(msg),
735+
):
734736
mock_load.return_value = _TEST_PIPELINE_SPEC_JSON.encode()
735737

736738
job = pipeline_jobs.PipelineJob(
@@ -740,9 +742,7 @@ def test_block_until_complete_logs_symbolic_state_name(
740742
)
741743
job.run(sync=True, create_request_timeout=None)
742744

743-
state_log = next(
744-
(m for m in logged_messages if "current state" in m), None
745-
)
745+
state_log = next((m for m in logged_messages if "current state" in m), None)
746746
assert state_log is not None, "No 'current state' log message found"
747747
assert "PIPELINE_STATE_RUNNING" in state_log
748748
assert "current state:\n3" not in state_log

tests/unit/aiplatform/test_training_jobs.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1307,7 +1307,9 @@ def test_block_until_complete_logs_symbolic_state_name(
13071307
), mock.patch.object(
13081308
training_jobs, "_JOB_WAIT_TIME", 0
13091309
), mock.patch.object(
1310-
training_jobs._LOGGER, "info", side_effect=lambda msg, *a, **kw: logged_messages.append(msg)
1310+
training_jobs._LOGGER,
1311+
"info",
1312+
side_effect=lambda msg, *a, **kw: logged_messages.append(msg),
13111313
):
13121314
mock_pkg.return_value = _TEST_OUTPUT_PYTHON_PACKAGE_PATH
13131315
mock_create.return_value = gca_training_pipeline.TrainingPipeline(
@@ -1334,9 +1336,7 @@ def test_block_until_complete_logs_symbolic_state_name(
13341336
)
13351337
job.run(base_output_dir=_TEST_BASE_OUTPUT_DIR, sync=True)
13361338

1337-
state_log = next(
1338-
(m for m in logged_messages if "current state" in m), None
1339-
)
1339+
state_log = next((m for m in logged_messages if "current state" in m), None)
13401340
assert state_log is not None, "No 'current state' log message found"
13411341
assert "PIPELINE_STATE_RUNNING" in state_log
13421342
assert "current state:\n3" not in state_log

0 commit comments

Comments
 (0)