tests: split YAML navigation tests by operation

Author: ynqa

promkit-widgets/tests/tests.rs

@@ -13,7 +13,11 @@ mod jsonz {
 }
 
 mod yamlz {
+    mod down;
     mod extract;
+    mod head;
     mod multi_documents;
-    mod navigation;
+    mod tail;
+    mod toggle;
+    mod up;
 }

promkit-widgets/tests/yamlz/down.rs

@@ -0,0 +1,23 @@
+use promkit_widgets::yaml::yamlz::{Row, RowOperation, create_rows};
+
+fn sequence_mapping_rows() -> Vec<Row> {
+    let input = serde_yaml::from_str::<serde_yaml::Value>(
+        r#"
+- name: alice
+  age: 20
+- tail
+"#,
+    )
+    .unwrap();
+
+    create_rows([&input])
+}
+
+#[test]
+fn down_skips_a_mapping_key_merged_into_its_sequence_item() {
+    let rows = sequence_mapping_rows();
+
+    // Row 1 and row 2 render together as "- name: alice", so moving down
+    // from that line should select "age: 20" at row 3.
+    assert_eq!(rows.down(1), 3);
+}

promkit-widgets/tests/yamlz/head.rs

@@ -0,0 +1,24 @@
+use promkit_widgets::yaml::{
+    Document,
+    yamlz::{RowOperation, create_rows},
+};
+
+#[test]
+fn document_starts_on_the_first_visible_root_mapping_key() {
+    let input = serde_yaml::from_str::<serde_yaml::Value>("apiVersion: v1\nkind: Pod\n").unwrap();
+    let document = Document::new([&input]);
+
+    let extracted = document.extract_rows_from_current(1);
+
+    assert_eq!(extracted[0].key.as_deref(), Some("apiVersion"));
+}
+
+#[test]
+fn a_collapsed_root_container_remains_navigable() {
+    let input = serde_yaml::from_str::<serde_yaml::Value>("apiVersion: v1\nkind: Pod\n").unwrap();
+    let mut rows = create_rows([&input]);
+
+    rows.toggle(0);
+
+    assert_eq!(rows.head(), 0);
+}

promkit-widgets/tests/yamlz/tail.rs

@@ -0,0 +1,14 @@
+use promkit_widgets::yaml::yamlz::{RowOperation, create_rows};
+
+#[test]
+fn tail_does_not_stop_on_a_merged_mapping_key() {
+    let input = serde_yaml::from_str::<serde_yaml::Value>(
+        r#"
+- name: alice
+"#,
+    )
+    .unwrap();
+    let rows = create_rows([&input]);
+
+    assert_eq!(rows.tail(), 1);
+}

promkit-widgets/tests/yamlz/navigation.rs promkit-widgets/tests/yamlz/toggle.rspromkit-widgets/tests/yamlz/navigation.rs renamed to promkit-widgets/tests/yamlz/toggle.rs

@@ -17,24 +17,6 @@ fn sequence_mapping_rows() -> Vec<Row> {
     create_rows([&input])
 }
 
-#[test]
-fn down_skips_a_mapping_key_merged_into_its_sequence_item() {
-    let rows = sequence_mapping_rows();
-
-    // Row 1 and row 2 render together as "- name: alice", so moving down
-    // from that line should select "age: 20" at row 3.
-    assert_eq!(rows.down(1), 3);
-}
-
-#[test]
-fn up_skips_a_mapping_key_merged_into_its_sequence_item() {
-    let rows = sequence_mapping_rows();
-
-    // Moving up from "age: 20" should select the combined
-    // "- name: alice" line at row 1, not its key row at row 2.
-    assert_eq!(rows.up(3), 1);
-}
-
 #[test]
 fn toggle_on_a_scalar_sequence_item_key_is_a_no_op() {
     let mut rows = sequence_mapping_rows();
@@ -69,48 +51,6 @@ fn toggle_from_a_merged_scalar_key_is_a_no_op() {
     assert_eq!(rendered, vec!["- name: alice"]);
 }
 
-#[test]
-fn tail_does_not_stop_on_a_merged_mapping_key() {
-    let input = serde_yaml::from_str::<serde_yaml::Value>(
-        r#"
-- name: alice
-"#,
-    )
-    .unwrap();
-    let rows = create_rows([&input]);
-
-    assert_eq!(rows.tail(), 1);
-}
-
-#[test]
-fn document_starts_on_the_first_visible_root_mapping_key() {
-    let input = serde_yaml::from_str::<serde_yaml::Value>("apiVersion: v1\nkind: Pod\n").unwrap();
-    let document = Document::new([&input]);
-
-    let extracted = document.extract_rows_from_current(1);
-
-    assert_eq!(extracted[0].key.as_deref(), Some("apiVersion"));
-}
-
-#[test]
-fn up_from_the_first_root_mapping_key_does_not_select_the_invisible_root() {
-    let input = serde_yaml::from_str::<serde_yaml::Value>("apiVersion: v1\nkind: Pod\n").unwrap();
-    let rows = create_rows([&input]);
-
-    assert_eq!(rows.head(), 1);
-    assert_eq!(rows.up(1), 1);
-}
-
-#[test]
-fn a_collapsed_root_container_remains_navigable() {
-    let input = serde_yaml::from_str::<serde_yaml::Value>("apiVersion: v1\nkind: Pod\n").unwrap();
-    let mut rows = create_rows([&input]);
-
-    rows.toggle(0);
-
-    assert_eq!(rows.head(), 0);
-}
-
 #[test]
 fn toggle_on_the_first_root_scalar_key_is_a_no_op() {
     let input = serde_yaml::from_str::<serde_yaml::Value>("apiVersion: v1\nkind: Pod\n").unwrap();

promkit-widgets/tests/yamlz/up.rs

@@ -0,0 +1,32 @@
+use promkit_widgets::yaml::yamlz::{Row, RowOperation, create_rows};
+
+fn sequence_mapping_rows() -> Vec<Row> {
+    let input = serde_yaml::from_str::<serde_yaml::Value>(
+        r#"
+- name: alice
+  age: 20
+- tail
+"#,
+    )
+    .unwrap();
+
+    create_rows([&input])
+}
+
+#[test]
+fn up_skips_a_mapping_key_merged_into_its_sequence_item() {
+    let rows = sequence_mapping_rows();
+
+    // Moving up from "age: 20" should select the combined
+    // "- name: alice" line at row 1, not its key row at row 2.
+    assert_eq!(rows.up(3), 1);
+}
+
+#[test]
+fn up_from_the_first_root_mapping_key_does_not_select_the_invisible_root() {
+    let input = serde_yaml::from_str::<serde_yaml::Value>("apiVersion: v1\nkind: Pod\n").unwrap();
+    let rows = create_rows([&input]);
+
+    assert_eq!(rows.head(), 1);
+    assert_eq!(rows.up(1), 1);
+}