Project

General

Profile

« Previous | Next » 

Revision 2dec184c

Added by Eric Helms over 8 years ago

Fixes #11303: Allow scoped search by name for permissions.

This both provides this functionality and fixes an issue with creating
filters using hammer. Hammer attempts to search for a permission by
name using scoped search syntax when creating a filter with the
--permission=view_domains style syntax. However, this was defaulting to
returning all permissions instead of scoped permissions.

View differences:

test/unit/permission_test.rb
Permission.stubs(:with_translations).returns([['Z', 'z'], ['A', 'b'], ['H', 'a']])
assert_equal [['A', 'b'], ['H', 'a'], ['Z', 'z']], Permission.resources_with_translations
end
test "can search permissions by name" do
permission = FactoryGirl.create :permission, :domain, :name => 'view_all_domains'
as_admin do
permissions = Permission.search_for('name = view_all_domains')
assert_includes permissions, permission
end
end
test "can search permissions by resource_type" do
permission = FactoryGirl.create :permission, :domain, :name => 'view_all_domains'
as_admin do
permissions = Permission.search_for('resource_type = Domain')
assert_includes permissions, permission
end
end
end

Also available in: Unified diff