apply feedback

Author: fredcarle

cli/test/action/p2p_replicator_delete.go

@@ -16,7 +16,7 @@ import (
 	"github.com/stretchr/testify/require"
 )
 
-// P2PReplicatorDelete executes the `client p2p replicator set` command.
+// P2PReplicatorDelete executes the `client p2p replicator delete` command.
 type P2PReplicatorDelete struct {
 	stateful
 	augmented

cli/test/action/p2p_replicator_getall.go

@@ -18,7 +18,7 @@ import (
 	"github.com/sourcenetwork/defradb/client"
 )
 
-// P2PReplicatorGetAll executes the `client p2p replicator set` command.
+// P2PReplicatorGetAll executes the `client p2p replicator getall` command.
 type P2PReplicatorGetAll struct {
 	stateful
 	augmented

cli/test/integration/p2p/connect_test.go

@@ -62,7 +62,7 @@ func TestConnect_WithInvalidIP_ShouldFail(t *testing.T) {
 // However, it does at least verify that a single address can be passed in.
 //
 // TODO: Add capability to have multiple defradb instances in tests, so we can
-// actually test successful connections. #4021
+// actually test successful connections. https://github.com/sourcenetwork/defradb/issues/4021
 func TestConnect_WithSinglePeer_ShouldSucceed(t *testing.T) {
 	test := &integration.Test{
 		Actions: []action.Action{
@@ -80,7 +80,7 @@ func TestConnect_WithSinglePeer_ShouldSucceed(t *testing.T) {
 // However, it does at least verify that multiple addresses can be passed in.
 //
 // TODO: Add capability to have multiple defradb instances in tests, so we can
-// actually test successful connections. #4021
+// actually test successful connections. https://github.com/sourcenetwork/defradb/issues/4021
 func TestConnect_WithMultiplePeers_ShouldSucceed(t *testing.T) {
 	test := &integration.Test{
 		Actions: []action.Action{