Adjust format and remove comment

Signed-off-by: hwware <[email protected]>

Author: hwware

src/pubsub.c

@@ -279,14 +279,11 @@ void pubsubSubscribeChannel(client *c, robj *channel, pubsubtype type) {
             clients = dictCreate(&clientDictType);
             kvstoreDictSetVal(*type.serverPubSubChannels, slot, de, clients);
             incrRefCount(channel);
-            serverLog(LL_WARNING, "The channel count is %d", type.subscriptionCount(c));
         }
 
         serverAssert(dictAdd(clients, c, NULL) != DICT_ERR);
         serverAssert(dictInsertAtPosition(type.clientPubSubChannels(c), channel, position));
         incrRefCount(channel);
-        serverLog(LL_WARNING, "The channel count is %d", type.subscriptionCount(c));
-	
     }
 }
 

tests/unit/pubsub.tcl

@@ -45,7 +45,6 @@ start_server {tags {"pubsub network"}} {
         set rd1 [valkey_deferring_client]
 
         # subscribe to two channels
-        # assert_equal {1 2} [subscribe $rd1 {chan1 chan2}]
         assert_equal {1} [subscribe $rd1 {chan1}]
         assert_equal {2} [subscribe $rd1 {chan2}]
         assert_equal 1 [r publish chan1 hello]
@@ -85,7 +84,6 @@ start_server {tags {"pubsub network"}} {
 
     test "PUBLISH/SUBSCRIBE after UNSUBSCRIBE without arguments" {
         set rd1 [valkey_deferring_client]
-        # assert_equal {1 2 3} [subscribe $rd1 {chan1 chan2 chan3}]
         assert_equal {1} [subscribe $rd1 {chan1}]
         assert_equal {2} [subscribe $rd1 {chan2}]
         assert_equal {3} [subscribe $rd1 {chan3}]
@@ -109,20 +107,6 @@ start_server {tags {"pubsub network"}} {
         assert_equal {1} [subscribe $rd1 {chan1}]
         assert_equal {2} [subscribe $rd1 {chan2}]
         assert_equal {3} [subscribe $rd1 {chan3}]
-        #assert_equal {1 1 1} [subscribe $rd1 {chan1 chan1 chan1}]
-        assert_equal 1 [r publish chan1 hello]
-        assert_equal {message chan1 hello} [$rd1 read]
-
-        # clean up clients
-        $rd1 close
-    }
-
-    test "SUBSCRIBE to one channel more than once" {
-        set rd1 [valkey_deferring_client]
-        assert_equal {1} [subscribe $rd1 {chan1}]
-        assert_equal {2} [subscribe $rd1 {chan2}]
-        assert_equal {3} [subscribe $rd1 {chan3}]
-        #assert_equal {1 1 1} [subscribe $rd1 {chan1 chan1 chan1}]
         assert_equal 1 [r publish chan1 hello]
         assert_equal {message chan1 hello} [$rd1 read]
 
@@ -504,10 +488,7 @@ start_server {tags {"pubsub network"}} {
         # Note: SUBSCRIBE and UNSUBSCRIBE with multiple channels in the same command,
         # Only one response is returned 
 	# This update matches with Redis response: one command always returns one response
-
         assert_equal "subscribe foo 1 subscribe bar 2 subscribe baz 3" [r subscribe foo bar baz]
-        # assert_equal "subscribe bar 2" [r read]
-        # assert_equal "subscribe baz 3" [r read]
 
         r multi
         r ping abc

tests/unit/pubsubshard.tcl

@@ -64,7 +64,6 @@ start_server {tags {"pubsubshard external:skip"}} {
 
     test "SSUBSCRIBE to one channel more than once" {
         set rd1 [valkey_deferring_client]
-        #assert_equal {1 1 1} [ssubscribe $rd1 {chan1 chan1 chan1}]
         assert_equal {1} [ssubscribe $rd1 {chan1}]
         assert_equal {1} [ssubscribe $rd1 {chan1}]
         assert_equal {1} [ssubscribe $rd1 {chan1}]