CI: Skip demo framework tests when no changes detected or 'force all tests' is false (#31296)

Author: EugeniyKiyashko

.github/workflows/demos_visual_tests_frameworks.yml

@@ -136,10 +136,12 @@ jobs:
 #      run: pnpm install
 
     - name: Prepare bundles
+      if: github.event_name != 'pull_request' || contains(github.event.pull_request.labels.*.name, 'force all tests')
       working-directory: apps/demos
       run: pnpx nx prepare-bundles
 
     - name: Demos - Run tsc
+      if: github.event_name != 'pull_request' || contains(github.event.pull_request.labels.*.name, 'force all tests')
       working-directory: apps/demos
       run: tsc --noEmit
 
@@ -162,12 +164,22 @@ jobs:
 
       - name: Get changed files
         uses: DevExpress/github-actions/get-changed-files@v1
-        if: github.event_name == 'pull_request' && !contains(github.event.pull_request.labels.*.name, 'force all tests')
         with:
           gh-token: ${{ secrets.GITHUB_TOKEN }}
           paths: 'apps/demos/Demos/**/*'
           output: apps/demos/changed-files.json
 
+      - name: Display changed files
+        run: |
+          if [ -f "apps/demos/changed-files.json" ]; then
+            echo "Found changed-files.json"
+            echo "Content of changed-files.json:"
+            cat apps/demos/changed-files.json
+            echo "Number of changed files: $(jq length apps/demos/changed-files.json)"
+          else
+            echo "changed-files.json not found"
+          fi
+
       - name: Upload artifacts
         uses: actions/upload-artifact@v4
         with:
@@ -263,7 +275,6 @@ jobs:
       uses: actions/checkout@v4
 
     - name: Download artifacts
-      if: github.event_name == 'pull_request' && !contains(github.event.pull_request.labels.*.name, 'force all tests')
       uses: actions/download-artifact@v4
       with:
         name: changed-demos
@@ -305,47 +316,78 @@ jobs:
     - name: Install tgz
       run: pnpm add -w ./devextreme-installer.tgz ./devextreme-dist-installer.tgz ./devextreme-react-installer.tgz ./devextreme-vue-installer.tgz ./devextreme-angular-installer.tgz
 
-    - uses: actions/setup-dotnet@v4
-      with:
-        dotnet-version: 5.0.408
+    - name: Run lint on all demos
+      if: github.event_name != 'pull_request' || contains(github.event.pull_request.labels.*.name, 'force all tests')
+      working-directory: apps/demos
+      env:
+        DEBUG: 'eslint:cli-engine,stylelint:standalone'
+      run: ${{ matrix.command }}
 
-    - name: Prepare dotnet
+    - name: Run lint on changed demos
+      if: github.event_name == 'pull_request' && !contains(github.event.pull_request.labels.*.name, 'force all tests')
+      working-directory: apps/demos
+      env:
+        DEBUG: 'eslint:cli-engine,stylelint:standalone'
       run: |
-        dotnet new globaljson --sdk-version 5.0.408
-        dotnet tool install -g dotnet-format --version 5.1.225507
+        pnpx nx lint-non-demos
+
+        if [ -f "changed-files.json" ]; then
+          echo "Running lint-demos on changed files"
+          CHANGED_DEMOS=$(jq -r '.[].filename' changed-files.json | grep '^apps/demos/Demos/' | sed 's|^apps/demos/||' | tr '\n' ' ')
+          if [ ! -z "$CHANGED_DEMOS" ]; then
+            echo "Changed demo files: $CHANGED_DEMOS"
+            pnpx eslint $CHANGED_DEMOS
+          else
+            echo "No demo files changed, skipping lint-demos"
+          fi
+        else
+          echo "changed-files.json not found"
+          pnpm run lint-demos
+        fi
 
-    - name: Run lint
+    - name: Run lint on changed demos
+      if: github.event_name != 'pull_request' && !contains(github.event.pull_request.labels.*.name, 'force all tests')
       working-directory: apps/demos
       env:
-        CHANGEDFILEINFOSPATH: changed-files.json
         DEBUG: 'eslint:cli-engine,stylelint:standalone'
-      run: ${{ matrix.command }}
+      run: |
+        pnpx nx lint-non-demos
+
+        if [ -f "changed-files.json" ]; then
+          echo "Running lint-demos on changed files"
+          CHANGED_DEMOS=$(jq -r '.[].filename' changed-files.json | grep '^apps/demos/Demos/' | sed 's|^apps/demos/||' | tr '\n' ' ')
+          if [ ! -z "$CHANGED_DEMOS" ]; then
+            echo "Changed demo files: $CHANGED_DEMOS"
+            pnpx eslint $CHANGED_DEMOS
+          else
+            echo "No demo files changed, skipping lint-demos"
+          fi
 
   check_generated_demos:
     name: ${{ matrix.name }}
     runs-on: ubuntu-22.04
     timeout-minutes: 10
-    needs: build-demos
+    needs:
+      - build-devextreme
+      - get-changes
 
     strategy:
       fail-fast: false
       matrix:
         include:
-          - name: Check generated demos (1/5)
-            command: CONSTEL=1/5 pnpm run convert-to-js
-          - name: Check generated demos (2/5)
-            command: CONSTEL=2/5 pnpm run convert-to-js
-          - name: Check generated demos (3/5)
-            command: CONSTEL=3/5 pnpm run convert-to-js
-          - name: Check generated demos (4/5)
-            command: CONSTEL=4/5 pnpm run convert-to-js
-          - name: Check generated demos (5/5)
-            command: CONSTEL=5/5 pnpm run convert-to-js
+          - name: Check generated demos
+            command: pnpm run convert-to-js
 
     steps:
     - name: Get sources
       uses: actions/checkout@v4
 
+    - name: Download artifacts
+      uses: actions/download-artifact@v4
+      with:
+        name: changed-demos
+        path: apps/demos
+
     - uses: pnpm/action-setup@v3
       with:
         version: 9
@@ -357,21 +399,6 @@ jobs:
         node-version: '20'
         cache: 'pnpm'
 
-#    - name: Get pnpm store directory
-#      shell: bash
-#      run: |
-#        echo "STORE_PATH=$(pnpm store path --silent)" >> $GITHUB_ENV
-#
-#    - uses: actions/cache@v4
-#      name: Setup pnpm cache
-#      with:
-#        path: |
-#          ${{ env.STORE_PATH }}
-#          .nx/cache
-#        key: ${{ runner.os }}-pnpm-store-${{ hashFiles('**/pnpm-lock.yaml') }}
-#        restore-keys: |
-#          ${{ runner.os }}-pnpm-store
-
     - name: Download devextreme sources
       uses: actions/download-artifact@v4
       with:
@@ -388,10 +415,37 @@ jobs:
       working-directory: apps/demos
       run: pnpm run prepare-js
 
+    - name: Download changed demos info
+      uses: actions/download-artifact@v4
+      with:
+        name: changed-demos
+        path: apps/demos
+
     - name: Check generated JS demos
       working-directory: apps/demos
       run: |
-        ${{ matrix.command }}
+        if [ -f "changed-files.json" ]; then
+          echo "Running convert-to-js on changed files only"
+          
+          CHANGED_DEMOS=$(jq -r '.[].filename' changed-files.json | grep '/React/' | grep '\.tsx$' | sed 's|^apps/demos/||' | sed 's|/[^/]*\.tsx$||' | sort | uniq)
+          
+          if [ -z "$CHANGED_DEMOS" ]; then
+            echo "No React demos found in changed files, skipping conversion"
+          else
+            echo "Changed React demos:"
+            echo "$CHANGED_DEMOS"
+            
+            echo "$CHANGED_DEMOS" | while read -r demo_dir; do
+              if [ ! -z "$demo_dir" ]; then
+                echo "Converting: $demo_dir"
+                pnpm run convert-to-js "$demo_dir"
+              fi
+            done
+          fi
+        else
+          echo "Running convert-to-js on all files"
+          ${{ matrix.command }}
+        fi
 
         git add ./Demos -N
 
@@ -405,6 +459,7 @@ jobs:
 
   testcafe:
     needs: build-demos
+    if: github.event_name != 'pull_request' || contains(github.event.pull_request.labels.*.name, 'force all tests')
     strategy:
       fail-fast: false
       matrix:
@@ -436,7 +491,6 @@ jobs:
         name: devextreme-sources
 
     - name: Download artifacts
-      if: github.event_name == 'pull_request' && !contains(github.event.pull_request.labels.*.name, 'force all tests')
       uses: actions/download-artifact@v4
       with:
         name: changed-demos
@@ -469,9 +523,6 @@ jobs:
       working-directory: apps/demos
       run: pnpm add ../../devextreme-installer.tgz ../../devextreme-dist-installer.tgz ../../devextreme-react-installer.tgz ../../devextreme-vue-installer.tgz ../../devextreme-angular-installer.tgz
 
-#    - name: Build wrappers
-#      run: pnpm exec nx run-many -t pack -p devextreme-angular devextreme-react devetreme-vue
-
     - name: Prepare JS
       working-directory: apps/demos
       run: pnpm run prepare-js