luxolus
/
open-nomad
2
0
Fork 0
Code Issues 1 Pull Requests Packages Releases Wiki Activity

nomad: fix leftover missed refactoring in consul policy checking

This commit is contained in:
Seth Hoenig 2020-01-15 13:08:10 -06:00
parent 057f117592
commit ee89a754f1
2 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
nomad/consul_policy.go
View File

@ -94,7 +94,7 @@ func (c *consulACLsAPI) policyAllowsServiceWrite(task string, policyID string) (
return false, err return false, err
} }
if c.allowsServiceWrite(task, cp) { if cp.allowsServiceWrite(task) {
return true, nil return true, nil
} }

3
nomad/consul_policy_test.go
View File

@ -94,8 +94,7 @@ func TestConsulACLsAPI_allowsServiceWrite(t *testing.T) {
t.Parallel() t.Parallel()
try := func(t *testing.T, task string, cp *ConsulPolicy, exp bool) { try := func(t *testing.T, task string, cp *ConsulPolicy, exp bool) {
cAPI := new(consulACLsAPI) result := cp.allowsServiceWrite(task)
result := cAPI.allowsServiceWrite(task, cp)
require.Equal(t, exp, result) require.Equal(t, exp, result)
} }