From b0a597ec8f5ae8da95d60177a82a10cf25124c3c Mon Sep 17 00:00:00 2001 From: Brian Jing Date: Fri, 15 Oct 2021 16:22:21 +1100 Subject: [PATCH] Do not force QueryValidator to use List --- .../main/scala/sangria/validation/QueryValidator.scala | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/modules/core/src/main/scala/sangria/validation/QueryValidator.scala b/modules/core/src/main/scala/sangria/validation/QueryValidator.scala index 0934fe48..bd09f5e6 100644 --- a/modules/core/src/main/scala/sangria/validation/QueryValidator.scala +++ b/modules/core/src/main/scala/sangria/validation/QueryValidator.scala @@ -16,7 +16,7 @@ trait QueryValidator { } object QueryValidator { - val allRules: List[ValidationRule] = List( + val allRules: Seq[ValidationRule] = Seq( new ValuesOfCorrectType, new ExecutableDefinitions, new FieldsOnCorrectType, @@ -46,7 +46,7 @@ object QueryValidator { new SingleFieldSubscriptions ) - def ruleBased(rules: List[ValidationRule]) = new RuleBasedQueryValidator(rules) + def ruleBased(rules: Seq[ValidationRule]) = new RuleBasedQueryValidator(rules) val empty = new QueryValidator { def validateQuery(schema: Schema[_, _], queryAst: ast.Document): Vector[Violation] = @@ -56,7 +56,7 @@ object QueryValidator { val default: RuleBasedQueryValidator = ruleBased(allRules) } -class RuleBasedQueryValidator(rules: List[ValidationRule]) extends QueryValidator { +class RuleBasedQueryValidator(rules: Seq[ValidationRule]) extends QueryValidator { def validateQuery(schema: Schema[_, _], queryAst: ast.Document): Vector[Violation] = { val ctx = new ValidationContext(schema, queryAst, queryAst.sourceMapper, new TypeInfo(schema)) @@ -93,7 +93,7 @@ class RuleBasedQueryValidator(rules: List[ValidationRule]) extends QueryValidato def validateUsingRules( queryAst: ast.AstNode, ctx: ValidationContext, - visitors: List[ValidationRule#AstValidatingVisitor], + visitors: Iterable[ValidationRule#AstValidatingVisitor], topLevel: Boolean): Unit = AstVisitor.visitAstRecursive( doc = queryAst, onEnter = node => {