Skip to content

Commit

Permalink
Fixup ACL spec
Browse files Browse the repository at this point in the history
Due to an bug in our rspec matcher, we weren't correctly checking the
results of various ACL tests. This corrects the test to match what we
actually return, as the actual permissions seem sane and reasonable.
  • Loading branch information
markan committed Mar 14, 2017
1 parent 0965712 commit 55c643e
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions oc-chef-pedant/spec/api/account/account_acl_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -931,6 +931,8 @@ def rand_id
}
}
}}
let(:groups) { ["users", "admins"] }
let(:read_groups) { ["users", "clients", "admins"] }
when "policy_groups"
let(:creation_url) {
api_url("#{type}/#{new_object}/policies/acl_test_policy")
Expand All @@ -946,6 +948,8 @@ def rand_id
}
}
}}
let(:groups) { ["users", "admins"] }
let(:read_groups) { ["users", "clients", "admins"] }
end

before :each do
Expand Down

0 comments on commit 55c643e

Please sign in to comment.