Skip to content

Commit c510b26

Browse files
committed
[fix] analyzer warnings and XML comments
1 parent a369004 commit c510b26

File tree

3 files changed

+13
-21
lines changed

3 files changed

+13
-21
lines changed

src/TestClient/Program.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,8 @@
1313

1414
var result = client.ExecuteAsync(request).Result;
1515

16-
if (result.IsSuccessful != true)
16+
if (!result.IsSuccessful)
1717
throw new InvalidOperationException("Error sending file: " + result.Content);
1818

1919
Console.WriteLine("HTTP status: " + result.StatusCode);
20-
Console.ReadLine();
20+
Console.ReadLine();

src/TestServer/Controllers/Api/v1/TestInController.cs

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
using System;
22
using System.Diagnostics;
33
using System.IO;
4-
using System.Linq;
54
using System.Threading.Tasks;
65
using Simplify.Web;
76
using Simplify.Web.Attributes;
@@ -14,17 +13,17 @@ public class TestInController : Controller2<MultipartViewModel>
1413
{
1514
public async Task<ControllerResponse> Invoke()
1615
{
17-
var file = Model.Files.FirstOrDefault() ?? throw new ArgumentException("No files in model");
16+
var file = Model.Files[0] ?? throw new ArgumentException("No files in model");
1817
using var stream = new StreamReader(file.Data);
1918
var fileData = await stream.ReadToEndAsync();
2019

21-
Trace.WriteLine($"Files count: '{Model.Files.Count}'");
20+
Trace.TraceInformation($"Files count: '{Model.Files.Count}'");
2221
Console.WriteLine($"Files count: '{Model.Files.Count}'");
2322

24-
Trace.WriteLine($"File name: '{file.FileName}'");
23+
Trace.TraceInformation($"File name: '{file.FileName}'");
2524
Console.WriteLine($"File name: '{file.FileName}'");
2625

27-
Trace.WriteLine($"File content: '{fileData}'");
26+
Trace.TraceInformation($"File content: '{fileData}'");
2827
Console.WriteLine($"File content: '{fileData}'");
2928

3029
// Assert

src/TestServer/Properties/launchSettings.json

Lines changed: 7 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -1,29 +1,22 @@
11
{
2-
"iisSettings":
3-
{
2+
"iisSettings": {
43
"windowsAuthentication": false,
54
"anonymousAuthentication": true,
6-
"iisExpress":
7-
{
5+
"iisExpress": {
86
"applicationUrl": "http://localhost:5000",
97
"sslPort": 0
108
}
119
},
12-
"profiles":
13-
{
14-
"IIS Express":
15-
{
10+
"profiles": {
11+
"IIS Express": {
1612
"commandName": "IISExpress",
17-
"environmentVariables":
18-
{
13+
"environmentVariables": {
1914
"ASPNETCORE_ENVIRONMENT": "Development"
2015
}
2116
},
22-
"TesterApp":
23-
{
17+
"TesterApp": {
2418
"commandName": "Project",
25-
"environmentVariables":
26-
{
19+
"environmentVariables": {
2720
"ASPNETCORE_ENVIRONMENT": "Development"
2821
},
2922
"launchBrowser": true,

0 commit comments

Comments
 (0)