Merge pull request #32 from sshaw/master

More descriptive error message for NotAuthorizedError
exAspArk · Oct 24, 2019 · 83e08a7 · 83e08a7
2 parents 807bcbb + 034891f
commit 83e08a7
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/lib/graphql/guard.rb b/lib/graphql/guard.rb
@@ -6,7 +6,7 @@
 module GraphQL
   class Guard
     ANY_FIELD_NAME = :'*'
-    DEFAULT_NOT_AUTHORIZED = ->(type, field) { raise NotAuthorizedError.new("#{type}.#{field}") }
+    DEFAULT_NOT_AUTHORIZED = ->(type, field) { raise NotAuthorizedError.new("Not authorized to access: #{type}.#{field}") }
 
     NotAuthorizedError = Class.new(StandardError)
 

diff --git a/spec/graphql/guard_spec.rb b/spec/graphql/guard_spec.rb
@@ -24,7 +24,7 @@
 
       expect {
         Inline::Schema.execute(query, variables: {'userId' => '2'}, context: {current_user: user})
-      }.to raise_error(GraphQL::Guard::NotAuthorizedError, 'Query.posts')
+      }.to raise_error(GraphQL::Guard::NotAuthorizedError, 'Not authorized to access: Query.posts')
     end
 
     it 'does not authorize a field with a policy on the type' do
@@ -33,7 +33,7 @@
 
       expect {
         Inline::Schema.execute(query, variables: {'userId' => '1'}, context: {current_user: user})
-      }.to raise_error(GraphQL::Guard::NotAuthorizedError, 'Post.id')
+      }.to raise_error(GraphQL::Guard::NotAuthorizedError, 'Not authorized to access: Post.id')
     end
 
     it 'does not authorize a field and returns an error' do
@@ -91,7 +91,7 @@
 
       expect {
         PolicyObject::Schema.execute(query, variables: {'userId' => '2'}, context: {current_user: user})
-      }.to raise_error(GraphQL::Guard::NotAuthorizedError, 'Query.posts')
+      }.to raise_error(GraphQL::Guard::NotAuthorizedError, 'Not authorized to access: Query.posts')
     end
 
     it 'does not authorize a field with a policy on the type' do
@@ -100,7 +100,7 @@
 
       expect {
         PolicyObject::Schema.execute(query, variables: {'userId' => '1'}, context: {current_user: user})
-      }.to raise_error(GraphQL::Guard::NotAuthorizedError, 'Post.id')
+      }.to raise_error(GraphQL::Guard::NotAuthorizedError, 'Not authorized to access: Post.id')
     end
   end
 end