handle extra WIPs

tests/integration/cli/client_ping_test.go

@@ -11,6 +11,7 @@
 package clitest
 
 import (
+	"fmt"
 	"strings"
 	"testing"
 
@@ -54,6 +55,9 @@ func TestPingCommandToInvalidHost(t *testing.T) {
 
 func TestPingCommandNoHost(t *testing.T) {
 	conf := NewDefraNodeDefaultConfig(t)
-	_, stderr := runDefraCommand(t, conf, []string{"client", "ping", "--url", "localhost:56788"}) //WIP
+	p, err := findFreePortInRange(49152, 65535)
+	assert.NoError(t, err)
+	addr := fmt.Sprintf("localhost:%d", p)
+	_, stderr := runDefraCommand(t, conf, []string{"client", "ping", "--url", addr})
 	assertContainsSubstring(t, stderr, "failed to send ping")
 }

tests/integration/cli/serverdump_test.go

@@ -26,12 +26,3 @@ func TestServerDumpInvalidStoreErrs(t *testing.T) {
 	// assertContainsSubstring(t, stderr, "invalid datastore type")
 	assertContainsSubstring(t, stderr, "server-side dump is only supported for the Badger datastore")
 }
-
-func TestServerDump(t *testing.T) {
-	conf := NewDefraNodeDefaultConfig(t)
-	stopDefra := runDefraNode(t, conf)
-	_ = stopDefra()
-
-	_, _ = runDefraCommand(t, conf, []string{"server-dump"})
-	// WIP unsure why there is no output
-}

tests/integration/cli/utils.go

@@ -122,8 +122,6 @@ func runDefraCommand(t *testing.T, conf DefraNodeConfig, args []string) (stdout,
 		defraCmd.RootCmd.SetArgs(args)
 		_ = defraCmd.Execute(ctx)
 	})
-	t.Log("⭐️ stdout", stdout) //WIP
-	t.Log("⭐️ stderr", stderr) //WIP
 	return stdout, stderr
 }