Skip to content

Commit

Permalink
Try adding videos to e2e tests
Browse files Browse the repository at this point in the history
  • Loading branch information
Luke Brody committed Sep 17, 2024
1 parent dd59eea commit c797755
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 1 deletion.
9 changes: 8 additions & 1 deletion .github/workflows/check.yml
Original file line number Diff line number Diff line change
Expand Up @@ -98,12 +98,19 @@ jobs:
fluxbox >/dev/null 2>&1 & # needed for window resizing in Testcafe
npm ci
npx ts-node test/ci_proxy.ts &
npx testcafe -e "chromium '--window-size=1400,800' --hide-scrollbars --disable-gpu" -s thumbnails=false test/${{ matrix.test-file }}.ts
npx testcafe -e "chromium '--window-size=1400,800' --hide-scrollbars --disable-gpu" \
-s thumbnails=false \
--video videos \
test/${{ matrix.test-file }}.ts
working-directory: react
- uses: actions/upload-artifact@v4
with:
name: screenshots-${{ matrix.test-file }}
path: react/screenshots
- uses: actions/upload-artifact@v4
with:
name: videos-${{ matrix.test-file }}
path: react/videos

check-screenshots:
runs-on: ubuntu-22.04
Expand Down
3 changes: 3 additions & 0 deletions react/test/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,9 @@ RUN apt-get -y install fluxbox
# needed for downloads directory in TestCafe
RUN apt-get -y install xdg-user-dirs

# needed for video recording in TestCafe
RUN apt-get -y install ffmpeg

COPY ./requirements.txt ./
RUN pip3 install --break-system-packages -r requirements.txt

Expand Down

0 comments on commit c797755

Please sign in to comment.