OU-1472: run only changed tests #1134
+64
−16
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
| @@ -0,0 +1,62 @@ | ||||||
| #!/bin/bash | ||||||
|
|
||||||
| CHANGED_FILES=$(git diff --name-only origin/main...HEAD) | ||||||
|
|
||||||
| RUN_ALL=false | ||||||
| declare -A TAGS_TO_RUN | ||||||
|
Contributor
There was a problem hiding this comment. We might need to change this into something POSIX compatible, I got the following issue while trying to run on mac: |
||||||
|
|
||||||
| # Create frontend folder to tag map | ||||||
| declare -A FEATURE_MAP | ||||||
| FEATURE_MAP["incidents"]="@cluster-health-analyzer" | ||||||
| FEATURE_MAP["alerts"]="@alerting" | ||||||
| FEATURE_MAP["legacy-dashboards"]="@legacy-dashboards" | ||||||
| FEATURE_MAP["metrics"]="@metrics" | ||||||
| FEATURE_MAP["perses-dashboards"]="@perses-dashboards" | ||||||
| FEATURE_MAP["targets"]="@targets" | ||||||
|
|
||||||
| for FILE in $CHANGED_FILES; do | ||||||
|
Contributor
There was a problem hiding this comment. not sure if we have file paths with white spaces, but this will split on white space |
||||||
|
|
||||||
| # Check if any file is outside of web/src/features/ | ||||||
| if [[ ! "$FILE" =~ ^web/src/features/ ]]; then | ||||||
| echo "Global or shared file changed: $FILE" | ||||||
| RUN_ALL=true | ||||||
| break | ||||||
| fi | ||||||
|
|
||||||
| # File is in features and looks like: web/src/features/incidents/components/App.tsx | ||||||
| # awk splits by '/' and grabs the 4th segment | ||||||
| FEATURE_DIR=$(echo "$FILE" | awk -F'/' '{print $4}') | ||||||
| CYPRESS_TAG=${FEATURE_MAP[$FEATURE_DIR]} | ||||||
|
|
||||||
| if [ -n "$CYPRESS_TAG" ]; then | ||||||
| TAGS_TO_RUN["$CYPRESS_TAG"]=1 | ||||||
| else | ||||||
| echo "Error: Unmapped feature directory changed: $FEATURE_DIR" | ||||||
| exit 1 | ||||||
| fi | ||||||
| done | ||||||
|
|
||||||
| if [ "$RUN_ALL" = true ]; then | ||||||
| echo "Changes affect global scope or unmapped features. Running ALL monitoring tests..." | ||||||
|
Contributor
There was a problem hiding this comment. unmapped features already exits
Suggested change
|
||||||
|
|
||||||
| DEFAULT_TAGS="@alerting @legacy-dashboards @metrics @targets acm-alerting @cluster-health-analyzer @perses-dashboards --@flaky --@xfail" | ||||||
|
Contributor
There was a problem hiding this comment.
Suggested change
|
||||||
| npm run test-cypress-monitoring:base -- --env grepTags="${DEFAULT_TAGS}" | ||||||
| else | ||||||
| UNIQUE_TAGS=("${!TAGS_TO_RUN[@]}") | ||||||
|
|
||||||
| if [ ${#UNIQUE_TAGS[@]} -eq 0 ]; then | ||||||
| echo "No relevant source files changed. Skipping tests." | ||||||
| exit 0 | ||||||
| fi | ||||||
|
|
||||||
| FEATURE_TAGS=$( | ||||||
| IFS=" " | ||||||
| echo "${UNIQUE_TAGS[*]}" | ||||||
| ) | ||||||
|
|
||||||
| GREP_TAGS="${FEATURE_TAGS} --@flaky --@xfail" | ||||||
|
|
||||||
| echo "Running specific tests for tags: $GREP_TAGS" | ||||||
|
|
||||||
| npm run test-cypress-monitoring:base -- --env grepTags="${GREP_TAGS}" | ||||||
| fi | ||||||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
The target branch might be different we should provide it from env defaulting to main
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.
we should add error handling, in shallow checkouts the main branch might not exist.