python-getpaid-paynow 3.0.0a5__tar.gz → 3.1.0__tar.gz

python_getpaid_paynow-3.1.0/.github/release-drafter.yml

@@ -0,0 +1,31 @@
+name-template: 'v$RESOLVED_VERSION'
+tag-template: 'v$RESOLVED_VERSION'
+categories:
+  - title: ':boom: Breaking Changes'
+    label: 'breaking'
+  - title: ':rocket: Features'
+    label: 'enhancement'
+  - title: ':fire: Removals and Deprecations'
+    label: 'removal'
+  - title: ':beetle: Fixes'
+    label: 'bug'
+  - title: ':racehorse: Performance'
+    label: 'performance'
+  - title: ':rotating_light: Testing'
+    label: 'testing'
+  - title: ':construction_worker: Continuous Integration'
+    label: 'ci'
+  - title: ':books: Documentation'
+    label: 'documentation'
+  - title: ':hammer: Refactoring'
+    label: 'refactoring'
+  - title: ':lipstick: Style'
+    label: 'style'
+  - title: ':package: Dependencies'
+    labels:
+      - 'dependencies'
+      - 'build'
+template: |
+  ## Changes
+
+  $CHANGES

{python_getpaid_paynow-3.0.0a5 → python_getpaid_paynow-3.1.0}/.github/workflows/ci.yml

@@ -29,5 +29,11 @@ jobs:
       - name: Lint with ruff
         run: uv run ruff check .
 
+      - name: Type check with ty
+        run: uv run ty check
+
+      - name: Audit dependencies
+        run: uv run pip-audit --strict
+
       - name: Run tests
-        run: uv run pytest --tb=short
+        run: uv run pytest --tb=short --ignore tests/test_simulator_plugin.py

python_getpaid_paynow-3.1.0/.github/workflows/release.yml

@@ -0,0 +1,70 @@
+name: Release
+
+on:
+  push:
+    branches:
+      - main
+      - master
+
+jobs:
+  release:
+    name: Release
+    runs-on: ubuntu-latest
+    steps:
+      - name: Check out the repository
+        uses: actions/checkout@v4
+        with:
+          fetch-depth: 0  # Full history needed for version detection
+
+      - name: Set up Python
+        uses: actions/setup-python@v5
+        with:
+          python-version: "3.12"
+
+      - name: Install uv
+        run: pip install uv
+
+      # Version is read from __init__.py (dynamic via hatch)
+      - name: Detect version from __init__.py
+        id: get-version
+        run: |
+          init_py=$(grep -A2 '\[tool\.hatch\.version\]' pyproject.toml | grep '^path\s*=' | head -1 | sed -E "s/path\s*=\s*['\"]([^'\"]+)['\"].*/\1/")
+          version=$(grep '__version__' "$init_py" | head -1 | sed -E "s/.*= ['\"]([^'\"]+)['\"].*/\1/")
+          echo "version=$version" >> "$GITHUB_OUTPUT"
+
+      - name: Check if tag already exists
+        id: check-tag
+        run: |
+          tag="v${{ steps.get-version.outputs.version }}"
+          if git rev-parse "$tag" >/dev/null 2>&1; then
+            echo "already_tagged=true" >> "$GITHUB_OUTPUT"
+          else
+            echo "already_tagged=false" >> "$GITHUB_OUTPUT"
+          fi
+
+      - name: Tag new version
+        if: steps.check-tag.outputs.already_tagged == 'false'
+        run: |
+          tag="v${{ steps.get-version.outputs.version }}"
+          git config user.name "github-actions[bot]"
+          git config user.email "github-actions[bot]@users.noreply.github.com"
+          git tag -a "$tag" -m "Release $tag"
+          git push origin "$tag"
+
+      - name: Build package
+        run: uv build
+
+      - name: Publish package on PyPI
+        uses: pypa/gh-action-pypi-publish@release/v1
+        with:
+          password: ${{ secrets.PYPI_TOKEN }}
+
+      - name: Publish the release notes
+        if: steps.check-tag.outputs.already_tagged == 'false'
+        uses: release-drafter/release-drafter@v5.20.0
+        with:
+          publish: true
+          name: v${{ steps.get-version.outputs.version }}
+          tag: v${{ steps.get-version.outputs.version }}
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

python_getpaid_paynow-3.1.0/.python-version

@@ -0,0 +1 @@
+3.12

{python_getpaid_paynow-3.0.0a5 → python_getpaid_paynow-3.1.0}/PKG-INFO

@@ -1,14 +1,15 @@
 Metadata-Version: 2.4
 Name: python-getpaid-paynow
-Version: 3.0.0a5
+Version: 3.1.0
 Summary: Paynow payment gateway integration for python-getpaid ecosystem.
 Project-URL: Homepage, https://github.com/django-getpaid/python-getpaid-paynow
 Project-URL: Repository, https://github.com/django-getpaid/python-getpaid-paynow
+Project-URL: Documentation, https://python-getpaid-paynow.readthedocs.io
 Project-URL: Changelog, https://github.com/django-getpaid/python-getpaid-paynow/releases
 Author-email: Dominik Kozaczko <dominik@kozaczko.info>
 License: MIT
 License-File: LICENSE
