change: fixuture renaming

Author: benieric

tests/integ/test_auto_ml.py

@@ -49,10 +49,10 @@
 
 
 # use a succeeded AutoML job to test describe and list candidates method, otherwise tests will run too long
-# reusable-job will be created once if it doesn't exist, and be reused in relevant tests.
+# test-session-job will be created once per session if it doesn't exist, and be reused in relevant tests.
 @pytest.fixture(scope="module")
-def reusable_job_name():
-    job_name = unique_name_from_base("reusable-job", max_length=32)
+def test_session_job_name():
+    job_name = unique_name_from_base("test-session-job", max_length=32)
     return job_name
 
 
@@ -199,7 +199,7 @@ def test_auto_ml_invalid_target_attribute(sagemaker_session):
     tests.integ.test_region() in tests.integ.NO_AUTO_ML_REGIONS,
     reason="AutoML is not supported in the region yet.",
 )
-def test_auto_ml_describe_auto_ml_job(sagemaker_session, reusable_job_name):
+def test_auto_ml_describe_auto_ml_job(sagemaker_session, test_session_job_name):
     expected_default_input_config = [
         {
             "DataSource": {
@@ -221,13 +221,13 @@ def test_auto_ml_describe_auto_ml_job(sagemaker_session, reusable_job_name):
         )
     }
 
-    auto_ml_utils.create_auto_ml_job_if_not_exist(sagemaker_session, reusable_job_name)
+    auto_ml_utils.create_auto_ml_job_if_not_exist(sagemaker_session, test_session_job_name)
     auto_ml = AutoML(
         role=ROLE, target_attribute_name=TARGET_ATTRIBUTE_NAME, sagemaker_session=sagemaker_session
     )
 
-    desc = auto_ml.describe_auto_ml_job(job_name=reusable_job_name)
-    assert desc["AutoMLJobName"] == reusable_job_name
+    desc = auto_ml.describe_auto_ml_job(job_name=test_session_job_name)
+    assert desc["AutoMLJobName"] == test_session_job_name
     assert desc["AutoMLJobStatus"] == "Completed"
     assert isinstance(desc["BestCandidate"], dict)
     assert desc["InputDataConfig"] == expected_default_input_config
@@ -239,7 +239,7 @@ def test_auto_ml_describe_auto_ml_job(sagemaker_session, reusable_job_name):
     tests.integ.test_region() in tests.integ.NO_AUTO_ML_REGIONS,
     reason="AutoML is not supported in the region yet.",
 )
-def test_auto_ml_attach(sagemaker_session, reusable_job_name):
+def test_auto_ml_attach(sagemaker_session, test_session_job_name):
     expected_default_input_config = [
         {
             "DataSource": {
@@ -261,13 +261,13 @@ def test_auto_ml_attach(sagemaker_session, reusable_job_name):
         )
     }
 
-    auto_ml_utils.create_auto_ml_job_if_not_exist(sagemaker_session, reusable_job_name)
+    auto_ml_utils.create_auto_ml_job_if_not_exist(sagemaker_session, test_session_job_name)
 
     attached_automl_job = AutoML.attach(
-        auto_ml_job_name=reusable_job_name, sagemaker_session=sagemaker_session
+        auto_ml_job_name=test_session_job_name, sagemaker_session=sagemaker_session
     )
     attached_desc = attached_automl_job.describe_auto_ml_job()
-    assert attached_desc["AutoMLJobName"] == reusable_job_name
+    assert attached_desc["AutoMLJobName"] == test_session_job_name
     assert attached_desc["AutoMLJobStatus"] == "Completed"
     assert isinstance(attached_desc["BestCandidate"], dict)
     assert attached_desc["InputDataConfig"] == expected_default_input_config
@@ -279,28 +279,28 @@ def test_auto_ml_attach(sagemaker_session, reusable_job_name):
     tests.integ.test_region() in tests.integ.NO_AUTO_ML_REGIONS,
     reason="AutoML is not supported in the region yet.",
 )
-def test_list_candidates(sagemaker_session, reusable_job_name):
-    auto_ml_utils.create_auto_ml_job_if_not_exist(sagemaker_session, reusable_job_name)
+def test_list_candidates(sagemaker_session, test_session_job_name):
+    auto_ml_utils.create_auto_ml_job_if_not_exist(sagemaker_session, test_session_job_name)
 
     auto_ml = AutoML(
         role=ROLE, target_attribute_name=TARGET_ATTRIBUTE_NAME, sagemaker_session=sagemaker_session
     )
 
