Skip to content

Commit db880bc

Browse files
committed
error message for skipped tests
1 parent 50dbf8d commit db880bc

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

.ci/magician/cmd/collect_nightly_test_status.go

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -158,8 +158,9 @@ func createTestReport(pVersion provider.Version, tc TeamcityClient, gcs Cloudsto
158158
var errorMessage string
159159
// Get test debug log gcs link
160160
logLink := fmt.Sprintf("https://storage.cloud.google.com/teamcity-logs/nightly/%s/%s/%s/debug-%s-%s-%s-%s.txt", pVersion.TeamCityNightlyProjectName(), date, build.Number, pVersion.ProviderName(), build.Number, strconv.Itoa(build.Id), testResult.Name)
161-
// Get concise error message
162-
if testResult.Status == "FAILURE" {
161+
// Get concise error message for failed and skipped tests
162+
// Skipped tests have a status of "UNKNOWN" on TC
163+
if testResult.Status == "FAILURE" || testResult.Status == "UNKNOWN" {
163164
errorMessage = convertErrorMessage(testResult.ErrorMessage)
164165
}
165166
testInfoList = append(testInfoList, TestInfo{

0 commit comments

Comments
 (0)