luxolus/open-consul
2
0
Fork 0
Code Issues 1 Pull requests Packages Releases Wiki Activity

Fix broken api test for service Meta (logical conflict rom OSS). Add test that would make this much easier to catch in future.

Browse source
This commit is contained in:
Paul Banks 2018-05-12 11:27:44 +01:00 committed by Mitchell Hashimoto
parent bd5eb8b749
commit 73f2a49ef1
No known key found for this signature in database
GPG key ID: 744E147AA52F5B0A
2 changed files with 6 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
agent/agent_endpoint.go
View file

@ -175,6 +175,7 @@ func (s *HTTPServer) AgentServices(resp http.ResponseWriter, req *http.Request)
ID: s.ID,
Service: s.Service,
Tags: s.Tags,
Meta: s.Meta,
Port: s.Port,
Address: s.Address,
EnableTagOverride: s.EnableTagOverride,

6
agent/agent_endpoint_test.go
View file

@ -56,7 +56,10 @@ func TestAgent_Services(t *testing.T) {
ID: "mysql",
Service: "mysql",
Tags: []string{"master"},
Port: 5000,
Meta: map[string]string{
"foo": "bar",
},
Port: 5000,
}
require.NoError(t, a.State.AddService(srv1, ""))
@ -81,6 +84,7 @@ func TestAgent_Services(t *testing.T) {
val := obj.(map[string]*api.AgentService)
assert.Lenf(t, val, 1, "bad services: %v", obj)
assert.Equal(t, 5000, val["mysql"].Port)
assert.Equal(t, srv1.Meta, val["mysql"].Meta)
assert.NotNil(t, val["mysql"].Connect)
assert.NotNil(t, val["mysql"].Connect.Proxy)
assert.Equal(t, prxy1.ExecMode.String(), string(val["mysql"].Connect.Proxy.ExecMode))