fix: sklearn integ tests, remove swallowing exception on feature group delete attempt (#2563)

* Remove swallowing exception on feature group delete attempt

* fix black format issues and update test for sklearn

* fix passing in python3 as command

* fix passing in python3 as command for network config test

* remove asserts that no longer hold

Co-authored-by: Ahsan Khan <[email protected]>

Author: shreyapandit

tests/integ/test_feature_store.py

@@ -312,4 +312,4 @@ def cleanup_feature_group(feature_group: FeatureGroup):
         try:
             feature_group.delete()
         except Exception:
-            pass
+            raise RuntimeError(f"Failed to delete feature group with name {feature_group.name}")

tests/integ/test_processing.py

@@ -162,6 +162,7 @@ def test_sklearn_with_customizations(
     sklearn_processor = SKLearnProcessor(
         framework_version=sklearn_latest_version,
         role=ROLE,
+        command=["python3"],
         instance_type=cpu_instance_type,
         instance_count=1,
         volume_size_in_gb=100,
@@ -287,7 +288,6 @@ def test_sklearn_with_custom_default_bucket(
 
     assert job_description["ProcessingInputs"][0]["InputName"] == "dummy_input"
     assert custom_bucket_name in job_description["ProcessingInputs"][0]["S3Input"]["S3Uri"]
-
     assert job_description["ProcessingJobName"].startswith("test-sklearn-with-customizations")
 
     assert job_description["ProcessingJobStatus"] == "Completed"
@@ -677,6 +677,7 @@ def test_sklearn_with_network_config(sagemaker_session, sklearn_latest_version,
     sklearn_processor = SKLearnProcessor(
         framework_version=sklearn_latest_version,
         role=ROLE,
+        command=["python3"],
         instance_type=cpu_instance_type,
         instance_count=1,
         sagemaker_session=sagemaker_session,