From c05545775521f094d96bdf827aba720823889618 Mon Sep 17 00:00:00 2001 From: Dotan Simha Date: Tue, 14 Dec 2021 18:06:23 +0200 Subject: [PATCH] added more rules --- Cargo.lock | 4 ++-- graph/Cargo.toml | 2 +- graphql/Cargo.toml | 2 +- graphql/src/runner.rs | 5 ++++- 4 files changed, 8 insertions(+), 5 deletions(-) diff --git a/Cargo.lock b/Cargo.lock index ca579691042..854f90201c3 100644 --- a/Cargo.lock +++ b/Cargo.lock @@ -1857,9 +1857,9 @@ dependencies = [ [[package]] name = "graphql-tools" -version = "0.0.5" +version = "0.0.6" source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "7e05409e7d4dc676134c13a187506100b3ceb1d32690e3841091ff7134589e4f" +checksum = "29ddc6865d21d4b832729812edc8df50a1688b8dd63a9ea7ea548e354b83eee7" dependencies = [ "graphql-parser", ] diff --git a/graph/Cargo.toml b/graph/Cargo.toml index 1d3e7d1cc93..2e928a3c7bd 100644 --- a/graph/Cargo.toml +++ b/graph/Cargo.toml @@ -21,7 +21,7 @@ http = "0.2.3" fail = { version = "0.5", features = ["failpoints"] } futures = "0.1.21" graphql-parser = "0.4.0" -graphql-tools = "0.0.5" +graphql-tools = "0.0.6" lazy_static = "1.4.0" mockall = "0.8.3" num-bigint = { version = "^0.2.6", features = ["serde"] } diff --git a/graphql/Cargo.toml b/graphql/Cargo.toml index 747558cadf6..b486271984a 100644 --- a/graphql/Cargo.toml +++ b/graphql/Cargo.toml @@ -8,7 +8,7 @@ crossbeam = "0.8" futures01 = { package="futures", version="0.1.29" } graph = { path = "../graph" } graphql-parser = "0.4.0" -graphql-tools = "0.0.5" +graphql-tools = "0.0.6" indexmap = "1.7" Inflector = "0.11.3" lazy_static = "1.2.0" diff --git a/graphql/src/runner.rs b/graphql/src/runner.rs index f86dd755eb5..2931081d750 100644 --- a/graphql/src/runner.rs +++ b/graphql/src/runner.rs @@ -22,7 +22,8 @@ use graph::{ prelude::QueryStore, }; use graphql_tools::validation::rules::{ - FragmentsOnCompositeTypes, LoneAnonymousOperation, OverlappingFieldsCanBeMerged, + FragmentsOnCompositeTypes, KnownFragmentNamesRule, LoneAnonymousOperation, NoUnusedFragments, + OverlappingFieldsCanBeMerged, }; use graphql_tools::validation::validate::{validate, ValidationPlan}; @@ -145,6 +146,8 @@ where graphql_validation_plan.add_rule(Box::new(LoneAnonymousOperation {})); graphql_validation_plan.add_rule(Box::new(FragmentsOnCompositeTypes {})); graphql_validation_plan.add_rule(Box::new(OverlappingFieldsCanBeMerged {})); + graphql_validation_plan.add_rule(Box::new(KnownFragmentNamesRule {})); + graphql_validation_plan.add_rule(Box::new(NoUnusedFragments {})); GraphQlRunner { logger,