actions/runner
1
0
Fork 0
mirror of https://github.com/actions/runner.git synced 2025-12-17 15:59:37 +00:00
Code Issues Packages Projects Releases Wiki Activity

If conclusion is not set, we cannot get default value (#2535)

Browse Source
This commit is contained in:
Yang Cao
2023-04-13 13:03:57 -04:00
committed by GitHub
parent 766cefe599
commit b9a0b5dba9
1 changed files with 7 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
src/Sdk/WebApi/WebApi/ResultsHttpClient.cs
View File

@@ -330,7 +330,7 @@ namespace GitHub.Services.Results.Client
Status = ConvertStateToStatus(r.State.GetValueOrDefault()), Status = ConvertStateToStatus(r.State.GetValueOrDefault()),
StartedAt = r.StartTime?.ToString(Constants.TimestampFormat), StartedAt = r.StartTime?.ToString(Constants.TimestampFormat),
CompletedAt = r.FinishTime?.ToString(Constants.TimestampFormat), CompletedAt = r.FinishTime?.ToString(Constants.TimestampFormat),
Conclusion = ConvertResultToConclusion(r.Result.GetValueOrDefault()) Conclusion = ConvertResultToConclusion(r.Result)
}; };
} }
@@ -349,8 +349,13 @@ namespace GitHub.Services.Results.Client
} }
} }
private Conclusion ConvertResultToConclusion(TaskResult r) private Conclusion ConvertResultToConclusion(TaskResult? r)
{ {
if (!r.HasValue)
{
return Conclusion.ConclusionUnknown;
}
switch (r) switch (r)
{ {
case TaskResult.Succeeded: case TaskResult.Succeeded: