{"action":"created","issue":{"url":"https://api.github.com/repos/solidusio/solidus/issues/449","labels_url":"https://api.github.com/repos/solidusio/solidus/issues/449/labels{/name}","comments_url":"https://api.github.com/repos/solidusio/solidus/issues/449/comments","events_url":"https://api.github.com/repos/solidusio/solidus/issues/449/events","html_url":"https://github.com/solidusio/solidus/pull/449","id":111855493,"number":449,"title":"Clean up authorization test helpers.","user":{"login":"athal7","id":467872,"avatar_url":"https://avatars.githubusercontent.com/u/467872?v=3","gravatar_id":"","url":"https://api.github.com/users/athal7","html_url":"https://github.com/athal7","followers_url":"https://api.github.com/users/athal7/followers","following_url":"https://api.github.com/users/athal7/following{/other_user}","gists_url":"https://api.github.com/users/athal7/gists{/gist_id}","starred_url":"https://api.github.com/users/athal7/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/athal7/subscriptions","organizations_url":"https://api.github.com/users/athal7/orgs","repos_url":"https://api.github.com/users/athal7/repos","events_url":"https://api.github.com/users/athal7/events{/privacy}","received_events_url":"https://api.github.com/users/athal7/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":18,"created_at":"2015-10-16T15:33:01Z","updated_at":"2015-10-29T22:53:01Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/solidusio/solidus/pulls/449","html_url":"https://github.com/solidusio/solidus/pull/449","diff_url":"https://github.com/solidusio/solidus/pull/449.diff","patch_url":"https://github.com/solidusio/solidus/pull/449.patch"},"body":"The current way that authorization is stubbed on controller and feature\r\nspecs only allows non-superuser configuration through anonymous blocks.\r\nThe addition of permission sets gives us a type of user that we\r\nwant to assert against, but we don't yet have a great way of testing the\r\nabilities of that type of user at the controller / feature level, plus\r\nour anonymous authorization changes might not be a great representation\r\nof the behavior we are trying to test.\r\n\r\nThis change simplifies the authorization helper code, \r\nand allows for the testing of permission sets at a controller level more easily."},"comment":{"url":"https://api.github.com/repos/solidusio/solidus/issues/comments/152350087","html_url":"https://github.com/solidusio/solidus/pull/449#issuecomment-152350087","issue_url":"https://api.github.com/repos/solidusio/solidus/issues/449","id":152350087,"user":{"login":"mbj","id":117966,"avatar_url":"https://avatars.githubusercontent.com/u/117966?v=3","gravatar_id":"","url":"https://api.github.com/users/mbj","html_url":"https://github.com/mbj","followers_url":"https://api.github.com/users/mbj/followers","following_url":"https://api.github.com/users/mbj/following{/other_user}","gists_url":"https://api.github.com/users/mbj/gists{/gist_id}","starred_url":"https://api.github.com/users/mbj/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mbj/subscriptions","organizations_url":"https://api.github.com/users/mbj/orgs","repos_url":"https://api.github.com/users/mbj/repos","events_url":"https://api.github.com/users/mbj/events{/privacy}","received_events_url":"https://api.github.com/users/mbj/received_events","type":"User","site_admin":false},"created_at":"2015-10-29T22:53:01Z","updated_at":"2015-10-29T22:53:01Z","body":"@athal7 You are referencing: https://github.com/solidusio/solidus/pull/467#issuecomment-152281968 in particular? If so. +1."}}