From 449c6856758f19102fd32934922ad7d606682d29 Mon Sep 17 00:00:00 2001
From: Ben Wiederhake <BenWiederhake.GitHub@gmx.de>
Date: Tue, 7 May 2024 21:32:24 +0200
Subject: [PATCH 1/2] uucore+timeout: accept signals of any casing

---
 src/uu/kill/src/kill.rs                |  3 +--
 src/uucore/src/lib/features/signals.rs |  5 +++--
 tests/by-util/test_timeout.rs          | 14 ++++++++------
 3 files changed, 12 insertions(+), 10 deletions(-)

diff --git a/src/uu/kill/src/kill.rs b/src/uu/kill/src/kill.rs
index 6048acfb9..1e60847c0 100644
--- a/src/uu/kill/src/kill.rs
+++ b/src/uu/kill/src/kill.rs
@@ -200,8 +200,7 @@ fn list(signals: &Vec<String>) {
 }
 
 fn parse_signal_value(signal_name: &str) -> UResult<usize> {
-    let signal_name_upcase = signal_name.to_uppercase();
-    let optional_signal_value = signal_by_name_or_value(&signal_name_upcase);
+    let optional_signal_value = signal_by_name_or_value(signal_name);
     match optional_signal_value {
         Some(x) => Ok(x),
         None => Err(USimpleError::new(
diff --git a/src/uucore/src/lib/features/signals.rs b/src/uucore/src/lib/features/signals.rs
index 8c9ca6195..41dda3e9f 100644
--- a/src/uucore/src/lib/features/signals.rs
+++ b/src/uucore/src/lib/features/signals.rs
@@ -340,14 +340,15 @@ pub static ALL_SIGNALS: [&str; 37] = [
 ];
 
 pub fn signal_by_name_or_value(signal_name_or_value: &str) -> Option<usize> {
-    if let Ok(value) = signal_name_or_value.parse() {
+    let signal_name_upcase = signal_name_or_value.to_uppercase();
+    if let Ok(value) = signal_name_upcase.parse() {
         if is_signal(value) {
             return Some(value);
         } else {
             return None;
         }
     }
-    let signal_name = signal_name_or_value.trim_start_matches("SIG");
+    let signal_name = signal_name_upcase.trim_start_matches("SIG");
 
     ALL_SIGNALS.iter().position(|&s| s == signal_name)
 }
diff --git a/tests/by-util/test_timeout.rs b/tests/by-util/test_timeout.rs
index 896065a20..1ba6445c8 100644
--- a/tests/by-util/test_timeout.rs
+++ b/tests/by-util/test_timeout.rs
@@ -97,12 +97,14 @@ fn test_preserve_status() {
 fn test_preserve_status_even_when_send_signal() {
     // When sending CONT signal, process doesn't get killed or stopped.
     // So, expected result is success and code 0.
-    new_ucmd!()
-        .args(&["-s", "CONT", "--preserve-status", ".1", "sleep", "5"])
-        .succeeds()
-        .code_is(0)
-        .no_stderr()
-        .no_stdout();
+    for cont_spelling in ["CONT", "cOnT", "SIGcont"] {
+        new_ucmd!()
+            .args(&["-s", cont_spelling, "--preserve-status", ".1", "sleep", "2"])
+            .succeeds()
+            .code_is(0)
+            .no_stderr()
+            .no_stdout();
+    }
 }
 
 #[test]

From 8f6eb578561a8a323a8e3589689cfbabebc72af6 Mon Sep 17 00:00:00 2001
From: Ben Wiederhake <BenWiederhake.GitHub@gmx.de>
Date: Thu, 9 May 2024 04:00:27 +0200
Subject: [PATCH 2/2] common: permit signal names of any casing in tests

---
 tests/common/util.rs | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)

diff --git a/tests/common/util.rs b/tests/common/util.rs
index 82f49556b..a5ee8a017 100644
--- a/tests/common/util.rs
+++ b/tests/common/util.rs
@@ -3193,8 +3193,6 @@ mod tests {
     #[cfg(feature = "sleep")]
     #[cfg(unix)]
     #[rstest]
-    #[case::signal_full_name_lower_case("sigkill")]
-    #[case::signal_short_name_lower_case("kill")]
     #[case::signal_only_part_of_name("IGKILL")] // spell-checker: disable-line
     #[case::signal_just_sig("SIG")]
     #[case::signal_value_too_high("100")]
@@ -3207,6 +3205,17 @@ mod tests {
         result.signal_name_is(signal_name);
     }
 
+    #[test]
+    #[cfg(feature = "sleep")]
+    #[cfg(unix)]
+    fn test_cmd_result_signal_name_is_accepts_lowercase() {
+        let mut child = TestScenario::new("sleep").ucmd().arg("60").run_no_wait();
+        child.kill();
+        let result = child.wait().unwrap();
+        result.signal_name_is("sigkill");
+        result.signal_name_is("kill");
+    }
+
     #[test]
     #[cfg(unix)]
     fn test_parse_coreutil_version() {