Minor test changes (#1027)

Author: kblok

lib/PuppeteerSharp.Tests/ConnectionTests/ConnectionTests.cs

@@ -11,17 +11,6 @@ public ConnectionTests(ITestOutputHelper output) : base(output)
         {
         }
 
-        [Fact]
-        public async Task ShouldThrowNiceErrors()
-        {
-            var exception = await Assert.ThrowsAsync<MessageException>(async () =>
-            {
-                await TheSourceOfTheProblems();
-            });
-            Assert.Contains("TheSourceOfTheProblems", exception.StackTrace);
-            Assert.Contains("ThisCommand.DoesNotExist", exception.Message);
-        }
-
         [Fact]
         public async Task ShouldCleanCallbackList()
         {
@@ -33,7 +22,5 @@ public async Task ShouldCleanCallbackList()
             await Page.SetJavaScriptEnabledAsync(true);
             Assert.False(Page.Client.HasPendingCallbacks());
         }
-
-        private async Task TheSourceOfTheProblems() => await Page.Client.SendAsync("ThisCommand.DoesNotExist");
     }
 }

lib/PuppeteerSharp.Tests/InputTests/KeyboardTests.cs

@@ -182,20 +182,6 @@ await Page.EvaluateExpressionAsync(@"{
             Assert.Equal("He Wrd!", await Page.EvaluateExpressionAsync<string>("textarea.value"));
         }
 
-        [Fact]
-        public async Task KeyboardModifiers()
-        {
-            var keyboard = Page.Keyboard;
-            Assert.Equal(0, keyboard.Modifiers);
-            await keyboard.DownAsync(Key.Shift);
-            Assert.Equal(8, keyboard.Modifiers);
-            await keyboard.DownAsync(Key.Alt);
-            Assert.Equal(9, keyboard.Modifiers);
-            await keyboard.UpAsync(Key.Shift);
-            await keyboard.UpAsync(Key.Alt);
-            Assert.Equal(0, keyboard.Modifiers);
-        }
-
         [Fact]
         public async Task ShouldSpecifyRepeatProperty()
         {

lib/PuppeteerSharp.Tests/TargetTests/CreateCDPSessionTests.cs

@@ -87,5 +87,19 @@ public async Task ShouldBeAbleToDetachSession()
                 }));
             Assert.Contains("Session closed.", exception.Message);
         }
+
+        [Fact]
+        public async Task ShouldThrowNiceErrors()
+        {
+            var client = await Page.Target.CreateCDPSessionAsync();
+            async Task TheSourceOfTheProblems() => await client.SendAsync("ThisCommand.DoesNotExist");
+
+            var exception = await Assert.ThrowsAsync<MessageException>(async () =>
+            {
+                await TheSourceOfTheProblems();
+            });
+            Assert.Contains("TheSourceOfTheProblems", exception.StackTrace);
+            Assert.Contains("ThisCommand.DoesNotExist", exception.Message);
+        }
     }
 }