From fc5fce6d30541c06835a45ffb6e2d71e306a492e Mon Sep 17 00:00:00 2001 From: Marco Gil Date: Wed, 26 Nov 2025 09:59:41 +0100 Subject: [PATCH] PTHMINT-87: Fix ruff D103 --- pyproject.toml | 1 - .../e2e/examples/auth_manager/test_get_api_token.py | 1 + .../capture_manager/test_capture_reservation_cancel.py | 1 + .../e2e/examples/category_manager/test_get_categories.py | 1 + .../e2e/examples/gateway_manager/test_get_by_code.py | 1 + .../e2e/examples/gateway_manager/test_get_gateways.py | 1 + .../issuer_manager/test_get_issuers_by_gateway_code.py | 1 + .../multisafepay/e2e/examples/me_manager/test_get_me.py | 1 + .../e2e/examples/order_manager/test_capture.py | 1 + .../e2e/examples/order_manager/test_create.py | 1 + .../e2e/examples/order_manager/test_get_order.py | 1 + .../e2e/examples/order_manager/test_refund.py | 1 + .../e2e/examples/order_manager/test_refund_by_item.py | 1 + .../order_manager/test_refund_by_shopping_cart.py | 1 + .../e2e/examples/order_manager/test_update.py | 1 + .../payment_method_manager/test_get_by_gateway_code.py | 1 + .../payment_method_manager/test_get_payment_methods.py | 1 + .../e2e/examples/recurring_manager/test_recurring.py | 1 + .../transaction_manager/test_get_transactions.py | 1 + .../test_integration_order_response_afterpay_direct.py | 1 + .../request/components/test_unit_payment_options.py | 9 +++++++++ 21 files changed, 28 insertions(+), 1 deletion(-) diff --git a/pyproject.toml b/pyproject.toml index c688365..0c6f0bd 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -100,7 +100,6 @@ extend-safe-fixes = [ ] ignore = [ "D101", - "D103", "D104", "D107", "D200", diff --git a/tests/multisafepay/e2e/examples/auth_manager/test_get_api_token.py b/tests/multisafepay/e2e/examples/auth_manager/test_get_api_token.py index 09592b1..4c61511 100644 --- a/tests/multisafepay/e2e/examples/auth_manager/test_get_api_token.py +++ b/tests/multisafepay/e2e/examples/auth_manager/test_get_api_token.py @@ -22,6 +22,7 @@ @pytest.fixture(scope="module") def auth_manager() -> AuthManager: + """Fixture that provides an AuthManager instance for testing.""" load_dotenv() api_key = os.getenv("API_KEY") multisafepay_sdk = Sdk(api_key, False) diff --git a/tests/multisafepay/e2e/examples/capture_manager/test_capture_reservation_cancel.py b/tests/multisafepay/e2e/examples/capture_manager/test_capture_reservation_cancel.py index 4388221..bfc7c33 100644 --- a/tests/multisafepay/e2e/examples/capture_manager/test_capture_reservation_cancel.py +++ b/tests/multisafepay/e2e/examples/capture_manager/test_capture_reservation_cancel.py @@ -50,6 +50,7 @@ @pytest.fixture(scope="module") def sdk() -> Sdk: + """Fixture that provides an SDK instance for testing.""" load_dotenv() api_key = os.getenv("API_KEY") return Sdk(api_key, False) diff --git a/tests/multisafepay/e2e/examples/category_manager/test_get_categories.py b/tests/multisafepay/e2e/examples/category_manager/test_get_categories.py index 8e36acb..3f8019a 100644 --- a/tests/multisafepay/e2e/examples/category_manager/test_get_categories.py +++ b/tests/multisafepay/e2e/examples/category_manager/test_get_categories.py @@ -25,6 +25,7 @@ @pytest.fixture(scope="module") def category_manager() -> "CategoryManager": + """Fixture that provides a CategoryManager instance for testing.""" load_dotenv() api_key = os.getenv("API_KEY") multisafepay_sdk = Sdk(api_key, False) diff --git a/tests/multisafepay/e2e/examples/gateway_manager/test_get_by_code.py b/tests/multisafepay/e2e/examples/gateway_manager/test_get_by_code.py index ae0334e..733b27f 100644 --- a/tests/multisafepay/e2e/examples/gateway_manager/test_get_by_code.py +++ b/tests/multisafepay/e2e/examples/gateway_manager/test_get_by_code.py @@ -21,6 +21,7 @@ @pytest.fixture(scope="module") def gateway_manager() -> GatewayManager: + """Fixture that provides a GatewayManager instance for testing.""" load_dotenv() api_key = os.getenv("API_KEY") multisafepay_sdk = Sdk(api_key, False) diff --git a/tests/multisafepay/e2e/examples/gateway_manager/test_get_gateways.py b/tests/multisafepay/e2e/examples/gateway_manager/test_get_gateways.py index 99c53cd..66f3062 100644 --- a/tests/multisafepay/e2e/examples/gateway_manager/test_get_gateways.py +++ b/tests/multisafepay/e2e/examples/gateway_manager/test_get_gateways.py @@ -21,6 +21,7 @@ @pytest.fixture(scope="module") def gateway_manager() -> GatewayManager: + """Fixture that provides a GatewayManager instance for testing.""" load_dotenv() api_key = os.getenv("API_KEY") multisafepay_sdk = Sdk(api_key, False) diff --git a/tests/multisafepay/e2e/examples/issuer_manager/test_get_issuers_by_gateway_code.py b/tests/multisafepay/e2e/examples/issuer_manager/test_get_issuers_by_gateway_code.py index 81fa145..8170258 100644 --- a/tests/multisafepay/e2e/examples/issuer_manager/test_get_issuers_by_gateway_code.py +++ b/tests/multisafepay/e2e/examples/issuer_manager/test_get_issuers_by_gateway_code.py @@ -22,6 +22,7 @@ @pytest.fixture(scope="module") def issuer_manager() -> IssuerManager: + """Fixture that provides an IssuerManager instance for testing.""" load_dotenv() api_key = os.getenv("API_KEY") multisafepay_sdk = Sdk(api_key, False) diff --git a/tests/multisafepay/e2e/examples/me_manager/test_get_me.py b/tests/multisafepay/e2e/examples/me_manager/test_get_me.py index a24cefb..57a6b8e 100644 --- a/tests/multisafepay/e2e/examples/me_manager/test_get_me.py +++ b/tests/multisafepay/e2e/examples/me_manager/test_get_me.py @@ -25,6 +25,7 @@ @pytest.fixture(scope="module") def me_manager() -> "MeManager": + """Fixture that provides a MeManager instance for testing.""" load_dotenv() api_key = os.getenv("API_KEY") multisafepay_sdk = Sdk(api_key, False) diff --git a/tests/multisafepay/e2e/examples/order_manager/test_capture.py b/tests/multisafepay/e2e/examples/order_manager/test_capture.py index 9ba8c8b..124ad18 100644 --- a/tests/multisafepay/e2e/examples/order_manager/test_capture.py +++ b/tests/multisafepay/e2e/examples/order_manager/test_capture.py @@ -46,6 +46,7 @@ @pytest.fixture(scope="module") def order_manager() -> OrderManager: + """Fixture that provides an OrderManager instance for testing.""" load_dotenv() api_key = os.getenv("API_KEY") multisafepay_sdk = Sdk(api_key, False) diff --git a/tests/multisafepay/e2e/examples/order_manager/test_create.py b/tests/multisafepay/e2e/examples/order_manager/test_create.py index 61c06d5..88ef015 100644 --- a/tests/multisafepay/e2e/examples/order_manager/test_create.py +++ b/tests/multisafepay/e2e/examples/order_manager/test_create.py @@ -44,6 +44,7 @@ @pytest.fixture(scope="module") def order_manager() -> "OrderManager": + """Fixture that provides an OrderManager instance for testing.""" load_dotenv() api_key = os.getenv("API_KEY") multisafepay_sdk = Sdk(api_key, False) diff --git a/tests/multisafepay/e2e/examples/order_manager/test_get_order.py b/tests/multisafepay/e2e/examples/order_manager/test_get_order.py index e960ab7..0f3ce3c 100644 --- a/tests/multisafepay/e2e/examples/order_manager/test_get_order.py +++ b/tests/multisafepay/e2e/examples/order_manager/test_get_order.py @@ -41,6 +41,7 @@ @pytest.fixture(scope="module") def order_manager() -> OrderManager: + """Fixture that provides an OrderManager instance for testing.""" load_dotenv() api_key = os.getenv("API_KEY") multisafepay_sdk = Sdk(api_key, False) diff --git a/tests/multisafepay/e2e/examples/order_manager/test_refund.py b/tests/multisafepay/e2e/examples/order_manager/test_refund.py index d2bcdc4..0c7d344 100644 --- a/tests/multisafepay/e2e/examples/order_manager/test_refund.py +++ b/tests/multisafepay/e2e/examples/order_manager/test_refund.py @@ -38,6 +38,7 @@ @pytest.fixture(scope="module") def order_manager() -> OrderManager: + """Fixture that provides an OrderManager instance for testing.""" load_dotenv() api_key = os.getenv("API_KEY") multisafepay_sdk = Sdk(api_key, False) diff --git a/tests/multisafepay/e2e/examples/order_manager/test_refund_by_item.py b/tests/multisafepay/e2e/examples/order_manager/test_refund_by_item.py index 4cf5caa..1503a4e 100644 --- a/tests/multisafepay/e2e/examples/order_manager/test_refund_by_item.py +++ b/tests/multisafepay/e2e/examples/order_manager/test_refund_by_item.py @@ -41,6 +41,7 @@ @pytest.fixture(scope="module") def order_manager() -> OrderManager: + """Fixture that provides an OrderManager instance for testing.""" load_dotenv() api_key = os.getenv("API_KEY") multisafepay_sdk = Sdk(api_key, False) diff --git a/tests/multisafepay/e2e/examples/order_manager/test_refund_by_shopping_cart.py b/tests/multisafepay/e2e/examples/order_manager/test_refund_by_shopping_cart.py index 172800e..3a56268 100644 --- a/tests/multisafepay/e2e/examples/order_manager/test_refund_by_shopping_cart.py +++ b/tests/multisafepay/e2e/examples/order_manager/test_refund_by_shopping_cart.py @@ -41,6 +41,7 @@ @pytest.fixture(scope="module") def order_manager() -> OrderManager: + """Fixture that provides an OrderManager instance for testing.""" load_dotenv() api_key = os.getenv("API_KEY") multisafepay_sdk = Sdk(api_key, False) diff --git a/tests/multisafepay/e2e/examples/order_manager/test_update.py b/tests/multisafepay/e2e/examples/order_manager/test_update.py index 7f8f6bf..459da18 100644 --- a/tests/multisafepay/e2e/examples/order_manager/test_update.py +++ b/tests/multisafepay/e2e/examples/order_manager/test_update.py @@ -44,6 +44,7 @@ @pytest.fixture(scope="module") def order_manager() -> OrderManager: + """Fixture that provides an OrderManager instance for testing.""" load_dotenv() api_key = os.getenv("API_KEY") multisafepay_sdk = Sdk(api_key, False) diff --git a/tests/multisafepay/e2e/examples/payment_method_manager/test_get_by_gateway_code.py b/tests/multisafepay/e2e/examples/payment_method_manager/test_get_by_gateway_code.py index f6aa04d..7361c6e 100644 --- a/tests/multisafepay/e2e/examples/payment_method_manager/test_get_by_gateway_code.py +++ b/tests/multisafepay/e2e/examples/payment_method_manager/test_get_by_gateway_code.py @@ -26,6 +26,7 @@ @pytest.fixture(scope="module") def payment_method_manager() -> PaymentMethodManager: + """Fixture that provides a PaymentMethodManager instance for testing.""" load_dotenv() api_key = os.getenv("API_KEY") multisafepay_sdk = Sdk(api_key, False) diff --git a/tests/multisafepay/e2e/examples/payment_method_manager/test_get_payment_methods.py b/tests/multisafepay/e2e/examples/payment_method_manager/test_get_payment_methods.py index 0cc260f..26e998a 100644 --- a/tests/multisafepay/e2e/examples/payment_method_manager/test_get_payment_methods.py +++ b/tests/multisafepay/e2e/examples/payment_method_manager/test_get_payment_methods.py @@ -25,6 +25,7 @@ @pytest.fixture(scope="module") def payment_method_manager(): + """Fixture that provides a PaymentMethodManager instance for testing.""" load_dotenv() api_key = os.getenv("API_KEY") multisafepay_sdk = Sdk(api_key, False) diff --git a/tests/multisafepay/e2e/examples/recurring_manager/test_recurring.py b/tests/multisafepay/e2e/examples/recurring_manager/test_recurring.py index 5502970..7f02202 100644 --- a/tests/multisafepay/e2e/examples/recurring_manager/test_recurring.py +++ b/tests/multisafepay/e2e/examples/recurring_manager/test_recurring.py @@ -36,6 +36,7 @@ @pytest.fixture(scope="module") def sdk() -> Sdk: + """Fixture that provides an SDK instance for testing.""" load_dotenv() api_key = os.getenv("API_KEY") return Sdk(api_key, False) diff --git a/tests/multisafepay/e2e/examples/transaction_manager/test_get_transactions.py b/tests/multisafepay/e2e/examples/transaction_manager/test_get_transactions.py index fa94a3f..96bb10a 100644 --- a/tests/multisafepay/e2e/examples/transaction_manager/test_get_transactions.py +++ b/tests/multisafepay/e2e/examples/transaction_manager/test_get_transactions.py @@ -27,6 +27,7 @@ @pytest.fixture(scope="module") def transaction_manager(): + """Fixture that provides a TransactionManager instance for testing.""" load_dotenv() api_key = os.getenv("API_KEY") multisafepay_sdk = Sdk(api_key, False) diff --git a/tests/multisafepay/integration/api/path/orders/response/test_integration_order_response_afterpay_direct.py b/tests/multisafepay/integration/api/path/orders/response/test_integration_order_response_afterpay_direct.py index 95cb727..2481bbe 100644 --- a/tests/multisafepay/integration/api/path/orders/response/test_integration_order_response_afterpay_direct.py +++ b/tests/multisafepay/integration/api/path/orders/response/test_integration_order_response_afterpay_direct.py @@ -27,6 +27,7 @@ def test_afterpay_response(): + """Test AfterPay order response parsing and validation.""" data = { "amount": 37485, "amount_refunded": 0, diff --git a/tests/multisafepay/unit/api/path/orders/request/components/test_unit_payment_options.py b/tests/multisafepay/unit/api/path/orders/request/components/test_unit_payment_options.py index 3c1ab74..96e7fa9 100644 --- a/tests/multisafepay/unit/api/path/orders/request/components/test_unit_payment_options.py +++ b/tests/multisafepay/unit/api/path/orders/request/components/test_unit_payment_options.py @@ -15,6 +15,7 @@ def test_initializes_payment_options_correctly(): + """Test that PaymentOptions initializes correctly with all parameters.""" payment_options = PaymentOptions( notification_url="https://example.com/notify", settings={"key": "value"}, @@ -33,6 +34,7 @@ def test_initializes_payment_options_correctly(): def test_initializes_payment_options_with_empty_values(): + """Test that PaymentOptions initializes with None values when no parameters are provided.""" payment_options = PaymentOptions() assert payment_options.notification_url is None @@ -44,6 +46,7 @@ def test_initializes_payment_options_with_empty_values(): def test_add_notification_url_updates_value(): + """Test that add_notification_url updates the notification_url value.""" payment_options = PaymentOptions() payment_options_updated = payment_options.add_notification_url( "https://example.com/notify", @@ -54,6 +57,7 @@ def test_add_notification_url_updates_value(): def test_add_settings_updates_value(): + """Test that add_settings updates the settings value.""" payment_options = PaymentOptions() payment_options_updated = payment_options.add_settings({"key": "value"}) @@ -62,6 +66,7 @@ def test_add_settings_updates_value(): def test_add_notification_method_updates_value(): + """Test that add_notification_method updates the notification_method value.""" payment_options = PaymentOptions() payment_options_updated = payment_options.add_notification_method("GET") @@ -70,6 +75,7 @@ def test_add_notification_method_updates_value(): def test_add_notification_method_raises_exception_for_invalid_value(): + """Test that add_notification_method raises exception for invalid values.""" payment_options = PaymentOptions() try: @@ -79,6 +85,7 @@ def test_add_notification_method_raises_exception_for_invalid_value(): def test_add_redirect_url_updates_value(): + """Test that add_redirect_url updates the redirect_url value.""" payment_options = PaymentOptions() payment_options_updated = payment_options.add_redirect_url( "https://example.com/redirect", @@ -89,6 +96,7 @@ def test_add_redirect_url_updates_value(): def test_add_cancel_url_updates_value(): + """Test that add_cancel_url updates the cancel_url value.""" payment_options = PaymentOptions() payment_options_updated = payment_options.add_cancel_url( "https://example.com/cancel", @@ -99,6 +107,7 @@ def test_add_cancel_url_updates_value(): def test_add_close_window_updates_value(): + """Test that add_close_window updates the close_window value.""" payment_options = PaymentOptions() payment_options_updated = payment_options.add_close_window(True)