Skip to content

Commit

Permalink
Merge pull request #106 from GAIA-X4PLC-AAD/feature/105-Git-workflow-…
Browse files Browse the repository at this point in the history
…Check-missing-tests

Git workflow: Check missing tests
  • Loading branch information
devbysp authored Oct 8, 2024
2 parents 708a41d + 833443b commit cda8dc2
Show file tree
Hide file tree
Showing 2 changed files with 42 additions and 4 deletions.
15 changes: 11 additions & 4 deletions .github/workflows/branch-push.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -20,23 +20,30 @@ jobs:
- name: Checkout repository
uses: actions/checkout@v4

# Step 2: Set up Node.js with the required version
# Step 2: Fail the job if there are uncovered files (js, ts, jsx, tsx)
- name: Make script 'check-missing-tests.sh' executable
run: chmod +x ./check-missing-tests.sh

- name: Check missing tests
run: ./check-missing-tests.sh

# Step 3: Set up Node.js with the required version
- name: Set up Node.js
uses: actions/setup-node@v4
with:
node-version: ${{ matrix.node-version }}

# Step 3: Install dependencies
# Step 4: Install dependencies
- name: Install dependencies
run: npm install

# Step 4: Run Jest tests
# Step 5: Run Jest tests
- name: Run Jest tests
run: npm test -- --coverage
env:
CI: true # Ensures Jest runs in CI mode

# Step 5: Upload test coverage results (optional, if you want to store it in the workflow run artifacts)
# Step 6: Upload test coverage results (optional, if you want to store it in the workflow run artifacts)
- name: Upload coverage report
uses: actions/upload-artifact@v4
with:
Expand Down
31 changes: 31 additions & 0 deletions check-missing-tests.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,31 @@
#!/bin/bash
git fetch origin main

# Step 1: Get the list of changed JS, TS, JSX and TSX files in the src directory
CHANGED_FILES=$(git diff --name-only origin/main..HEAD -- 'src/*.[jt]s' 'src/*.[jt]sx')

# Initialize an empty array to hold files without tests
MISSING_TEST_FILES=()

# Step 2: Loop through each changed file and check if it has a corresponding test file
for FILE in $CHANGED_FILES; do
# Extract the file path and replace "src/" with "tests/" and add ".test" postfix
TEST_FILE=$(echo "$FILE" | sed 's|^src/|tests/|' | sed 's|\.[jt]sx\?$|.test&|')

# Check if the corresponding test file exists
if [[ ! -f "$TEST_FILE" ]]; then
MISSING_TEST_FILES+=("$FILE") # Add to missing test files array if test is not found
fi
done

# Step 3: Report and fail if there are any missing test files
if [ ${#MISSING_TEST_FILES[@]} -ne 0 ]; then
echo "The following source files do not have corresponding test files:"
for MISSING_FILE in "${MISSING_TEST_FILES[@]}"; do
echo "$MISSING_FILE"
done
exit 1 # Fail the workflow
else
echo "All changed files have corresponding test files."
exit 0 # Success
fi

0 comments on commit cda8dc2

Please sign in to comment.