-    candidates = auto_ml.list_candidates(job_name=reusable_job_name)
+    candidates = auto_ml.list_candidates(job_name=test_session_job_name)
     assert len(candidates) == 3
 
 
 @pytest.mark.skipif(
     tests.integ.test_region() in tests.integ.NO_AUTO_ML_REGIONS,
     reason="AutoML is not supported in the region yet.",
 )
-def test_best_candidate(sagemaker_session, reusable_job_name):
-    auto_ml_utils.create_auto_ml_job_if_not_exist(sagemaker_session, reusable_job_name)
+def test_best_candidate(sagemaker_session, test_session_job_name):
+    auto_ml_utils.create_auto_ml_job_if_not_exist(sagemaker_session, test_session_job_name)
 
     auto_ml = AutoML(
         role=ROLE, target_attribute_name=TARGET_ATTRIBUTE_NAME, sagemaker_session=sagemaker_session
     )
-    best_candidate = auto_ml.best_candidate(job_name=reusable_job_name)
+    best_candidate = auto_ml.best_candidate(job_name=test_session_job_name)
     assert len(best_candidate["InferenceContainers"]) == 3
     assert len(best_candidate["CandidateSteps"]) == 4
     assert best_candidate["CandidateStatus"] == "Completed"
@@ -311,13 +311,13 @@ def test_best_candidate(sagemaker_session, reusable_job_name):
     reason="AutoML is not supported in the region yet.",
 )
 @pytest.mark.release
-def test_deploy_best_candidate(sagemaker_session, cpu_instance_type, reusable_job_name):
-    auto_ml_utils.create_auto_ml_job_if_not_exist(sagemaker_session, reusable_job_name)
+def test_deploy_best_candidate(sagemaker_session, cpu_instance_type, test_session_job_name):
+    auto_ml_utils.create_auto_ml_job_if_not_exist(sagemaker_session, test_session_job_name)
 
     auto_ml = AutoML(
         role=ROLE, target_attribute_name=TARGET_ATTRIBUTE_NAME, sagemaker_session=sagemaker_session
     )
-    best_candidate = auto_ml.best_candidate(job_name=reusable_job_name)
+    best_candidate = auto_ml.best_candidate(job_name=test_session_job_name)
     endpoint_name = unique_name_from_base("sagemaker-auto-ml-best-candidate-test")
 
     with timeout(minutes=AUTO_ML_DEFAULT_TIMEMOUT_MINUTES):
@@ -343,15 +343,15 @@ def test_deploy_best_candidate(sagemaker_session, cpu_instance_type, reusable_jo
     reason="",
 )
 def test_candidate_estimator_default_rerun_and_deploy(
-    sagemaker_session, cpu_instance_type, reusable_job_name
+    sagemaker_session, cpu_instance_type, test_session_job_name
 ):
-    auto_ml_utils.create_auto_ml_job_if_not_exist(sagemaker_session, reusable_job_name)
+    auto_ml_utils.create_auto_ml_job_if_not_exist(sagemaker_session, test_session_job_name)
 
     auto_ml = AutoML(
         role=ROLE, target_attribute_name=TARGET_ATTRIBUTE_NAME, sagemaker_session=sagemaker_session
     )
 
-    candidates = auto_ml.list_candidates(job_name=reusable_job_name)
+    candidates = auto_ml.list_candidates(job_name=test_session_job_name)
     candidate = candidates[1]
 
     candidate_estimator = CandidateEstimator(candidate, sagemaker_session)
@@ -377,13 +377,13 @@ def test_candidate_estimator_default_rerun_and_deploy(
     tests.integ.test_region() in tests.integ.NO_AUTO_ML_REGIONS,
     reason="AutoML is not supported in the region yet.",
 )
-def test_candidate_estimator_get_steps(sagemaker_session, reusable_job_name):
-    auto_ml_utils.create_auto_ml_job_if_not_exist(sagemaker_session, reusable_job_name)
+def test_candidate_estimator_get_steps(sagemaker_session, test_session_job_name):
+    auto_ml_utils.create_auto_ml_job_if_not_exist(sagemaker_session, test_session_job_name)
 
     auto_ml = AutoML(
         role=ROLE, target_attribute_name=TARGET_ATTRIBUTE_NAME, sagemaker_session=sagemaker_session
     )
-    candidates = auto_ml.list_candidates(job_name=reusable_job_name)
+    candidates = auto_ml.list_candidates(job_name=test_session_job_name)
     candidate = candidates[1]
 
     candidate_estimator = CandidateEstimator(candidate, sagemaker_session)