Skip to content

Commit d7ae37b

Browse files
author
Marius Moisescu
committed
Updated test_workflow and est_training_step
1 parent cb855b1 commit d7ae37b

File tree

2 files changed

+0
-14
lines changed

2 files changed

+0
-14
lines changed

tests/integ/sagemaker/workflow/test_workflow.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1269,8 +1269,6 @@ def test_caching_behavior(
12691269
# create pipeline
12701270
pipeline.create(role)
12711271
definition = json.loads(pipeline.definition())
1272-
# delete profiler config for assertions as it will contain a timestamp
1273-
del definition["Steps"][1]["Arguments"]["ProfilerRuleConfigurations"]
12741272

12751273
# verify input path
12761274
expected_abalone_input_path = f"{pipeline_name}/{step_process.name}" f"/input/abalone_data"

tests/unit/sagemaker/workflow/test_training_step.py

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -532,9 +532,6 @@ def test_training_step_with_framework_estimator(
532532
del expected_step_args["OutputDataConfig"]["S3OutputPath"]
533533
del step_def["Arguments"]["OutputDataConfig"]["S3OutputPath"]
534534

535-
# delete profiler rule configurations because of timestamp collision
536-
del expected_step_args["ProfilerRuleConfigurations"]
537-
538535
if "sagemaker_s3_output" in step_args["HyperParameters"]:
539536
del expected_step_args["HyperParameters"]["sagemaker_s3_output"]
540537
del step_def["Arguments"]["HyperParameters"]["sagemaker_s3_output"]
@@ -602,9 +599,6 @@ def test_training_step_with_framework_estimator_local_code(
602599
del expected_step_args["OutputDataConfig"]["S3OutputPath"]
603600
del step_def["Arguments"]["OutputDataConfig"]["S3OutputPath"]
604601

605-
# delete profiler rule configurations because of timestamp collision
606-
del expected_step_args["ProfilerRuleConfigurations"]
607-
608602
if "sagemaker_s3_output" in step_args["HyperParameters"]:
609603
del expected_step_args["HyperParameters"]["sagemaker_s3_output"]
610604
del step_def["Arguments"]["HyperParameters"]["sagemaker_s3_output"]
@@ -694,9 +688,6 @@ def test_training_step_with_algorithm_base(algo_estimator, training_input, pipel
694688
del step_args["InputDataConfig"][0]["DataSource"]["S3DataSource"]["S3Uri"]
695689
del step_def["Arguments"]["InputDataConfig"][0]["DataSource"]["S3DataSource"]["S3Uri"]
696690

697-
# delete profiler rule configurations because of timestamp collision
698-
del step_args["ProfilerRuleConfigurations"]
699-
700691
assert step_def == {
701692
"Name": "MyTrainingStep",
702693
"Type": "Training",
@@ -781,9 +772,6 @@ def test_training_step_with_algorithm_base_local_code(
781772
del step_args["InputDataConfig"][0]["DataSource"]["S3DataSource"]["S3Uri"]
782773
del step_def["Arguments"]["InputDataConfig"][0]["DataSource"]["S3DataSource"]["S3Uri"]
783774

784-
# delete profiler rule configurations because of timestamp collision
785-
del step_args["ProfilerRuleConfigurations"]
786-
787775
assert step_def == {
788776
"Name": "MyTrainingStep",
789777
"Type": "Training",

0 commit comments

Comments
 (0)