From 023a9d207904613615ceb23733d378d79efb0a79 Mon Sep 17 00:00:00 2001
From: Danilo Bargen <mail@dbrgn.ch>
Date: Sun, 5 Dec 2021 14:31:03 +0100
Subject: [PATCH] Lowercase page names before lookup (#227)

---
 src/main.rs  |  5 ++++-
 tests/lib.rs | 19 +++++++++++++++++++
 2 files changed, 23 insertions(+), 1 deletion(-)

diff --git a/src/main.rs b/src/main.rs
index 2ef2071..c90c128 100644
--- a/src/main.rs
+++ b/src/main.rs
@@ -493,7 +493,10 @@ fn main() {
 
     // Show command from cache
     if !args.command.is_empty() {
-        let command = args.command.join("-");
+        // Note: According to the TLDR client spec, page names must be transparently
+        // lowercased before lookup:
+        // https://github.com/tldr-pages/tldr/blob/main/CLIENT-SPECIFICATION.md#page-names
+        let command = args.command.join("-").to_lowercase();
 
         let languages = args
             .language
diff --git a/tests/lib.rs b/tests/lib.rs
index de212d2..7d52beb 100644
--- a/tests/lib.rs
+++ b/tests/lib.rs
@@ -690,3 +690,22 @@ fn test_pager_warning() {
         .success()
         .stderr(contains("pager flag not available on Windows"));
 }
+
+/// Ensure that page lookup is case insensitive, so a page lookup for `eyed3`
+/// and `eyeD3` should return the same page.
+#[test]
+fn test_lowercased_page_lookup() {
+    let testenv = TestEnv::new();
+
+    // Lookup `eyed3`, initially fails
+    testenv.command().args(["eyed3"]).assert().failure();
+
+    // Add entry
+    testenv.add_entry("eyed3", "contents");
+
+    // Lookup `eyed3` again
+    testenv.command().args(["eyed3"]).assert().success();
+
+    // Lookup `eyeD3`, should succeed as well
+    testenv.command().args(["eyeD3"]).assert().success();
+}