Make EsprimaExtensions.TryGetKey[ more resilient to missing execution…

… context

Jint.Tests/Runtime/EvaluationContextTests.cs

@@ -0,0 +1,14 @@
+namespace Jint.Tests.Runtime;
+
+public class EvaluationContextTests
+{
+    [Fact]
+    public void ShouldThrowJavaScriptException()
+    {
+        var mockedEngine = new Engine();
+
+        Expression expression = new Identifier(NodeType.MemberExpression.ToString());
+
+        Assert.Throws<Jint.Runtime.JavaScriptException>(() => AstExtensions.TryGetComputedPropertyKey(expression, mockedEngine));
+    }
+}

Jint/AstExtensions.cs

@@ -62,7 +62,7 @@ internal static JsValue TryGetKey<T>(this T expression, Engine engine, bool reso
             return key;
         }
 
-        private static JsValue TryGetComputedPropertyKey<T>(T expression, Engine engine)
+        internal static JsValue TryGetComputedPropertyKey<T>(T expression, Engine engine)
             where T : Expression
         {
             if (expression.Type is NodeType.Identifier
@@ -79,8 +79,8 @@ or NodeType.FunctionExpression
                 or NodeType.YieldExpression
                 or NodeType.TemplateLiteral)
             {
-                var context = engine._activeEvaluationContext;
-                return JintExpression.Build(expression).GetValue(context!);
+                var context = engine._activeEvaluationContext ?? new EvaluationContext(engine);
+                return JintExpression.Build(expression).GetValue(context);
             }
 
             return JsValue.Undefined;
@@ -152,7 +152,7 @@ internal static bool IsAnonymousFunctionDefinition<T>(this T node) where T : Nod
         internal static bool IsOptional<T>(this T node) where T : Expression
         {
             return node is IChainElement { Optional: true };
-            }
+        }
 
         [MethodImpl(MethodImplOptions.AggressiveInlining)]
         internal static string LiteralKeyToString(Literal literal)
@@ -318,7 +318,7 @@ internal static Record DefineMethod<T>(this T m, ObjectInstance obj, ObjectInsta
 
             var runningExecutionContext = engine.ExecutionContext;
             var env = runningExecutionContext.LexicalEnvironment;
-            var privateEnv= runningExecutionContext.PrivateEnvironment;
+            var privateEnv = runningExecutionContext.PrivateEnvironment;
 
             var prototype = functionPrototype ?? intrinsics.Function.PrototypeObject;
             var function = m.Value as IFunction;

Jint/Engine.cs

@@ -62,7 +62,7 @@ public sealed partial class Engine : IDisposable
         internal readonly Dictionary<string, Type?> TypeCache = new(StringComparer.Ordinal);
 
         // we use registered type reference as prototype if it's known
-        internal Dictionary<Type,TypeReference>? _typeReferences;
+        internal Dictionary<Type, TypeReference>? _typeReferences;
 
         // cache for already wrapped CLR objects to keep object identity
         internal ConditionalWeakTable<object, ObjectInstance>? _objectWrapperCache;
@@ -459,7 +459,7 @@ private Engine ScriptEvaluation(ScriptRecord scriptRecord, ParserOptions parserO
                 // TODO what about callstack and thrown exceptions?
                 RunAvailableContinuations();
 
-               return this;
+                return this;
             }
             finally
             {
@@ -1243,7 +1243,7 @@ internal void EvalDeclarationInstantiation(
             {
                 foreach (var name in pointer.Names)
                 {
-                    privateIdentifiers??= new HashSet<PrivateIdentifier>(PrivateIdentifierNameComparer._instance);
+                    privateIdentifiers ??= new HashSet<PrivateIdentifier>(PrivateIdentifierNameComparer._instance);
                     privateIdentifiers.Add(name.Key);
                 }
 

Jint/Native/Function/EvalFunction.cs

@@ -1,6 +1,7 @@
 using Jint.Runtime;
 using Jint.Runtime.Descriptors;
 using Jint.Runtime.Environments;
+using Jint.Runtime.Interpreter;
 using Jint.Runtime.Interpreter.Statements;
 using Environment = Jint.Runtime.Environments.Environment;
 
@@ -178,7 +179,9 @@ internal JsValue PerformEval(JsValue x, Realm callerRealm, bool strictCaller, bo
                 Engine.EvalDeclarationInstantiation(script, varEnv, lexEnv, privateEnv, strictEval);
 
                 var statement = new JintScript(script);
-                var result = statement.Execute(_engine._activeEvaluationContext!);
+                var context = _engine._activeEvaluationContext ?? new EvaluationContext(_engine);
+                var result = statement.Execute(context);
+
                 var value = result.GetValueOrDefault();
 
                 if (result.Type == CompletionType.Throw)

Jint/Runtime/Interpreter/Expressions/JintExpression.cs

@@ -132,7 +132,7 @@ protected internal static JintExpression Build(Expression expression)
                     : new JintMemberExpression((MemberExpression) ((ChainExpression) expression).Expression),
                 NodeType.AwaitExpression => new JintAwaitExpression((AwaitExpression) expression),
                 NodeType.YieldExpression => new JintYieldExpression((YieldExpression) expression),
-                _ =>  null
+                _ => null
             };
 
             if (result is null)