From 980e82a1b6a7eaa085732c25d330d67b1279ea21 Mon Sep 17 00:00:00 2001 From: Mayuki Sawatari Date: Fri, 24 Jan 2025 12:05:55 +0900 Subject: [PATCH 1/2] Fix unit tests --- tests/MagicOnion.Client.Tests/UnaryTest.cs | 40 +++++++++++----------- 1 file changed, 20 insertions(+), 20 deletions(-) diff --git a/tests/MagicOnion.Client.Tests/UnaryTest.cs b/tests/MagicOnion.Client.Tests/UnaryTest.cs index 93987aec0..a1b4e9614 100644 --- a/tests/MagicOnion.Client.Tests/UnaryTest.cs +++ b/tests/MagicOnion.Client.Tests/UnaryTest.cs @@ -41,7 +41,7 @@ public async Task Clone_WithOptions() // Assert Assert.NotNull(client); - callInvokerMock.Received(); + callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, Box>)!, default, default, default!); Assert.NotNull(actualCallOptions.Headers); Assert.Contains(actualCallOptions.Headers, x => x.Key == "foo" && x.Value == "bar"); } @@ -61,7 +61,7 @@ public async Task Clone_WithHost() // Assert Assert.NotNull(client); - callInvokerMock.Received(); + callInvokerMock.Received().AsyncUnaryCall(Arg.Any, Box>>(), "www.example.com", Arg.Any(), Arg.Any>()); } [Fact] @@ -87,7 +87,7 @@ public async Task Clone_WithCancellationToken() // Assert Assert.NotNull(client); - callInvokerMock.Received(); + callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, Box>)!, default, default!, default!); Assert.Equal(cts.Token, actualCancellationToken); } @@ -113,7 +113,7 @@ public async Task Clone_WithHeaders() // Assert Assert.NotNull(client); - callInvokerMock.Received(); + callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, Box>)!, default, default, default!); Assert.NotNull(actualHeaders); Assert.Contains(actualHeaders, x => x.Key == "foo" && x.Value == "bar"); } @@ -142,7 +142,7 @@ public async Task ParameterlessRequestsNil() // Assert Assert.Equal(Nil.Default, result); - callInvokerMock.Received(); + callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, Box>)!, default, default, default!); Assert.Equal(new[] { MessagePackCode.Nil }, serializedResponse.ToArray()); } @@ -160,7 +160,7 @@ public async Task ParameterlessReturnNil() // Assert Assert.Equal(Nil.Default, result); - callInvokerMock.Received(); + callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, Box>)!, default, default, default!); } [Fact] @@ -177,7 +177,7 @@ public async Task ParameterlessReturnValueType() // Assert Assert.Equal(123, result); - callInvokerMock.Received(); + callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, Box>)!, default, default, default!); } [Fact] @@ -194,7 +194,7 @@ public async Task ParameterlessReturnRefType() // Assert Assert.Equal("FooBar", result); - callInvokerMock.Received(); + callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, string>)!, default, default, default!); } [Fact] @@ -213,7 +213,7 @@ public async Task OneRefTypeParameterReturnValueType() // Assert Assert.Equal(123, result); - callInvokerMock.Received(); + callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method>)!, default, default, default!); } [Fact] @@ -232,7 +232,7 @@ public async Task OneValueTypeParameterReturnValueType() // Assert Assert.Equal(456, result); - callInvokerMock.Received(); + callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, Box>)!, default, default, default!); } [Fact] @@ -258,7 +258,7 @@ public async Task OneRefTypeParameterReturnRefType() // Assert Assert.Equal("Ok", result); - callInvokerMock.Received(); + callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method)!, default, default, default!); Assert.Equal("RequestValue", sentRequest); } @@ -285,7 +285,7 @@ public async Task OneValueTypeParameterReturnRefType() // Assert Assert.Equal("OK", result); - callInvokerMock.Received(); + callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, string>)!, default, default, default!); Assert.Equal(123, (sentRequest?.Value)); } @@ -313,7 +313,7 @@ public async Task TwoParametersReturnRefType() // Assert Assert.Equal("OK", result); - callInvokerMock.Received(); + callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method>, string>)!, default, default, default!); Assert.Equal(123, (sentRequest?.Value.Item1)); Assert.Equal("Foo", (sentRequest?.Value.Item2)); } @@ -342,7 +342,7 @@ public async Task TwoParametersReturnValueType() // Assert Assert.Equal(987, result); - callInvokerMock.Received(); + callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method>, Box>)!, default, default, default!); Assert.Equal(123, (sentRequest?.Value.Item1)); Assert.Equal("Foo", (sentRequest?.Value.Item2)); } @@ -360,7 +360,7 @@ public async Task ParameterlessNonGenericReturnType() await client.ParameterlessNonGenericReturnType(); // Assert - callInvokerMock.Received(); + callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, Box>)!, default, default, default!); } [Fact] @@ -377,7 +377,7 @@ public async Task OneRefTypeParameterNonGenericReturnType() await client.OneRefTypeParameterNonGenericReturnType(request); // Assert - callInvokerMock.Received(); + callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method>)!, default, default, default!); } [Fact] @@ -394,7 +394,7 @@ public async Task OneValueTypeParameterNonGenericReturnType() await client.OneValueTypeParameterNonGenericReturnType(request); // Assert - callInvokerMock.Received(); + callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, Box>)!, default, default, default!); } [Fact] @@ -419,7 +419,7 @@ public async Task TwoParametersNonGenericReturnType() await client.TwoParametersNonGenericReturnType(requestArg1, requestArg2); // Assert - callInvokerMock.Received(); + callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method>, Box>)!, default, default, default!); Assert.Equal(123, (sentRequest?.Value.Item1)); Assert.Equal("Foo", (sentRequest?.Value.Item2)); } @@ -444,7 +444,7 @@ public async Task ThrowsResponseHeaders() // Assert Assert.Equal(StatusCode.Unknown, result.StatusCode); Assert.Equal("FaultedOnResponseHeaders", result.Message); - callInvokerMock.Received(); + callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, Box>)!, default, default, default!); } [Fact] @@ -462,7 +462,7 @@ public async Task ThrowsResponse() // Assert Assert.Equal(StatusCode.Unknown, result.StatusCode); Assert.Equal("Faulted", result.Message); - callInvokerMock.Received(); + callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, Box>)!, default, default, default!); } public interface IUnaryTestService : IService From 025753c4ccc9eb5d257856ce757f3a65534a67d4 Mon Sep 17 00:00:00 2001 From: Mayuki Sawatari Date: Fri, 24 Jan 2025 12:12:17 +0900 Subject: [PATCH 2/2] Fix warnings --- tests/MagicOnion.Client.Tests/UnaryTest.cs | 40 +++++++++++----------- 1 file changed, 20 insertions(+), 20 deletions(-) diff --git a/tests/MagicOnion.Client.Tests/UnaryTest.cs b/tests/MagicOnion.Client.Tests/UnaryTest.cs index a1b4e9614..b0742519b 100644 --- a/tests/MagicOnion.Client.Tests/UnaryTest.cs +++ b/tests/MagicOnion.Client.Tests/UnaryTest.cs @@ -41,7 +41,7 @@ public async Task Clone_WithOptions() // Assert Assert.NotNull(client); - callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, Box>)!, default, default, default!); + _ = callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, Box>)!, default, default, default!); Assert.NotNull(actualCallOptions.Headers); Assert.Contains(actualCallOptions.Headers, x => x.Key == "foo" && x.Value == "bar"); } @@ -61,7 +61,7 @@ public async Task Clone_WithHost() // Assert Assert.NotNull(client); - callInvokerMock.Received().AsyncUnaryCall(Arg.Any, Box>>(), "www.example.com", Arg.Any(), Arg.Any>()); + _ = callInvokerMock.Received().AsyncUnaryCall(Arg.Any, Box>>(), "www.example.com", Arg.Any(), Arg.Any>()); } [Fact] @@ -87,7 +87,7 @@ public async Task Clone_WithCancellationToken() // Assert Assert.NotNull(client); - callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, Box>)!, default, default!, default!); + _ = callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, Box>)!, default, default!, default!); Assert.Equal(cts.Token, actualCancellationToken); } @@ -113,7 +113,7 @@ public async Task Clone_WithHeaders() // Assert Assert.NotNull(client); - callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, Box>)!, default, default, default!); + _ = callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, Box>)!, default, default, default!); Assert.NotNull(actualHeaders); Assert.Contains(actualHeaders, x => x.Key == "foo" && x.Value == "bar"); } @@ -142,7 +142,7 @@ public async Task ParameterlessRequestsNil() // Assert Assert.Equal(Nil.Default, result); - callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, Box>)!, default, default, default!); + _ = callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, Box>)!, default, default, default!); Assert.Equal(new[] { MessagePackCode.Nil }, serializedResponse.ToArray()); } @@ -160,7 +160,7 @@ public async Task ParameterlessReturnNil() // Assert Assert.Equal(Nil.Default, result); - callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, Box>)!, default, default, default!); + _ = callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, Box>)!, default, default, default!); } [Fact] @@ -177,7 +177,7 @@ public async Task ParameterlessReturnValueType() // Assert Assert.Equal(123, result); - callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, Box>)!, default, default, default!); + _ = callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, Box>)!, default, default, default!); } [Fact] @@ -194,7 +194,7 @@ public async Task ParameterlessReturnRefType() // Assert Assert.Equal("FooBar", result); - callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, string>)!, default, default, default!); + _ = callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, string>)!, default, default, default!); } [Fact] @@ -213,7 +213,7 @@ public async Task OneRefTypeParameterReturnValueType() // Assert Assert.Equal(123, result); - callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method>)!, default, default, default!); + _ = callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method>)!, default, default, default!); } [Fact] @@ -232,7 +232,7 @@ public async Task OneValueTypeParameterReturnValueType() // Assert Assert.Equal(456, result); - callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, Box>)!, default, default, default!); + _ = callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, Box>)!, default, default, default!); } [Fact] @@ -258,7 +258,7 @@ public async Task OneRefTypeParameterReturnRefType() // Assert Assert.Equal("Ok", result); - callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method)!, default, default, default!); + _ = callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method)!, default, default, default!); Assert.Equal("RequestValue", sentRequest); } @@ -285,7 +285,7 @@ public async Task OneValueTypeParameterReturnRefType() // Assert Assert.Equal("OK", result); - callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, string>)!, default, default, default!); + _ = callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, string>)!, default, default, default!); Assert.Equal(123, (sentRequest?.Value)); } @@ -313,7 +313,7 @@ public async Task TwoParametersReturnRefType() // Assert Assert.Equal("OK", result); - callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method>, string>)!, default, default, default!); + _ = callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method>, string>)!, default, default, default!); Assert.Equal(123, (sentRequest?.Value.Item1)); Assert.Equal("Foo", (sentRequest?.Value.Item2)); } @@ -342,7 +342,7 @@ public async Task TwoParametersReturnValueType() // Assert Assert.Equal(987, result); - callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method>, Box>)!, default, default, default!); + _ = callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method>, Box>)!, default, default, default!); Assert.Equal(123, (sentRequest?.Value.Item1)); Assert.Equal("Foo", (sentRequest?.Value.Item2)); } @@ -360,7 +360,7 @@ public async Task ParameterlessNonGenericReturnType() await client.ParameterlessNonGenericReturnType(); // Assert - callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, Box>)!, default, default, default!); + _ = callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, Box>)!, default, default, default!); } [Fact] @@ -377,7 +377,7 @@ public async Task OneRefTypeParameterNonGenericReturnType() await client.OneRefTypeParameterNonGenericReturnType(request); // Assert - callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method>)!, default, default, default!); + _ = callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method>)!, default, default, default!); } [Fact] @@ -394,7 +394,7 @@ public async Task OneValueTypeParameterNonGenericReturnType() await client.OneValueTypeParameterNonGenericReturnType(request); // Assert - callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, Box>)!, default, default, default!); + _ = callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, Box>)!, default, default, default!); } [Fact] @@ -419,7 +419,7 @@ public async Task TwoParametersNonGenericReturnType() await client.TwoParametersNonGenericReturnType(requestArg1, requestArg2); // Assert - callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method>, Box>)!, default, default, default!); + _ = callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method>, Box>)!, default, default, default!); Assert.Equal(123, (sentRequest?.Value.Item1)); Assert.Equal("Foo", (sentRequest?.Value.Item2)); } @@ -444,7 +444,7 @@ public async Task ThrowsResponseHeaders() // Assert Assert.Equal(StatusCode.Unknown, result.StatusCode); Assert.Equal("FaultedOnResponseHeaders", result.Message); - callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, Box>)!, default, default, default!); + _ = callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, Box>)!, default, default, default!); } [Fact] @@ -462,7 +462,7 @@ public async Task ThrowsResponse() // Assert Assert.Equal(StatusCode.Unknown, result.StatusCode); Assert.Equal("Faulted", result.Message); - callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, Box>)!, default, default, default!); + _ = callInvokerMock.ReceivedWithAnyArgs().AsyncUnaryCall(default(Method, Box>)!, default, default, default!); } public interface IUnaryTestService : IService