Fix attributes naming on fields and arguments for interfaces and unions (#806)

Additionally:
- revive macros/tests/object
- revive executor_tests/interfaces_unions

Author: tyranron

docs/book/content/types/interfaces.md

@@ -167,7 +167,7 @@ use juniper::{graphql_interface, GraphQLObject};
 trait Character {
     fn id(&self) -> &str;
 
-    #[graphql_interface(ignore)] // or `#[graphql_interface(skip)]`, your choice
+    #[graphql(ignore)] // or `#[graphql(skip)]`, your choice
     fn ignored(&self) -> u32 { 0 }
 }
 
@@ -205,26 +205,26 @@ use juniper::graphql_interface;
 /// This doc is absent in GraphQL schema.  
 trait Character {
     // Renames the field in GraphQL schema.
-    #[graphql_interface(name = "myId")]
+    #[graphql(name = "myId")]
     // Deprecates the field in GraphQL schema.
     // Usual Rust `#[deprecated]` attribute is supported too as field deprecation,
     // but `deprecated` attribute argument takes precedence over it, if specified.
-    #[graphql_interface(deprecated = "Do not use it.")]
+    #[graphql(deprecated = "Do not use it.")]
     // Describes the field in GraphQL schema.
-    #[graphql_interface(description = "ID of my own character.")]
+    #[graphql(description = "ID of my own character.")]
     // Usual Rust docs are supported too as field description, 
     // but `description` attribute argument takes precedence over them, if specified.
     /// This description is absent in GraphQL schema.  
     fn id(
         &self,
         // Renames the argument in GraphQL schema.
-        #[graphql_interface(name = "myNum")]
+        #[graphql(name = "myNum")]
         // Describes the argument in GraphQL schema.
-        #[graphql_interface(description = "ID number of my own character.")]
+        #[graphql(description = "ID number of my own character.")]
         // Specifies the default value for the argument.
         // The concrete value may be omitted, and the `Default::default` one 
         // will be used in such case.
-        #[graphql_interface(default = 5)]
+        #[graphql(default = 5)]
         num: i32,
     ) -> &str;
 }
@@ -254,7 +254,7 @@ trait Character {                 // while still can be specified via `Context =
     fn id(&self, context: &Database) -> Option<&str>;
 
     // Otherwise, you may mark it explicitly as a context argument.
-    fn name(&self, #[graphql_interface(context)] db: &Database) -> Option<&str>;
+    fn name(&self, #[graphql(context)] db: &Database) -> Option<&str>;
 }
 
 #[derive(GraphQLObject)]
@@ -307,7 +307,7 @@ trait Character<S: ScalarValue> {
     // Otherwise, you may mark it explicitly as an executor argument.
     async fn name<'b>(
         &'b self,
-        #[graphql_interface(executor)] another: &Executor<'_, '_, (), S>,
+        #[graphql(executor)] another: &Executor<'_, '_, (), S>,
     ) -> &'b str
     where
         S: Send + Sync;
@@ -319,7 +319,7 @@ struct Human {
     id: String,
     name: String,
 }
-#[graphql_interface(Scalar = S)]
+#[graphql_interface(scalar = S)]
 impl<S: ScalarValue> Character<S> for Human {
     async fn id<'a>(&self, executor: &'a Executor<'_, '_, (), S>) -> &'a str
     where
@@ -356,12 +356,12 @@ struct Database {
 }
 impl juniper::Context for Database {}
 
-#[graphql_interface(for = [Human, Droid], Context = Database)]
+#[graphql_interface(for = [Human, Droid], context = Database)]
 #[graphql_interface(on Droid = get_droid)] // enables downcasting `Droid` via `get_droid()` function
 trait Character {
     fn id(&self) -> &str;
 
-    #[graphql_interface(downcast)] // makes method a downcast to `Human`, not a field 
+    #[graphql(downcast)] // makes method a downcast to `Human`, not a field 
     // NOTICE: The method signature may optionally contain `&Database` context argument.
     fn as_human(&self) -> Option<&Human> {
         None
@@ -419,7 +419,7 @@ By default, `#[graphql_interface]` macro generates code, which is generic over a
 use juniper::{graphql_interface, DefaultScalarValue, GraphQLObject};
 
 #[graphql_interface(for = [Human, Droid])]
-#[graphql_interface(Scalar = DefaultScalarValue)] // removing this line will fail compilation
+#[graphql_interface(scalar = DefaultScalarValue)] // removing this line will fail compilation
 trait Character {
     fn id(&self) -> &str;
 }
@@ -430,7 +430,7 @@ struct Human {
     id: String,
     home_planet: String,
 }
-#[graphql_interface(Scalar = DefaultScalarValue)]
+#[graphql_interface(scalar = DefaultScalarValue)]
 impl Character for Human {
     fn id(&self) -> &str {
         &self.id
@@ -443,7 +443,7 @@ struct Droid {
     id: String,
     primary_function: String,
 }
-#[graphql_interface(Scalar = DefaultScalarValue)]
+#[graphql_interface(scalar = DefaultScalarValue)]
 impl Character for Droid {
     fn id(&self) -> &str {
         &self.id

docs/book/content/types/unions.md

@@ -316,7 +316,7 @@ struct Database {
 }
 impl juniper::Context for Database {}
 
-#[graphql_union(Context = Database)]
+#[graphql_union(context = Database)]
 trait Character {
     // NOTICE: The method signature may optionally contain `&Context`.
     fn as_human<'db>(&self, ctx: &'db Database) -> Option<&'db Human> { None }
@@ -363,7 +363,7 @@ struct Droid {
 trait Character {
     fn as_human(&self) -> Option<&Human> { None }
     fn as_droid(&self) -> Option<&Droid> { None }
-    #[graphql_union(ignore)]  // or `#[graphql_union(skip)]`, your choice
+    #[graphql(ignore)]  // or `#[graphql(skip)]`, your choice
     fn id(&self) -> &str;
 }
 
@@ -410,13 +410,13 @@ struct Database {
 }
 impl juniper::Context for Database {}
 
-#[graphql_union(Context = Database)]
+#[graphql_union(context = Database)]
 #[graphql_union(
     on Human = DynCharacter::get_human,
     on Droid = get_droid,
 )]
 trait Character {
-    #[graphql_union(ignore)]  // or `#[graphql_union(skip)]`, your choice
+    #[graphql(ignore)]  // or `#[graphql(skip)]`, your choice
     fn id(&self) -> &str;
 }
 

integration_tests/codegen_fail/fail/interface/downcast_method_conflicts_with_external_downcast_fn.rs

@@ -22,7 +22,7 @@ impl Character for ObjA {
 trait Character {
     fn id(&self, num: i32) -> &str;
 
-    #[graphql_interface(downcast)]
+    #[graphql(downcast)]
     fn as_obja(&self) -> Option<&ObjA>;
 }
 

integration_tests/codegen_fail/fail/interface/downcast_method_conflicts_with_external_downcast_fn.stderr

@@ -5,7 +5,7 @@ error: GraphQL interface trait method `as_obja` conflicts with the external down
    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
    = note: https://spec.graphql.org/June2018/#sec-Interfaces
-   = note: use `#[graphql_interface(ignore)]` attribute argument to ignore this trait method for interface implementers downcasting
+   = note: use `#[graphql(ignore)]` attribute argument to ignore this trait method for interface implementers downcasting
 
 error[E0412]: cannot find type `CharacterValue` in this scope
  --> $DIR/downcast_method_conflicts_with_external_downcast_fn.rs:4:18

integration_tests/codegen_fail/fail/interface/downcast_method_wrong_input_args.rs

@@ -6,7 +6,7 @@ trait Character {
         0
     }
 
-    #[graphql_interface(downcast)]
+    #[graphql(downcast)]
     fn a(&self, ctx: &(), rand: u8) -> Option<&Human> {
         None
     }

integration_tests/codegen_fail/fail/interface/downcast_method_wrong_return_type.rs

@@ -6,7 +6,7 @@ trait Character {
         0
     }
 
-    #[graphql_interface(downcast)]
+    #[graphql(downcast)]
     fn a(&self, ctx: &(), rand: u8) -> &Human {
         unimplemented!()
     }

integration_tests/codegen_fail/fail/interface/fields_duplicate.rs

@@ -15,7 +15,7 @@ trait Character {
         "funA"
     }
 
-    #[graphql_interface(name = "id")]
+    #[graphql(name = "id")]
     fn id2(&self) -> &str {
         "funB"
     }

integration_tests/codegen_fail/fail/union/trait_method_conflicts_with_external_resolver_fn.stderr

@@ -5,4 +5,4 @@ error: GraphQL union trait method `a` conflicts with the external resolver funct
   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   |
   = note: https://spec.graphql.org/June2018/#sec-Unions
-  = note: use `#[graphql_union(ignore)]` attribute to ignore this trait method for union variants resolution
+  = note: use `#[graphql(ignore)]` attribute to ignore this trait method for union variants resolution

integration_tests/codegen_fail/fail/union/trait_with_attr_on_method.rs

@@ -2,7 +2,7 @@ use juniper::{graphql_union, GraphQLObject};
 
 #[graphql_union]
 trait Character {
-    #[graphql_union(with = something)]
+    #[graphql(with = something)]
     fn a(&self) -> Option<&Human>;
 }
 

integration_tests/codegen_fail/fail/union/trait_with_attr_on_method.stderr

@@ -1,8 +1,8 @@
-error: GraphQL union cannot use #[graphql_union(with = ...)] attribute on a trait method
- --> $DIR/trait_with_attr_on_method.rs:5:21
+error: GraphQL union cannot use #[graphql(with = ...)] attribute on a trait method
+ --> $DIR/trait_with_attr_on_method.rs:5:15
   |
-5 |     #[graphql_union(with = something)]
-  |                     ^^^^
+5 |     #[graphql(with = something)]
+  |               ^^^^
   |
   = note: https://spec.graphql.org/June2018/#sec-Unions
-  = note: instead use #[graphql_union(ignore)] on the method with #[graphql_union(on ... = ...)] on the trait itself
+  = note: instead use #[graphql(ignore)] on the method with #[graphql_union(on ... = ...)] on the trait itself