PR feedback

Author: RenderMichael

dotnet/test/common/BiDi/Script/CallFunctionLocalValueTest.cs

@@ -127,7 +127,7 @@ await context.Script.CallFunctionAsync($$"""
     }
 
     [Test]
-    public void CanCallFunctionWithArgumentNumber5()
+    public void CanCallFunctionWithArgumentNumberFive()
     {
         var number5 = new LocalValue.Number(5);
 
@@ -144,7 +144,7 @@ await context.Script.CallFunctionAsync($$"""
     }
 
     [Test]
-    public void CanCallFunctionWithArgumentNumberNegative5()
+    public void CanCallFunctionWithArgumentNumberNegativeFive()
     {
         var numberMinus5 = new LocalValue.Number(-5);
 
@@ -161,7 +161,7 @@ await context.Script.CallFunctionAsync($$"""
     }
 
     [Test]
-    public void CanCallFunctionWithArgumentNumber0()
+    public void CanCallFunctionWithArgumentNumberZero()
     {
         var number0 = new LocalValue.Number(0);
 
@@ -180,15 +180,15 @@ await context.Script.CallFunctionAsync($$"""
     [Test]
     [IgnoreBrowser(Selenium.Browser.Edge, "Chromium can't handle -0 argument as a number: https://github.com/w3c/webdriver-bidi/issues/887")]
     [IgnoreBrowser(Selenium.Browser.Chrome, "Chromium can't handle -0 argument as a number: https://github.com/w3c/webdriver-bidi/issues/887")]
-    public void CanCallFunctionWithArgumentNumberNegative0()
+    public void CanCallFunctionWithArgumentNumberNegativeZero()
     {
         var minus0 = new LocalValue.Number(double.NegativeZero);
 
         Assert.That(async () =>
         {
             await context.Script.CallFunctionAsync($$"""
             (arg) => {
-              if (arg !== 0 || arg.toLocaleString()[0] !== '-') {
+              if (!Object.is(arg, -0)) {
                 throw new Error("Assert failed: " + arg.toLocaleString());
               }
             }

dotnet/test/common/BiDi/Script/CallFunctionRemoteValueTest.cs

@@ -84,7 +84,7 @@ public async Task CanCallFunctionAndReturnUnixEpoch()
     }
 
     [Test]
-    public async Task CanCallFunctionAndReturnNumber5()
+    public async Task CanCallFunctionAndReturnNumberFive()
     {
         var response = await context.Script.CallFunctionAsync("() => { return 5; }", false);
 
@@ -93,7 +93,7 @@ public async Task CanCallFunctionAndReturnNumber5()
     }
 
     [Test]
-    public async Task CanCallFunctionAndReturnNumberNegative5()
+    public async Task CanCallFunctionAndReturnNumberNegativeFive()
     {
         var response = await context.Script.CallFunctionAsync("() => { return -5; }", false);
 
@@ -102,7 +102,7 @@ public async Task CanCallFunctionAndReturnNumberNegative5()
     }
 
     [Test]
-    public async Task CanCallFunctionAndReturnNumber0()
+    public async Task CanCallFunctionAndReturnNumberZero()
     {
         var response = await context.Script.CallFunctionAsync("() => { return 0; }", false);
 
@@ -111,7 +111,7 @@ public async Task CanCallFunctionAndReturnNumber0()
     }
 
     [Test]
-    public async Task CanCallFunctionAndReturnNumberNegative0()
+    public async Task CanCallFunctionAndReturnNumberNegativeZero()
     {
         var response = await context.Script.CallFunctionAsync("() => { return -0; }", false);