Add more tests of actual schemas to cynic-parser

cynic-parser/Cargo.toml

@@ -23,6 +23,7 @@ pretty = { version = "0.12", optional = true }
 criterion = "0.4"
 graphql-parser = "0.4"
 insta = "1.29"
+paste = "1"
 similar-asserts = "1.5"
 
 # Tests need the `print` functionality so enable it here

cynic-parser/src/errors/report.rs

@@ -45,7 +45,7 @@ impl Error {
  expected,
  } => (
  format!("unexpected {}", token),
- Label::new(*start..*end).with_message("didn't expect to see this"),
+ Label::new(*dbg!(start)..*dbg!(end)).with_message("didn't expect to see this"),
  Some(format!("expected one of {}", expected.join(", "))),
  ),
  Error::ExtraToken {

cynic-parser/src/printer.rs

@@ -460,16 +460,17 @@ impl<'a> Pretty<'a, BoxAllocator> for NodeDisplay<DirectiveDefinition<'a>> {
  .parens()
  .flat_alt(arguments.parens());
 
- builder = builder.append(arguments).append(allocator.space())
+ builder = builder.append(arguments)
  }
 
  if self.0.is_repeatable() {
  builder = builder
- .append(allocator.text("repeatable"))
  .append(allocator.space())
+ .append(allocator.text("repeatable"))
  }
 
  builder
+ .append(allocator.space())
  .append(allocator.text("on"))
  .append(allocator.space())
  .append(allocator.intersperse(

cynic-parser/tests/actual_schemas.rs

@@ -0,0 +1,46 @@
+macro_rules! schema_tests {
+ ($name: ident, $path: literal) => {
+ mod $name {
+ use similar_asserts::assert_eq;
+
+ #[allow(non_upper_case_globals)]
+ const SCHEMA: &str = include_str!($path);
+
+ #[test]
+ fn snapshot() {
+ let parsed = cynic_parser::parse_type_system_document(SCHEMA)
+ .map_err(|error| error.to_report(SCHEMA))
+ .unwrap();
+ insta::assert_snapshot!(parsed.to_sdl());
+ }
+
+ #[test]
+ fn double_roundtrip() {
+ let parsed = cynic_parser::parse_type_system_document(SCHEMA)
+ .map_err(|error| error.to_report(SCHEMA))
+ .unwrap();
+ let sdl = parsed.to_sdl();
+
+ let reparsed = cynic_parser::parse_type_system_document(&sdl)
+ .map_err(|error| error.to_report(&sdl))
+ .unwrap();
+
+ assert_eq!(sdl, reparsed.to_sdl());
+ }
+ }
+ };
+}
+
+schema_tests!(github, "../../schemas/github.graphql");
+
+schema_tests!(books, "../../schemas/books.graphql");
+
+schema_tests!(graphql_jobs, "../../schemas/graphql.jobs.graphql");
+
+schema_tests!(raindancer, "../../schemas/raindancer.graphql");
+
+schema_tests!(simple, "../../schemas/simple.graphql");
+
+schema_tests!(starwars, "../../schemas/starwars.schema.graphql");
+
+schema_tests!(test_cases, "../../schemas/test_cases.graphql");

cynic-parser/tests/snapshots/actual_schemas__books__snapshot.snap

@@ -0,0 +1,43 @@
+---
+source: cynic-parser/tests/actual_schemas.rs
+expression: parsed.to_sdl()
+---
+schema {
+ query: QueryRoot
+ mutation: MutationRoot
+ subscription: SubscriptionRoot
+}
+
+directive @ifdef on FIELD
+
+type Book {
+ id: String!
+ name: String!
+ author: String!
+}
+
+type BookChanged {
+ mutationType: MutationType!
+ id: ID!
+ book: Book
+}
+
+type MutationRoot {
+ createBook(name: String!, author: String!): ID!
+ deleteBook(id: ID!): Boolean!
+}
+
+enum MutationType {
+ CREATED
+ DELETED
+}
+
+type QueryRoot {
+ books: [Book!]!
+}
+
+type SubscriptionRoot {
+ interval(n: Int! = 1): Int!
+ books(mutationType: MutationType): BookChanged!
+}
+