Handle Symbol.toPrimitive and valueOf calls better under interop (#1470)

Jint.Tests/Runtime/InteropTests.cs

@@ -24,7 +24,7 @@ namespace Jint.Tests.Runtime
             _engine = new Engine(cfg => cfg.AllowClr(
                         typeof(Shape).GetTypeInfo().Assembly,
                         typeof(Console).GetTypeInfo().Assembly,
-                        typeof(System.IO.File).GetTypeInfo().Assembly))
+                        typeof(File).GetTypeInfo().Assembly))
                     .SetValue("log", new Action<object>(Console.WriteLine))
                     .SetValue("assert", new Action<bool>(Assert.True))
                     .SetValue("equal", new Action<object, object>(Assert.Equal))
@@ -3154,6 +3154,25 @@ try {
             }
         }
 
+        private class ClassWithIndexerAndProperty
+        {
+            public string MyProp { get; } = "from property";
+
+            public string this[string name] => name != nameof(MyProp) ? "from indexer" : null;
+        }
+
+        [Fact]
+        public void CanToStringObjectWithoutToPrimitiveSymbol()
+        {
+            var engine = new Engine();
+
+            engine.SetValue("obj", new ClassWithIndexerAndProperty());
+            Assert.Equal("Jint.Tests.Runtime.InteropTests+ClassWithIndexerAndProperty", engine.Evaluate("obj + ''").AsString());
+
+            engine.SetValue("obj", new Company("name"));
+            Assert.Equal("Jint.Tests.Runtime.Domain.Company", engine.Evaluate("obj + ''").AsString());
+        }
+
         [Fact]
         public void CanConstructOptionalRecordClass()
         {

Jint/Runtime/Interop/ObjectWrapper.cs

@@ -187,18 +187,24 @@ namespace Jint.Runtime.Interop
             }
 
             // if we have array-like or dictionary or expando, we can provide iterator
-            if (property.IsSymbol() && property == GlobalSymbolRegistry.Iterator && _typeDescriptor.Iterable)
+            if (property.IsSymbol())
             {
-                var iteratorFunction = new ClrFunctionInstance(
-                    Engine,
-                    "iterator",
-                    Iterator,
-                    1,
-                    PropertyFlag.Configurable);
-
-                var iteratorProperty = new PropertyDescriptor(iteratorFunction, PropertyFlag.Configurable | PropertyFlag.Writable);
-                SetProperty(GlobalSymbolRegistry.Iterator, iteratorProperty);
-                return iteratorProperty;
+                if (property == GlobalSymbolRegistry.Iterator && _typeDescriptor.Iterable)
+                {
+                    var iteratorFunction = new ClrFunctionInstance(
+                        Engine,
+                        "iterator",
+                        Iterator,
+                        1,
+                        PropertyFlag.Configurable);
+
+                    var iteratorProperty = new PropertyDescriptor(iteratorFunction, PropertyFlag.Configurable | PropertyFlag.Writable);
+                    SetProperty(GlobalSymbolRegistry.Iterator, iteratorProperty);
+                    return iteratorProperty;
+                }
+
+                // not that safe
+                return PropertyDescriptor.Undefined;
             }
 
             var member = property.ToString();

Jint/Runtime/Interop/Reflection/IndexerAccessor.cs

@@ -139,7 +139,14 @@ namespace Jint.Runtime.Interop.Reflection
                 }
             }
 
-            return _getter.Invoke(target, parameters);
+            try
+            {
+                return _getter.Invoke(target, parameters);
+            }
+            catch (TargetInvocationException tie) when (tie.InnerException is KeyNotFoundException)
+            {
+                return JsValue.Undefined;
+            }
         }
 
         protected override void DoSetValue(object target, object? value)