ladybird/Libraries/LibJS/Tests/assignment-evaluation-order.js
Luke Wilde 18c0739bbb LibJS: Copy base object of LHS of assignment to preserve eval order
Previously, the given test would create an object with the test
property that pointed to itself.

This is because `temp = temp.test || {}` overwrote the `temp` local
register, and `temp.test = temp` used the new object instead of the
original one it fetched.

Allows https://www.yorkshiretea.co.uk/ to load, which was failing in
Gsap library initialization.
2025-09-02 12:59:52 +01:00

34 lines
719 B
JavaScript

test("Assignment should always evaluate LHS first", () => {
function go(a) {
let i = 0;
a[i] = a[++i];
}
let a = [1, 2, 3];
go(a);
expect(a).toEqual([2, 2, 3]);
});
test("Binary assignment should always evaluate LHS first", () => {
function go(a) {
let i = 0;
a[i] |= a[++i];
}
let a = [1, 2];
go(a);
expect(a).toEqual([3, 2]);
});
test("Base object of lhs of assignment is copied to preserve evaluation order", () => {
let topLevel = {};
function go() {
let temp = topLevel;
temp.test = temp = temp.test || {};
}
go();
expect(topLevel.test).not.toBeUndefined();
expect(topLevel.test).toEqual({});
});