better tests. 9 failing

Author: jowharshamshiri

Cargo.toml

@@ -92,7 +92,7 @@ license = "MIT"
 name = "boxmux"
 readme = "README.md"
 repository = "https://github.com/jowharshamshiri/boxmux"
-version = "0.228.277519"
+version = "0.229.278442"
 
 [package.metadata.deb]
 assets = [["target/release/boxmux", "usr/bin/", "755"], ["README.md", "usr/share/doc/boxmux/README", "644"], ["examples/*", "usr/share/doc/boxmux/examples/", "644"]]

src/components/selection_styles.rs

@@ -494,7 +494,7 @@ impl SelectionStyleRenderer {
 #[cfg(test)]
 mod tests {
     use super::*;
-    use crate::model::common::{Bounds, ScreenBuffer};
+    use crate::model::common::ScreenBuffer;
 
     #[test]
     fn test_selection_style_renderer_creation() {
@@ -701,8 +701,8 @@ mod tests {
 
         let (fg, bg, text) = renderer.calculate_style_colors_and_text(
             &choice,
-            "white",
-            "black",
+            &Some("white".to_string()),
+            &Some("black".to_string()),
             &Some("red".to_string()),
             &Some("blue".to_string()),
             true, // is_focused = true

src/tests/box_renderer_tests.rs

@@ -15,7 +15,7 @@ mod tests {
     #[test]
     fn test_box_renderer_creation() {
         let muxbox = TestDataFactory::create_test_muxbox("test_box");
-        let renderer = BoxRenderer::new(&muxbox, "test_renderer".to_string());
+        let mut renderer = BoxRenderer::new(&muxbox, "test_renderer".to_string());
 
         // BoxRenderer created successfully - component encapsulates muxbox reference
         assert_eq!(muxbox.id, "test_box");
@@ -42,7 +42,7 @@ mod tests {
         );
         adjusted_bounds.insert("test_layout".to_string(), layout_bounds);
 
-        let renderer = BoxRenderer::new(&muxbox, "test_renderer".to_string());
+        let mut renderer = BoxRenderer::new(&muxbox, "test_renderer".to_string());
         let mut buffer = crate::ScreenBuffer::new();
 
         // Should render without errors
@@ -107,7 +107,7 @@ mod tests {
         );
         adjusted_bounds.insert("test_layout".to_string(), layout_bounds);
 
-        let renderer = BoxRenderer::new(&muxbox, "content_renderer".to_string());
+        let mut renderer = BoxRenderer::new(&muxbox, "content_renderer".to_string());
         let mut buffer = crate::ScreenBuffer::new();
 
         let result = renderer.render(
@@ -182,7 +182,7 @@ mod tests {
         );
         adjusted_bounds.insert("test_layout".to_string(), layout_bounds);
 
-        let renderer = BoxRenderer::new(&muxbox, "choice_renderer".to_string());
+        let mut renderer = BoxRenderer::new(&muxbox, "choice_renderer".to_string());
         let mut buffer = crate::ScreenBuffer::new();
 
         let result = renderer.render(
@@ -215,7 +215,7 @@ mod tests {
     #[test]
     fn test_box_renderer_preserves_muxbox_state() {
         let muxbox = TestDataFactory::create_test_muxbox("state_box");
-        let renderer = BoxRenderer::new(&muxbox, "state_renderer".to_string());
+        let mut renderer = BoxRenderer::new(&muxbox, "state_renderer".to_string());
 
         // BoxRenderer preserves muxbox state - it's a read-only rendering component
         assert_eq!(muxbox.id, "state_box");
@@ -240,7 +240,7 @@ mod tests {
         // Empty adjusted bounds - should handle gracefully
         let adjusted_bounds = HashMap::new();
 
-        let renderer = BoxRenderer::new(&muxbox, "missing_bounds_renderer".to_string());
+        let mut renderer = BoxRenderer::new(&muxbox, "missing_bounds_renderer".to_string());
         let mut buffer = crate::ScreenBuffer::new();
 
         let result = renderer.render(
@@ -294,7 +294,7 @@ mod tests {
         );
         adjusted_bounds.insert("test_layout".to_string(), layout_bounds);
 
-        let renderer = BoxRenderer::new(&muxbox, "integration_renderer".to_string());
+        let mut renderer = BoxRenderer::new(&muxbox, "integration_renderer".to_string());
         let mut buffer = crate::ScreenBuffer::new();
 
         let result = renderer.render(
@@ -330,7 +330,7 @@ mod tests {
         );
         adjusted_bounds.insert("test_layout".to_string(), layout_bounds);
 
-        let renderer = BoxRenderer::new(&muxbox, "pty_renderer".to_string());
+        let mut renderer = BoxRenderer::new(&muxbox, "pty_renderer".to_string());
         let mut buffer = crate::ScreenBuffer::new();
 
         let result = renderer.render(

src/tests/choice_click_bounds_tests.rs

@@ -21,7 +21,7 @@ app:
             y1: '10%' 
             x2: '90%'
             y2: '80%'
-          border: true
+          border_color: "white"
           title: 'Choices'
           choices:
             - id: 'build'
@@ -153,7 +153,7 @@ app:
             y1: '10%'
             x2: '90%'
             y2: '80%'
-          border: true
+          border_color: "white"
           title: 'X'
           choices:
             - id: 'x'
@@ -199,7 +199,7 @@ app:
             y1: '10%'
             x2: '90%' 
             y2: '80%'
-          border: true
+          border_color: "white"
           title: 'Deploy'
           choices:
             - id: 'deploy'