mirror of
https://github.com/featurebasedb/featurebase.git
synced 2026-08-28 10:54:59 +00:00
f627199acb
stdout/stderr around A lot of functions in the cmd and ctl packages were passing these around and barely using them. Replaced them with a logger for most functions. Some functions get an io.Writer instead so that their tests can find the output they're looking for. More cleanup on fb-1766: reworked the tests that were using io.Pipe or os.Pipe to check their results so they now use a bytes.Buffer. Unexported some variables that didn't need to be exported. Fixed NewConfigCommand to use the provided stderr, not os.Stderr. Added tests for rbf_dump, rbf_page, and keygen, since those weren't being tested at all. Added chksum_test, final cleanup.
60 lines
1.5 KiB
Go
60 lines
1.5 KiB
Go
// Copyright 2021 Molecula Corp. All rights reserved.
|
|
package ctl
|
|
|
|
import (
|
|
"context"
|
|
"io"
|
|
"net/http"
|
|
"os"
|
|
"strings"
|
|
"testing"
|
|
|
|
pilosa "github.com/molecula/featurebase/v3"
|
|
"github.com/molecula/featurebase/v3/logger"
|
|
"github.com/molecula/featurebase/v3/test"
|
|
)
|
|
|
|
func TestExportCommand_Validation(t *testing.T) {
|
|
cmLog := logger.NewStandardLogger(os.Stderr)
|
|
|
|
cm := NewExportCommand(cmLog)
|
|
|
|
err := cm.Run(context.Background())
|
|
if !errContains(err, pilosa.ErrIndexRequired) {
|
|
t.Fatalf("wrong error, expected %q, got: '%s'", pilosa.ErrIndexRequired, err)
|
|
}
|
|
|
|
cm.Index = "i"
|
|
err = cm.Run(context.Background())
|
|
if !errContains(err, pilosa.ErrFieldRequired) {
|
|
t.Fatalf("wrong error, expected %q, got: '%s'", pilosa.ErrFieldRequired, err)
|
|
}
|
|
}
|
|
|
|
func TestExportCommand_Run(t *testing.T) {
|
|
cluster := test.MustRunCluster(t, 1)
|
|
defer cluster.Close()
|
|
cmd := cluster.GetNode(0)
|
|
|
|
cmLog := logger.NewStandardLogger(io.Discard)
|
|
cm := NewExportCommand(cmLog)
|
|
hostport := cmd.API.Node().URI.HostPort()
|
|
cm.Host = hostport
|
|
|
|
resp, err := http.DefaultClient.Do(test.MustNewHTTPRequest("POST", "http://"+hostport+"/index/i", strings.NewReader("")))
|
|
if err != nil {
|
|
t.Fatalf("making http request: %v", err)
|
|
}
|
|
resp.Body.Close()
|
|
resp, err = http.DefaultClient.Do(test.MustNewHTTPRequest("POST", "http://"+hostport+"/index/i/field/f", strings.NewReader("")))
|
|
if err != nil {
|
|
t.Fatalf("making http request: %v", err)
|
|
}
|
|
resp.Body.Close()
|
|
|
|
cm.Index = "i"
|
|
cm.Field = "f"
|
|
if err := cm.Run(context.Background()); err != nil {
|
|
t.Fatalf("Export Run doesn't work: %s", err)
|
|
}
|
|
}
|