✨ Create required scripts file and other workflow files

Author: dinushchathurya

.github/workflows/security.yaml

@@ -2,7 +2,7 @@ name: Security
 
 on:
   push:
-    branches: [ main, develop ]
+    branches: [ main ]
     tags:
       - 'v*.*.*'
   pull_request:

.github/workflows/test.yaml

@@ -31,6 +31,7 @@ permissions:
 env:
   DOCKER_BUILDKIT: 1
   COMPOSE_DOCKER_CLI_BUILD: 1
+  DOCKER_IMAGE_TAG: "eks-helm-client:test" # Define image tag for all jobs
 
 jobs:
   unit-tests:
@@ -41,16 +42,16 @@ jobs:
       - name: Checkout
         uses: actions/checkout@v4
 
-      - name: Setup Scripts
+      - name: Make scripts executable
+        # It's best practice to set executable permissions in your Git repository.
+        # This step ensures they are executable if not already set.
         run: |
           chmod +x .github/scripts/testing/*.sh
           chmod +x scripts/*.sh
 
       - name: Run Unit Tests
-        # run: .github/scripts/testing/run-unit-tests.sh
-        run: |
-          bash .github/scripts/testing/run-unit-tests.sh || true
-          cat /tmp/eks-helm-test-*/test-results.txt || echo "No test results found"
+        # CRITICAL FIX: Removed '|| true' to ensure the workflow fails if tests fail.
+        run: bash .github/scripts/testing/run-unit-tests.sh
 
       - name: Upload Test Results
         if: always()
@@ -65,22 +66,6 @@ jobs:
     runs-on: ubuntu-latest
     needs: unit-tests
 
-    strategy:
-      matrix:
-        test-scenario:
-          - name: "Basic"
-            cluster: "test-cluster"
-            region: "us-west-2"
-          - name: "Private Cluster"
-            cluster: "private-cluster"
-            region: "us-east-1"
-            private: "true"
-          - name: "Custom Versions"
-            cluster: "version-test"
-            region: "eu-west-1"
-            kubectl: "1.27.1"
-            helm: "3.12.0"
-
     steps:
       - name: Checkout
         uses: actions/checkout@v4
@@ -94,28 +79,14 @@ jobs:
           context: .
           push: false
           load: true
-          tags: eks-helm-client:test
+          tags: ${{ env.DOCKER_IMAGE_TAG }}
           cache-from: type=gha
           cache-to: type=gha,mode=max
 
-      - name: Run Integration Test - ${{ matrix.test-scenario.name }}
-        env:
-          TEST_CLUSTER_NAME: ${{ matrix.test-scenario.cluster }}
-          TEST_REGION: ${{ matrix.test-scenario.region }}
-          PRIVATE_CLUSTER: ${{ matrix.test-scenario.private }}
-          KUBECTL_VERSION: ${{ matrix.test-scenario.kubectl }}
-          HELM_VERSION: ${{ matrix.test-scenario.helm }}
-        run: |
-          docker run --rm \
-            -e INPUT_CLUSTER_NAME="${TEST_CLUSTER_NAME}" \
-            -e INPUT_REGION="${TEST_REGION}" \
-            -e INPUT_PRIVATE_CLUSTER="${PRIVATE_CLUSTER:-false}" \
-            -e INPUT_KUBECTL_VERSION="${KUBECTL_VERSION:-}" \
-            -e INPUT_HELM_VERSION="${HELM_VERSION:-}" \
-            -e INPUT_DRY_RUN="true" \
-            -e INPUT_DEBUG="${{ inputs.debug || 'false' }}" \
-            -e INPUT_ARGS="echo 'Test: ${{ matrix.test-scenario.name }}'" \
-            eks-helm-client:test
+      - name: Run Integration Tests
+        # CORRECTION: This now runs the actual integration test script.
+        # The previous matrix only ran a simple 'echo' command inside the container.
+        run: .github/scripts/testing/run-integration-tests.sh
 
   action-validation:
     name: Validate Action
@@ -125,10 +96,11 @@ jobs:
       - name: Checkout
         uses: actions/checkout@v4
 
+      - name: Make validation script executable
+        run: chmod +x .github/scripts/testing/validate-action.sh
+
       - name: Validate Action Configuration
-        run: |
-          chmod +x .github/scripts/testing/validate-action.sh
-          .github/scripts/testing/validate-action.sh
+        run: .github/scripts/testing/validate-action.sh
 
       - name: Upload Validation Report
         if: always()
@@ -178,7 +150,8 @@ jobs:
     if: always()
 
     steps:
-      - name: Check Test Results
+      - name: Generate Job Summary
+        id: summary
         run: |
           echo "## 🧪 Test Summary" >> $GITHUB_STEP_SUMMARY
           echo "" >> $GITHUB_STEP_SUMMARY
@@ -189,15 +162,20 @@ jobs:
           echo "| Action Validation | ${{ needs.action-validation.result == 'success' && '✅ Passed' || '❌ Failed' }} |" >> $GITHUB_STEP_SUMMARY
           echo "| Action Test | ${{ needs.test-action.result == 'success' && '✅ Passed' || '❌ Failed' }} |" >> $GITHUB_STEP_SUMMARY
           echo "" >> $GITHUB_STEP_SUMMARY
-          
-          if [[ "${{ contains(needs.*.result, 'failure') }}" == "true" ]]; then
+
+          if [[ "${{ needs.unit-tests.result }}" != "success" || "${{ needs.integration-tests.result }}" != "success" || "${{ needs.action-validation.result }}" != "success" || "${{ needs.test-action.result }}" != "success" ]]; then
+            echo "status=failure" >> $GITHUB_OUTPUT
             echo "❌ **Some tests failed!**" >> $GITHUB_STEP_SUMMARY
-            exit 1
           else
+            echo "status=success" >> $GITHUB_OUTPUT
             echo "✅ **All tests passed!**" >> $GITHUB_STEP_SUMMARY
           fi
 
-      - name: Comment PR
+      - name: Fail workflow on test failure
+        if: steps.summary.outputs.status == 'failure'
+        run: exit 1
+
+      - name: Comment on PR
         if: github.event_name == 'pull_request'
         uses: actions/github-script@v7
         with:
@@ -211,7 +189,7 @@ jobs:
             | Action Validation | ${{ needs.action-validation.result == 'success' && '✅ Passed' || '❌ Failed' }} |
             | Action Test | ${{ needs.test-action.result == 'success' && '✅ Passed' || '❌ Failed' }} |
             
-            **Commit**: ${{ github.event.pull_request.head.sha || github.sha }}
+            **Commit**: \`${{ github.event.pull_request.head.sha || github.sha }}\`
             **Workflow**: [Run #${{ github.run_number }}](${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }})
             `;