Skip to content

Commit f041659

Browse files
Demo Tests: fix check generated demos on push or dispatch (#31896)
1 parent 4ec17e1 commit f041659

File tree

1 file changed

+8
-16
lines changed

1 file changed

+8
-16
lines changed

.github/workflows/demos_visual_tests.yml

Lines changed: 8 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -420,13 +420,8 @@ jobs:
420420
name: Check generated demos
421421
runs-on: devextreme-shr2
422422
timeout-minutes: 10
423-
needs: [check-should-run, build-devextreme, determine-framework-tests-scope]
424-
if: |
425-
always() &&
426-
needs.check-should-run.outputs.should-run == 'true' &&
427-
needs.determine-framework-tests-scope.result == 'success' &&
428-
needs.determine-framework-tests-scope.outputs.framework-tests-scope != 'none' &&
429-
needs.build-devextreme.result == 'success'
423+
needs: [check-should-run, get-changes, build-devextreme, determine-framework-tests-scope]
424+
if: needs.check-should-run.outputs.should-run == 'true' && needs.determine-framework-tests-scope.outputs.framework-tests-scope != 'none'
430425

431426
steps:
432427
- name: Get sources
@@ -483,35 +478,32 @@ jobs:
483478
run: |
484479
if [ -f "changed-files.json" ]; then
485480
echo "Running convert-to-js on changed files only"
486-
481+
487482
CHANGED_DEMOS=$(jq -r '.[].filename' changed-files.json | grep '/React/' | grep '\.tsx$' | sed 's|^apps/demos/||' | sed 's|/[^/]*\.tsx$||' | sort | uniq)
488-
483+
489484
if [ -z "$CHANGED_DEMOS" ]; then
490485
echo "No React demos found in changed files, skipping conversion"
491486
else
492487
echo "Changed React demos:"
493488
echo "$CHANGED_DEMOS"
494-
489+
495490
echo "$CHANGED_DEMOS" | while read -r demo_dir; do
496491
if [ ! -z "$demo_dir" ]; then
497492
echo "Converting: $demo_dir"
498493
pnpm run convert-to-js "$demo_dir"
499494
fi
500495
done
501496
fi
502-
else
503-
echo "Running convert-to-js on all files"
504-
pnpm run convert-to-js
505497
fi
506498
507499
git add ./Demos -N
508500
509501
if git diff --exit-code . ':!package.json' ; then
510502
echo "Generated JS demos are up-to-date"
511503
else
512-
echo "Generated JS demos are outdated. Execute 'pnpm run convert-to-js split' and commit changes."
513-
echo "If you see another diff, ensure that extra listed files have LF endings."
514-
exit 1
504+
echo "Generated JS demos are outdated. Execute 'pnpm run convert-to-js split' and commit changes."
505+
echo "If you see another diff, ensure that extra listed files have LF endings."
506+
exit 1
515507
fi
516508
517509
testcafe-jquery:

0 commit comments

Comments
 (0)