Skip to content

Commit 0968e97

Browse files
Mourya Baddambmouryakr
authored andcommitted
Remove redundant unit test
1 parent d3171d4 commit 0968e97

1 file changed

Lines changed: 0 additions & 19 deletions

File tree

tests/unit/sagemaker/workflow/test_pipeline.py

Lines changed: 0 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -790,7 +790,6 @@ def test_pipeline_disable_experiment_config(sagemaker_session_mock):
790790
}
791791
)
792792

793-
794793
@patch("sagemaker.workflow.pipeline.EXPERIMENTS_REGIONS", {"us-east-1", "us-west-2"})
795794
def test_pipeline_init_with_default_experiment_config_in_supported_region(sagemaker_session_mock):
796795
"""Test that default experiment config is preserved in regions where Experiments is available."""
@@ -866,24 +865,6 @@ def test_pipeline_init_with_none_experiment_config_in_supported_region(sagemaker
866865
# Explicitly set None should be preserved
867866
assert pipeline.pipeline_experiment_config is None
868867

869-
870-
@patch("sagemaker.workflow.pipeline.EXPERIMENTS_REGIONS", {"us-east-1"})
871-
def test_pipeline_definition_without_experiment_config_in_unsupported_region(
872-
sagemaker_session_mock,
873-
):
874-
"""Test that pipeline definition has no experiment config in unsupported regions."""
875-
sagemaker_session_mock.boto_region_name = "ap-southeast-4" # Not in EXPERIMENTS_REGIONS
876-
877-
pipeline = Pipeline(
878-
name="MyPipeline",
879-
steps=[CustomStep(name="MyStep", input_data="input")],
880-
sagemaker_session=sagemaker_session_mock,
881-
)
882-
883-
definition = json.loads(pipeline.definition())
884-
assert definition["PipelineExperimentConfig"] is None
885-
886-
887868
def test_pipeline_list_executions(sagemaker_session_mock):
888869
sagemaker_session_mock.sagemaker_client.list_pipeline_executions.return_value = {
889870
"PipelineExecutionSummaries": [Mock()],

0 commit comments

Comments
 (0)