From c40ade484968d130070ff5f8d089fe24f981e86d Mon Sep 17 00:00:00 2001 From: Shad Lords Date: Thu, 19 Sep 2024 08:23:23 -0600 Subject: [PATCH] Add test for oracle profile --- .../profiles/oracle/test_oracle_user_pass.py | 160 ++++++++++++++++++ 1 file changed, 160 insertions(+) create mode 100644 tests/profiles/oracle/test_oracle_user_pass.py diff --git a/tests/profiles/oracle/test_oracle_user_pass.py b/tests/profiles/oracle/test_oracle_user_pass.py new file mode 100644 index 000000000..20793e393 --- /dev/null +++ b/tests/profiles/oracle/test_oracle_user_pass.py @@ -0,0 +1,160 @@ +"""Tests for the Oracle profile.""" + +from unittest.mock import patch + +import pytest +from airflow.models.connection import Connection + +from cosmos.profiles import get_automatic_profile_mapping +from cosmos.profiles.oracle.user_pass import OracleUserPasswordProfileMapping + + +@pytest.fixture() +def mock_oracle_conn(): # type: ignore + """ + Sets the Oracle connection as an environment variable. + """ + conn = Connection( + conn_id="my_oracle_connection", + conn_type="oracle", + host="my_host", + login="my_user", + password="my_password", + port=1521, + extra='{"service_name": "my_service"}', + ) + + with patch("airflow.hooks.base.BaseHook.get_connection", return_value=conn): + yield conn + + +@pytest.fixture() +def mock_oracle_conn_custom_port(): # type: ignore + """ + Sets the Oracle connection with a custom port as an environment variable. + """ + conn = Connection( + conn_id="my_oracle_connection", + conn_type="oracle", + host="my_host", + login="my_user", + password="my_password", + port=1600, + extra='{"service_name": "my_service"}', + ) + + with patch("airflow.hooks.base.BaseHook.get_connection", return_value=conn): + yield conn + + +def test_connection_claiming() -> None: + """ + Tests that the Oracle profile mapping claims the correct connection type. + """ + potential_values = { + "conn_type": "oracle", + "host": "my_host", + "login": "my_user", + "password": "my_password", + "extra": '{"service_name": "my_service"}', + } + + # if we're missing any of the required values, it shouldn't claim + for key in potential_values: + values = potential_values.copy() + del values[key] + conn = Connection(**values) # type: ignore + + with patch("airflow.hooks.base.BaseHook.get_connection", return_value=conn): + profile_mapping = OracleUserPasswordProfileMapping(conn, {"schema": "my_schema"}) + assert not profile_mapping.can_claim_connection() + + # if we have all the required values, it should claim + conn = Connection(**potential_values) # type: ignore + with patch("airflow.hooks.base.BaseHook.get_connection", return_value=conn): + profile_mapping = OracleUserPasswordProfileMapping(conn, {"schema": "my_schema"}) + assert profile_mapping.can_claim_connection() + + +def test_profile_mapping_selected( + mock_oracle_conn: Connection, +) -> None: + """ + Tests that the correct profile mapping is selected. + """ + profile_mapping = get_automatic_profile_mapping( + mock_oracle_conn.conn_id, + {"schema": "my_schema"}, + ) + assert isinstance(profile_mapping, OracleUserPasswordProfileMapping) + + +def test_profile_mapping_keeps_custom_port(mock_oracle_conn_custom_port: Connection) -> None: + profile = OracleUserPasswordProfileMapping(mock_oracle_conn_custom_port.conn_id, {"schema": "my_schema"}) + assert profile.profile["port"] == 1600 + + +def test_profile_args( + mock_oracle_conn: Connection, +) -> None: + """ + Tests that the profile values are set correctly. + """ + profile_mapping = get_automatic_profile_mapping( + mock_oracle_conn.conn_id, + profile_args={"schema": "my_schema"}, + ) + assert profile_mapping.profile_args == { + "schema": "my_schema", + } + + assert profile_mapping.profile == { + "type": mock_oracle_conn.conn_type, + "host": mock_oracle_conn.host, + "user": mock_oracle_conn.login, + "password": "{{ env_var('COSMOS_CONN_ORACLE_PASSWORD') }}", + "port": mock_oracle_conn.port, + "service_name": "my_service", + "schema": "my_schema", + } + + +def test_profile_args_overrides( + mock_oracle_conn: Connection, +) -> None: + """ + Tests that profile values can be overridden. + """ + profile_mapping = get_automatic_profile_mapping( + mock_oracle_conn.conn_id, + profile_args={"schema": "my_schema", "service_name": "my_service_override"}, + ) + assert profile_mapping.profile_args == { + "schema": "my_schema", + "service_name": "my_service_override", + } + + assert profile_mapping.profile == { + "type": mock_oracle_conn.conn_type, + "host": mock_oracle_conn.host, + "user": mock_oracle_conn.login, + "password": "{{ env_var('COSMOS_CONN_ORACLE_PASSWORD') }}", + "port": mock_oracle_conn.port, + "service_name": "my_service_override", + "schema": "my_schema", + } + + +def test_profile_env_vars( + mock_oracle_conn: Connection, +) -> None: + """ + Tests that environment variables are set correctly. + """ + profile_mapping = get_automatic_profile_mapping( + mock_oracle_conn.conn_id, + profile_args={"schema": "my_schema"}, + ) + assert profile_mapping.env_vars == { + "COSMOS_CONN_ORACLE_PASSWORD": mock_oracle_conn.password, + }