Skip to content

Commit

Permalink
make query validation check variables #major
Browse files Browse the repository at this point in the history
QueryValidator needs access to variables now, so this causes a breaking change.
  • Loading branch information
Amanda Steinwedel committed Jul 19, 2024
1 parent cf734e4 commit d0c0e9d
Show file tree
Hide file tree
Showing 15 changed files with 609 additions and 232 deletions.
6 changes: 5 additions & 1 deletion build.sbt
Original file line number Diff line number Diff line change
Expand Up @@ -195,7 +195,11 @@ lazy val core = project
ProblemFilters.exclude[IncompatibleResultTypeProblem](
"sangria.schema.WithInputTypeRendering.deprecationTracker"),
ProblemFilters.exclude[ReversedMissingMethodProblem](
"sangria.schema.WithInputTypeRendering.deprecationTracker")
"sangria.schema.WithInputTypeRendering.deprecationTracker"),
ProblemFilters.exclude[DirectMissingMethodProblem](
"sangria.validation.RuleBasedQueryValidator.validateInputDocument"),
ProblemFilters.exclude[DirectMissingMethodProblem](
"sangria.validation.RuleBasedQueryValidator.validateInputDocument")
),
Test / testOptions += Tests.Argument(TestFrameworks.ScalaTest, "-oF"),
libraryDependencies ++= Seq(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,7 @@ class OverlappingFieldsCanBeMergedBenchmark {
bh.consume(doValidate(validator, deepAbstractConcrete))

private def doValidate(validator: QueryValidator, document: Document): Vector[Violation] = {
val result = validator.validateQuery(schema, document, None)
val result = validator.validateQuery(schema, document, Map.empty, None)
require(result.isEmpty)
result
}
Expand Down
Loading

0 comments on commit d0c0e9d

Please sign in to comment.