From f70a394912789d59c45531d7fc812331a0034a85 Mon Sep 17 00:00:00 2001 From: Abubakar Abid Date: Tue, 26 Sep 2023 07:08:12 -0700 Subject: [PATCH] Fix functional tests, part 2 (#5687) * remove build * intentionally break * add changeset * testing string * change to false * throw error * revert break * delete changeset * add client js files to cache --------- Co-authored-by: gradio-pr-bot --- .github/actions/install-all-deps/action.yml | 4 ++-- .github/actions/install-frontend-deps/action.yml | 6 +++--- .github/workflows/backend.yml | 4 ++-- .github/workflows/ui.yml | 1 - 4 files changed, 7 insertions(+), 8 deletions(-) diff --git a/.github/actions/install-all-deps/action.yml b/.github/actions/install-all-deps/action.yml index fbd7a5c376..87755a5066 100644 --- a/.github/actions/install-all-deps/action.yml +++ b/.github/actions/install-all-deps/action.yml @@ -4,7 +4,7 @@ description: 'Install all deps' inputs: always-install-pnpm: description: 'Dictates whether or not we should install pnpm & dependencies, regardless of the cache' - default: false + default: 'false' node_auth_token: description: 'Node auth token' default: "" @@ -13,7 +13,7 @@ inputs: default: "" skip_build: description: 'Skip build' - default: false + default: 'false' runs: using: "composite" diff --git a/.github/actions/install-frontend-deps/action.yml b/.github/actions/install-frontend-deps/action.yml index 7198ea91f2..e2d3b64416 100644 --- a/.github/actions/install-frontend-deps/action.yml +++ b/.github/actions/install-frontend-deps/action.yml @@ -4,7 +4,7 @@ description: 'Install frontend deps' inputs: always-install-pnpm: description: 'Dictates whether or not we should install pnpm & dependencies, regardless of the cache' - default: false + default: 'false' node_auth_token: description: 'Node auth token' default: "" @@ -13,7 +13,7 @@ inputs: default: "" skip_build: description: 'Skip build' - default: false + default: 'false' runs: using: "composite" @@ -23,7 +23,7 @@ runs: with: path: | gradio/templates/* - key: gradio-lib-front-end-${{ hashFiles('js/**')}} + key: gradio-lib-front-end-${{ hashFiles('js/**', 'client/js/**')}} - name: Install pnpm if: steps.frontend-cache.outputs.cache-hit != 'true' || inputs.always-install-pnpm == 'true' uses: pnpm/action-setup@v2 diff --git a/.github/workflows/backend.yml b/.github/workflows/backend.yml index 9344e7daff..90b0f796e2 100644 --- a/.github/workflows/backend.yml +++ b/.github/workflows/backend.yml @@ -76,7 +76,7 @@ jobs: with: path: | gradio/templates/* - key: gradio-lib-front-end-${{ hashFiles('js/**')}} + key: gradio-lib-front-end-${{ hashFiles('js/**', 'client/js/**')}} - name: Install pnpm if: steps.frontend-cache.outputs.cache-hit != 'true' uses: pnpm/action-setup@v2 @@ -168,7 +168,7 @@ jobs: with: path: | gradio/templates/* - key: gradio-lib-front-end-${{ hashFiles('js/**')}} + key: gradio-lib-front-end-${{ hashFiles('js/**', 'client/js/**')}} - name: Install pnpm if: steps.frontend-cache.outputs.cache-hit != 'true' uses: pnpm/action-setup@v2 diff --git a/.github/workflows/ui.yml b/.github/workflows/ui.yml index a63d0f625d..cdd3c8ed8d 100644 --- a/.github/workflows/ui.yml +++ b/.github/workflows/ui.yml @@ -65,7 +65,6 @@ jobs: run: | . venv/bin/activate python -m pip install -r demo/outbreak_forecast/requirements.txt - - run: pnpm build - run: pnpm exec playwright install chromium - name: run browser tests run: |