adding redefine existing property test to LambdaAccessorSlotTests to …

…cover that code path
mozilla · Dec 9, 2024 · 474c3ba · 474c3ba
1 parent d350e81
commit 474c3ba
Showing 1 changed file with 29 additions and 0 deletions.
diff --git a/tests/src/test/java/org/mozilla/javascript/tests/LambdaAccessorSlotTest.java b/tests/src/test/java/org/mozilla/javascript/tests/LambdaAccessorSlotTest.java
@@ -117,6 +117,35 @@ public void testOnlyGetterCanBeAccessed() {
                 });
     }
 
+    @Test
+    public void testRedefineExistingProperty() {
+        Utils.runWithAllOptimizationLevels(
+                cx -> {
+                    Scriptable scope = cx.initStandardObjects();
+                    var sh = new StatusHolder("PENDING");
+
+                    sh.defineProperty("value", "oldValueOfValue", DONTENUM);
+
+                    sh.defineProperty(cx, "value", (thisObj) -> "valueOfValue", null, DONTENUM);
+
+                    sh.defineProperty(cx, "status", (thisObj) -> 42, null, DONTENUM);
+
+                    sh.defineProperty(
+                            cx,
+                            "status",
+                            (thisObj) -> self(thisObj).getStatus(),
+                            (thisObj, value) -> self(thisObj).setStatus(value),
+                            DONTENUM);
+
+                    var status = sh.get("status", sh);
+                    assertEquals("PENDING", status);
+
+                    var value = sh.get("value", sh);
+                    assertEquals("valueOfValue", value);
+                    return null;
+                });
+    }
+
     @Test
     public void testWhenNoSetterDefined_InStrictMode_WillThrowException() {
         Utils.runWithAllOptimizationLevels(