wip: test: fixing up the tests

- should bring back the rest of the mental model
- and allow me to prune it!

output/src/out_structs.rs

@@ -1,3 +1,4 @@
+#[cfg(test)] use proptest_derive::Arbitrary;
 use crate::*;
 
 /// A point (X, Y).
@@ -6,7 +7,9 @@ use crate::*;
 /// let xy: XY<u16> = XY(0, 0);
 /// ```
 #[cfg_attr(test, derive(Arbitrary))]
-#[derive(Copy, Clone, Debug, Default)] pub struct XY<C: Coord>(pub C, pub C);
+#[derive(Copy, Clone, Debug, Default, PartialEq)] pub struct XY<C: Coord>(
+    pub C, pub C
+);
 
 /// A size (Width, Height).
 ///
@@ -14,7 +17,9 @@ use crate::*;
 /// let wh: WH<u16> = WH(0, 0);
 /// ```
 #[cfg_attr(test, derive(Arbitrary))]
-#[derive(Copy, Clone, Debug, Default)] pub struct WH<C: Coord>(pub C, pub C);
+#[derive(Copy, Clone, Debug, Default, PartialEq)] pub struct WH<C: Coord>(
+    pub C, pub C
+);
 
 /// Point with size.
 ///
@@ -25,7 +30,9 @@ use crate::*;
 /// * [ ] TODO: anchor field (determines at which corner/side is X0 Y0)
 ///
 #[cfg_attr(test, derive(Arbitrary))]
-#[derive(Copy, Clone, Debug, Default)] pub struct XYWH<C: Coord>(pub C, pub C, pub C, pub C);
+#[derive(Copy, Clone, Debug, Default, PartialEq)] pub struct XYWH<C: Coord>(
+    pub C, pub C, pub C, pub C
+);
 
 /// A cardinal direction.
 ///

output/src/out_tests.rs

@@ -3,7 +3,7 @@ use proptest_derive::Arbitrary;
 use crate::*;
 
 #[test] fn test_area () {
-    assert_eq!(Area::center(&[10u16, 10, 20, 20]), [20, 20]);
+    assert_eq!(XYWH(10u16, 10, 20, 20).center(), XY(20, 20));
 }
 
 proptest! {
@@ -19,7 +19,7 @@ proptest! {
         h in u16::MIN..u16::MAX,
         a in u16::MIN..u16::MAX,
     ) {
-        let _ = d.split_fixed([x, y, w, h], a);
+        let _ = d.split_fixed(XYWH(x, y, w, h), a);
     }
 }
 
@@ -32,27 +32,27 @@ proptest! {
         a in u16::MIN..u16::MAX,
         b in u16::MIN..u16::MAX,
     ) {
-        let _: [u16;4] = <[u16;4] as Area<u16>>::zero();
+        let _: XYWH<u16> = XYWH::zero();
-        let _: [u16;4] = <[u16;4] as Area<u16>>::from_position([a, b]);
+        //let _: XYWH<u16> = XYWH::from_position([a, b]);
-        let _: [u16;4] = <[u16;4] as Area<u16>>::from_size([a, b]);
+        //let _: XYWH<u16> = XYWH::from_size([a, b]);
-        let area: [u16;4] = [x, y, w, h];
+        let area: XYWH<u16> = XYWH(x, y, w, h);
-        let _ = area.expect_min(a, b);
+        //let _ = area.expect_min(a, b);
         let _ = area.xy();
         let _ = area.wh();
-        let _ = area.xywh();
+        //let _ = area.xywh();
-        let _ = area.clip_h(a);
+        let _ = area.clipped_h(a);
-        let _ = area.clip_w(b);
+        let _ = area.clipped_w(b);
-        let _ = area.clip([a, b]);
+        let _ = area.clipped(WH(a, b));
-        let _ = area.set_w(a);
+        //let _ = area.set_w(a);
-        let _ = area.set_h(b);
+        //let _ = area.set_h(b);
         let _ = area.x2();
         let _ = area.y2();
         let _ = area.lrtb();
         let _ = area.center();
-        let _ = area.center_x(a);
-        let _ = area.center_y(b);
-        let _ = area.center_xy([a, b]);
         let _ = area.centered();
+        let _ = area.centered_x(a);
+        let _ = area.centered_y(b);
+        let _ = area.centered_xy([a, b]);
     }
 }
 
