IpcMainTests: Properly test for the types of values (must not be JsonElement)

Author: softworkz

src/ElectronNET.IntegrationTests/ElectronNET.IntegrationTests.csproj

@@ -10,7 +10,7 @@
   <PropertyGroup>
     <TargetFramework>net10.0</TargetFramework>
     <ImplicitUsings>enable</ImplicitUsings>
-    <Nullable>enable</Nullable>
+    <Nullable>disable</Nullable>
     <IsPackable>false</IsPackable>
     <CopyLocalLockFileAssemblies>true</CopyLocalLockFileAssemblies> <!-- https://github.com/Tyrrrz/GitHubActionsTestLogger/issues/5 -->
   </PropertyGroup>

src/ElectronNET.IntegrationTests/Tests/IpcMainTests.cs

@@ -15,10 +15,21 @@ public IpcMainTests(ElectronFixture fx)
         [Fact(Timeout = 20000)]
         public async Task Ipc_On_receives_message_from_renderer()
         {
+            object received = null;
+
             var tcs = new TaskCompletionSource<string>();
-            await Electron.IpcMain.On("ipc-on-test", obj => tcs.TrySetResult(obj?.ToString() ?? string.Empty));
+            await Electron.IpcMain.On("ipc-on-test", obj =>
+            {
+                received = obj;
+                tcs.TrySetResult(obj as string);
+            });
+
             await this.fx.MainWindow.WebContents.ExecuteJavaScriptAsync<string>("require('electron').ipcRenderer.send('ipc-on-test','payload123')");
+
             var result = await tcs.Task.WaitAsync(TimeSpan.FromSeconds(5));
+
+            received.Should().BeOfType<string>();
+            received.Should().Be("payload123");
             result.Should().Be("payload123");
         }
 
@@ -46,12 +57,18 @@ public async Task Ipc_RemoveAllListeners_stops_receiving()
         [Fact(Timeout = 20000)]
         public async Task Ipc_OnSync_returns_value()
         {
+            object received = null;
+
             Electron.IpcMain.OnSync("ipc-sync-test", (obj) =>
             {
-                obj.Should().NotBeNull();
+                received = obj;
                 return "pong";
             });
             var ret = await this.fx.MainWindow.WebContents.ExecuteJavaScriptAsync<string>("require('electron').ipcRenderer.sendSync('ipc-sync-test','ping')");
+
+            received.Should().BeOfType<string>();
+            received.Should().Be("ping");
+
             ret.Should().Be("pong");
         }