Merge pull request #105 from mathstuf/inline-fragment-fields

Improve fragment support

Author: tomhoule

graphql_client_codegen/src/shared.rs

@@ -132,11 +132,11 @@ pub(crate) fn response_fields_for_selection(
                 let type_name = Ident::new(&fragment.fragment_name, Span::call_site());
                 Ok(quote!{
                     #[serde(flatten)]
-                    #field_name: #type_name
+                    pub #field_name: #type_name
                 })
             }
             SelectionItem::InlineFragment(_) => {
-                Err(format_err!("inline fragment on object field"))?
+                Err(format_err!("unimplemented: inline fragment on object field"))?
             }
         })
         .filter(|x| match x {

tests/fragments.rs

@@ -0,0 +1,30 @@
+#[macro_use]
+extern crate graphql_client;
+extern crate serde;
+#[macro_use]
+extern crate serde_derive;
+#[macro_use]
+extern crate serde_json;
+
+#[derive(GraphQLQuery)]
+#[graphql(
+    query_path = "tests/fragments/query.graphql",
+    schema_path = "tests/fragments/schema.graphql"
+)]
+#[allow(dead_code)]
+struct FragmentReference;
+
+#[test]
+fn fragment_reference() {
+    let valid_response = json!({
+        "inFragment": "value",
+    });
+
+    let valid_fragment_reference =
+        serde_json::from_value::<fragment_reference::ResponseData>(valid_response).unwrap();
+
+    assert_eq!(
+        valid_fragment_reference.fragment_reference.in_fragment.unwrap(),
+        "value"
+    );
+}

tests/fragments/query.graphql

@@ -0,0 +1,7 @@
+fragment FragmentReference on QueryRoot {
+  inFragment
+}
+
+query FragmentReference {
+  ...FragmentReference
+}

tests/fragments/schema.graphql

@@ -0,0 +1,8 @@
+schema {
+  query: QueryRoot
+}
+
+type QueryRoot {
+  extra: String
+  inFragment: String
+}