From 3e2611fa7ffaa685c45101b237a7438e33b3e875 Mon Sep 17 00:00:00 2001 From: "harshitha.d" Date: Wed, 2 Apr 2025 17:22:56 +0530 Subject: [PATCH] Refactor unit test workflow to simplify plugin testing and remove unused steps --- .github/workflows/unit-test.yml | 74 +++++++++++++++++---------------- 1 file changed, 39 insertions(+), 35 deletions(-) diff --git a/.github/workflows/unit-test.yml b/.github/workflows/unit-test.yml index c395556936..87959b7a1c 100644 --- a/.github/workflows/unit-test.yml +++ b/.github/workflows/unit-test.yml @@ -19,40 +19,44 @@ jobs: - name: Install dependencies for all plugins run: | npm run setup-repo-old - - - name: Fetch latest references - run: | - git fetch --prune - - - name: Identify Changed Plugins - id: changes - run: | - echo "Finding changed files..." - # Ensure both commit references are valid - if [[ -z "${{ github.event.before }}" || -z "${{ github.sha }}" ]]; then - echo "Error: Missing commit references" - exit 1 - fi - - CHANGED_FILES=$(git diff --name-only ${{ github.event.before }} ${{ github.sha }}) - echo "Changed files:" - echo "$CHANGED_FILES" + + - name: Run tests for Audit plugin + working-directory: ./packages/contentstack-audit + run: npm run test:unit + + # - name: Fetch latest references + # run: | + # git fetch --prune + + # - name: Identify Changed Plugins + # id: changes + # run: | + # echo "Finding changed files..." + # # Ensure both commit references are valid + # if [[ -z "${{ github.event.before }}" || -z "${{ github.sha }}" ]]; then + # echo "Error: Missing commit references" + # exit 1 + # fi + + # CHANGED_FILES=$(git diff --name-only ${{ github.event.before }} ${{ github.sha }}) + # echo "Changed files:" + # echo "$CHANGED_FILES" - # Identify affected plugins - AFFECTED_PLUGINS=$(echo "$CHANGED_FILES" | grep -oP '(?<=^packages/)([^/]+)' | sort -u | tr '\n' ' ') - echo "Affected plugins: $AFFECTED_PLUGINS" + # # Identify affected plugins + # AFFECTED_PLUGINS=$(echo "$CHANGED_FILES" | grep -oP '(?<=^packages/)([^/]+)' | sort -u | tr '\n' ' ') + # echo "Affected plugins: $AFFECTED_PLUGINS" - # Set output for the next step - echo "::set-output name=affected_plugins::$AFFECTED_PLUGINS" - - - name: Run Unit Tests for Affected Plugins - run: | - for plugin in ${{ steps.changes.outputs.affected_plugins }}; do - echo "Checking if tests should run for $plugin..." - if [[ "$plugin" == "contentstack-audit" ]]; then - echo "Running tests for contentstack-audit..." - npm run test:unit --prefix ./packages/contentstack-audit - else - echo "contentstack-audit has not changed. Skipping tests." - fi - done \ No newline at end of file + # # Set output for the next step + # echo "::set-output name=affected_plugins::$AFFECTED_PLUGINS" + + # - name: Run Unit Tests for Affected Plugins + # run: | + # for plugin in ${{ steps.changes.outputs.affected_plugins }}; do + # echo "Checking if tests should run for $plugin..." + # if [[ "$plugin" == "contentstack-audit" ]]; then + # echo "Running tests for contentstack-audit..." + # npm run test:unit --prefix ./packages/contentstack-audit + # else + # echo "contentstack-audit has not changed. Skipping tests." + # fi + # done \ No newline at end of file