feat: mutualized build for test

README.md

@@ -12,7 +12,6 @@
  </a>
   <a href="https://github.com/0xBradock/actix-real-world/blob/master/openapi.yaml"><img src="https://img.shields.io/badge/Docs-OpenAPI%203.0-success" alt="OpenAPI Docs"></a>
   <a href="https://github.com/qdrant/qdrant/blob/master/LICENSE"><img src="https://img.shields.io/github/license/0xBradock/actix-real-world" alt="MIT"></a>
-
 </p>
 
 

src/main.rs

@@ -1,12 +1,12 @@
-use std::net::TcpListener;
-
-use actix_real_world::{config, run::run};
+use actix_real_world::{config, run::build};
 
+/// main instantiates the configuration and build the server
 #[actix_web::main]
 async fn main() -> std::io::Result<()> {
     let conf = config::Settings::new().expect("Failed to read config file");
-    let addr = format!("{}:{}", conf.server.host, conf.server.port);
-    let listener = TcpListener::bind(addr)?;
+    let server = build(&conf).await?;
+
+    server.await?;
 
-    run(listener)?.await
+    Ok(())
 }

src/run.rs

@@ -2,8 +2,17 @@ use std::net::TcpListener;
 
 use actix_web::{dev::Server, App, HttpServer};
 
-use crate::routes::health;
+use crate::{config::Settings, routes::health};
 
+/// build is responsible to start all services and call the server runner
+pub async fn build(conf: &Settings) -> Result<Server, std::io::Error> {
+    let addr = format!("{}:{}", conf.server.host, conf.server.port);
+    let listener = TcpListener::bind(addr)?;
+
+    run(listener)
+}
+
+/// run only runs the server with the services provided as parameters
 pub fn run(listener: TcpListener) -> Result<Server, std::io::Error> {
     let s = HttpServer::new(|| App::new().service(health))
         .listen(listener)?

tests/health.rs

@@ -1,30 +1,25 @@
-// use actix_real_world::routes::health;
+use actix_real_world::{config, run::build};
 
-use std::net::TcpListener;
-
-use actix_real_world::{config, run::run};
-
-fn spawn_app() -> String {
+async fn spawn_app() -> String {
     let conf = config::Settings::new().expect("Failed to read config file");
+    let server = build(&conf).await;
+    match server {
+        Ok(server) => tokio::spawn(server),
+        Err(_) => panic!("Failed to create server"),
+    };
 
-    let listener =
-        TcpListener::bind(format!("{}:0", conf.server.host)).expect("Failed to bind to port");
-    let port = listener.local_addr().unwrap().port();
-    let server = run(listener).expect("Failed to load server");
-    let _ = tokio::spawn(server);
-
-    format!("http://{}:{}", conf.server.host, port)
+    format!("http://{}:{}", conf.server.host, conf.server.port)
 }
 
 #[actix_web::test]
 async fn health_check_works() {
     // Arrange
-    let address = spawn_app();
+    let server = spawn_app().await;
     let client = reqwest::Client::new();
 
     // Act
     let response = client
-        .get(&format!("{}/health", &address))
+        .get(&format!("{}/health", server))
         .send()
         .await
         .expect("Failed to execute response");