Merge pull request #1657 from weather-gov/mgwalker/1653-afd-wfo-select

Select default WFO correctly, regardless of capitalization of WFO in path
weather-gov · Aug 29, 2024 · ba57fd5 · ba57fd5
2 parents 5d7f0fc + 14b4507
commit ba57fd5
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 0 deletions.
diff --git a/tests/playwright/e2e/afd-page.spec.js b/tests/playwright/e2e/afd-page.spec.js
@@ -99,6 +99,19 @@ describe("AFD Page Tests", () => {
       await page.goto("http://localhost:8080/afd/OKX");
     });
 
+    describe("WFO selector correctly populates based on path", () => {
+      test("with uppercase WFO code", async ({ page }) => {
+        const selected = page.locator("wx-afd-selector #wfo-selector");
+        await expect(selected).toHaveValue("OKX");
+      });
+
+      test("with lowercase WFO code", async ({ page }) => {
+        await page.goto("http://localhost:8080/afd/okx");
+        const selected = page.locator("wx-afd-selector #wfo-selector");
+        await expect(selected).toHaveValue("OKX");
+      });
+    });
+
     test("Selecting third version and clicking button will load that version", async ({page}) => {
       const thirdId = OKX_REFERENCES["@graph"][2].id;
       const versionSelect = await page.locator('select#version-selector');

diff --git a/web/themes/new_weather_theme/templates/layout/afd-page.html.twig b/web/themes/new_weather_theme/templates/layout/afd-page.html.twig
@@ -1,3 +1,5 @@
+{% set wfo = wfo | upper %}
+
 <div class="layout-container grid-row height-full flex-column flex-no-wrap">
     <header role="banner">
         <div class="grid-container">