浏览代码

LibJS: Uncomment Object.{freeze,seal}() tests that now pass :^)

Linus Groh 4 年之前
父节点
当前提交
1856400547

+ 2 - 3
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");

+ 2 - 3
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");