Partial implementation of Tasks and Logs and test refactor

src/app/mod.rs

@@ -9,8 +9,11 @@ use crate::app::radarr::{ActiveRadarrBlock, RadarrData};
 use crate::models::{HorizontallyScrollableText, Route, TabRoute, TabState};
 use crate::network::NetworkEvent;
 
-pub(crate) mod key_binding;
-pub(crate) mod radarr;
+#[cfg(test)]
+#[path = "app_tests.rs"]
+mod app_tests;
+pub mod key_binding;
+pub mod radarr;
 
 const DEFAULT_ROUTE: Route = Route::Radarr(ActiveRadarrBlock::Movies, None);
 
@@ -174,216 +177,3 @@ impl Default for RadarrConfig {
     }
   }
 }
-
-#[cfg(test)]
-mod tests {
-  use anyhow::anyhow;
-  use pretty_assertions::{assert_eq, assert_str_eq};
-  use tokio::sync::mpsc;
-
-  use crate::app::radarr::{ActiveRadarrBlock, RadarrData};
-  use crate::app::{App, Data, RadarrConfig, DEFAULT_ROUTE};
-  use crate::models::{HorizontallyScrollableText, Route, TabRoute};
-  use crate::network::radarr_network::RadarrEvent;
-  use crate::network::NetworkEvent;
-
-  #[test]
-  fn test_app_default() {
-    let app = App::default();
-
-    assert_eq!(app.navigation_stack, vec![DEFAULT_ROUTE]);
-    assert!(app.network_tx.is_none());
-    assert_eq!(app.error, HorizontallyScrollableText::default());
-    assert!(app.response.is_empty());
-    assert_eq!(app.server_tabs.index, 0);
-    assert_eq!(
-      app.server_tabs.tabs,
-      vec![
-        TabRoute {
-          title: "Radarr",
-          route: ActiveRadarrBlock::Movies.into(),
-          help: "<↑↓> scroll | ←→ change tab | <tab> change servarr | <q> quit  ",
-          contextual_help: None,
-        },
-        TabRoute {
-          title: "Sonarr",
-          route: Route::Sonarr,
-          help: "<tab> change servarr | <q> quit  ",
-          contextual_help: None,
-        }
-      ]
-    );
-    assert_str_eq!(app.title, "Managarr");
-    assert_eq!(app.tick_until_poll, 400);
-    assert_eq!(app.ticks_until_scroll, 4);
-    assert_eq!(app.tick_count, 0);
-    assert!(!app.is_loading);
-    assert!(!app.is_routing);
-    assert!(!app.should_refresh);
-    assert!(!app.should_ignore_quit_key);
-  }
-
-  #[test]
-  fn test_navigation_stack_methods() {
-    let mut app = App::default();
-
-    assert_eq!(app.get_current_route(), &DEFAULT_ROUTE);
-
-    app.push_navigation_stack(ActiveRadarrBlock::Downloads.into());
-
-    assert_eq!(
-      app.get_current_route(),
-      &ActiveRadarrBlock::Downloads.into()
-    );
-    assert!(app.is_routing);
-
-    app.is_routing = false;
-    app.pop_and_push_navigation_stack(ActiveRadarrBlock::Collections.into());
-
-    assert_eq!(
-      app.get_current_route(),
-      &ActiveRadarrBlock::Collections.into()
-    );
-    assert!(app.is_routing);
-
-    app.is_routing = false;
-    app.pop_navigation_stack();
-
-    assert_eq!(app.get_current_route(), &DEFAULT_ROUTE);
-    assert!(app.is_routing);
-
-    app.is_routing = false;
-    app.pop_navigation_stack();
-
-    assert_eq!(app.get_current_route(), &DEFAULT_ROUTE);
-    assert!(app.is_routing);
-  }
-
-  #[test]
-  fn test_reset_tick_count() {
-    let mut app = App {
-      tick_count: 2,
-      ..App::default()
-    };
-
-    app.reset_tick_count();
-
-    assert_eq!(app.tick_count, 0);
-  }
-
-  #[test]
-  fn test_reset() {
-    let mut app = App {
-      tick_count: 2,
-      error: "Test error".to_owned().into(),
-      data: Data {
-        radarr_data: RadarrData {
-          version: "test".to_owned(),
-          ..RadarrData::default()
-        },
-      },
-      ..App::default()
-    };
-
-    app.reset();
-
-    assert_eq!(app.tick_count, 0);
-    assert_eq!(app.error, HorizontallyScrollableText::default());
-    assert!(app.data.radarr_data.version.is_empty());
-  }
-
-  #[test]
-  fn test_handle_error() {
-    let mut app = App::default();
-    let test_string = "Testing";
-
-    app.handle_error(anyhow!(test_string));
-
-    assert_eq!(app.error.text, test_string);
-
-    app.handle_error(anyhow!("Testing a different error"));
-
-    assert_eq!(app.error.text, test_string);
-  }
-
-  #[tokio::test]
-  async fn test_on_tick_first_render() {
-    let (sync_network_tx, mut sync_network_rx) = mpsc::channel::<NetworkEvent>(500);
-
-    let mut app = App {
-      tick_until_poll: 2,
-      network_tx: Some(sync_network_tx),
-      ..App::default()
-    };
-
-    assert_eq!(app.tick_count, 0);
-
-    app.on_tick(true).await;
-    assert_eq!(
-      sync_network_rx.recv().await.unwrap(),
-      RadarrEvent::GetQualityProfiles.into()
-    );
-    assert_eq!(
-      sync_network_rx.recv().await.unwrap(),
-      RadarrEvent::GetTags.into()
-    );
-    assert_eq!(
-      sync_network_rx.recv().await.unwrap(),
-      RadarrEvent::GetRootFolders.into()
-    );
-    assert_eq!(
-      sync_network_rx.recv().await.unwrap(),
-      RadarrEvent::GetOverview.into()
-    );
-    assert_eq!(
-      sync_network_rx.recv().await.unwrap(),
-      RadarrEvent::GetStatus.into()
-    );
-    assert_eq!(
-      sync_network_rx.recv().await.unwrap(),
-      RadarrEvent::GetMovies.into()
-    );
-    assert_eq!(
-      sync_network_rx.recv().await.unwrap(),
-      RadarrEvent::GetDownloads.into()
-    );
-    assert!(!app.is_routing);
-    assert!(!app.should_refresh);
-    assert_eq!(app.tick_count, 1);
-  }
-
-  #[tokio::test]
-  async fn test_on_tick_routing() {
-    let mut app = App {
-      tick_until_poll: 2,
-      tick_count: 2,
-      is_routing: true,
-      ..App::default()
-    };
-
-    app.on_tick(false).await;
-    assert!(!app.is_routing);
-  }
-
-  #[tokio::test]
-  async fn test_on_tick_should_refresh() {
-    let mut app = App {
-      tick_until_poll: 2,
-      tick_count: 2,
-      should_refresh: true,
-      ..App::default()
-    };
-
-    app.on_tick(false).await;
-    assert!(!app.should_refresh);
-  }
-
-  #[test]
-  fn test_radarr_config_default() {
-    let radarr_config = RadarrConfig::default();
-
-    assert_str_eq!(radarr_config.host, "localhost");
-    assert_eq!(radarr_config.port, Some(7878));
-    assert!(radarr_config.api_token.is_empty());
-  }
-}