config test and validate

config.go

@@ -89,6 +89,26 @@ func NewConfig() *Config {
 	return c
 }
 
+// Validate that all configuration permutations are compatible with each other.
+func (c *Config) Validate() error {
+	if !foundItem(c.Cluster.Hosts, c.Bind) {
+		return ErrConfigHosts
+	}
+
+	// Validate cluster types
+	// TODO cluster types
+	// TODO validate len hosts
+	// TODO vaidate replica num and host len
+	// TODO  internal-hosts and hosts len must match
+
+	if c.Cluster.Type == "http" || c.Cluster.Type == "gossip" {
+		if !ContainsSubstring(c.Cluster.InternalPort, c.Cluster.InternalHosts) {
+			return ErrConfigBroadcastPort
+		}
+	}
+	return nil
+}
+
 // Duration is a TOML wrapper type for time.Duration.
 type Duration time.Duration
 

config_test.go

@@ -0,0 +1,18 @@
+package pilosa_test
+
+import (
+	"testing"
+
+	"github.com/pilosa/pilosa"
+)
+
+func Test_NewConfig(t *testing.T) {
+	x := pilosa.NewConfig()
+
+	// Check for bind addres in cluster hosts
+	if err := x.Validate(); err != pilosa.ErrConfigHosts {
+		t.Fatal(err)
+	}
+
+	x.Cluster.Type = "http"
+}

server/server.go

@@ -107,7 +107,11 @@ func (m *Command) Run(args ...string) (err error) {
 
 // SetupServer use the cluster configuration to setup this server
 func (m *Command) SetupServer() error {
-	var err error
+	err := m.Config.Validate()
+	if err != nil {
+		return err
+	}
+
 	cluster := pilosa.NewCluster()
 	cluster.ReplicaN = m.Config.Cluster.ReplicaN