Skip to content

Commit

Permalink
fix: anvil snapshots
Browse files Browse the repository at this point in the history
  • Loading branch information
KelvinOm committed Sep 18, 2024
1 parent 9f7e624 commit 8c41914
Show file tree
Hide file tree
Showing 5 changed files with 15 additions and 14 deletions.
10 changes: 5 additions & 5 deletions .github/workflows/ci-test-custom-script.yml
Original file line number Diff line number Diff line change
Expand Up @@ -242,7 +242,7 @@ jobs:
- uses: browser-actions/setup-chrome@latest
with:
chrome-version: stable
chrome-version: "128.0.6613.137"
- run: |
echo "BROWSER_PATH=$(which chrome)" >> $GITHUB_ENV
Expand Down Expand Up @@ -441,20 +441,20 @@ jobs:
name: server-logs-${{ matrix.job }}
path: app/server/server-logs.log
overwrite: true

- name: Collect docker log as file
if: always()
run: |
docker logs appsmith >& app/server/docker-logs.log
- name: Upload server docker logs bundle on failure
- name: Upload server docker logs bundle on failure
uses: actions/upload-artifact@v4
if: always()
with:
name: docker-logs-${{ matrix.job }}
path: app/server/docker-logs.log
overwrite: true

# Set status = success
- name: Save the status of the run
run: |
Expand Down
3 changes: 2 additions & 1 deletion .github/workflows/ci-test-hosted.yml
Original file line number Diff line number Diff line change
Expand Up @@ -139,7 +139,7 @@ jobs:
- uses: browser-actions/setup-chrome@latest
with:
chrome-version: stable
chrome-version: "128.0.6613.137"
- run: |
echo "BROWSER_PATH=$(which chrome)" >> $GITHUB_ENV
Expand Down Expand Up @@ -243,6 +243,7 @@ jobs:
CYPRESS_S3_SECRET: ${{ secrets.CYPRESS_S3_SECRET }}
CYPRESS_STATIC_ALLOCATION: true
with:
browser: ${{ env.BROWSER_PATH }}
install: false
config-file: cypress_ci_hosted.config.ts
working-directory: app/client
Expand Down
12 changes: 6 additions & 6 deletions .github/workflows/ci-test-limited-with-count.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ on:
run_count:
description: 'Number of times to repeat the test run'
required: false
type: number
type: number
default: 1
workflow_call:
inputs:
Expand Down Expand Up @@ -234,7 +234,7 @@ jobs:
- uses: browser-actions/setup-chrome@latest
with:
chrome-version: stable
chrome-version: "128.0.6613.137"
- run: |
echo "BROWSER_PATH=$(which chrome)" >> $GITHUB_ENV
Expand Down Expand Up @@ -349,18 +349,18 @@ jobs:
npx cypress-repeat-pro run -n ${{ inputs.run_count }} --force \
--spec ${{ env.specs_to_run }} \
--config-file "cypress_ci_custom.config.ts"
cat cy-repeat-summary.txt
cat cy-repeat-summary.txt
# Define the path for the failure flag file
FAILURE_FLAG_FILE="ci_test_status.txt"
# Check for test results and store the status in the file
if ! grep -q "Total Failed: 0" cy-repeat-summary.txt; then
echo "ci_test_failed=true" > "$FAILURE_FLAG_FILE"
else
echo "ci_test_failed=false" > "$FAILURE_FLAG_FILE"
fi
cat "$FAILURE_FLAG_FILE"
- name: Trim number of cypress log files
if: failure()
run: |
Expand All @@ -373,7 +373,7 @@ jobs:
name: cypress-repeat-logs
path: ${{ github.workspace }}/app/client/cy-repeat-summary.txt
overwrite: true

- name: Upload ci_test_status.txt artifact
if: always()
uses: actions/upload-artifact@v4
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/ci-test-limited.yml
Original file line number Diff line number Diff line change
Expand Up @@ -224,7 +224,7 @@ jobs:
- uses: browser-actions/setup-chrome@latest
with:
chrome-version: stable
chrome-version: "128.0.6613.137"
- run: |
echo "BROWSER_PATH=$(which chrome)" >> $GITHUB_ENV
Expand Down
2 changes: 1 addition & 1 deletion app/client/cypress/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,6 @@ FROM cypress/factory:4.0.2

# Install chromium in this way since there is no browsers in the docker container for the arm64 architecture
# https://github.com/cypress-io/cypress-docker-images/issues/695
RUN apt update && apt install -y chromium
RUN apt-get update && apt-get install -y chromium=128.0.6613.137-1~deb12u1

ENTRYPOINT ["yarn", "cypress:snapshot"]

0 comments on commit 8c41914

Please sign in to comment.