String wasn't defaulting to empty when called as String() with no arg…

…ument

boa/src/builtins/string/mod.rs

@@ -32,15 +32,14 @@ use std::{
 /// [[Construct]] - Creates a new instance `this`
 ///
 /// [[Call]] - Returns a new native `string`
+/// <https://tc39.es/ecma262/#sec-string-constructor-string-value>
 pub fn make_string(this: &mut Value, args: &[Value], _: &mut Interpreter) -> ResultValue {
     // This value is used by console.log and other routines to match Obexpecty"failed to parse argument for String method"pe
     // to its Javascript Identifier (global constructor method name)
     this.set_kind(ObjectKind::String);
     this.set_internal_slot(
         "StringData",
-        args.get(0)
-            .expect("failed to get StringData for make_string()")
-            .clone(),
+        args.get(0).unwrap_or(&Value::string("")).clone(),
     );
 
     let arg = match args.get(0) {