Fixed outdated dummy

Author: gammelalf

src/dummy_impl/database.rs

@@ -1,12 +1,11 @@
 use rorm_sql::value::Value;
 
+use super::{no_sqlx, NotInstantiable};
 use crate::database::{Database, DatabaseConfiguration};
 use crate::error::Error;
 use crate::row::Row;
 use crate::transaction::Transaction;
 
-use super::{no_sqlx, NotInstantiable};
-
 pub(crate) type Impl = NotInstantiable;
 
 /// Implementation of [Database::connect]
@@ -19,7 +18,7 @@ pub async fn raw_sql<'a>(
     db: &Database,
     _query_string: &'a str,
     _bind_params: Option<&[Value<'a>]>,
-    _transaction: Option<&mut Transaction<'_>>,
+    _transaction: Option<&mut Transaction>,
 ) -> Result<Vec<Row>, Error> {
     // "Read" pool at least once
     let _ = db.pool;

src/dummy_impl/executor.rs

@@ -12,7 +12,7 @@ use crate::executor::{
     AffectedRows, All, Executor, Nothing, One, Optional, QueryStrategy, QueryStrategyResult, Stream,
 };
 use crate::row::Row;
-use crate::transaction::Transaction;
+use crate::transaction::{Transaction, TransactionGuard};
 
 impl<'executor> Executor<'executor> for &'executor Database {
     fn execute<'data, 'result, Q>(
@@ -40,7 +40,7 @@ impl<'executor> Executor<'executor> for &'executor Database {
         no_sqlx();
     }
 }
-impl<'executor> Executor<'executor> for &'executor mut Transaction<'_> {
+impl<'executor> Executor<'executor> for &'executor mut Transaction {
     fn execute<'data, 'result, Q>(
         self,
         _query: String,

src/dummy_impl/transaction.rs

@@ -1,18 +1,17 @@
+use super::{no_sqlx, NotInstantiable};
 use crate::transaction::Transaction;
 use crate::Error;
 
-use super::{no_sqlx, NotInstantiable};
-
 pub(crate) type Impl = NotInstantiable;
 
 /// Implementation of [Transaction::commit]
-pub(crate) async fn commit(transaction: Transaction<'_>) -> Result<(), Error> {
+pub(crate) async fn commit(transaction: Transaction) -> Result<(), Error> {
     // "Read" tx at least once
     let _ = transaction.tx;
     no_sqlx();
 }
 
 /// Implementation of [Transaction::rollback]
-pub(crate) async fn rollback(_transaction: Transaction<'_>) -> Result<(), Error> {
+pub(crate) async fn rollback(_transaction: Transaction) -> Result<(), Error> {
     no_sqlx();
 }

src/error.rs

@@ -2,25 +2,24 @@
 
 use std::{error, fmt};
 
-#[cfg(feature = "sqlx")]
-use sqlx::Error as SqlxError;
-
 #[cfg(not(feature = "sqlx"))]
-const _: () = {
+mod sqlx {
     /// Represent all ways a method can fail within SQLx.
     ///
     /// I.e. not at all since sqlx isn't a dependency.
     #[derive(Debug)]
-    pub enum SqlxError {}
+    pub enum Error {}
 
-    impl error::Error for Error {}
+    impl std::error::Error for Error {}
 
-    impl fmt::Display for Error {
-        fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
+    impl std::fmt::Display for Error {
+        fn fmt(&self, _f: &mut std::fmt::Formatter) -> std::fmt::Result {
             unreachable!("You shouldn't be able to get an instance of an empty enum!");
         }
     }
-};
+}
+
+use sqlx::Error as SqlxError;
 
 /// Error type to simplify propagating different error types.
 #[derive(Debug)]