fix: remove invalid event access from subclass; fix describe chain

- Remove if (Console != null) check in CdpWebWorker since events can only
  be accessed from their declaring class (CS0070)
- Fix PuppeteerTest describe chain from 'Workers > console' to 'Workers console'

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: kblok

lib/PuppeteerSharp.Tests/WorkerTests/PageWorkerTests.cs

@@ -143,7 +143,7 @@ public async Task CanBeClosed()
             Assert.That(await workerClosedTcs.Task, Is.SameAs(worker));
         }
 
-        [Test, PuppeteerTest("worker.spec", "Workers > console", "should work")]
+        [Test, PuppeteerTest("worker.spec", "Workers console", "should work")]
         public async Task ConsoleShouldWork()
         {
             var worker = await CreateWorkerAsync();
@@ -160,7 +160,7 @@ public async Task ConsoleShouldWork()
             Assert.That(await message.Args[1].JsonValueAsync<int>(), Is.EqualTo(5));
         }
 
-        [Test, PuppeteerTest("worker.spec", "Workers > console", "should work for Error instances")]
+        [Test, PuppeteerTest("worker.spec", "Workers console", "should work for Error instances")]
         public async Task ConsoleShouldWorkForErrorInstances()
         {
             var worker = await CreateWorkerAsync();
@@ -175,7 +175,7 @@ public async Task ConsoleShouldWorkForErrorInstances()
             Assert.That(message.Args, Has.Count.EqualTo(1));
         }
 
-        [Test, PuppeteerTest("worker.spec", "Workers > console", "should return the first line of the error message in text()")]
+        [Test, PuppeteerTest("worker.spec", "Workers console", "should return the first line of the error message in text()")]
         public async Task ConsoleShouldReturnFirstLineOfErrorMessageInText()
         {
             var worker = await CreateWorkerAsync();
@@ -189,7 +189,7 @@ public async Task ConsoleShouldReturnFirstLineOfErrorMessageInText()
             Assert.That(message.Type, Is.EqualTo(ConsoleType.Log));
         }
 
-        [Test, PuppeteerTest("worker.spec", "Workers > console", "should work for console.trace")]
+        [Test, PuppeteerTest("worker.spec", "Workers console", "should work for console.trace")]
         public async Task ConsoleShouldWorkForConsoleTrace()
         {
             var worker = await CreateWorkerAsync();
@@ -203,7 +203,7 @@ public async Task ConsoleShouldWorkForConsoleTrace()
             Assert.That(message.Text, Is.EqualTo("calling console.trace"));
         }
 
-        [Test, PuppeteerTest("worker.spec", "Workers > console", "should work for console.dir")]
+        [Test, PuppeteerTest("worker.spec", "Workers console", "should work for console.dir")]
         public async Task ConsoleShouldWorkForConsoleDir()
         {
             var worker = await CreateWorkerAsync();
@@ -217,7 +217,7 @@ public async Task ConsoleShouldWorkForConsoleDir()
             Assert.That(message.Text, Is.EqualTo("calling console.dir"));
         }
 
-        [Test, PuppeteerTest("worker.spec", "Workers > console", "should work for console.warn")]
+        [Test, PuppeteerTest("worker.spec", "Workers console", "should work for console.warn")]
         public async Task ConsoleShouldWorkForConsoleWarn()
         {
             var worker = await CreateWorkerAsync();
@@ -231,7 +231,7 @@ public async Task ConsoleShouldWorkForConsoleWarn()
             Assert.That(message.Text, Is.EqualTo("calling console.warn"));
         }
 
-        [Test, PuppeteerTest("worker.spec", "Workers > console", "should work for console.error")]
+        [Test, PuppeteerTest("worker.spec", "Workers console", "should work for console.error")]
         public async Task ConsoleShouldWorkForConsoleError()
         {
             var worker = await CreateWorkerAsync();
@@ -245,7 +245,7 @@ public async Task ConsoleShouldWorkForConsoleError()
             Assert.That(message.Text, Is.EqualTo("calling console.error"));
         }
 
-        [Test, PuppeteerTest("worker.spec", "Workers > console", "should work for console.log with promise")]
+        [Test, PuppeteerTest("worker.spec", "Workers console", "should work for console.log with promise")]
         public async Task ConsoleShouldWorkForConsoleLogWithPromise()
         {
             var worker = await CreateWorkerAsync();
@@ -259,7 +259,7 @@ public async Task ConsoleShouldWorkForConsoleLogWithPromise()
             Assert.That(message.Text, Does.Contain("promise").Or.Contain("Promise"));
         }
 
