Skip to content

Commit 8ed56e9

Browse files
committed
resolving nullability warnings
1 parent 17f299d commit 8ed56e9

File tree

3 files changed

+14
-10
lines changed

3 files changed

+14
-10
lines changed

msgraph-chaos-proxy/ChaosEngine.cs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -295,16 +295,16 @@ private void UpdateProxyResponse(SessionEventArgs e, ResponseComponents response
295295
}
296296

297297
if ((int)responseComponents.ErrorStatus >= 400 && string.IsNullOrEmpty(responseComponents.Body)) {
298-
responseComponents.Body = JsonSerializer.Serialize(new ErrorResponseBody {
299-
Error = new ErrorResponseError {
298+
responseComponents.Body = JsonSerializer.Serialize(new ErrorResponseBody(
299+
new ErrorResponseError {
300300
Code = new Regex("([A-Z])").Replace(responseComponents.ErrorStatus.ToString(), m => { return $" {m.Groups[1]}"; }).Trim(),
301301
Message = "Some error happened",
302302
InnerError = new ErrorResponseInnerError {
303303
RequestId = responseComponents.RequestId,
304304
Date = responseComponents.RequestDate
305305
}
306-
}
307-
});
306+
})
307+
);
308308
}
309309
Console.WriteLine($"\t{(matchingResponse is not null ? "Mocked" : "Failed")} {e.HttpClient.Request.RequestUri.AbsolutePath} with {responseComponents.ErrorStatus}");
310310
e.GenericResponse(responseComponents.Body ?? string.Empty, responseComponents.ErrorStatus, responseComponents.Headers);

msgraph-chaos-proxy/ChaosProxyConfiguration.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,7 @@ public void Dispose()
9393

9494
public class ChaosProxyMockResponse {
9595
[JsonPropertyName("url")]
96-
public string Url { get; set; }
96+
public string Url { get; set; } = string.Empty;
9797
[JsonPropertyName("responseCode")]
9898
public int? ResponseCode { get; set; } = 200;
9999
[JsonPropertyName("responseBody")]

msgraph-chaos-proxy/ErrorResponseBody.cs

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4,21 +4,25 @@ namespace Microsoft.Graph.ChaosProxy {
44
internal class ErrorResponseBody {
55
[JsonPropertyName("error")]
66
public ErrorResponseError Error { get; set; }
7+
8+
public ErrorResponseBody(ErrorResponseError error) {
9+
Error = error;
10+
}
711
}
812

913
internal class ErrorResponseError {
1014
[JsonPropertyName("code")]
11-
public string Code { get; set; }
15+
public string Code { get; set; } = string.Empty;
1216
[JsonPropertyName("message")]
13-
public string Message { get; set; }
17+
public string Message { get; set; } = string.Empty;
1418
[JsonPropertyName("innerError")]
15-
public ErrorResponseInnerError InnerError { get; set; }
19+
public ErrorResponseInnerError? InnerError { get; set; }
1620
}
1721

1822
internal class ErrorResponseInnerError {
1923
[JsonPropertyName("request-id")]
20-
public string RequestId { get; set; }
24+
public string RequestId { get; set; } = string.Empty;
2125
[JsonPropertyName("date")]
22-
public string Date { get; set; }
26+
public string Date { get; set; } = string.Empty;
2327
}
2428
}

0 commit comments

Comments
 (0)