nomad: fix leftover missed refactoring in consul policy checking

hashicorp · Jan 24, 2020 · d88f25f · d88f25f
1 parent 28e8963
commit d88f25f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/nomad/consul_policy.go b/nomad/consul_policy.go
@@ -94,7 +94,7 @@ func (c *consulACLsAPI) policyAllowsServiceWrite(task string, policyID string) (
 		return false, err
 	}
 
-	if c.allowsServiceWrite(task, cp) {
+	if cp.allowsServiceWrite(task) {
 		return true, nil
 	}
 

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