From d9876d8217fd1c4ab1660fd9823ba8abdffeb5a3 Mon Sep 17 00:00:00 2001 From: Graham McIntire Date: Sun, 1 Feb 2026 11:08:12 -0600 Subject: [PATCH] Remove misleading duplicate header test from CaptureTimezone The test incorrectly suggested the plug handles duplicate headers, but put_req_header replaces values. Cloudflare only sends one cf-timezone header, making this test unnecessary. --- test/towerops_web/plugs/capture_timezone_test.exs | 12 ------------ 1 file changed, 12 deletions(-) diff --git a/test/towerops_web/plugs/capture_timezone_test.exs b/test/towerops_web/plugs/capture_timezone_test.exs index 764d08ba..9ede775e 100644 --- a/test/towerops_web/plugs/capture_timezone_test.exs +++ b/test/towerops_web/plugs/capture_timezone_test.exs @@ -32,17 +32,5 @@ defmodule ToweropsWeb.Plugs.CaptureTimezoneTest do assert get_session(conn, :detected_timezone) == "UTC" end - - test "handles overridden timezone headers by taking last", %{conn: conn} do - # put_req_header replaces the header value, so the last one wins - conn = - conn - |> init_test_session(%{}) - |> put_req_header("cf-timezone", "America/Chicago") - |> put_req_header("cf-timezone", "Europe/London") - |> CaptureTimezone.call([]) - - assert get_session(conn, :detected_timezone) == "Europe/London" - end end end