diff --git a/Userland/Libraries/LibJS/Tests/builtins/Object/Object.freeze.js b/Userland/Libraries/LibJS/Tests/builtins/Object/Object.freeze.js index f8b7a89f69..39d5720055 100644 --- a/Userland/Libraries/LibJS/Tests/builtins/Object/Object.freeze.js +++ b/Userland/Libraries/LibJS/Tests/builtins/Object/Object.freeze.js @@ -29,9 +29,8 @@ describe("normal behavior", () => { test("prevents changing attributes of existing properties", () => { const o = { foo: "bar" }; Object.freeze(o); - // FIXME: These don't change anything and should not throw! - // expect(Object.defineProperty(o, "foo", {})).toBe(o); - // expect(Object.defineProperty(o, "foo", { configurable: false })).toBe(o); + expect(Object.defineProperty(o, "foo", {})).toBe(o); + expect(Object.defineProperty(o, "foo", { configurable: false })).toBe(o); expect(() => { Object.defineProperty(o, "foo", { configurable: true }); }).toThrowWithMessage(TypeError, "Object's [[DefineOwnProperty]] method returned false"); diff --git a/Userland/Libraries/LibJS/Tests/builtins/Object/Object.seal.js b/Userland/Libraries/LibJS/Tests/builtins/Object/Object.seal.js index d48b0bd9f6..20d6239d82 100644 --- a/Userland/Libraries/LibJS/Tests/builtins/Object/Object.seal.js +++ b/Userland/Libraries/LibJS/Tests/builtins/Object/Object.seal.js @@ -29,9 +29,8 @@ describe("normal behavior", () => { test("prevents changing attributes of existing properties", () => { const o = { foo: "bar" }; Object.seal(o); - // FIXME: These don't change anything and should not throw! - // expect(Object.defineProperty(o, "foo", {})).toBe(o); - // expect(Object.defineProperty(o, "foo", { configurable: false })).toBe(o); + expect(Object.defineProperty(o, "foo", {})).toBe(o); + expect(Object.defineProperty(o, "foo", { configurable: false })).toBe(o); expect(() => { Object.defineProperty(o, "foo", { configurable: true }); }).toThrowWithMessage(TypeError, "Object's [[DefineOwnProperty]] method returned false");