Merge pull request #42 from exploreriii/test-2-test-workflow

ci: test testing workflow

Author: exploreriii

.github/scripts/check_test_files.sh

@@ -0,0 +1,53 @@
+#!/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")
+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
+    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'. ${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,25 @@
+name: Test Files Naming Check
+
+on:
+  push
+
+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: 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

@@ -48,6 +48,7 @@ This changelog is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.
 - Added Issue Reminder (no-PR) bot, .github/scripts/issue_reminder_no_pr.sh and .github/workflows/bot-issue-reminder-no-pr.yml to automatically detect assigned issues with no linked pull requests for 7+ days and post a gentle ReminderBot comment.(#951)
 - 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 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)