Replace OnceLock with LazyLock

Resolve clippies

fpx-lib/src/api/handlers/spans.rs

@@ -7,7 +7,7 @@ use fpx_macros::ApiError;
 use http::StatusCode;
 use serde::{Deserialize, Serialize};
 use thiserror::Error;
-use tracing::{error, info};
+use tracing::error;
 
 #[tracing::instrument(skip_all)]
 pub async fn span_get_handler(

fpx-lib/src/data.rs

@@ -10,6 +10,7 @@ type Result<T, E = DbError> = anyhow::Result<T, E>;
 
 pub type BoxedStore = Arc<dyn Store>;
 
+#[derive(Clone, Default, Debug)]
 pub struct Transaction {}
 
 impl Transaction {

fpx-lib/src/data/fake_store.rs

@@ -1,28 +1,19 @@
 use super::{models, DbError, Result, Store, Transaction};
 use async_trait::async_trait;
 use std::sync::{Arc, RwLock};
-use tracing::info;
 
 /// A simple in-memory [`Store`] implementation. Currently only intended for
 /// proof of concept.
 ///
 /// This implementation does not provide any Transaction support, nor will it
 /// work concurrently.
-#[derive(Clone, Debug)]
+#[derive(Clone, Default, Debug)]
 pub struct FakeStore {
     /// Spans are stored in a [`RwLock`] so that we can always mutate the inner
     /// [`Vec`], even with a reference to this [`FakeStore`].
     spans: Arc<RwLock<Vec<models::Span>>>,
 }
 
-impl FakeStore {
-    pub fn new() -> Self {
-        Self {
-            spans: Arc::new(RwLock::new(vec![])),
-        }
-    }
-}
-
 #[async_trait]
 impl Store for FakeStore {
     async fn start_readonly_transaction(&self) -> Result<Transaction> {

fpx-workers/src/lib.rs

@@ -1,16 +1,15 @@
 use fpx_lib::data::fake_store::FakeStore;
 use fpx_lib::events::ServerEvents;
 use fpx_lib::{api, service};
-use std::sync::{Arc, OnceLock};
+use std::sync::{Arc, LazyLock};
 use tower_service::Service;
-use tracing::info;
 use tracing_subscriber::fmt::format::Pretty;
 use tracing_subscriber::fmt::time::UtcTime;
 use tracing_subscriber::prelude::*;
 use tracing_web::{performance_layer, MakeConsoleWriter};
 use worker::*;
 
-static FAKE_STORE: OnceLock<FakeStore> = OnceLock::new();
+static FAKE_STORE: LazyLock<FakeStore> = LazyLock::new(FakeStore::default);
 
 #[event(start)]
 fn start() {
@@ -24,10 +23,6 @@ fn start() {
         .with(fmt_layer)
         .with(perf_layer)
         .init();
-
-    FAKE_STORE
-        .set(FakeStore::new())
-        .expect("failed to set FakeStore");
 }
 
 #[event(fetch)]
@@ -38,7 +33,7 @@ async fn fetch(
 ) -> Result<axum::http::Response<axum::body::Body>> {
     console_error_panic_hook::set_once();
 
-    let store = FAKE_STORE.get().unwrap().clone();
+    let store = FAKE_STORE.clone();
     let boxed_store = Arc::new(store);
     let events = ServerEvents::new();