Upd code

Author: tyranron

juniper/src/http/mod.rs

@@ -164,27 +164,35 @@ where
 /// whether to send a 200 or 400 HTTP status code.
 #[derive(Clone, Debug, PartialEq)]
 pub struct GraphQLResponse<S = DefaultScalarValue>(
-    pub Result<(Value<S>, Vec<ExecutionError<S>>), GraphQLError>,
+    Result<(Value<S>, Vec<ExecutionError<S>>), GraphQLError>,
 );
 
 impl<S> GraphQLResponse<S>
 where
     S: ScalarValue,
 {
-    /// Constructs new `GraphQLResponse` using the given result
+    /// Constructs a new [`GraphQLResponse`] from the provided execution [`Result`].
+    #[must_use]
     pub fn from_result(r: Result<(Value<S>, Vec<ExecutionError<S>>), GraphQLError>) -> Self {
         Self(r)
     }
 
-    /// Constructs an error response outside of the normal execution flow
+    /// Unwraps this [`GraphQLResponse`] into its underlying execution [`Result`].
+    pub fn into_result(self) -> Result<(Value<S>, Vec<ExecutionError<S>>), GraphQLError> {
+        self.0
+    }
+
+    /// Constructs an error [`GraphQLResponse`] outside the normal execution flow.
+    #[must_use]
     pub fn error(error: FieldError<S>) -> Self {
-        GraphQLResponse(Ok((Value::null(), vec![ExecutionError::at_origin(error)])))
+        Self(Ok((Value::null(), vec![ExecutionError::at_origin(error)])))
     }
 
-    /// Was the request successful or not?
+    /// Indicates whether this [`GraphQLResponse`] contains a successful execution [`Result`].
     ///
-    /// Note that there still might be errors in the response even though it's
-    /// considered OK. This is by design in GraphQL.
+    /// **NOTE**: There still might be errors in the response even though it's considered OK.
+    ///           This is by design in GraphQL.
+    #[must_use]
     pub fn is_ok(&self) -> bool {
         self.0.is_ok()
     }