Skip to content

Commit

Permalink
Merge pull request #447 from cherylking/skipTestsChanges
Browse files Browse the repository at this point in the history
Update messages related to running tests when skipTests is set to true
  • Loading branch information
cherylking authored Jul 17, 2024
2 parents 4bb43e6 + 409d413 commit b2ca3c9
Showing 1 changed file with 10 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -691,6 +691,12 @@ public void runTests(boolean waitForApplicationUpdate, int messageOccurrences, T
error(e.getMessage(), e);
}
}
} else {
if (projectName != null) {
info("Tests will not run on demand for " + projectName + " because skipTests is set to true.");
} else {
info("Tests will not run on demand because skipTests is set to true.");
}
}
}

Expand Down Expand Up @@ -2522,6 +2528,10 @@ private void printDevModeMessages(boolean inputUnavailable, boolean startup) thr
}

private void printTestsMessage(boolean formatForAttention) {
// setting skipTests to true overrides the setting of hotTests, and prevents any tests from running in dev mode (automatically or on demand)
if (skipTests) {
return;
}
if (hotTests) {
String message = "Tests will run automatically when changes are detected. You can also press the Enter key to run tests on demand.";
info(formatForAttention ? formatAttentionMessage("Enter - " + message) : message);
Expand Down

0 comments on commit b2ca3c9

Please sign in to comment.