@@ -63,15 +63,15 @@ proptest! {
         a in u16::MIN..u16::MAX,
         b in u16::MIN..u16::MAX,
     ) {
-        let size = [x, y];
+        let size = WH(x, y);
         let _ = size.w();
         let _ = size.h();
         let _ = size.wh();
         let _ = size.clip_w(a);
         let _ = size.clip_h(b);
-        let _ = size.expect_min(a, b);
+        //let _ = size.expect_min(a, b);
-        let _ = size.to_area_pos();
+        //let _ = size.to_area_pos();
-        let _ = size.to_area_size();
+        //let _ = size.to_area_size();
     }
 }
 

tui/src/lib.rs

@@ -1,23 +1,16 @@
 #![feature(type_changing_struct_update, trait_alias)]
-
+pub extern crate dizzle;
+pub extern crate tengri_input;
+pub extern crate tengri_output;
+pub extern crate ratatui;
+pub extern crate crossterm;
+pub extern crate palette;
+pub extern crate better_panic;
+pub use ::tengri_output::PerfModel;
 use std::{time::Duration, thread::{spawn, JoinHandle}, io::Write};
 use unicode_width::*;
-
-pub use ::{
-    dizzle,
-    tengri_input,
-    tengri_output::{self, PerfModel},
-    ratatui,
-    crossterm,
-    palette,
-    better_panic,
-};
-
 pub(crate) use ::{
-    dizzle::*,
+    dizzle::*, tengri_input::*, tengri_output::*,
-    tengri_input::*,
-    tengri_output::*,
-    atomic_float::AtomicF64,
     std::{io::{stdout, Stdout}, sync::{Arc, RwLock, atomic::{AtomicBool, Ordering::*}}},
     better_panic::{Settings, Verbosity},
     palette::{*, convert::*, okhsl::*},
@@ -38,8 +31,7 @@ pub(crate) use ::{
 #[macro_export] macro_rules! tui_main {
     ($expr:expr) => {
         fn main () -> Usually<()> {
-            let state = Arc::new(RwLock::new($expr));
+            tengri_tui::Tui::run(true, $expr)?;
-            tengri_tui::Tui::new().unwrap().run(&state)?;
             Ok(())
         }
     };
@@ -82,10 +74,12 @@ macro_rules! border {
 
 mod tui_structs; pub use self::tui_structs::*;
 mod tui_traits;  pub use self::tui_traits::*;
-mod tui_impls;    pub use self::tui_impls::*;
+mod tui_impls;
+#[cfg(test)] mod tui_test;
 
 /// Run an app in the main loop.
 pub fn tui_run <T: Send + Sync + Draw<TuiOut> + Handle<TuiIn> + 'static> (
+    join: bool,
     state: &Arc<RwLock<T>>
 ) -> Usually<Arc<RwLock<Tui>>> {
     let backend = CrosstermBackend::new(stdout());
@@ -100,6 +94,7 @@ pub fn tui_run <T: Send + Sync + Draw<TuiOut> + Handle<TuiIn> + 'static> (
     let _input_thread = tui_input(tui.clone(), state, Duration::from_millis(100));
     tui.write().unwrap().setup()?;
     let render_thread = tui_output(tui.clone(), state, Duration::from_millis(10))?;
+    if join {
         match render_thread.join() {
             Ok(result) => {
                 tui.write().unwrap().teardown()?;
@@ -110,6 +105,7 @@ pub fn tui_run <T: Send + Sync + Draw<TuiOut> + Handle<TuiIn> + 'static> (
                 panic!("\n\rDraw thread failed: error={error:?}.\n\r")
             },
         }
+    }
     Ok(tui)
 }
 
@@ -492,41 +488,3 @@ pub(crate) fn width_chars_max (max: u16, text: impl AsRef<str>) -> u16 {
     }
     return width
 }
-
-#[cfg(test)] mod tui_test {
-    use crate::*;
-    #[test] fn test_tui_engine () -> Usually<()> {
-        //use std::sync::{Arc, RwLock};
-        struct TestComponent(String);
-        impl HasContent<TuiOut> for TestComponent {
-            fn content (&self) -> impl Content<TuiOut> {
-                Some(self.0.as_str())
-            }
-        }
-        impl Handle<TuiIn> for TestComponent {
-            fn handle (&mut self, _from: &TuiIn) -> Perhaps<bool> {
-                Ok(None)
-            }
-        }
-        let engine = Tui::new()?;
-        engine.read().unwrap().exited.store(true, std::sync::atomic::Ordering::Relaxed);
-        let state = TestComponent("hello world".into());
-        let _state = std::sync::Arc::new(std::sync::RwLock::new(state));
-        //engine.run(&state)?;
-        Ok(())
-    }
-    //#[test] fn test_parse_key () {
-        ////use KeyModifiers as Mods;
-        //let _test = |x: &str, y|assert_eq!(KeyMatcher::new(x).build(), Some(Event::Key(y)));
-        ////test(":x",
-            ////KeyEvent::new(KeyCode::Char('x'), Mods::NONE));
-        ////test(":ctrl-x",
-            ////KeyEvent::new(KeyCode::Char('x'), Mods::CONTROL));
-        ////test(":alt-x",
-            ////KeyEvent::new(KeyCode::Char('x'), Mods::ALT));
-        ////test(":shift-x",
-            ////KeyEvent::new(KeyCode::Char('x'), Mods::SHIFT));
-        ////test(":ctrl-alt-shift-x",
-            ////KeyEvent::new(KeyCode::Char('x'), Mods::CONTROL | Mods::ALT | Mods::SHIFT ));
-    //}
-}

