fix unit test (bad payload)

Author: mgravell

src/StackExchange.Redis/KeyNotification.SubKeys.cs

@@ -204,8 +204,8 @@ internal SubKeyEnumerator(scoped KeyNotification notification)
             {
                 case KeyNotificationKind.SubKeySpace:
                 case KeyNotificationKind.SubKeyEvent:
-                    // Payload: <event>|<len>:<subkey>[|<len>:<subkey>...] or <key_len>:<key>|<len>:<subkey>[|<len>:<subkey>...]
-                    // We need to skip to the first | and then iterate through length-prefixed subkeys
+                    // Payload: <event>|<len>:<subkey>,<len>:<subkey>,... or <key_len>:<key>|<len>:<subkey>,<len>:<subkey>,...
+                    // We need to skip to the first | and then iterate through comma-separated length-prefixed subkeys
                     _data = CopyAndLeaseValue(notification._value, out _lease);
 
                     // Find the first pipe to skip the event/key part
@@ -242,7 +242,7 @@ internal SubKeyEnumerator(scoped KeyNotification notification)
                     break;
 
                 case KeyNotificationKind.SubKeySpaceEvent:
-                    // Payload: <len>:<subkey>[|<len>:<subkey>...]
+                    // Payload: <len>:<subkey>,<len>:<subkey>,...
                     _data = CopyAndLeaseValue(notification._value, out _lease);
                     _position = 0;
                     break;

src/StackExchange.Redis/KeyNotification.cs

@@ -811,7 +811,7 @@ public KeyNotificationType Type
                     return KeyNotificationTypeMetadata.Parse(ChannelSuffix);
 
                 case KeyNotificationKind.SubKeySpace:
-                    // Payload contains <event>|<len>:<subkey>[|<len>:<subkey>...]
+                    // Payload contains <event>|<len>:<subkey>,<len>:<subkey>,...
                     if (_value.TryGetSpan(out var directSub))
                     {
                         var pipeIndexSub = directSub.IndexOf((byte)'|');

…ange.Redis.Tests/KeyNotificationTests.cs ….Redis.Tests/KeyNotificationUnitTests.cstests/StackExchange.Redis.Tests/KeyNotificationTests.cs renamed to tests/StackExchange.Redis.Tests/KeyNotificationUnitTests.cs tests/StackExchange.Redis.Tests/KeyNotificationTests.cs renamed to tests/StackExchange.Redis.Tests/KeyNotificationUnitTests.cs

@@ -7,7 +7,7 @@
 
 namespace StackExchange.Redis.Tests;
 
-public class KeyNotificationTests(ITestOutputHelper log)
+public class KeyNotificationUnitTests(ITestOutputHelper log)
 {
     [Theory]
     [InlineData("foo", "foo")]
@@ -1677,9 +1677,10 @@ public void SubKeyEvent_RealWorldPayload_ParsesCorrectly()
     [Fact]
     public void SubKeySpace_MultipleFields_ParsesCorrectly()
     {
-        // __subkeyspace@0__:mykey with payload "4:hset|6:field1,6:field2"
+        // __subkeyspace@0__:mykey with payload "hset|6:field1,6:field2"
+        // Format: <event>|<len>:<subkey>,<len>:<subkey>...
         var channel = RedisChannel.Literal("__subkeyspace@0__:mykey");
-        RedisValue value = "4:hset|6:field1,6:field2";
+        RedisValue value = "hset|6:field1,6:field2";
 
         Assert.True(KeyNotification.TryParse(in channel, in value, out var notification));