Partially fix flaky tests (#308)

* Minor code style changes

* Disable rendering of timestamps in tests

* Update test snapshots

* Test everything with insta macros (no more `assert(_eq)?`)

- This has the benefit of creating snapshots for everything, allowing for later diffing

* Don't use `assert_debug_snapshot` for large string outputs

- This makes snapshots more human-inspectable

* Code style improvement on two tests

- `pause_by_space`
- `rearranged_by_tab`

* Minor code style improvements

1 files changed, 10 insertions, 17 deletions

diff --git a/src/display/ui.rs b/src/display/ui.rs
index 3241b0b..00532e2 100644
--- a/src/display/ui.rs
+++ b/src/display/ui.rs
@@ -123,24 +123,17 @@ where
     }
 
     pub fn draw(&mut self, paused: bool, show_dns: bool, elapsed_time: Duration, ui_offset: usize) {
-        let state = &self.state;
-        let children = self.get_tables_to_display();
+        let layout = Layout {
+            header: HeaderDetails {
+                state: &self.state,
+                elapsed_time,
+                paused,
+            },
+            children: self.get_tables_to_display(),
+            footer: HelpText { paused, show_dns },
+        };
         self.terminal
-            .draw(|frame| {
-                let size = frame.size();
-                let total_bandwidth = HeaderDetails {
-                    state,
-                    elapsed_time,
-                    paused,
-                };
-                let help_text = HelpText { paused, show_dns };
-                let layout = Layout {
-                    header: total_bandwidth,
-                    children,
-                    footer: help_text,
-                };
-                layout.render(frame, size, ui_offset);
-            })
+            .draw(|frame| layout.render(frame, frame.size(), ui_offset))
             .unwrap();
     }