Skip to content

chore(azure-ai-ml): remove stale live_test_only markers, re-enable e2e tests in playback CI #48230

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
Copilot wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

chore(azure-ai-ml): remove stale live_test_only markers, re-enable e2e tests in playback CI #48230

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 0 additions & 1 deletion sdk/ml/azure-ai-ml/tests/component/e2etests/test_component.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1222,7 +1222,6 @@ def test_load_component_from_flow(self, client: MLClient, randstr, params_overri

assert component._get_origin_code_value() == created_component._get_origin_code_value()

@pytest.mark.live_test_only("Needs re-recording to work with new test proxy sanitizers")
def test_load_component_from_flow_in_registry(self, registry_client: MLClient, randstr):
target_path: str = "./tests/test_configs/flows/runs/with_environment.yml"
component = load_component(
Expand Down
3 changes: 0 additions & 3 deletions sdk/ml/azure-ai-ml/tests/connection/e2etests/test_connections.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,6 @@
@pytest.mark.core_sdk_test
@pytest.mark.usefixtures("recorded_test")
class TestWorkspaceConnections(AzureRecordedTestCase):
@pytest.mark.live_test_only("Needs re-recording to work with new common sanitizers")
def test_secret_population(
self,
client: MLClient,
Expand Down Expand Up @@ -122,7 +121,6 @@ def test_workspace_connections_create_update_and_delete_git_pat(
with pytest.raises(Exception):
client.connections.get(name=wps_connection_name)

@pytest.mark.live_test_only("Needs re-recording to work with new common sanitizers")
def test_workspace_connections_create_update_and_delete_snowflake_user_pwd(
self,
client: MLClient,
Expand Down Expand Up @@ -898,7 +896,6 @@ def test_python_feed_crud(
with pytest.raises(Exception):
client.connections.get(name=wps_connection_name)

@pytest.mark.live_test_only("Needs re-recording to work with new common sanitizers")
def test_container_registry_managed_id_crud(
self,
client: MLClient,
Expand Down
4 changes: 0 additions & 4 deletions sdk/ml/azure-ai-ml/tests/datastore/e2etests/test_datastore.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,6 @@ def test_hdfs_keytab(
with pytest.raises(Exception):
client.datastores.get(random_name)

@pytest.mark.live_test_only("Needs re-recording to work with new common sanitizers")
def test_blob_store(
self,
client: MLClient,
Expand All @@ -106,7 +105,6 @@ def test_blob_store(
with pytest.raises(Exception):
client.datastores.get(random_name)

@pytest.mark.live_test_only("Needs re-recording to work with new common sanitizers")
def test_blob_store_credential_less(
self,
client: MLClient,
Expand All @@ -129,7 +127,6 @@ def test_blob_store_credential_less(
with pytest.raises(Exception):
client.datastores.get(random_name)

@pytest.mark.live_test_only("Needs re-recording to work with new common sanitizers")
def test_file_store(
self,
client: MLClient,
Expand Down Expand Up @@ -217,7 +214,6 @@ def test_adls_gen2_store(
with pytest.raises(Exception):
client.datastores.get(random_name)

@pytest.mark.live_test_only("Needs re-recording to work with new common sanitizers")
def test_credential_less_adls_gen2_store(
self,
client: MLClient,
Expand Down
2 changes: 0 additions & 2 deletions sdk/ml/azure-ai-ml/tests/environment/e2etests/test_environment.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,6 @@ def test_environment_create_or_update_docker(self, client: MLClient, env_name: C
assert env_dump["id"] == ARM_ID_PREFIX + environment_id
assert env_dump["image"] == environment.image

@pytest.mark.live_test_only("Needs re-recording to work with new test proxy sanitizers")
def test_environment_create_or_update_docker_context(
self, client: MLClient, env_name: Callable[[str], str]
) -> None:
Expand Down Expand Up @@ -218,7 +217,6 @@ def test_environment_get_latest_label(self, client: MLClient, randstr: Callable[
sleep_if_live(2)
assert client.environments.get(name, label="latest").version == version

@pytest.mark.live_test_only("Needs re-recording to work with new test proxy sanitizers")
def test_registry_environment_create_conda_and_get(
self, only_registry_client: MLClient, env_name: Callable[[str], str]
) -> None:
Expand Down
1 change: 0 additions & 1 deletion sdk/ml/azure-ai-ml/tests/internal/e2etests/test_pipeline_job.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -287,7 +287,6 @@ def pipeline_with_command_components(tsv_file, content):

assert_job_cancel(pipeline_job, client, experiment_name="v15_v2_interop")

@pytest.mark.live_test_only("Needs re-recording to work with new common sanitizers")
def test_pipeline_with_setting_node_output_mode(self, client: MLClient):
# get dataset
training_data = Input(type=AssetTypes.URI_FILE, path="https://dprepdata.blob.core.windows.net/demo/Titanic.csv")
Expand Down
Loading