fix the fake_test

config/config.go → config.go

@@ -1,7 +1,10 @@
-package config
+package main
 
-import kingpin "gopkg.in/alecthomas/kingpin.v2"
-import "net/url"
+import (
+	"net/url"
+
+	kingpin "gopkg.in/alecthomas/kingpin.v2"
+)
 
 type MateConfig struct {
 	Producer string
@@ -27,7 +30,7 @@ type MateConfig struct {
 	GoogleRecordGroupID string
 }
 
-func New(version string) *MateConfig {
+func NewConfig(version string) *MateConfig {
 	kingpin.Version(version)
 	return &MateConfig{}
 }

main.go

@@ -5,7 +5,6 @@ import (
 
 	log "github.com/Sirupsen/logrus"
 
-	"github.com/zalando-incubator/mate/config"
 	"github.com/zalando-incubator/mate/consumers"
 	"github.com/zalando-incubator/mate/controller"
 	"github.com/zalando-incubator/mate/producers"
@@ -14,7 +13,7 @@ import (
 var version = "Unknown"
 
 func main() {
-	cfg := config.New(version)
+	cfg := NewConfig(version)
 
 	cfg.ParseFlags()
 
@@ -47,7 +46,7 @@ func main() {
 	ctrl.Wait()
 }
 
-func newSyncConsumer(cfg *config.MateConfig) (consumers.Consumer, error) {
+func newSyncConsumer(cfg *MateConfig) (consumers.Consumer, error) {
 	// New returns a Consumer implementation.
 	var consumer consumers.Consumer
 	var err error
@@ -67,7 +66,7 @@ func newSyncConsumer(cfg *config.MateConfig) (consumers.Consumer, error) {
 	return consumers.NewSynced(consumer)
 }
 
-func newProducer(cfg *config.MateConfig) (producers.Producer, error) {
+func newProducer(cfg *MateConfig) (producers.Producer, error) {
 	switch cfg.Producer {
 	case "kubernetes":
 		kubeConfig := &producers.KubernetesOptions{

producers/fake.go

@@ -15,6 +15,9 @@ import (
 const (
 	defaultFakeDomain = "example.org."
 	defaultFakeMode   = "ip"
+	ipMode            = "ip"
+	hostnameMode      = "hostname"
+	fixedMode         = "fixed"
 )
 
 type fakeProducer struct {

producers/fake_test.go

@@ -63,11 +63,12 @@ func TestFakeEndpointsResolveToHostnamesInHostnameMode(t *testing.T) {
 }
 
 func TestNewFakeReadsConfigurationFromParams(t *testing.T) {
-	fakeParams.dnsName = "dnsName"
-	fakeParams.mode = "mode"
-	fakeParams.targetDomain = "targetDomain"
+	fakeParams := &FakeProducerOptions{}
+	fakeParams.DNSName = "dnsName"
+	fakeParams.Mode = "mode"
+	fakeParams.TargetDomain = "targetDomain"
 
-	producer, err := NewFake()
+	producer, err := NewFake(fakeParams)
 	if err != nil {
 		t.Fatal(err)
 	}