-
Notifications
You must be signed in to change notification settings - Fork 98
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: add regression tests for server-side-rendering #1174
base: master
Are you sure you want to change the base?
chore: add regression tests for server-side-rendering #1174
Conversation
|
Preview URL: https://1174--preview-instui.netlify.app |
c52de80
to
c351f84
Compare
regression-test/package.json
Outdated
@@ -6,6 +6,8 @@ | |||
"scripts": { | |||
"build:webpack4": "webpack", | |||
"build:esbuild": "node esbuild.cjs", | |||
"build:next": "next build ./src", | |||
"start:next": "next start ./src ", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This has to be changed to next dev ./src
, since we wan't to test the development build of the next.js app.
server-side-rendering-regression-test: | ||
needs: release | ||
if: "!startsWith(github.event.head_commit.message, 'chore(release)')" | ||
name: Regression testing | ||
runs-on: ubuntu-latest | ||
strategy: | ||
matrix: | ||
browser: [chrome, firefox] | ||
defaults: | ||
run: | ||
working-directory: ./regression-test | ||
steps: | ||
- uses: actions/checkout@v3 | ||
- uses: actions/setup-node@v3 | ||
with: | ||
node-version: '16' | ||
cache: 'yarn' | ||
- name: Retrieve latest snapshot version | ||
run: node prepare.js | ||
- name: Install dependencies for regression testing | ||
run: yarn install --no-immutable | ||
- name: Build project and start the server | ||
run: | | ||
yarn build:next | ||
yarn start:next & | ||
- name: Run regression tests | ||
run: npx testcafe ${{ matrix.browser }}:headless ./src/test.ts |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since the next.js app throws multiple errors, I would turn off this job until those errors are resolved in the regression-test
application, because of those errors the whole workflow will always fail.
You can turn off a job by adding an if
statement to the job itself:
server-side-rendering-regression-test:
needs: release
if: false # "!startsWith(github.event.head_commit.message, 'chore(release)')"
name: Regression testing
...
c351f84
to
d86d7b3
Compare
d86d7b3
to
28eb65b
Compare
No description provided.