Moved server initialization outside of main for easier testing

src/main.rs

@@ -54,12 +54,13 @@ use unix_daemonize::{daemonize_redirect, ChdirMode};
 use core::ops::Deref;
 use crate::errors::*;
 use getopts::Options;
-use rocket_contrib::serve::StaticFiles;
 use simplelog::{Level, LevelFilter, SimpleLogger, TermLogger};
 use std::path::Path;
 use std::sync::Arc;
 
 mod api;
+#[cfg(test)]
+mod api_tests;
 mod config;
 mod db;
 mod ddns;
@@ -69,6 +70,7 @@ mod lastfm;
 mod metadata;
 mod playlist;
 mod serve;
+mod server;
 mod thumbnails;
 mod ui;
 mod user;
@@ -233,18 +235,16 @@ fn run() -> Result<()> {
 		.parse()
 		.or(Err("invalid port number"))?;
 
-	let config = rocket::Config::build(rocket::config::Environment::Production)
+	let server = server::get_server(
-		.port(port)
+		port,
-		.finalize()?;
+		&static_url,
-
+		&api_url,
-	let db_server = db.clone();
+		&web_dir_path,
+		db.clone(),
+		command_sender,
+	)?;
 	std::thread::spawn(move || {
-		rocket::custom(config)
+		server.launch();
-			.manage(db_server)
-			.manage(command_sender)
-			.mount(&static_url, StaticFiles::from(web_dir_path))
-			.mount(&api_url, api::get_routes())
-			.launch();
 	});
 
 	// Start DDNS updates

src/server.rs

@@ -0,0 +1,28 @@
+use rocket;
+use rocket_contrib::serve::StaticFiles;
+use std::path::PathBuf;
+use std::sync::Arc;
+
+use crate::api;
+use crate::db::DB;
+use crate::errors;
+use crate::index::CommandSender;
+
+pub fn get_server(
+	port: u16,
+	static_url: &str,
+	api_url: &str,
+	web_dir_path: &PathBuf,
+	db: Arc<DB>,
+	command_sender: Arc<CommandSender>,
+) -> Result<rocket::Rocket, errors::Error> {
+	let config = rocket::Config::build(rocket::config::Environment::Production)
+		.port(port)
+		.finalize()?;
+
+	Ok(rocket::custom(config)
+		.manage(db)
+		.manage(command_sender)
+		.mount(&static_url, StaticFiles::from(web_dir_path))
+		.mount(&api_url, api::get_routes()))
+}