mirror of
				https://github.com/RGBCube/serenity
				synced 2025-10-26 04:22:07 +00:00 
			
		
		
		
	 abc7b31079
			
		
	
	
		abc7b31079
		
	
	
	
	
		
			
			The new default return_type argument is GetOwnPropertyReturnType::All, which returns properties with both string and symbol keys (which is also the default for [[OwnPropertyKeys]]). This means that in some cases we need to iterate the ordered property table twice, as we don't store string and symbol properties separately but symbols must - there's certainly room for (performance) improvements here. On the other hand this makes Reflect.ownKeys() return symbol properties now :^)
		
			
				
	
	
		
			53 lines
		
	
	
	
		
			1.9 KiB
		
	
	
	
		
			JavaScript
		
	
	
	
	
	
			
		
		
	
	
			53 lines
		
	
	
	
		
			1.9 KiB
		
	
	
	
		
			JavaScript
		
	
	
	
	
	
| test("length is 1", () => {
 | |
|     expect(Reflect.ownKeys).toHaveLength(1);
 | |
| });
 | |
| 
 | |
| describe("errors", () => {
 | |
|     test("target must be an object", () => {
 | |
|         [null, undefined, "foo", 123, NaN, Infinity].forEach(value => {
 | |
|             expect(() => {
 | |
|                 Reflect.ownKeys(value);
 | |
|             }).toThrowWithMessage(
 | |
|                 TypeError,
 | |
|                 "First argument of Reflect.ownKeys() must be an object"
 | |
|             );
 | |
|         });
 | |
|     });
 | |
| });
 | |
| 
 | |
| describe("normal behavior", () => {
 | |
|     test("regular empty object has no own keys", () => {
 | |
|         var objectOwnKeys = Reflect.ownKeys({});
 | |
|         expect(objectOwnKeys instanceof Array).toBeTrue();
 | |
|         expect(objectOwnKeys).toHaveLength(0);
 | |
|     });
 | |
| 
 | |
|     test("regular object with some properties has own keys", () => {
 | |
|         var symbol = Symbol("symbol");
 | |
|         var objectOwnKeys = Reflect.ownKeys({ foo: "bar", [symbol]: "qux", bar: "baz", 0: 42 });
 | |
|         expect(objectOwnKeys instanceof Array).toBeTrue();
 | |
|         expect(objectOwnKeys).toHaveLength(4);
 | |
|         expect(objectOwnKeys[0]).toBe("0");
 | |
|         expect(objectOwnKeys[1]).toBe("foo");
 | |
|         expect(objectOwnKeys[2]).toBe("bar");
 | |
|         expect(objectOwnKeys[3]).toBe(symbol);
 | |
|     });
 | |
| 
 | |
|     test("empty array has only 'length' own key", () => {
 | |
|         var arrayOwnKeys = Reflect.ownKeys([]);
 | |
|         expect(arrayOwnKeys instanceof Array).toBeTrue();
 | |
|         expect(arrayOwnKeys).toHaveLength(1);
 | |
|         expect(arrayOwnKeys[0]).toBe("length");
 | |
|     });
 | |
| 
 | |
|     test("array with some values has 'length' and indices own keys", () => {
 | |
|         var arrayOwnKeys = Reflect.ownKeys(["foo", [], 123, undefined]);
 | |
|         expect(arrayOwnKeys instanceof Array).toBeTrue();
 | |
|         expect(arrayOwnKeys).toHaveLength(5);
 | |
|         expect(arrayOwnKeys[0]).toBe("0");
 | |
|         expect(arrayOwnKeys[1]).toBe("1");
 | |
|         expect(arrayOwnKeys[2]).toBe("2");
 | |
|         expect(arrayOwnKeys[3]).toBe("3");
 | |
|         expect(arrayOwnKeys[4]).toBe("length");
 | |
|     });
 | |
| });
 |