refactor: Add Query::sql_display instead.

`Debug` for `Query` isn't a good idea.

Author: pravic

src/query.rs

@@ -1,6 +1,6 @@
 use hyper::{header::CONTENT_LENGTH, Method, Request};
 use serde::Deserialize;
-use std::fmt;
+use std::fmt::Display;
 use url::Url;
 
 use crate::headers::with_request_headers;
@@ -23,12 +23,6 @@ pub struct Query {
     sql: SqlBuilder,
 }
 
-impl fmt::Debug for Query {
-    fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
-        write!(f, "{sql}", sql = self.sql)
-    }
-}
-
 impl Query {
     pub(crate) fn new(client: &Client, template: &str) -> Self {
         Self {
@@ -37,6 +31,11 @@ impl Query {
         }
     }
 
+    /// Display SQL query as string.
+    pub fn sql_display(&self) -> &impl Display {
+        &self.sql
+    }
+
     /// Binds `value` to the next `?` in the query.
     ///
     /// The `value`, which must either implement [`Serialize`] or be an

tests/it/query.rs

@@ -221,7 +221,7 @@ async fn prints_query() {
 
     let q = client.query("SELECT ?fields FROM test WHERE a = ? AND b < ?");
     assert_eq!(
-        format!("{q:?}"),
+        format!("{}", q.sql_display()),
         "SELECT ?fields FROM test WHERE a = ? AND b < ?"
     );
 }