more tests

rbri · rbri · commit 1feda22e6c50 · 2025-04-30T09:32:39.000+02:00
diff --git a/src/main/java/org/htmlunit/javascript/host/Storage.java b/src/main/java/org/htmlunit/javascript/host/Storage.java
@@ -135,10 +135,13 @@ public void removeItem(final String key) {
      */
     @JsxFunction
     public String key(final int index) {
-        int counter = 0;
-        for (final String key : store_.keySet()) {
-            if (counter++ == index) {
-                return key;
+        if (index >= 0) {
+            int counter = 0;
+            for (final String key : store_.keySet()) {
+                if (counter == index) {
+                    return key;
+                }
+                counter++;
             }
         }
         return null;
diff --git a/src/test/java/org/htmlunit/javascript/host/StorageTest.java b/src/test/java/org/htmlunit/javascript/host/StorageTest.java
@@ -190,6 +190,34 @@ public void localStorageSizeManyEntries() throws Exception {
         loadPageVerifyTitle2(firstHtml);
     }
 
+    /**
+     * @throws Exception if the test fails
+     */
+    @Test
+    @Alerts({"true", "true", "null", "null"})
+    public void localStorageKey() throws Exception {
+        final String html = DOCTYPE_HTML
+            + "<html>\n"
+            + "<body>\n"
+            + "<script>\n"
+            + LOG_TITLE_FUNCTION
+            + "  if (window.localStorage) {\n"
+            + "    localStorage.clear();\n"
+
+            + "    localStorage.setItem('HtmlUnit', '0');\n"
+            + "    localStorage.setItem('HtmlUnit 1', '1');\n"
+            + "    localStorage.setItem('HtmlUnit 2', '2');\n"
+
+            + "    log(localStorage.key(0).startsWith('HtmlUnit'));\n"
+            + "    log(localStorage.key(1).startsWith('HtmlUnit'));\n"
+            + "    log(localStorage.key(3));\n"
+            + "    log(localStorage.key(-1));\n"
+            + "  }\n"
+            + "</script>\n"
+            + "</body></html>";
+        loadPageVerifyTitle2(html);
+    }
+
     /**
      * @throws Exception if the test fails
      */