-Classifier: Development Status :: 3 - Alpha
+Classifier: Development Status :: 5 - Production/Stable
 Classifier: Intended Audience :: Developers
 Classifier: License :: OSI Approved :: MIT License
 Classifier: Programming Language :: Python :: 3.12
@@ -18,10 +19,10 @@ Classifier: Topic :: Office/Business :: Financial :: Point-Of-Sale
 Classifier: Typing :: Typed
 Requires-Python: >=3.12
 Requires-Dist: httpx>=0.27.0
-Requires-Dist: python-getpaid-core>=3.0.0a4
+Requires-Dist: python-getpaid-core>=3.0.0
 Provides-Extra: simulator
 Requires-Dist: litestar>=2.0; extra == 'simulator'
-Requires-Dist: python-getpaid-simulator>=3.0.0a3; extra == 'simulator'
+Requires-Dist: python-getpaid-simulator>=3.0.0; extra == 'simulator'
 Description-Content-Type: text/markdown
 
 # python-getpaid-paynow
@@ -169,7 +170,7 @@ GETPAID_BACKEND_SETTINGS = {
 ## Requirements
 
 - Python 3.12+
-- `python-getpaid-core >= 3.0.0a4`
+- `python-getpaid-core >= 3.0.0`
 - `httpx >= 0.27.0`
 
 ## Links

{python_getpaid_paynow-3.0.0a5 → python_getpaid_paynow-3.1.0}/README.md

@@ -143,7 +143,7 @@ GETPAID_BACKEND_SETTINGS = {
 ## Requirements
 
 - Python 3.12+
-- `python-getpaid-core >= 3.0.0a4`
+- `python-getpaid-core >= 3.0.0`
 - `httpx >= 0.27.0`
 
 ## Links

python_getpaid_paynow-3.1.0/docs/changelog.md

@@ -0,0 +1,28 @@
+# Changelog
+
+## v3.0.0 (2026-06-04)
+
+Stable release of the Paynow payment gateway integration.
+
+### Breaking Changes
+
+- Version bumped from `3.0.0a4` to `3.0.0` (stable).
+- Development status changed from `Alpha` to `Production/Stable`.
+- Core dependency floor raised to `>=3.0.0` (from `>=3.0.0a4`).
+
+### Features
+
+- Full Paynow V3 REST API coverage with async HTTP client.
+- HMAC-SHA256 request and notification signature verification.
+- Payment creation with redirect URL support.
+- Notification (PUSH) callback handling with semantic payment updates.
+- Status polling (PULL) via API.
+- Refund lifecycle: create, check status, cancel.
+- Payment methods retrieval.
+- Simulator plugin for local testing via `python-getpaid-simulator`.
+- Support for 4 currencies: PLN, EUR, USD, GBP.
+
+### Migration from alpha
+
+- Update dependency from `python-getpaid-paynow>=3.0.0a4` to `python-getpaid-paynow>=3.0.0`.
+- No API changes — all public interfaces remain stable.

{python_getpaid_paynow-3.0.0a5 → python_getpaid_paynow-3.1.0}/docs/concepts.md

@@ -145,6 +145,22 @@ grosze for PLN, cents for EUR). The client handles conversion automatically:
 - `PaynowClient._to_lowest_unit(Decimal("49.99"))` → `4999`
 - `PaynowClient._from_lowest_unit(4999)` → `Decimal("49.99")`
 
+## Refund Notifications
+
+Paynow may send webhook notifications for **refund status changes** (NEW, PENDING,
+SUCCESSFUL, FAILED, CANCELLED). This plugin does **not** process refund
+notifications — the `handle_callback()` method only maps payment statuses.
+
+If your application needs to react to refund status changes, implement a
+custom webhook handler that calls `PaynowClient.get_refund_status()` to poll
+the current state.
+
+:::{note}
+This is a known limitation. Future versions may add refund notification
+support when the core `BaseProcessor` provides a dedicated refund callback
+hook.
+:::
+
 ## PUSH vs PULL Status Checking
 
 The plugin supports both notification models:

{python_getpaid_paynow-3.0.0a5 → python_getpaid_paynow-3.1.0}/pyproject.toml

@@ -9,7 +9,7 @@ authors = [
 ]
 requires-python = '>=3.12'
 classifiers = [
-    'Development Status :: 3 - Alpha',
+    'Development Status :: 5 - Production/Stable',
     'Intended Audience :: Developers',
     'License :: OSI Approved :: MIT License',
     'Programming Language :: Python :: 3.12',
@@ -19,13 +19,13 @@ classifiers = [
     'Typing :: Typed',
 ]
 dependencies = [
-    'python-getpaid-core>=3.0.0a4',
+    'python-getpaid-core>=3.0.0',
     'httpx>=0.27.0',
 ]
 
 [project.optional-dependencies]
 simulator = [
-    'python-getpaid-simulator>=3.0.0a3',
+    'python-getpaid-simulator>=3.0.0',
     'litestar>=2.0',
 ]
 
@@ -41,11 +41,13 @@ dev = [
     'furo>=2024.8.6',
     'sphinx>=8.0',
     'myst-parser>=4.0',
+    'pip-audit>=2.7.0',
 ]
 
 [project.urls]
 Homepage = 'https://github.com/django-getpaid/python-getpaid-paynow'
 Repository = 'https://github.com/django-getpaid/python-getpaid-paynow'
+Documentation = 'https://python-getpaid-paynow.readthedocs.io'
 Changelog = 'https://github.com/django-getpaid/python-getpaid-paynow/releases'
 
 [project.entry-points."getpaid.backends"]
@@ -120,8 +122,18 @@ include = ['tests/**']
 unresolved-attribute = 'ignore'
 invalid-argument-type = 'ignore'
 
+# processor.py no longer needs overrides — client signatures accept str | None
+# and runtime guards raise ValueError for None values.
+
+# Simulator files depend on optional extras (litestar, getpaid_simulator)
+[[tool.ty.overrides]]
+include = ['src/getpaid_paynow/simulator/**']
+[tool.ty.overrides.rules]
+unresolved-attribute = 'ignore'
+unresolved-import = 'ignore'
+
 [[tool.ty.overrides]]
-include = ['src/getpaid_paynow/processor.py']
+include = ['tests/test_simulator_plugin.py']
 [tool.ty.overrides.rules]
 unresolved-attribute = 'ignore'
-call-non-callable = 'ignore'
+unresolved-import = 'ignore'

python_getpaid_paynow-3.1.0/src/getpaid_paynow/__init__.py

@@ -0,0 +1,11 @@
+"""Paynow V3 payment gateway integration for python-getpaid ecosystem."""
+
+from getpaid_paynow.client import PaynowClient
+from getpaid_paynow.processor import PaynowProcessor
+
+__all__ = [
+    "PaynowClient",
+    "PaynowProcessor",
+]
+
+__version__ = "3.1.0"

{python_getpaid_paynow-3.0.0a5 → python_getpaid_paynow-3.1.0}/src/getpaid_paynow/client.py

@@ -4,8 +4,11 @@ import base64
 import hashlib
 import hmac
 import json
+import logging
 import uuid
+from decimal import ROUND_HALF_UP
 from decimal import Decimal
+from typing import NoReturn
 
 import httpx
 from getpaid_core.exceptions import CommunicationError
@@ -19,6 +22,9 @@ from .types import PaymentStatusResponse
 from .types import RefundStatusResponse
 
 
+logger = logging.getLogger(__name__)
+
+
 class PaynowClient:
     """Async client for Paynow V3 REST API.
 
@@ -30,23 +36,24 @@ class PaynowClient:
             await client.create_payment(...)
     """
 
-    last_response: httpx.Response | None = None
-
     def __init__(
         self,
         *,
         api_key: str,
         signature_key: str,
         api_url: str,
+        timeout: float = 10.0,
     ) -> None:
         self.api_key = api_key
         self.signature_key = signature_key
         self.api_url = api_url.rstrip("/")
+        self.timeout = timeout
+        self.last_response: httpx.Response | None = None
         self._client: httpx.AsyncClient | None = None
         self._owns_client: bool = False
 
     async def __aenter__(self) -> "PaynowClient":
-        self._client = httpx.AsyncClient()
+        self._client = httpx.AsyncClient(timeout=self.timeout)
         self._owns_client = True
         return self
 
@@ -124,8 +131,7 @@ class PaynowClient:
         body: str = "",
         parameters: dict | None = None,
     ) -> dict[str, str]:
-        """Build request headers with authentication and
-        signature."""
+        """Build request headers with authentication and signature."""
         params = parameters or {}
         signature = self._calculate_request_signature(
             api_key=self.api_key,
@@ -148,41 +154,110 @@ class PaynowClient:
         *,
         body: str | None = None,
         params: dict | None = None,
+        retries: int = 3,
+        backoff: float = 0.5,
     ) -> httpx.Response:
-        """Execute an authenticated HTTP request."""
-        url = f"{self.api_url}{path}"
-        idempotency_key = self._generate_idempotency_key()
+        """Execute an authenticated HTTP request with retry for transient
+        failures.
 
-        # Convert params to string values for signature
-        str_params: dict = {}
-        if params:
-            str_params = {k: str(v) for k, v in params.items()}
-
-        headers = self._build_headers(
-            idempotency_key=idempotency_key,
-            body=body or "",
-            parameters=str_params,
-        )
-
-        if self._client is not None:
-            return await self._client.request(
-                method,
-                url,
-                headers=headers,
-                content=body,
-                params=params,
+        Retries on 5xx server errors, timeouts, and connection errors.
+        Does not retry on 4xx client errors or credential failures.
+        """
+        last_exc: Exception | None = None
+        for attempt in range(retries + 1):
+            url = f"{self.api_url}{path}"
+            idempotency_key = self._generate_idempotency_key()
+
+            # Convert params to string values for signature
+            str_params: dict = {}
+            if params:
+                str_params = {k: str(v) for k, v in params.items()}
+
+            headers = self._build_headers(
+                idempotency_key=idempotency_key,
+                body=body or "",
+                parameters=str_params,
             )
-        async with httpx.AsyncClient() as client:
-            return await client.request(
-                method,
-                url,
-                headers=headers,
-                content=body,
-                params=params,
+
+            try:
+                if self._client is not None:
+                    response = await self._client.request(
+                        method,
+                        url,
+                        headers=headers,
+                        content=body,
+                        params=params,
+                    )
+                else:
+                    async with httpx.AsyncClient(
+                        timeout=self.timeout
+                    ) as client:
+                        response = await client.request(
+                            method,
+                            url,
+                            headers=headers,
+                            content=body,
+                            params=params,
+                        )
+
+                # Retry only on transient failures (5xx, timeouts, conn errors)
+                if response.status_code < 500:
+                    self.last_response = response
+                    return response
+
+                # 5xx — retry
+                last_exc = CommunicationError(
+                    f"Paynow API returned {response.status_code}. "
+                    f"Attempt {attempt + 1}/{retries + 1}."
+                )
+                if attempt < retries:
+                    await self._sleep(backoff * (2**attempt))
+                    logger.warning(
+                        "Paynow %s %s returned %d, retrying in %.1fs",
+                        method,
+                        path,
+                        response.status_code,
+                        backoff * (2**attempt),
+                    )
+                continue
+
+            except (httpx.TimeoutException, httpx.ConnectError) as exc:
+                last_exc = exc
+                if attempt < retries:
+                    await self._sleep(backoff * (2**attempt))
+                    logger.warning(
+                        "Paynow %s %s failed (%s), retrying in %.1fs",
+                        method,
+                        path,
+                        type(exc).__name__,
+                        backoff * (2**attempt),
+                    )
+                continue
+
+        # All retries exhausted
+        if isinstance(last_exc, CommunicationError):
+            last_exc.args = (
+                f"Paynow API request failed after {retries + 1} attempts: "
+                f"{last_exc.args[0]}",
             )
+            raise last_exc
+        raise CommunicationError(
+            f"Paynow API request failed after {retries + 1} attempts: "
+            f"{last_exc}"
+        ) from last_exc
+
+    @staticmethod
+    async def _sleep(seconds: float) -> None:
+        """Sleep without blocking the event loop."""
+        import asyncio
+
+        await asyncio.sleep(seconds)
+
+    def _handle_error(self, response: httpx.Response) -> NoReturn:
+        """Raise appropriate exception based on status code.
 
-    def _handle_error(self, response: httpx.Response) -> None:
-        """Raise appropriate exception based on status code."""
+        This method always raises, so it is annotated as NoReturn.
+        """
         if response.status_code == 401:
             raise CredentialsError(
                 "Paynow API authentication failed.",
@@ -195,9 +270,11 @@ class PaynowClient:
 
     @staticmethod
     def _to_lowest_unit(amount: Decimal) -> int:
-        """Convert a Decimal amount to integer lowest currency
-        unit."""
-        return int(amount * 100)
+        """Convert a Decimal amount to integer lowest currency unit,
+        rounding half-up to avoid silent truncation."""
+        return int(
+            (amount * 100).quantize(Decimal("1"), rounding=ROUND_HALF_UP)
+        )
 
     @staticmethod
     def _from_lowest_unit(amount: int) -> Decimal:
@@ -225,8 +302,7 @@ class PaynowClient:
 
         :param amount: Payment amount in main currency unit.
         :param currency: ISO 4217 currency code.
-        :param external_id: Unique external ID (maps to
-            payment.id).
+        :param external_id: Unique external ID (maps to payment.id).
         :param description: Payment description.
         :param buyer_email: Buyer email address.
         :return: Response with redirectUrl and paymentId.
@@ -263,31 +339,34 @@ class PaynowClient:
         if self.last_response.status_code in (200, 201):
             return self.last_response.json()
         self._handle_error(self.last_response)
-        # unreachable — _handle_error always raises
-        raise AssertionError  # pragma: no cover
 
     async def get_payment_status(
         self,
-        payment_id: str,
+        payment_id: str | None,
     ) -> PaymentStatusResponse:
         """Get payment status.
 
         GET /v3/payments/{paymentId}/status
 
-        :param payment_id: Paynow payment ID.
+        :param payment_id: Paynow payment ID (may be None for uncreated
+            payments; raises a clear error in that case).
         :return: Payment status response.
         """
+        if payment_id is None:
+            raise ValueError(
+                "payment_id must not be None. "
+                "Call create_payment() first to obtain a payment ID."
+            )
         path = f"/v3/payments/{payment_id}/status"
         self.last_response = await self._request("GET", path)
         if self.last_response.status_code == 200:
             return self.last_response.json()
         self._handle_error(self.last_response)
-        raise AssertionError  # pragma: no cover
 
     async def create_refund(
         self,
         *,
-        payment_id: str,
+        payment_id: str | None,
         amount: Decimal,
         reason: str | None = None,
     ) -> CreateRefundResponse:
@@ -295,11 +374,17 @@ class PaynowClient:
 
         POST /v3/payments/{paymentId}/refunds
 
-        :param payment_id: Paynow payment ID.
+        :param payment_id: Paynow payment ID (may be None for uncreated
+            payments; raises a clear error in that case).
         :param amount: Refund amount in main currency unit.
         :param reason: Refund reason code.
         :return: Refund response.
         """
+        if payment_id is None:
+            raise ValueError(
+                "payment_id must not be None. "
+                "Call create_payment() first to obtain a payment ID."
+            )
         amount_int = self._to_lowest_unit(amount)
         data: dict = {"amount": amount_int}
         if reason is not None:
@@ -326,21 +411,26 @@ class PaynowClient:
 
     async def get_refund_status(
         self,
-        refund_id: str,
+        refund_id: str | None,
     ) -> RefundStatusResponse:
         """Get refund status.
 
         GET /v3/refunds/{refundId}/status
 
-        :param refund_id: Paynow refund ID.
+        :param refund_id: Paynow refund ID (may be None for uncreated
+            refunds; raises a clear error in that case).
         :return: Refund status response.
         """
+        if refund_id is None:
+            raise ValueError(
+                "refund_id must not be None. "
+                "Call create_refund() first to obtain a refund ID."
+            )
         path = f"/v3/refunds/{refund_id}/status"
         self.last_response = await self._request("GET", path)
         if self.last_response.status_code == 200:
             return self.last_response.json()
         self._handle_error(self.last_response)
-        raise AssertionError  # pragma: no cover
 
     async def cancel_refund(
         self,
@@ -352,6 +442,11 @@ class PaynowClient:
 
         :param refund_id: Paynow refund ID.
         """
+        if not refund_id:
+            raise ValueError(
+                "refund_id must not be empty. "
+                "Call create_refund() first to obtain a refund ID."
+            )
         path = f"/v3/refunds/{refund_id}/cancel"
         self.last_response = await self._request("POST", path)
         if self.last_response.status_code in (200, 202):
@@ -368,8 +463,7 @@ class PaynowClient:
 
         GET /v3/payments/paymentmethods
 
-        :param amount: Optional amount filter (lowest currency
-            unit).
+        :param amount: Optional amount filter (lowest currency unit).
         :param currency: Optional currency filter.
         :return: List of payment method groups.
         """
@@ -386,4 +480,3 @@ class PaynowClient:
         if self.last_response.status_code == 200:
             return self.last_response.json()
         self._handle_error(self.last_response)
-        raise AssertionError  # pragma: no cover

{python_getpaid_paynow-3.0.0a5 → python_getpaid_paynow-3.1.0}/src/getpaid_paynow/processor.py

@@ -8,7 +8,7 @@ from typing import ClassVar
 from getpaid_core.enums import PaymentEvent
 from getpaid_core.exceptions import InvalidCallbackError
 from getpaid_core.processor import BaseProcessor
-from getpaid_core.types import ChargeResponse
+from getpaid_core.types import ChargeResult as ChargeResponse
 from getpaid_core.types import PaymentUpdate
 from getpaid_core.types import RefundResult
 from getpaid_core.types import TransactionResult
@@ -45,6 +45,7 @@ class PaynowProcessor(BaseProcessor):
             api_key=str(self.get_setting("api_key", "")),
             signature_key=str(self.get_setting("signature_key", "")),
             api_url=self.get_paywall_baseurl(),
+            timeout=float(self.get_setting("timeout", 10.0)),
         )
 
     def _resolve_url(self, url_template: str) -> str:
@@ -241,7 +242,18 @@ class PaynowProcessor(BaseProcessor):
     ) -> RefundResult:
         """Start a refund via Paynow API."""
         client = self._get_client()
-        refund_amount = amount or self.payment.amount_paid
+        refund_amount = (
+            amount if amount is not None else self.payment.amount_paid
+        )
+        if refund_amount <= 0:
+            raise ValueError(
+                f"Refund amount must be positive. Got {refund_amount}."
+            )
+        if refund_amount > self.payment.amount_paid:
+            raise ValueError(
+                f"Refund amount ({refund_amount}) exceeds paid amount "
+                f"({self.payment.amount_paid})."
+            )
         response = await client.create_refund(
             payment_id=self.payment.external_id,
             amount=refund_amount,
@@ -253,12 +265,21 @@ class PaynowProcessor(BaseProcessor):
         return RefundResult(amount=refund_amount, provider_data=provider_data)
 
     async def cancel_refund(self, **kwargs) -> bool:
-        """Cancel an awaiting refund via Paynow API."""
+        """Cancel an awaiting refund via Paynow API.
+
+        The refund identifier is stored in provider_data["refund_id"]
+        by ``start_refund()``. If it is missing, raise an error —
+        the Payment protocol does not define an ``external_refund_id``
+        attribute, so falling back to ``getattr`` was a silent no-op
+        that could silently cancel the wrong refund or do nothing at
+        all.
+        """
         client = self._get_client()
         refund_id = self.payment.provider_data.get("refund_id")
         if not refund_id:
-            refund_id = getattr(self.payment, "external_refund_id", "")
-        if not refund_id:
-            raise InvalidCallbackError("Missing refund identifier")
+            raise InvalidCallbackError(
+                "Missing refund identifier. "
+                'Expected provider_data["refund_id"] set by start_refund().'
+            )
         await client.cancel_refund(refund_id)
         return True

{python_getpaid_paynow-3.0.0a5 → python_getpaid_paynow-3.1.0}/src/getpaid_paynow/types.py

@@ -1,15 +1,10 @@
 """Paynow V3 API types and enums."""
 
-from enum import StrEnum
 from enum import auto
 from enum import unique
 from typing import TypedDict
 
-
-class AutoName(StrEnum):
-    @staticmethod
-    def _generate_next_value_(name, start, count, last_values):
-        return name.strip("_")
+from getpaid_core import AutoName
 
 
 @unique

{python_getpaid_paynow-3.0.0a5 → python_getpaid_paynow-3.1.0}/tests/conftest.py

@@ -5,7 +5,6 @@ from decimal import Decimal
 from typing import Any
 
 import pytest
-
 from getpaid_core.enums import PaymentStatus
 from getpaid_core.types import BuyerInfo
 from getpaid_core.types import ItemInfo

{python_getpaid_paynow-3.0.0a5 → python_getpaid_paynow-3.1.0}/tests/test_callback.py

@@ -6,7 +6,6 @@ import hmac as hmac_mod
 import json
 
 import pytest
-
 from getpaid_core.enums import PaymentEvent
 from getpaid_core.exceptions import InvalidCallbackError
 

{python_getpaid_paynow-3.0.0a5 → python_getpaid_paynow-3.1.0}/tests/test_client.py

@@ -3,6 +3,7 @@
 import json
 from decimal import Decimal
 
+import httpx
 import pytest
 from getpaid_core.exceptions import CommunicationError
 from getpaid_core.exceptions import CredentialsError
@@ -146,6 +147,19 @@ class TestAmountConversion:
     def test_to_lowest_unit_small(self):
         assert PaynowClient._to_lowest_unit(Decimal("0.01")) == 1
 
+    def test_to_lowest_unit_no_truncation(self):
+        """Regression: int() truncation must not silently lose money.
+
+        Decimal("1.005") * 100 = Decimal("100.5"), which must round to 101,
+        not 100. Using int() directly would have produced 100 — a real
+        money-loss bug (PAYNOW-001).
+        """
+        assert PaynowClient._to_lowest_unit(Decimal("1.005")) == 101
+
+    def test_to_lowest_unit_rounds_down(self):
+        """Fractional cents below .5 must round down."""
+        assert PaynowClient._to_lowest_unit(Decimal("1.004")) == 100
+
     def test_from_lowest_unit(self):
         assert PaynowClient._from_lowest_unit(123) == Decimal("1.23")
 
@@ -587,3 +601,120 @@ class TestAsyncContextManager:
         await client.get_payment_status("PAY-123")
         assert client.last_response is not None
         assert client.last_response.status_code == 200
+
+
+class TestNoneIdGuards:
+    """Tests for runtime guards on None/empty IDs."""
+
+    async def test_get_payment_status_raises_on_none(self):
+        client = _make_client()
+        with pytest.raises(ValueError, match="payment_id must not be None"):
+            await client.get_payment_status(None)
+
+    async def test_create_refund_raises_on_none_payment_id(self):
+        client = _make_client()
+        with pytest.raises(ValueError, match="payment_id must not be None"):
+            await client.create_refund(payment_id=None, amount=Decimal("10.00"))
+
+    async def test_get_refund_status_raises_on_none(self):
+        client = _make_client()
+        with pytest.raises(ValueError, match="refund_id must not be None"):
+            await client.get_refund_status(None)
+
+    async def test_cancel_refund_raises_on_empty_string(self):
+        client = _make_client()
+        with pytest.raises(ValueError, match="refund_id must not be empty"):
+            await client.cancel_refund("")
+
+
+class TestRetryLogic:
+    """Tests for transient failure retry behavior."""
+
+    async def test_retry_on_500(self, respx_mock):
+        """5xx errors should be retried."""
+        url = f"{SANDBOX_URL}/v3/payments/PAY-123/status"
+        route = respx_mock.get(url)
+        route.side_effect = [
+            httpx.Response(500, json={"error": "server"}),
+            httpx.Response(
+                200,
+                json={"paymentId": "PAY-123", "status": "CONFIRMED"},
+            ),
+        ]
+        client = _make_client()
+        result = await client.get_payment_status("PAY-123")
+        assert result["status"] == "CONFIRMED"
+        assert route.call_count == 2
+
+    async def test_retry_on_502(self, respx_mock):
+        """502 Bad Gateway should be retried."""
+        url = f"{SANDBOX_URL}/v3/payments/PAY-123/status"
+        route = respx_mock.get(url)
+        route.side_effect = [
+            httpx.Response(502),
+            httpx.Response(
+                200,
+                json={"paymentId": "PAY-123", "status": "CONFIRMED"},
+            ),
+        ]
+        client = _make_client()
+        result = await client.get_payment_status("PAY-123")
+        assert result["status"] == "CONFIRMED"
+        assert route.call_count == 2
+
+    async def test_no_retry_on_4xx(self, respx_mock):
+        """4xx errors should NOT be retried."""
+        url = f"{SANDBOX_URL}/v3/payments/PAY-123/status"
+        respx_mock.get(url).respond(
+            status_code=404,
+            json={"error": "not found"},
+        )
+        client = _make_client()
+        with pytest.raises(CommunicationError):
+            await client.get_payment_status("PAY-123")
+
+    async def test_no_retry_on_401(self, respx_mock):
+        """401 should not be retried (credentials error)."""
+        url = f"{SANDBOX_URL}/v3/payments/PAY-123/status"
+        respx_mock.get(url).respond(
+            status_code=401,
+            json={"error": "unauthorized"},
+        )
+        client = _make_client()
+        with pytest.raises(CredentialsError):
+            await client.get_payment_status("PAY-123")
+
+    async def test_exhaust_retries_on_500(self, respx_mock):
+        """Should raise after max retries exhausted."""
+        url = f"{SANDBOX_URL}/v3/payments/PAY-123/status"
+        respx_mock.get(url).respond(
+            status_code=500,
+            json={"error": "server"},
+        )
+        client = _make_client()
+        with pytest.raises(CommunicationError, match="attempts"):
+            await client.get_payment_status("PAY-123")
+
+    async def test_retry_on_timeout(self, respx_mock):
+        """Timeouts should be retried."""
+        url = f"{SANDBOX_URL}/v3/payments/PAY-123/status"
+        route = respx_mock.get(url)
+        route.side_effect = [
+            httpx.TimeoutException("timeout"),
+            httpx.Response(
+                200,
+                json={"paymentId": "PAY-123", "status": "CONFIRMED"},
+            ),
+        ]
+        client = _make_client()
+        result = await client.get_payment_status("PAY-123")
+        assert result["status"] == "CONFIRMED"
+        assert route.call_count == 2
+
+    async def test_exhaust_retries_on_timeout(self, respx_mock):
+        """Should raise after max retries on repeated timeouts."""
+        url = f"{SANDBOX_URL}/v3/payments/PAY-123/status"
+        respx_mock.get(url).side_effect = httpx.TimeoutException("timeout")
+        client = _make_client()
+        with pytest.raises(CommunicationError, match="attempts"):
+            await client.get_payment_status("PAY-123")

python_getpaid_paynow-3.1.0/tests/test_entry_points.py

@@ -0,0 +1,27 @@
+"""Tests for getpaid.backends entry point registration."""
+
+from getpaid_core.processor import BaseProcessor
+from getpaid_core.registry import PluginRegistry
+
+
+class TestEntryPoints:
+    """Verify entry points are correctly registered."""
+
+    def test_paynow_backend_entry_point(self):
+        """PaynowProcessor must be discoverable via entry points."""
+        registry = PluginRegistry()
+        registry.discover()
+
+        processor_class = registry.get_by_slug("paynow")
+        assert issubclass(processor_class, BaseProcessor)
+        assert processor_class.slug == "paynow"
+        assert processor_class.display_name == "Paynow"
+
+    def test_paynow_accepted_currencies(self):
+        """PaynowProcessor must list supported currencies."""
+        registry = PluginRegistry()
+        registry.discover()
+
+        processor_class = registry.get_by_slug("paynow")
+        assert len(processor_class.accepted_currencies) > 0
+        assert "PLN" in processor_class.accepted_currencies

{python_getpaid_paynow-3.0.0a5 → python_getpaid_paynow-3.1.0}/tests/test_processor.py

@@ -4,7 +4,6 @@ import json
 from decimal import Decimal
 
 import pytest
-
 from getpaid_core.enums import BackendMethod
 from getpaid_core.enums import PaymentEvent
 from getpaid_core.exceptions import CommunicationError
@@ -206,3 +205,88 @@ class TestRefunds:
 
         with pytest.raises(CommunicationError):
             await processor.cancel_refund()
+
+
+class TestRefundAmountGuard:
+    """Tests for PAYNOW-007: refund amount validation."""
+
+    async def test_refund_zero_amount_raises(self):
+        payment = make_mock_payment(external_id="PAY-123")
+        payment.amount_paid = Decimal("100.00")
+        processor = _make_processor(payment=payment)
+
+        with pytest.raises(ValueError, match="Refund amount must be positive"):
+            await processor.start_refund(amount=Decimal("0.00"))
+
+    async def test_refund_negative_amount_raises(self):
+        payment = make_mock_payment(external_id="PAY-123")
+        payment.amount_paid = Decimal("100.00")
+        processor = _make_processor(payment=payment)
+
+        with pytest.raises(ValueError, match="Refund amount must be positive"):
+            await processor.start_refund(amount=Decimal("-10.00"))
+
+    async def test_refund_exceeds_paid_raises(self):
+        payment = make_mock_payment(external_id="PAY-123")
+        payment.amount_paid = Decimal("50.00")
+        processor = _make_processor(payment=payment)
+
+        with pytest.raises(ValueError, match="exceeds paid amount"):
+            await processor.start_refund(amount=Decimal("100.00"))
+
+    async def test_refund_equal_to_paid_succeeds(self, respx_mock):
+        url = f"{SANDBOX_URL}/v3/payments/PAY-123/refunds"
+        respx_mock.post(url).respond(
+            json={"refundId": "REF-456", "status": "NEW"},
+            status_code=201,
+        )
+        payment = make_mock_payment(external_id="PAY-123")
+        payment.amount_paid = Decimal("100.00")
+        processor = _make_processor(payment=payment)
+
+        result = await processor.start_refund(amount=Decimal("100.00"))
+        assert result.amount == Decimal("100.00")
+
+
+class TestCancelRefundMissingId:
+    """Tests for PAYNOW-006: cancel_refund raises when refund_id is missing.
+
+    The Payment protocol does not define an ``external_refund_id``
+    attribute, so falling back to ``getattr`` was a silent no-op.
+    """
+
+    async def test_cancel_refund_raises_when_no_refund_id(self):
+        """cancel_refund raises on missing refund_id."""
+        from getpaid_core.exceptions import InvalidCallbackError
+
+        payment = make_mock_payment(provider_data={})
+        processor = _make_processor(payment=payment)
+
+        with pytest.raises(
+            InvalidCallbackError, match="Missing refund identifier"
+        ):
+            await processor.cancel_refund()
+
+    async def test_cancel_refund_raises_when_provider_data_empty(self):
+        """cancel_refund raises when provider_data is empty dict."""
+        from getpaid_core.exceptions import InvalidCallbackError
+
+        payment = make_mock_payment(provider_data=None)
+        processor = _make_processor(payment=payment)
+
+        with pytest.raises(
+            InvalidCallbackError, match="Missing refund identifier"
+        ):
+            await processor.cancel_refund()
+
+    async def test_cancel_refund_raises_when_provider_data_missing_key(self):
+        """cancel_refund raises when provider_data lacks refund_id key."""
+        from getpaid_core.exceptions import InvalidCallbackError
+
+        payment = make_mock_payment(provider_data={"other_key": "value"})
+        processor = _make_processor(payment=payment)
+
+        with pytest.raises(
+            InvalidCallbackError, match="Missing refund identifier"
+        ):
+            await processor.cancel_refund()

python_getpaid_paynow-3.1.0/tests/test_public_api.py

@@ -0,0 +1,33 @@
+"""Tests for the public package API."""
+
+import tomllib
+from pathlib import Path
+
+import getpaid_core
+
+import getpaid_paynow
+
+
+def test_version() -> None:
+    assert getpaid_paynow.__version__ == "3.0.0"
+
+
+def test_core_dependency_floor() -> None:
+    current_version = getpaid_paynow.__version__
+    pyproject_data = tomllib.loads(Path("pyproject.toml").read_text())
+    assert (
+        f"python-getpaid-core>={current_version}"
+        in pyproject_data["project"]["dependencies"]
+    )
+
+
+def test_core_version_major_minor_matches_paynow_version() -> None:
+    """Core and paynow must share the same major.minor version.
+
+    Patch-level drift is allowed because core may ship hotfixes
+    independently of backend plugins.
+    """
+    core_parts = getpaid_core.__version__.split(".")
+    paynow_parts = getpaid_paynow.__version__.split(".")
+    assert core_parts[0] == paynow_parts[0]
+    assert core_parts[1] == paynow_parts[1]

python_getpaid_paynow-3.0.0a5/docs/changelog.md

@@ -1,19 +0,0 @@
-# Changelog
-
-## v0.1.0 (2026-02-14)
-
-Initial release.
-
-### Features
-
-- Full Paynow V3 REST API coverage
-- Async HTTP client (`PaynowClient`) with API Key + HMAC-SHA256 signing
-- Payment processor (`PaynowProcessor`) implementing `BaseProcessor`
-- Payment creation with redirect URL
-- HMAC-SHA256 signature calculation and verification
-- Notification (PUSH) callback handling
-- Status polling (PULL) via API
-- Refund support (create, check status, cancel)
-- Payment methods retrieval
-- Amount conversion (`Decimal` ↔ integer lowest currency unit)
-- Support for PLN, EUR, USD, GBP currencies

python_getpaid_paynow-3.0.0a5/src/getpaid_paynow/__init__.py

@@ -1,21 +0,0 @@
-"""Paynow V3 payment gateway integration for python-getpaid ecosystem."""
-
-__version__ = "3.0.0a5"
-
-__all__ = [
-    "PaynowClient",
-    "PaynowProcessor",
-]
-
-
-def __getattr__(name: str):
-    if name == "PaynowClient":
-        from getpaid_paynow.client import PaynowClient
-
-        return PaynowClient
-    if name == "PaynowProcessor":
-        from getpaid_paynow.processor import PaynowProcessor
-
-        return PaynowProcessor
-    msg = f"module {__name__!r} has no attribute {name!r}"
-    raise AttributeError(msg)

python_getpaid_paynow-3.0.0a5/tests/test_public_api.py

@@ -1,18 +0,0 @@
-"""Tests for the public package API."""
-
-import tomllib
-from pathlib import Path
-
-import getpaid_paynow
-
-
-def test_version() -> None:
-    assert getpaid_paynow.__version__ == "3.0.0a4"
-
-
-def test_core_dependency_floor() -> None:
-    pyproject_data = tomllib.loads(Path("pyproject.toml").read_text())
-    assert (
-        "python-getpaid-core>=3.0.0a4"
-        in pyproject_data["project"]["dependencies"]
-    )