Skip to content

Commit

Permalink
ci: Modifying the CI workflows for ok-to-test to leverage Cypress tags (
Browse files Browse the repository at this point in the history
appsmithorg#28944)

## Description
If the user types `ok-to-test tags=Git`, Cypress should only run the
specs marked with the Git tag

Related to appsmithorg#28943

#### Type of change
- New feature (non-breaking change which adds functionality)

## Testing

#### How Has This Been Tested?
- [ ] Manual
- [ ] JUnit
- [ ] Jest
- [ ] Cypress

#### QA activity:
- [ ] [Speedbreak
features](https://github.com/appsmithorg/TestSmith/wiki/Guidelines-for-test-plans#speedbreakers-)
have been covered
- [ ] Test plan covers all impacted features and [areas of
interest](https://github.com/appsmithorg/TestSmith/wiki/Guidelines-for-test-plans#areas-of-interest-)
- [ ] Test plan has been peer reviewed by project stakeholders and other
QA members
- [ ] Manually tested functionality on DP
- [ ] We had an implementation alignment call with stakeholders post QA
Round 2
- [ ] Cypress test cases have been added and approved by SDET/manual QA
- [ ] Added `Test Plan Approved` label after Cypress tests were reviewed
- [ ] Added `Test Plan Approved` label after JUnit tests were reviewed
  • Loading branch information
mohanarpit authored Dec 1, 2023
1 parent 4ddb2a4 commit 08f625e
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 0 deletions.
5 changes: 5 additions & 0 deletions .github/workflows/ci-test-custom-script.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,10 @@ on:
description: "This is the PR number in case the workflow is being called in a pull request"
required: false
type: number
tags:
description: "These are the optional tags a developer can specify in order to run a subset of all the tests"
required: false
type: string

jobs:
ci-test:
Expand Down Expand Up @@ -383,6 +387,7 @@ jobs:
CYPRESS_DB_PWD: ${{ secrets.CYPRESS_DB_PWD }}
CYPRESS_S3_ACCESS: ${{ secrets.CYPRESS_S3_ACCESS }}
CYPRESS_S3_SECRET: ${{ secrets.CYPRESS_S3_SECRET }}
CYPRESS_grepTags: ${{ inputs.tags }} # This is a comma separated list of tags to run a subset of the suite
CYPRESS_SKIP_FLAKY: true
with:
browser: ${{ env.BROWSER_PATH }}
Expand Down
2 changes: 2 additions & 0 deletions .github/workflows/integration-tests-command.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@ jobs:
Workflow: `${{ github.workflow }}`.
Commit: `${{ github.event.client_payload.slash_command.args.named.sha }}`.
PR: ${{ github.event.client_payload.pull_request.number }}.
Tags: `${{ github.event.client_payload.slash_command.args.named.tags }}`.
server-build:
name: server-build
Expand Down Expand Up @@ -69,6 +70,7 @@ jobs:
secrets: inherit
with:
pr: ${{ github.event.client_payload.pull_request.number }}
tags: ${{ github.event.client_payload.slash_command.args.named.tags }}

ci-test-result:
needs: [ci-test]
Expand Down

0 comments on commit 08f625e

Please sign in to comment.