Implement Object.getOwnPropertyNames and Object.getOwnPropertySymbols

boa/src/builtins/object/mod.rs

@@ -85,6 +85,8 @@ impl BuiltIn for Object {
             "getOwnPropertyDescriptors",
             1,
         )
+        .static_method(Self::get_own_property_names, "getOwnPropertyNames", 1)
+        .static_method(Self::get_own_property_symbols, "getOwnPropertySymbols", 1)
         .build();
 
         (Self::NAME, object.into(), Self::attribute())
@@ -824,6 +826,42 @@ impl Object {
             Ok(JsValue::new(false))
         }
     }
+
+    /// `Object.getOwnPropertyNames( object )`
+    ///
+    /// More information:
+    ///  - [ECMAScript reference][spec]
+    ///  - [MDN documentation][mdn]
+    ///
+    /// [spec]: https://tc39.es/ecma262/#sec-object.getownpropertynames
+    /// [mdn]: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/getOwnPropertyNames
+    pub fn get_own_property_names(
+        _: &JsValue,
+        args: &[JsValue],
+        context: &mut Context,
+    ) -> JsResult<JsValue> {
+        // 1. Return ? GetOwnPropertyKeys(O, string).
+        let o = args.get_or_undefined(0);
+        get_own_property_keys(o, PropertyKeyType::String, context)
+    }
+
+    /// `Object.getOwnPropertySymbols( object )`
+    ///
+    /// More information:
+    ///  - [ECMAScript reference][spec]
+    ///  - [MDN documentation][mdn]
+    ///
+    /// [spec]: https://tc39.es/ecma262/#sec-object.getownpropertysymbols
+    /// [mdn]: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/getOwnPropertySymbols
+    pub fn get_own_property_symbols(
+        _: &JsValue,
+        args: &[JsValue],
+        context: &mut Context,
+    ) -> JsResult<JsValue> {
+        // 1. Return ? GetOwnPropertyKeys(O, symbol).
+        let o = args.get_or_undefined(0);
+        get_own_property_keys(o, PropertyKeyType::Symbol, context)
+    }
 }
 
 /// The abstract operation ObjectDefineProperties
@@ -877,3 +915,44 @@ fn object_define_properties(
     // 7. Return O.
     Ok(())
 }
+
+/// Type enum used in the abstract operation GetOwnPropertyKeys
+#[derive(Debug, Copy, Clone)]
+enum PropertyKeyType {
+    String,
+    Symbol,
+}
+
+/// The abstract operation GetOwnPropertyKeys
+///
+/// More information:
+///  - [ECMAScript reference][spec]
+///
+/// [spec]: https://tc39.es/ecma262/#sec-getownpropertykeys
+fn get_own_property_keys(
+    o: &JsValue,
+    r#type: PropertyKeyType,
+    context: &mut Context,
+) -> JsResult<JsValue> {
+    // 1. Let obj be ? ToObject(o).
+    let obj = o.to_object(context)?;
+
+    // 2. Let keys be ? obj.[[OwnPropertyKeys]]().
+    let keys = obj.__own_property_keys__(context)?;
+
+    // 3. Let nameList be a new empty List.
+    // 4. For each element nextKey of keys, do
+    let name_list = keys.iter().filter_map(|next_key| {
+        // a. If Type(nextKey) is Symbol and type is symbol or Type(nextKey) is String and type is string, then
+        // i. Append nextKey as the last element of nameList.
+        match (r#type, next_key) {
+            (PropertyKeyType::String, PropertyKey::String(key)) => Some(key.clone().into()),
+            (PropertyKeyType::String, PropertyKey::Index(index)) => Some(index.to_string().into()),
+            (PropertyKeyType::Symbol, PropertyKey::Symbol(symbol)) => Some(symbol.clone().into()),
+            _ => None,
+        }
+    });
+
+    // 5. Return CreateArrayFromList(nameList).
+    Ok(Array::create_array_from_list(name_list, context).into())
+}

boa/src/builtins/object/tests.rs

@@ -289,3 +289,51 @@ fn object_is_prototype_of() {
 
     assert_eq!(context.eval(init).unwrap(), JsValue::new(true));
 }
+
+#[test]
+fn object_get_own_property_names_invalid_args() {
+    let mut context = Context::new();
+
+    let error_message = r#"Uncaught "TypeError": "cannot convert 'null' or 'undefined' to object""#;
+
+    assert_eq!(
+        forward(&mut context, r#"Object.getOwnPropertyNames()"#),
+        error_message
+    );
+    assert_eq!(
+        forward(&mut context, r#"Object.getOwnPropertyNames(null)"#),
+        error_message
+    );
+    assert_eq!(
+        forward(&mut context, r#"Object.getOwnPropertyNames(undefined)"#),
+        error_message
+    );
+}
+
+#[test]
+fn object_get_own_property_names() {
+    let mut context = Context::new();
+
+    let init = r#"
+        const a = Object.getOwnPropertyNames(0);
+        const b = Object.getOwnPropertyNames(false);
+        const c = Object.getOwnPropertyNames(Symbol("a"));
+        const d = Object.getOwnPropertyNames({});
+        const e = Object.getOwnPropertyNames(NaN);
+
+        const f = Object.getOwnPropertyNames("abc");
+        const g = Object.getOwnPropertyNames([1, 2, 3]);
+        const h = Object.getOwnPropertyNames({ "a": 1, "b": 2, [ Symbol("c") ]: 3 });
+    "#;
+    forward(&mut context, init);
+
+    assert_eq!(forward(&mut context, "a"), "[]");
+    assert_eq!(forward(&mut context, "b"), "[]");
+    assert_eq!(forward(&mut context, "c"), "[]");
+    assert_eq!(forward(&mut context, "d"), "[]");
+    assert_eq!(forward(&mut context, "e"), "[]");
+
+    assert_eq!(forward(&mut context, "f"), r#"[ "0", "1", "2", "length" ]"#);
+    assert_eq!(forward(&mut context, "g"), r#"[ "0", "1", "2", "length" ]"#);
+    assert_eq!(forward(&mut context, "h"), r#"[ "a", "b" ]"#);
+}