Merge pull request #37 from RafaelJohn9/ci

CI updates

Author: RafaelJohn9

.github/workflows/code-quality.yml

@@ -36,7 +36,7 @@ jobs:
         run: uv pip install -e '.[dev]'
 
       - name: Run Ruff
-        run: uv run ruff check mpesa_sdk
+        run: uv run ruff check mpesa_sdk && uv run ruff check tests
 
   mypy:
     runs-on: ubuntu-latest

.github/workflows/publish.yml

@@ -0,0 +1,31 @@
+name: Publish Python Package
+
+on:
+  push:
+    tags:
+      - "v*"   # publish only on version tags like v0.1.0
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - name: Set up Python
+        uses: actions/setup-python@v5
+        with:
+          python-version: "3.12"
+
+      - name: Install Hatch
+        run: pip install hatch
+
+      - name: Build package
+        run: hatch build
+
+      - name: Publish to PyPI
+        env:
+          HATCH_INDEX_USER: __token__
+          HATCH_INDEX_AUTH: ${{ secrets.PYPI_API_TOKEN }}
+        run: hatch publish

.github/workflows/tests.yml

@@ -0,0 +1,31 @@
+name: Run Unit Tests
+
+on:
+  push:
+    branches: [ main ]
+  pull_request:
+  workflow_dispatch: # Allows manual triggering of the workflow
+
+jobs:
+  unit-tests:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - name: Set up Python
+        uses: actions/setup-python@v4
+        with:
+          python-version: '3.12'
+
+      - name: Install dependencies
+        run: |
+          python -m pip install --upgrade pip
+            pip install ".[dev,test]"
+            pip install pytest
+            pip install -e .
+
+      - name: Run unit tests
+        run: |
+          pytest tests/unit

pyproject.toml

@@ -8,7 +8,7 @@ dynamic = ["version"]
 description = "A Python SDK for integrating with M-Pesa APIs"
 readme = "README.md"
 license = { text = "APACHE-2.0" }
-requires-python = ">=3.10"
+requires-python = ">=3.12"
 authors = [
   { name = "John Kagunda", email = "johnmkagunda@gmail.com" }
 ]
@@ -19,43 +19,59 @@ classifiers = [
   "Development Status :: 4 - Beta",
   "Intended Audience :: Developers",
   "License :: OSI Approved :: Apache Software License",
-  "Programming Language :: Python :: 3.10",
-  "Programming Language :: Python :: 3.11",
   "Programming Language :: Python :: 3.12",
   "Operating System :: OS Independent",
   "Topic :: Software Development :: Libraries :: Python Modules",
   "Topic :: Internet :: WWW/HTTP :: Dynamic Content",
   "Typing :: Typed"
 ]
-keywords = ["mpesa", "sdk", "payment", "africa", "flask", "django", "fastapi", "rest", "api"]
-urls = { Homepage = "https://github.com/rafaeljohn9/mpesa-daraja-sdk", Repository = "https://github.com/rafaeljohn9/mpesa-daraja-sdk" }
+keywords = [
+  "mpesa",
+  "sdk",
+  "payment",
+  "africa",
+  "flask",
+  "django",
+  "fastapi",
+  "rest",
+  "api",
+  "mobile money",
+  "payments",
+]
+
+dependencies = [ 
+  "pydantic >=2.10.6,<3.0.0",
+  "pydantic[email] >=2.10.6,<3.0.0",
+  "requests >=2.32.3,<3.0.0",
+  "typing_extensions >= 4.12.2,<5.0.0"
+]
+
+[project.urls]
+Homepage = "https://github.com/rafaeljohn9/mpesa-sdk"
+Repository = "https://github.com/rafaeljohn9/mpesa-sdk"
+Issues = "https://github.com/rafaeljohn9/mpesa-sdk/issues"
+
 
 [project.optional-dependencies]
 dev = [
-  "bandit",
   "build",
   "coverage",
   "hatch",
   "hatch-vcs",
   "hatchling",
   "mypy",
-  "pydantic",
+  "bandit",
   "pytest",
   "pytest-cov",
-  "requests",
   "ruff",
   "types-requests"
 ]
-install = [
-  "pydantic",
-  "requests"
-]
 test = [
+  "bandit",
+  "mypy",
   "pyngrok",
   "pytest",
   "pytest-cov",
-  "requests",
-  "pydantic",
   "types-requests"
 ]
 
@@ -71,13 +87,18 @@ include = [
 [tool.hatch.build.targets.wheel]
 packages = ["mpesa_sdk"]
 
-# Ruff configuration
+# Ruff configuration-sdk/is
 [tool.ruff]
 target-version = "py312"
 fix = true  
 
 [tool.ruff.lint]
-select = ["D", "E", "W", "F"]  # Enable docstring checks, pycodestyle errors, and warnings
+select = [
+  "D", # pydocstyle
+  "E", # pycodestyle errors
+  "W", # pycodestyle warnings
+  "F"  # pyflakes
+  ]
 ignore = [
   "D104",  # Missing docstring in public package
   "D105",  # Missing docstring in magic method

tests/integration/auth/test_token_manager_e2e.py

@@ -70,7 +70,6 @@ def test_force_refresh_token(valid_credentials, http_client):
         consumer_secret=valid_credentials["consumer_secret"],
         http_client=http_client,
     )
-    token1 = tm.get_token()
     token2 = tm.get_token(force_refresh=True)
     assert isinstance(token2, str)
     # Token may or may not change, but should be valid

tests/integration/bill_manager/test_bill_integration_e2e.py

@@ -39,8 +39,6 @@ def test_bill_manager_opt_in_e2e(bill_manager_service):
     print("🔗 Starting E2E Test: Bill Manager Opt-In")
 
     shortcode = os.getenv("MPESA_BILL_MANAGER_SHORTCODE", "600999")
-    organization_name = os.getenv("MPESA_BILL_MANAGER_ORG_NAME", "TestOrg")
-    contact_person = os.getenv("MPESA_BILL_MANAGER_CONTACT_PERSON", "John Doe")
     contact_email = os.getenv("MPESA_BILL_MANAGER_CONTACT_EMAIL", "john@example.com")
     contact_phone = os.getenv("MPESA_BILL_MANAGER_CONTACT_PHONE", "254700000000")
     callbackurl = os.getenv(

tests/unit/B2C/test_B2C.py

@@ -225,6 +225,7 @@ def test_b2c_request_occasion_too_long_raises():
 
 
 def make_result_parameters(params):
+    """Helper to create list of B2CResultParameter from dict."""
     return [B2CResultParameter(Key=k, Value=v) for k, v in params.items()]
 
 

tests/unit/services/test_ratiba_service.py

@@ -6,7 +6,6 @@
 from mpesa_sdk.auth import TokenManager
 from mpesa_sdk.http_client import HttpClient
 from mpesa_sdk.mpesa_ratiba import (
-    StandingOrderRequest,
     StandingOrderResponse,
     TransactionTypeEnum,
     ReceiverPartyIdentifierTypeEnum,