Skip to content

Commit 47cef38

Browse files
author
Rohan Gujarathi
committed
change: use sagemaker_session in workflow tests
1 parent c5cc224 commit 47cef38

File tree

1 file changed

+3
-21
lines changed

1 file changed

+3
-21
lines changed

tests/integ/test_workflow.py

Lines changed: 3 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -74,21 +74,6 @@ def role(sagemaker_session):
7474
return get_execution_role(sagemaker_session)
7575

7676

77-
@pytest.fixture(scope="module")
78-
def workflow_session(region_name):
79-
boto_session = boto3.Session(region_name=region_name)
80-
81-
sagemaker_client_config = dict()
82-
sagemaker_client_config.setdefault("config", Config(retries=dict(max_attempts=2)))
83-
sagemaker_client = boto_session.client("sagemaker", **sagemaker_client_config)
84-
85-
return Session(
86-
boto_session=boto_session,
87-
sagemaker_client=sagemaker_client,
88-
sagemaker_runtime_client=None,
89-
)
90-
91-
9277
@pytest.fixture(scope="module")
9378
def script_dir():
9479
return os.path.join(DATA_DIR, "sklearn_processing")
@@ -119,7 +104,6 @@ def athena_dataset_definition(sagemaker_session):
119104

120105
def test_three_step_definition(
121106
sagemaker_session,
122-
workflow_session,
123107
region_name,
124108
role,
125109
script_dir,
@@ -205,7 +189,7 @@ def test_three_step_definition(
205189
name=pipeline_name,
206190
parameters=[instance_type, instance_count, output_prefix],
207191
steps=[step_process, step_train, step_model],
208-
sagemaker_session=workflow_session,
192+
sagemaker_session=sagemaker_session,
209193
)
210194

211195
definition = json.loads(pipeline.definition())
@@ -277,7 +261,6 @@ def test_three_step_definition(
277261

278262
def test_one_step_sklearn_processing_pipeline(
279263
sagemaker_session,
280-
workflow_session,
281264
role,
282265
sklearn_latest_version,
283266
cpu_instance_type,
@@ -313,7 +296,7 @@ def test_one_step_sklearn_processing_pipeline(
313296
name=pipeline_name,
314297
parameters=[instance_count],
315298
steps=[step_sklearn],
316-
sagemaker_session=workflow_session,
299+
sagemaker_session=sagemaker_session,
317300
)
318301

319302
try:
@@ -363,7 +346,6 @@ def test_one_step_sklearn_processing_pipeline(
363346

364347
def test_conditional_pytorch_training_model_registration(
365348
sagemaker_session,
366-
workflow_session,
367349
role,
368350
cpu_instance_type,
369351
pipeline_name,
@@ -433,7 +415,7 @@ def test_conditional_pytorch_training_model_registration(
433415
name=pipeline_name,
434416
parameters=[good_enough_input, instance_count, instance_type],
435417
steps=[step_cond],
436-
sagemaker_session=workflow_session,
418+
sagemaker_session=sagemaker_session,
437419
)
438420

439421
try:

0 commit comments

Comments
 (0)