-        [Test, PuppeteerTest("worker.spec", "Workers > console", "should work for different console API calls with timing functions")]
+        [Test, PuppeteerTest("worker.spec", "Workers console", "should work for different console API calls with timing functions")]
         public async Task ConsoleShouldWorkForTimingFunctions()
         {
             var worker = await CreateWorkerAsync();
@@ -276,7 +276,7 @@ await worker.EvaluateFunctionAsync(@"() => {
             Assert.That(messages[0].Text, Does.Contain("calling console.time"));
         }
 
-        [Test, PuppeteerTest("worker.spec", "Workers > console", "should work for different console API calls with group functions")]
+        [Test, PuppeteerTest("worker.spec", "Workers console", "should work for different console API calls with group functions")]
         public async Task ConsoleShouldWorkForGroupFunctions()
         {
             var worker = await CreateWorkerAsync();
@@ -294,7 +294,7 @@ await worker.EvaluateFunctionAsync(@"() => {
             Assert.That(messages[0].Text, Does.Contain("calling console.group"));
         }
 
-        [Test, PuppeteerTest("worker.spec", "Workers > console", "should return remote objects")]
+        [Test, PuppeteerTest("worker.spec", "Workers console", "should return remote objects")]
         public async Task ConsoleShouldReturnRemoteObjects()
         {
             var worker = await CreateWorkerAsync();

lib/PuppeteerSharp/Cdp/CdpWebWorker.cs

@@ -164,42 +164,39 @@ private async Task OnConsoleAPICalledAsync(MessageEventArgs e)
         var consoleData = e.MessageData.ToObject<PageConsoleResponse>();
         var handles = consoleData.Args.Select(i => new CdpJSHandle(World, i)).ToArray<IJSHandle>();
 
-        if (Console != null)
+        var tokens = handles.Select(i =>
         {
-            var tokens = handles.Select(i =>
+            var handle = (ICdpHandle)i;
+            if (handle.RemoteObject.Subtype == RemoteObjectSubtype.Error && !string.IsNullOrEmpty(handle.RemoteObject.Description))
             {
-                var handle = (ICdpHandle)i;
-                if (handle.RemoteObject.Subtype == RemoteObjectSubtype.Error && !string.IsNullOrEmpty(handle.RemoteObject.Description))
-                {
-                    return handle.RemoteObject.Description.Split('\n')[0];
-                }
+                return handle.RemoteObject.Description.Split('\n')[0];
+            }
 
-                return handle.RemoteObject.ObjectId != null || handle.RemoteObject.Type == RemoteObjectType.Object
-                    ? i.ToString()
-                    : RemoteObjectHelper.ValueFromRemoteObject<object>(handle.RemoteObject)?.ToString() ?? "null";
-            });
+            return handle.RemoteObject.ObjectId != null || handle.RemoteObject.Type == RemoteObjectType.Object
+                ? i.ToString()
+                : RemoteObjectHelper.ValueFromRemoteObject<object>(handle.RemoteObject)?.ToString() ?? "null";
+        });
 
-            var location = new ConsoleMessageLocation();
-            var stackTraceLocations = new List<ConsoleMessageLocation>();
-            if (consoleData.StackTrace?.CallFrames?.Length > 0)
+        var location = new ConsoleMessageLocation();
+        var stackTraceLocations = new List<ConsoleMessageLocation>();
+        if (consoleData.StackTrace?.CallFrames?.Length > 0)
+        {
+            foreach (var callFrame in consoleData.StackTrace.CallFrames)
             {
-                foreach (var callFrame in consoleData.StackTrace.CallFrames)
+                stackTraceLocations.Add(new ConsoleMessageLocation
                 {
-                    stackTraceLocations.Add(new ConsoleMessageLocation
-                    {
-                        URL = callFrame.URL,
-                        LineNumber = callFrame.LineNumber,
-                        ColumnNumber = callFrame.ColumnNumber,
-                    });
-                }
-
-                location = stackTraceLocations[0];
+                    URL = callFrame.URL,
+                    LineNumber = callFrame.LineNumber,
+                    ColumnNumber = callFrame.ColumnNumber,
+                });
             }
 
-            var consoleMessage = new ConsoleMessage(consoleData.Type, string.Join(" ", tokens), handles, location, stackTraceLocations);
-            OnConsole(new ConsoleEventArgs(consoleMessage));
+            location = stackTraceLocations[0];
         }
 
+        var consoleMessage = new ConsoleMessage(consoleData.Type, string.Join(" ", tokens), handles, location, stackTraceLocations);
+        OnConsole(new ConsoleEventArgs(consoleMessage));
+
         await _consoleAPICalled(consoleData.Type, handles, consoleData.StackTrace).ConfigureAwait(false);
     }