From 1ed8a016c1639903e274a81326f7286d7e031dc4 Mon Sep 17 00:00:00 2001
From: Zachary Dremann <dremann@gmail.com>
Date: Mon, 29 Apr 2019 21:32:28 -0400
Subject: [PATCH] Add tests for signals (including failing test for `is_signal`

---
 src/uucore/signals.rs | 36 ++++++++++++++++++++++++++++++++++++
 1 file changed, 36 insertions(+)

diff --git a/src/uucore/signals.rs b/src/uucore/signals.rs
index 8887ddfe1..a4a7551e8 100644
--- a/src/uucore/signals.rs
+++ b/src/uucore/signals.rs
@@ -343,3 +343,39 @@ pub fn signal_by_name_or_value(signal_name_or_value: &str) -> Option<usize> {
 pub fn is_signal(num: usize) -> bool {
     num < ALL_SIGNALS.len()
 }
+
+#[test]
+fn signals_all_contiguous() {
+    for (i, signal) in ALL_SIGNALS.iter().enumerate() {
+        assert_eq!(signal.value, i + 1);
+    }
+}
+
+#[test]
+fn signals_all_are_signal() {
+    for signal in &ALL_SIGNALS {
+        assert!(is_signal(signal.value));
+    }
+}
+
+#[test]
+fn signal_by_value() {
+    assert_eq!(signal_by_name_or_value("0"), Some(0));
+    for signal in &ALL_SIGNALS {
+        assert_eq!(signal_by_name_or_value(&signal.value.to_string()), Some(signal.value));
+    }
+}
+
+#[test]
+fn signal_by_short_name() {
+    for signal in &ALL_SIGNALS {
+        assert_eq!(signal_by_name_or_value(signal.name), Some(signal.value));
+    }
+}
+
+#[test]
+fn signal_by_long_name() {
+    for signal in &ALL_SIGNALS {
+        assert_eq!(signal_by_name_or_value(&format!("SIG{}", signal.name)), Some(signal.value));
+    }
+}