Skip to content

Commit 720eab9

Browse files
author
Robert Mosolgo
authored
Merge pull request rmosolgo#3690 from mattpage/1.11.x
use validation_max_errors in validation pipeline
2 parents 816e14d + b0a25a0 commit 720eab9

File tree

2 files changed

+8
-1
lines changed

2 files changed

+8
-1
lines changed

lib/graphql/query/validation_pipeline.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ def ensure_has_validated
7272
elsif @operation_name_error
7373
@validation_errors << @operation_name_error
7474
else
75-
validation_result = @schema.static_validator.validate(@query, validate: @validate, timeout: @schema.validate_timeout)
75+
validation_result = @schema.static_validator.validate(@query, validate: @validate, timeout: @schema.validate_timeout, max_errors: @schema.validate_max_errors)
7676
@validation_errors.concat(validation_result[:errors])
7777
@internal_representation = validation_result[:irep]
7878

spec/graphql/query_spec.rb

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,13 @@
4242
)}
4343
let(:result) { query.result }
4444

45+
it "applies the max validation errors config" do
46+
limited_schema = Class.new(schema) { validate_max_errors(2) }
47+
res = limited_schema.execute("{ a b c d }")
48+
assert_equal 2, res["errors"].size
49+
refute res.key?("data")
50+
end
51+
4552
describe "when passed both a query string and a document" do
4653
it "returns an error to the client when query kwarg is used" do
4754
assert_raises ArgumentError do

0 commit comments

Comments
 (0)