feat: refactor database locking model (#321)

* refactor database locking model

* fix prepare on strat up

* put DatabaseConnectionProvider behind test-utils feature flag

* Revert "put DatabaseConnectionProvider behind test-utils feature flag"

This reverts commit 9b0ed39.

---------

Co-authored-by: greg <[email protected]>

Author: frisitano

Cargo.lock

@@ -26,7 +26,10 @@ rollup-node-primitives.workspace = true
 async-trait.workspace = true
 auto_impl.workspace = true
 futures.workspace = true
+metrics.workspace = true
+metrics-derive.workspace = true
 sea-orm = { workspace = true, features = ["sqlx-sqlite", "runtime-tokio-native-tls", "macros"] }
+tempfile = { version = "3.20.0", optional = true }
 thiserror.workspace = true
 tokio = { workspace = true, features = ["macros", "sync"] }
 tracing.workspace = true
@@ -41,4 +44,4 @@ arbitrary.workspace = true
 rand.workspace = true
 
 [features]
-test-utils = ["dep:scroll-migration"]
+test-utils = ["dep:scroll-migration", "dep:tempfile"]

crates/chain-orchestrator/src/lib.rs

@@ -1,5 +1,4 @@
 /// The [`DatabaseConnectionProvider`] trait provides a way to get a connection to the database.
-/// This is implemented by the [`crate::Database`] and [`crate::DatabaseTransaction`] types.
 #[auto_impl::auto_impl(Arc)]
 pub trait DatabaseConnectionProvider {
     /// The type of the database connection.
@@ -9,3 +8,9 @@ pub trait DatabaseConnectionProvider {
     /// `StreamTrait` traits.
     fn get_connection(&self) -> &Self::Connection;
 }
+
+/// A trait for providing read-only access to the database.
+pub trait ReadConnectionProvider: DatabaseConnectionProvider {}
+
+/// A trait for providing read and write access to the database.
+pub trait WriteConnectionProvider: ReadConnectionProvider {}