Change times route-matching to be case-insensitive

While this applied with BLUE already it’s now a lot
more widespread.

Author: backspace

src/commands/times.rs

@@ -63,7 +63,12 @@ pub async fn handle_times_request(
             _ => panic!("Unexpected type parsing route number"),
         };
 
-        if !command.routes.is_empty() && !command.routes.contains(route_number) {
+        if !command.routes.is_empty()
+            && !command
+                .routes
+                .iter()
+                .any(|r| r.eq_ignore_ascii_case(route_number))
+        {
             continue;
         }
 

tests/times.rs

@@ -198,6 +198,49 @@ async fn stop_number_returns_single_route_stop_schedule_to_approved_number(db: P
     );
 }
 
+#[sqlx::test(fixtures("numbers-approved"))]
+async fn stop_number_returns_single_route_ignoring_case(db: PgPool) {
+    let mock_winnipeg_transit_api = MockServer::start().await;
+    let mock_stop_schedule_response = fs::read_to_string("tests/fixtures/times/stop_schedule.json")
+        .expect("Failed to read stop schedule fixture");
+
+    Mock::given(method("GET"))
+        .and(path_regex(r"^/v4/stops/.*/schedule.json$"))
+        .respond_with(
+            ResponseTemplate::new(200).set_body_string(mock_stop_schedule_response.clone()),
+        )
+        .expect(1)
+        .mount(&mock_winnipeg_transit_api)
+        .await;
+
+    let response = get(
+        "/twilio?Body=10619 blue&From=approved&To=textabus&MessageSid=SM1849",
+        InjectableServices {
+            db: db.clone(),
+            twilio_address: None,
+            winnipeg_transit_api_address: Some(mock_winnipeg_transit_api.uri()),
+        },
+    )
+    .await
+    .expect("Failed to execute request");
+
+    assert!(response.status().is_success());
+    assert_eq!(response.headers()["content-type"], "text/xml");
+
+    let document = Document::from(response.text().await.unwrap().as_str());
+    let body = &document.find(Name("body")).next().unwrap().text();
+
+    let expected_body = indoc! {"
+        10619 WB Graham@Vaughan (The Bay)
+        12:19p BLUE Downtown (8min late)
+        12:22p BLUE Downtown
+        12:33p BLUE Downtown
+        12:45p BLUE Downtown
+        "};
+
+    assert_that(body).contains(expected_body);
+}
+
 #[sqlx::test(fixtures("numbers-approved"))]
 async fn incorrect_stop_number_returns_error(db: PgPool) {
     let mock_winnipeg_transit_api = MockServer::start().await;