feat: added github workflow that checks test file names (#1054) (#1096)

Signed-off-by: HusseinYasser <[email protected]>

Author: HusseinYasser

.github/scripts/check_test_files.sh

@@ -0,0 +1,56 @@
+#!/usr/bin/env bash
+RED="\033[31m"
+YELLOW="\033[33m"
+RESET="\033[0m"
+
+# Base directories where test files should reside
+TEST_DIRS=("tests/unit" "tests/integration")
+EXCEPTION_NAMES=("conftest.py" "init.py" "__init__.py" "mock_server.py" "utils.py")
+DIFF_FILES=$(git diff --name-status origin/main)
+ERRORS=()
+
+function is_in_test_dir() {
+    local file="$1"
+    for dir in "${TEST_DIRS[@]}"; do
+        case "$file" in
+            "$dir"*)
+                return 0
+                ;;
+        esac
+    done
+    return 1
+}
+
+function check_test_file_name() {
+  local filename="$1"
+  if is_in_test_dir "$filename"; then
+    if [[ $(basename "$filename") != *.py ]]; then
+      return 0
+    fi
+    for exception in "${EXCEPTION_NAMES[@]}"; do
+      if [[ $(basename "$filename") == "$exception" ]]; then
+        return 0
+      fi
+    done
+    if [[ $(basename "$filename") != *_test.py ]]; then
+      ERRORS+=("${RED}ERROR${RESET}: Test file '$filename' doesn't end with '_test.py'. ${YELLOW}It has to follow the pytest naming convention.")
+      return 1
+    fi
+  fi
+  return 0
+}
+
+while IFS=$'\t' read -r status file1 file2; do
+  case "$status" in
+    A) check_test_file_name "$file1" ;;
+    R*) check_test_file_name "$file2" ;;
+    C*) check_test_file_name "$file2" ;;
+  esac
+done <<< "$DIFF_FILES"
+
+if (( ${#ERRORS[@]} > 0 )); then
+  for err in "${ERRORS[@]}"; do
+    echo -e "$err"
+  done
+  exit 1
+fi

.github/workflows/pr-check-test-files.yml

@@ -0,0 +1,31 @@
+name: Test Files Naming Check
+
+on:
+  push:
+    branches-ignore:
+      - main
+
+concurrency:
+  group: pr-checks-${{ github.workflow }}-${{ github.ref || github.run_id }}
+  cancel-in-progress: true
+
+jobs:
+  check-test-files:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8
+      
+      - name: Set up Hiero SDK Upstream
+        run: |
+          git remote set-url origin https://github.com/hiero-ledger/hiero-sdk-python.git
+
+      - name: Fetch main branch
+        run: |
+          git fetch origin main
+
+      - name: Check added test file names
+        run: |
+          chmod +x .github/scripts/check_test_files.sh
+          .github/scripts/check_test_files.sh

CHANGELOG.md

@@ -50,6 +50,7 @@ This changelog is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.
 - Add new `.github/ISSUE_TEMPLATE/05_intermediate_issue.yml` file (1072)(https://github.com/hiero-ledger/hiero-sdk-python/issues/1072)
 - Add a workflow to notify the team when issues are labeled as “good first issues” or identified as candidates for that label: `bot-gfi-notify-team.yml`(#1115)
 - Added __str__ and __repr__ to AccountBalance
+- Added GitHub workflow that makes sure newly added test files follow pytest test files naming conventions (#1054)
 
 ### Changed
 - Move `account_allowance_delete_transaction_hbar.py` from `examples/` to `examples/account/` for better organization (#1003)