Skip to content

Commit

Permalink
Ensure that test actually fails when things are broken
Browse files Browse the repository at this point in the history
Test could never actually fail, because the only thing it doess is
checks that `test_indicator` element exists and that element was shown
regardless of test result. Fixed that by giving different test ids to
success and failure indicators.

At the moment, tests deliberately fail because custom databases are not
supported.
  • Loading branch information
SevInf committed May 10, 2024
1 parent a02f4aa commit f94e37e
Showing 1 changed file with 10 additions and 4 deletions.
14 changes: 10 additions & 4 deletions .github/workflows/update-and-publish.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ on:
type: string
required: true
version:
description: npm version to publish
description: client version to update to and publish
type: string
required: true
secrets:
Expand All @@ -19,10 +19,14 @@ on:
inputs:
npmTag:
description: npm tag to publish to
type: string
type: choice
options:
- latest
- dev
- integration
required: true
version:
description: npm version to publish
description: client version to update to and publish
type: string
required: true

Expand Down Expand Up @@ -93,7 +97,9 @@ jobs:
run: yarn npm publish --tag "$NPM_TAG"

- name: Push
run: git push origin main
env:
GITHUB_REF: ${{ github.ref }}
run: git push origin "$GITHUB_REF"

notify-on-failure:
name: Notify on publish failure
Expand Down

0 comments on commit f94e37e

Please sign in to comment.