PTHMINT-87: Fix ruff D103 (#37)

zulquer · web-flow · commit ec4ccf1b13bd · 2025-11-26T12:29:47.000+01:00
diff --git a/pyproject.toml b/pyproject.toml
@@ -99,7 +99,6 @@ extend-safe-fixes = [
     "D415", # docstrings should end with a period, question mark, or exclamation point
 ]
 ignore = [
-    "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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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)
 

Original file line number	Diff line number	Diff line change
`@@ -99,7 +99,6 @@ extend-safe-fixes = [`
`99`	`99`	`"D415", # docstrings should end with a period, question mark, or exclamation point`
`100`	`100`	`]`
`101`	`101`	`ignore = [`
`102`		`- "D103",`
`103`	`102`	`"D104",`
`104`	`103`	`"D107",`
`105`	`104`	`"D200",`