fix: double Run create on load_run

fixes aws#3673

Author: danabens

src/sagemaker/experiments/run.py

@@ -633,7 +633,10 @@ def _extract_run_name_from_tc_name(trial_component_name: str, experiment_name: s
         Returns:
             str: The name of the Run object supplied by a user.
         """
-        return trial_component_name.replace("{}{}".format(experiment_name, DELIMITER), "", 1)
+        # TODO: we should revert the lower casting once backend fix reaches prod
+        return trial_component_name.replace(
+            "{}{}".format(experiment_name.lower(), DELIMITER), "", 1
+        )
 
     @staticmethod
     def _append_run_tc_label_to_tags(tags: Optional[List[Dict[str, str]]] = None) -> list:
@@ -869,6 +872,8 @@ def list_runs(
     Returns:
         list: A list of ``Run`` objects.
     """
+
+    # all trial components retrieved by default
     tc_summaries = _TrialComponent.list(
         experiment_name=experiment_name,
         created_before=created_before,

tests/unit/sagemaker/experiments/helpers.py

@@ -17,6 +17,7 @@
 
 
 TEST_EXP_NAME = "my-experiment"
+TEST_EXP_NAME_MIXED_CASE = "My-eXpeRiMeNt"
 TEST_RUN_NAME = "my-run"
 TEST_EXP_DISPLAY_NAME = "my-experiment-display-name"
 TEST_RUN_DISPLAY_NAME = "my-run-display-name"

tests/unit/sagemaker/experiments/test_run.py

@@ -48,6 +48,7 @@
     mock_trial_load_or_create_func,
     mock_tc_load_or_create_func,
     TEST_EXP_NAME,
+    TEST_EXP_NAME_MIXED_CASE,
     TEST_RUN_NAME,
     TEST_EXP_DISPLAY_NAME,
     TEST_RUN_DISPLAY_NAME,
@@ -779,7 +780,7 @@ def test_list(mock_tc_search, mock_tc_list, mock_tc_load, run_obj, sagemaker_ses
     ]
     mock_tc_list.return_value = [
         TrialComponentSummary(
-            trial_component_name=Run._generate_trial_component_name("A" + str(i), TEST_EXP_NAME),
+            trial_component_name=Run._generate_trial_component_name("A" + str(i), TEST_EXP_NAME_MIXED_CASE),
             trial_component_arn="b" + str(i),
             display_name="C" + str(i),
             source_arn="D" + str(i),
@@ -798,7 +799,7 @@ def test_list(mock_tc_search, mock_tc_list, mock_tc_load, run_obj, sagemaker_ses
         (
             _TrialComponent(
                 trial_component_name=Run._generate_trial_component_name(
-                    "a" + str(i), TEST_EXP_NAME
+                    "a" + str(i), TEST_EXP_NAME_MIXED_CASE
                 ),
                 trial_component_arn="b" + str(i),
                 display_name="C" + str(i),
@@ -818,14 +819,14 @@ def test_list(mock_tc_search, mock_tc_list, mock_tc_load, run_obj, sagemaker_ses
     ]
 
     run_list = list_runs(
-        experiment_name=TEST_EXP_NAME,
+        experiment_name=TEST_EXP_NAME_MIXED_CASE,
         sort_by=SortByType.CREATION_TIME,
         sort_order=SortOrderType.ASCENDING,
         sagemaker_session=sagemaker_session,
     )
 
     mock_tc_list.assert_called_once_with(
-        experiment_name=TEST_EXP_NAME,
+        experiment_name=TEST_EXP_NAME_MIXED_CASE,
         created_before=None,
         created_after=None,
         sort_by="CreationTime",