tui/src/tui_impls.rs

@@ -4,10 +4,10 @@ use rand::{thread_rng, distributions::uniform::UniformSampler};
 
 impl Tui {
     /// Create and launch a terminal user interface.
-    pub fn run <T: Send + Sync + Handle<TuiIn> + Draw<TuiOut> + 'static> (
+    pub fn run <T> (join: bool, state: T) -> Usually<Arc<RwLock<Self>>> where
-        state: &Arc<RwLock<T>>
+        T: Handle<TuiIn> + Draw<TuiOut> + Send + Sync + 'static
-    ) -> Usually<Arc<RwLock<Self>>> {
+    {
-        tui_run(state)
+        tui_run(join, &Arc::new(RwLock::new(state)))
     }
     /// True if done
     pub fn exited (&self) -> bool { self.exited.fetch_and(true, Relaxed) }

tui/src/tui_test.rs

@@ -0,0 +1,33 @@
+use crate::*;
+
+#[test] fn test_tui_engine () -> Usually<()> {
+    //use std::sync::{Arc, RwLock};
+    struct TestComponent(String);
+    impl Draw<TuiOut> for TestComponent {
+        fn draw (&self, _to: &mut TuiOut) {}
+    }
+    impl Handle<TuiIn> for TestComponent {
+        fn handle (&mut self, _from: &TuiIn) -> Perhaps<bool> {
+            Ok(None)
+        }
+    }
+    let engine = Tui::run(false, TestComponent("hello world".into()))?;
+    engine.read().unwrap().exited.store(true, std::sync::atomic::Ordering::Relaxed);
+    //engine.run(&state)?;
+    Ok(())
+}
+
+//#[test] fn test_parse_key () {
+    ////use KeyModifiers as Mods;
+    //let _test = |x: &str, y|assert_eq!(KeyMatcher::new(x).build(), Some(Event::Key(y)));
+    ////test(":x",
+        ////KeyEvent::new(KeyCode::Char('x'), Mods::NONE));
+    ////test(":ctrl-x",
+        ////KeyEvent::new(KeyCode::Char('x'), Mods::CONTROL));
+    ////test(":alt-x",
+        ////KeyEvent::new(KeyCode::Char('x'), Mods::ALT));
+    ////test(":shift-x",
+        ////KeyEvent::new(KeyCode::Char('x'), Mods::SHIFT));
+    ////test(":ctrl-alt-shift-x",
+        ////KeyEvent::new(KeyCode::Char('x'), Mods::CONTROL | Mods::ALT | Mods::SHIFT ));
+//}