From c7772361a51b403705c0c5940368d36e56a51ce9 Mon Sep 17 00:00:00 2001
From: wengchangcheng <wengchangcheng@huawei.com>
Date: Tue, 18 Oct 2022 23:41:01 +0800
Subject: [PATCH] fixed a007687 from
 https://gitee.com/weng-changcheng/arkcompiler_toolchain/pulls/35 Descriptor:
 Add testcase and bugfix for tooling

Details:
1. add variable testcase
2. delete unused variable of es2abc

Issue: https://gitee.com/openharmony/arkcompiler_toolchain/issues/I5WFGB

Signed-off-by: wengchangcheng <wengchangcheng@huawei.com>
Change-Id: Ic51c302edbc957aacea80e80a3b09619d184016f
---
 test/resource/tooling/ohos_test.xml           |   1 +
 tooling/agent/debugger_impl.cpp               |  10 +-
 tooling/base/pt_params.h                      |  18 ++
 tooling/test/BUILD.gn                         |   7 +-
 tooling/test/testcases/js/variable.js         |  34 +++
 .../test/testcases/js_breakpoint_arrow_test.h |   4 +-
 .../test/testcases/js_breakpoint_async_test.h |   4 +-
 tooling/test/testcases/js_breakpoint_test.h   |   4 +-
 tooling/test/testcases/js_exception_test.h    |   4 +-
 tooling/test/testcases/js_range_error_test.h  |   4 +-
 tooling/test/testcases/js_single_step_test.h  |   7 +-
 tooling/test/testcases/js_step_into_test.h    |   5 +-
 tooling/test/testcases/js_step_out_test.h     |   5 +-
 tooling/test/testcases/js_step_over_test.h    |   5 +-
 .../test/testcases/js_syntax_exception_test.h |   7 +-
 .../test/testcases/js_throw_exception_test.h  |   7 +-
 tooling/test/testcases/js_variable_test.h     | 239 ++++++++++++++++++
 tooling/test/utils/test_channel.h             |  54 ++++
 tooling/test/utils/test_events.h              |   4 +
 tooling/test/utils/test_hooks.h               |   5 +-
 tooling/test/utils/test_list.cpp              |   2 +
 tooling/test/utils/test_util.h                |   8 +-
 22 files changed, 409 insertions(+), 29 deletions(-)
 create mode 100644 tooling/test/testcases/js/variable.js
 create mode 100644 tooling/test/testcases/js_variable_test.h
 create mode 100644 tooling/test/utils/test_channel.h

diff --git a/test/resource/tooling/ohos_test.xml b/test/resource/tooling/ohos_test.xml
index 5a3cdfc9..84202e92 100755
--- a/test/resource/tooling/ohos_test.xml
+++ b/test/resource/tooling/ohos_test.xml
@@ -23,6 +23,7 @@
             <option name="push" value="obj/arkcompiler/toolchain/tooling/test/syntax_exception.abc -> /data/test" src="out"/>
             <option name="push" value="obj/arkcompiler/toolchain/tooling/test/throw_exception.abc -> /data/test" src="out"/>
             <option name="push" value="obj/arkcompiler/toolchain/tooling/test/step.abc -> /data/test" src="out"/>
+            <option name="push" value="obj/arkcompiler/toolchain/tooling/test/variable.abc -> /data/test" src="out"/>
             <option name="push" value="test/test/libjsdebugtest.so -> /data/test" src="out"/>
             <option name="push" value="test/test/libdebugger_entry.so -> /data/test" src="out"/>
             <option name="push" value="test/test/libark_jsruntime_test.so -> /data/test" src="out"/>
diff --git a/tooling/agent/debugger_impl.cpp b/tooling/agent/debugger_impl.cpp
index 11ee643c..610d0de4 100644
--- a/tooling/agent/debugger_impl.cpp
+++ b/tooling/agent/debugger_impl.cpp
@@ -1039,17 +1039,17 @@ void DebuggerImpl::GetLocalVariables(const FrameHandler *frameHandler, panda_fil
     bool hasThis = false;
     for (const auto &[varName, regIndex] : extractor->GetLocalVariableTable(methodId)) {
         value = DebuggerApi::GetVRegValue(vm_, frameHandler, regIndex);
-        if (varName == "4newTarget") {
+        if (varName == "4newTarget" || varName == "=nt") {
             continue;
         }
 
-        if (varName == "this") {
+        if (varName == "this" || varName == "=t") {
             thisVal = value;
             hasThis = true;
             continue;
         }
         Local<JSValueRef> name = JSValueRef::Undefined(vm_);
-        if (varName == "4funcObj") {
+        if (varName == "4funcObj" || varName == "=f") {
             if (value->IsFunction()) {
                 auto funcName = Local<FunctionRef>(value)->GetName(vm_)->ToString();
                 name = StringRef::NewFromUtf8(vm_, funcName.c_str());
@@ -1075,12 +1075,12 @@ void DebuggerImpl::GetLocalVariables(const FrameHandler *frameHandler, panda_fil
         JSThread *thread = vm_->GetJSThread();
         for (const auto &[varName, slot] : scopeDebugInfo->scopeInfo) {
             // skip possible duplicate variables both in local variable table and env
-            if (varName == "4newTarget") {
+            if (varName == "4newTarget" || varName == "=nt") {
                 continue;
             }
             value = JSNApiHelper::ToLocal<JSValueRef>(
                 JSHandle<JSTaggedValue>(thread, lexEnv->GetProperties(slot)));
-            if (varName == "this") {
+            if (varName == "this" || varName == "=t") {
                 if (!hasThis) {
                     thisVal = value;
                 }
diff --git a/tooling/base/pt_params.h b/tooling/base/pt_params.h
index 9503bd72..56bd93be 100644
--- a/tooling/base/pt_params.h
+++ b/tooling/base/pt_params.h
@@ -459,11 +459,23 @@ public:
         return objectId_;
     }
 
+    GetPropertiesParams &SetObjectId(RemoteObjectId id)
+    {
+        objectId_ = id;
+        return *this;
+    }
+
     bool GetOwnProperties() const
     {
         return ownProperties_.value_or(false);
     }
 
+    GetPropertiesParams &SetOwnProperties(bool ownProperties)
+    {
+        ownProperties_ = ownProperties;
+        return *this;
+    }
+
     bool HasOwnProperties() const
     {
         return ownProperties_.has_value();
@@ -474,6 +486,12 @@ public:
         return accessorPropertiesOnly_.value_or(false);
     }
 
+    GetPropertiesParams &SetAccessPropertiesOnly(bool accessorPropertiesOnly)
+    {
+        accessorPropertiesOnly_ = accessorPropertiesOnly;
+        return *this;
+    }
+
     bool HasAccessPropertiesOnly() const
     {
         return accessorPropertiesOnly_.has_value();
diff --git a/tooling/test/BUILD.gn b/tooling/test/BUILD.gn
index c8625b05..9d93e2e9 100644
--- a/tooling/test/BUILD.gn
+++ b/tooling/test/BUILD.gn
@@ -45,6 +45,7 @@ test_js_files = [
   "step",
   "syntax_exception",
   "throw_exception",
+  "variable",
 ]
 
 foreach(file, test_js_files) {
@@ -103,8 +104,12 @@ source_set("jsdebugtest_set") {
   if (target_toolchain == host_toolchain) {
     test_abc_dir = rebase_path(target_out_dir)
   }
+  test_js_dir = rebase_path(test_js_path)
 
-  defines = [ "DEBUGGER_ABC_DIR=\"${test_abc_dir}/\"" ]
+  defines = [
+    "DEBUGGER_ABC_DIR=\"${test_abc_dir}/\"",
+    "DEBUGGER_JS_DIR=\"${test_js_dir}\"",
+  ]
 
   if (is_ohos && is_standard_system) {
     if (enable_hilog) {
diff --git a/tooling/test/testcases/js/variable.js b/tooling/test/testcases/js/variable.js
new file mode 100644
index 00000000..875eda29
--- /dev/null
+++ b/tooling/test/testcases/js/variable.js
@@ -0,0 +1,34 @@
+/*
+ * Copyright (c) 2022 Huawei Device Co., Ltd.
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+var o = {
+    "foo" : function() {
+        var number0 = 1;
+        var number1 = 65535;
+
+        var string0 = "helloworld"
+        var string1 = ""
+
+        var obj0 = {
+            "key0": "value0",
+            "key1": 100
+        }
+        var obj1 = {}
+
+        var nop = undefined
+    }
+}
+
+o.foo()
\ No newline at end of file
diff --git a/tooling/test/testcases/js_breakpoint_arrow_test.h b/tooling/test/testcases/js_breakpoint_arrow_test.h
index b2d75da8..add38247 100644
--- a/tooling/test/testcases/js_breakpoint_arrow_test.h
+++ b/tooling/test/testcases/js_breakpoint_arrow_test.h
@@ -32,7 +32,8 @@ public:
         };
 
         loadModule = [this](std::string_view moduleName) {
-            location_ = TestUtil::GetLocation(18, 0, pandaFile_.c_str()); // 18: breakpointer line
+            // 18: breakpointer line
+            location_ = TestUtil::GetLocation(sourceFile_.c_str(), 18, 0, pandaFile_.c_str());
             ASSERT_TRUE(location_.GetMethodId().IsValid());
             TestUtil::SuspendUntilContinue(DebugEvent::LOAD_MODULE);
             ASSERT_EQ(moduleName, pandaFile_);
@@ -68,6 +69,7 @@ public:
 
 private:
     std::string pandaFile_ = DEBUGGER_ABC_DIR "arrow_func.abc";
+    std::string sourceFile_ = DEBUGGER_JS_DIR "arrow_func.js";
     std::string entryPoint_ = "_GLOBAL::func_main_0";
     JSPtLocation location_ {nullptr, JSPtLocation::EntityId(0), 0};
     size_t breakpointCounter_ = 0;
diff --git a/tooling/test/testcases/js_breakpoint_async_test.h b/tooling/test/testcases/js_breakpoint_async_test.h
index 129afdd5..9c46ea61 100644
--- a/tooling/test/testcases/js_breakpoint_async_test.h
+++ b/tooling/test/testcases/js_breakpoint_async_test.h
@@ -32,7 +32,8 @@ public:
         };
 
         loadModule = [this](std::string_view moduleName) {
-            location_ = TestUtil::GetLocation(18, 0, pandaFile_.c_str()); // 18: breakpointer line
+            // 18: breakpointer line
+            location_ = TestUtil::GetLocation(sourceFile_.c_str(), 18, 0, pandaFile_.c_str());
             ASSERT_TRUE(location_.GetMethodId().IsValid());
             TestUtil::SuspendUntilContinue(DebugEvent::LOAD_MODULE);
             ASSERT_EQ(moduleName, pandaFile_);
@@ -70,6 +71,7 @@ public:
 
 private:
     std::string pandaFile_ = DEBUGGER_ABC_DIR "async_func.abc";
+    std::string sourceFile_ = DEBUGGER_JS_DIR "async_func.js";
     std::string entryPoint_ = "_GLOBAL::func_main_0";
     JSPtLocation location_ {nullptr, JSPtLocation::EntityId(0), 0};
     size_t breakpointCounter_ = 0;
diff --git a/tooling/test/testcases/js_breakpoint_test.h b/tooling/test/testcases/js_breakpoint_test.h
index 489dd590..1cddb600 100644
--- a/tooling/test/testcases/js_breakpoint_test.h
+++ b/tooling/test/testcases/js_breakpoint_test.h
@@ -32,7 +32,8 @@ public:
         };
 
         loadModule = [this](std::string_view moduleName) {
-            location_ = TestUtil::GetLocation(22, 0, pandaFile_.c_str());  // 22: breakpointer line
+            // 22: breakpointer line
+            location_ = TestUtil::GetLocation(sourceFile_.c_str(), 22, 0, pandaFile_.c_str());
             ASSERT_TRUE(location_.GetMethodId().IsValid());
             TestUtil::SuspendUntilContinue(DebugEvent::LOAD_MODULE);
             ASSERT_EQ(moduleName, pandaFile_);
@@ -70,6 +71,7 @@ public:
 
 private:
     std::string pandaFile_ = DEBUGGER_ABC_DIR "sample.abc";
+    std::string sourceFile_ = DEBUGGER_JS_DIR "sample.js";
     std::string entryPoint_ = "_GLOBAL::func_main_0";
     JSPtLocation location_ {nullptr, JSPtLocation::EntityId(0), 0};
     size_t breakpointCounter_ = 0;
diff --git a/tooling/test/testcases/js_exception_test.h b/tooling/test/testcases/js_exception_test.h
index 3c204b2a..711f8614 100644
--- a/tooling/test/testcases/js_exception_test.h
+++ b/tooling/test/testcases/js_exception_test.h
@@ -55,7 +55,8 @@ public:
         };
 
         loadModule = [this](std::string_view moduleName) {
-            location_ = TestUtil::GetLocation(22, 0, pandaFile_.c_str());  // 22：breakpointer line
+            // 22：breakpointer line
+            location_ = TestUtil::GetLocation(sourceFile_.c_str(), 22, 0, pandaFile_.c_str());
             ASSERT_TRUE(location_.GetMethodId().IsValid());
             TestUtil::SuspendUntilContinue(DebugEvent::LOAD_MODULE);
             ASSERT_EQ(moduleName, pandaFile_);
@@ -94,6 +95,7 @@ public:
 
 private:
     std::string pandaFile_ = DEBUGGER_ABC_DIR "exception.abc";
+    std::string sourceFile_ = DEBUGGER_JS_DIR "exception.js";
     std::string entryPoint_ = "_GLOBAL::func_main_0";
     JSPtLocation location_ {nullptr, JSPtLocation::EntityId(0), 0};
     size_t breakpointCounter_ = 0;
diff --git a/tooling/test/testcases/js_range_error_test.h b/tooling/test/testcases/js_range_error_test.h
index ac358203..7ccdad38 100644
--- a/tooling/test/testcases/js_range_error_test.h
+++ b/tooling/test/testcases/js_range_error_test.h
@@ -55,7 +55,8 @@ public:
         };
 
         loadModule = [this](std::string_view moduleName) {
-            location_ = TestUtil::GetLocation(20, 0, pandaFile_.c_str());  // 20: breakpointer line
+            // 20: breakpointer line
+            location_ = TestUtil::GetLocation(sourceFile_.c_str(), 20, 0, pandaFile_.c_str());
             ASSERT_TRUE(location_.GetMethodId().IsValid());
             TestUtil::SuspendUntilContinue(DebugEvent::LOAD_MODULE);
             ASSERT_EQ(moduleName, pandaFile_);
@@ -94,6 +95,7 @@ public:
 
 private:
     std::string pandaFile_ = DEBUGGER_ABC_DIR "range_error.abc";
+    std::string sourceFile_ = DEBUGGER_JS_DIR "range_error.js";
     std::string entryPoint_ = "_GLOBAL::func_main_0";
     JSPtLocation location_ {nullptr, JSPtLocation::EntityId(0), 0};
     size_t breakpointCounter_ = 0;
diff --git a/tooling/test/testcases/js_single_step_test.h b/tooling/test/testcases/js_single_step_test.h
index 2b35dab6..56979338 100644
--- a/tooling/test/testcases/js_single_step_test.h
+++ b/tooling/test/testcases/js_single_step_test.h
@@ -30,8 +30,10 @@ public:
         };
 
         loadModule = [this](std::string_view moduleName) {
-            locationStart_ = TestUtil::GetLocation(19, 0, pandaFile_.c_str());  // 19: line number
-            locationEnd_ = TestUtil::GetLocation(22, 0, pandaFile_.c_str());  // 22: line number
+            // 19: line number
+            locationStart_ = TestUtil::GetLocation(sourceFile_.c_str(), 19, 0, pandaFile_.c_str());
+            // 22: line number
+            locationEnd_ = TestUtil::GetLocation(sourceFile_.c_str(), 22, 0, pandaFile_.c_str());
             TestUtil::SuspendUntilContinue(DebugEvent::LOAD_MODULE);
             ASSERT_EQ(moduleName, pandaFile_);
             auto condFuncRef = FunctionRef::Undefined(vm_);
@@ -82,6 +84,7 @@ public:
 
 private:
     std::string pandaFile_ = DEBUGGER_ABC_DIR "sample.abc";
+    std::string sourceFile_ = DEBUGGER_JS_DIR "sample.js";
     std::string entryPoint_ = "_GLOBAL::func_main_0";
     JSPtLocation locationStart_ {nullptr, JSPtLocation::EntityId(0), 0};
     JSPtLocation locationEnd_ {nullptr, JSPtLocation::EntityId(0), 0};
diff --git a/tooling/test/testcases/js_step_into_test.h b/tooling/test/testcases/js_step_into_test.h
index 31e9afff..e412df0e 100644
--- a/tooling/test/testcases/js_step_into_test.h
+++ b/tooling/test/testcases/js_step_into_test.h
@@ -99,6 +99,7 @@ private:
     static constexpr size_t STEP_SIZE = 9;
 
     std::string pandaFile_ = DEBUGGER_ABC_DIR "step.abc";
+    std::string sourceFile_ = DEBUGGER_JS_DIR "step.js";
     std::string entryPoint_ = "_GLOBAL::func_main_0";
     size_t breakpointCounter_ = 0;
     size_t stepCompleteCounter_ = 0;
@@ -108,8 +109,8 @@ private:
     void SetJSPtLocation(size_t *arr, size_t number, std::vector<JSPtLocation> &locations)
     {
         for (size_t i = 0; i < number; i++) {
-            JSPtLocation location =
-                TestUtil::GetLocation(arr[i * LINE_COLUMN], arr[i * LINE_COLUMN + 1], pandaFile_.c_str());
+            JSPtLocation location = TestUtil::GetLocation(sourceFile_.c_str(), arr[i * LINE_COLUMN],
+                                                          arr[i * LINE_COLUMN + 1], pandaFile_.c_str());
             locations.push_back(location);
         }
     };
diff --git a/tooling/test/testcases/js_step_out_test.h b/tooling/test/testcases/js_step_out_test.h
index 4d15728a..9c6950cb 100644
--- a/tooling/test/testcases/js_step_out_test.h
+++ b/tooling/test/testcases/js_step_out_test.h
@@ -96,6 +96,7 @@ private:
     static constexpr size_t STEP_SIZE = 4;
 
     std::string pandaFile_ = DEBUGGER_ABC_DIR "step.abc";
+    std::string sourceFile_ = DEBUGGER_JS_DIR "step.js";
     std::string entryPoint_ = "_GLOBAL::func_main_0";
     JSPtLocation location1_ {nullptr, JSPtLocation::EntityId(0), 0};
     JSPtLocation location2_ {nullptr, JSPtLocation::EntityId(0), 0};
@@ -107,8 +108,8 @@ private:
     void SetJSPtLocation(size_t *arr, size_t number, std::vector<JSPtLocation> &locations)
     {
         for (size_t i = 0; i < number; i++) {
-            JSPtLocation location_ =
-                TestUtil::GetLocation(arr[i * LINE_COLUMN], arr[i * LINE_COLUMN + 1], pandaFile_.c_str());
+            JSPtLocation location_ = TestUtil::GetLocation(sourceFile_.c_str(), arr[i * LINE_COLUMN],
+                                                           arr[i * LINE_COLUMN + 1], pandaFile_.c_str());
             locations.push_back(location_);
         }
     };
diff --git a/tooling/test/testcases/js_step_over_test.h b/tooling/test/testcases/js_step_over_test.h
index 6c93ed5b..1619ad47 100644
--- a/tooling/test/testcases/js_step_over_test.h
+++ b/tooling/test/testcases/js_step_over_test.h
@@ -96,6 +96,7 @@ private:
     static constexpr size_t STEP_SIZE = 7;
 
     std::string pandaFile_ = DEBUGGER_ABC_DIR "step.abc";
+    std::string sourceFile_ = DEBUGGER_JS_DIR "step.js";
     std::string entryPoint_ = "_GLOBAL::func_main_0";
     JSPtLocation location1_ {nullptr, JSPtLocation::EntityId(0), 0};
     JSPtLocation location2_ {nullptr, JSPtLocation::EntityId(0), 0};
@@ -107,8 +108,8 @@ private:
     void SetJSPtLocation(size_t *arr, size_t number, std::vector<JSPtLocation> &locations)
     {
         for (size_t i = 0; i < number; i++) {
-            JSPtLocation location_ =
-                TestUtil::GetLocation(arr[i * LINE_COLUMN], arr[i * LINE_COLUMN + 1], pandaFile_.c_str());
+            JSPtLocation location_ = TestUtil::GetLocation(sourceFile_.c_str(), arr[i * LINE_COLUMN],
+                                                           arr[i * LINE_COLUMN + 1], pandaFile_.c_str());
             locations.push_back(location_);
         }
     };
diff --git a/tooling/test/testcases/js_syntax_exception_test.h b/tooling/test/testcases/js_syntax_exception_test.h
index 7d9bdfef..6867b12c 100644
--- a/tooling/test/testcases/js_syntax_exception_test.h
+++ b/tooling/test/testcases/js_syntax_exception_test.h
@@ -55,7 +55,8 @@ public:
         };
 
         loadModule = [this](std::string_view moduleName) {
-            location_ = TestUtil::GetLocation(27, 0, pandaFile_.c_str()); // 27: breakpointer line
+            // 27: breakpointer line
+            location_ = TestUtil::GetLocation(sourceFile_.c_str(), 27, 0, pandaFile_.c_str());
             ASSERT_TRUE(location_.GetMethodId().IsValid());
             TestUtil::SuspendUntilContinue(DebugEvent::LOAD_MODULE);
             ASSERT_EQ(moduleName, pandaFile_);
@@ -93,7 +94,9 @@ public:
     ~JsSyntaxExceptionTest() = default;
 
 private:
-    std::string pandaFile_ = DEBUGGER_ABC_DIR "syntax_exception.abc";
+    const std::string testFile = "syntax_exception";
+    std::string pandaFile_ = DEBUGGER_ABC_DIR + testFile + ".abc";
+    std::string sourceFile_ = DEBUGGER_JS_DIR + testFile + ".js";
     std::string entryPoint_ = "_GLOBAL::func_main_0";
     JSPtLocation location_ {nullptr, JSPtLocation::EntityId(0), 0};
     size_t breakpointCounter_ = 0;
diff --git a/tooling/test/testcases/js_throw_exception_test.h b/tooling/test/testcases/js_throw_exception_test.h
index a8df9742..88f0fabc 100644
--- a/tooling/test/testcases/js_throw_exception_test.h
+++ b/tooling/test/testcases/js_throw_exception_test.h
@@ -56,7 +56,8 @@ public:
         };
 
         loadModule = [this](std::string_view moduleName) {
-            location_ = TestUtil::GetLocation(28, 0, pandaFile_.c_str()); // 28: breakpointer line
+            // 28: breakpointer line
+            location_ = TestUtil::GetLocation(sourceFile_.c_str(), 28, 0, pandaFile_.c_str());
             std::cout<<"vmStart1"<<std::endl;
             ASSERT_TRUE(location_.GetMethodId().IsValid());
             std::cout<<"vmStart2"<<std::endl;
@@ -96,7 +97,9 @@ public:
     ~JsThrowExceptionTest() = default;
 
 private:
-    std::string pandaFile_ = DEBUGGER_ABC_DIR "throw_exception.abc";
+    const std::string testFile = "throw_exception";
+    std::string pandaFile_ = DEBUGGER_ABC_DIR + testFile + ".abc";
+    std::string sourceFile_ = DEBUGGER_JS_DIR + testFile + ".js";
     std::string entryPoint_ = "_GLOBAL::func_main_0";
     JSPtLocation location_ {nullptr, JSPtLocation::EntityId(0), 0};
     size_t breakpointCounter_ = 0;
diff --git a/tooling/test/testcases/js_variable_test.h b/tooling/test/testcases/js_variable_test.h
new file mode 100644
index 00000000..cf2e1402
--- /dev/null
+++ b/tooling/test/testcases/js_variable_test.h
@@ -0,0 +1,239 @@
+/*
+ * Copyright (c) 2022 Huawei Device Co., Ltd.
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+#ifndef ECMASCRIPT_TOOLING_TEST_UTILS_TESTCASES_JS_VARIABLE_TEST_H
+#define ECMASCRIPT_TOOLING_TEST_UTILS_TESTCASES_JS_VARIABLE_TEST_H
+
+#include "test/utils/test_util.h"
+
+namespace panda::ecmascript::tooling::test {
+std::string g_pandaFile = DEBUGGER_ABC_DIR "variable.abc";
+std::string g_sourceFile = DEBUGGER_JS_DIR "variable.js";
+
+class JsVariableTest : public TestEvents {
+public:
+    JsVariableTest()
+    {
+        breakpoint = [this](const JSPtLocation &location) {
+            ASSERT_TRUE(location.GetMethodId().IsValid());
+            ASSERT_LOCATION_EQ(location, location_);
+            ++breakpointCounter_;
+            debugger_->NotifyPaused(location, PauseReason::INSTRUMENTATION);
+            TestUtil::SuspendUntilContinue(DebugEvent::BREAKPOINT, location);
+            return true;
+        };
+
+        loadModule = [this](std::string_view moduleName) {
+            static_cast<JsVariableTestChannel *>(channel_)->Initial(vm_, runtime_);
+            // 29: breakpointer line
+            location_ = TestUtil::GetLocation(g_sourceFile.c_str(), 29, 0, g_pandaFile.c_str());
+            ASSERT_TRUE(location_.GetMethodId().IsValid());
+            TestUtil::SuspendUntilContinue(DebugEvent::LOAD_MODULE);
+            ASSERT_EQ(moduleName, g_pandaFile);
+            ASSERT_TRUE(debugger_->NotifyScriptParsed(0, g_pandaFile));
+            auto condFuncRef = FunctionRef::Undefined(vm_);
+            auto ret = debugInterface_->SetBreakpoint(location_, condFuncRef);
+            ASSERT_TRUE(ret);
+            return true;
+        };
+
+        scenario = [this]() {
+            TestUtil::WaitForLoadModule();
+            TestUtil::Continue();
+            TestUtil::WaitForBreakpoint(location_);
+            TestUtil::Continue();
+            auto ret = debugInterface_->RemoveBreakpoint(location_);
+            ASSERT_TRUE(ret);
+            ASSERT_EXITED();
+            return true;
+        };
+
+        vmDeath = [this]() {
+            ASSERT_EQ(breakpointCounter_, 1U);  // 2: break point counter
+            return true;
+        };
+
+        channel_ = new JsVariableTestChannel();
+    }
+
+    std::pair<std::string, std::string> GetEntryPoint() override
+    {
+        return {g_pandaFile, entryPoint_};
+    }
+    ~JsVariableTest()
+    {
+        delete channel_;
+        channel_ = nullptr;
+    }
+
+private:
+    class JsVariableTestChannel : public TestChannel {
+    public:
+        JsVariableTestChannel() = default;
+        ~JsVariableTestChannel() = default;
+        void Initial(const EcmaVM *vm, RuntimeImpl *runtime)
+        {
+            vm_ = vm;
+            runtime_ = runtime;
+        }
+
+        void SendNotification(const PtBaseEvents &events) override
+        {
+            const static std::vector<std::function<bool(const PtBaseEvents &events)>> eventList = {
+                [](const PtBaseEvents &events) -> bool {
+                    auto parsed = static_cast<const ScriptParsed *>(&events);
+                    std::string str = parsed->ToJson()->Stringify();
+                    std::cout << "JsVariableTestChannel: SendNotification 0:\n" << str << std::endl;
+
+                    ASSERT_EQ(parsed->GetName(), "Debugger.scriptParsed");
+                    ASSERT_EQ(parsed->GetUrl(), g_sourceFile);
+                    return true;
+                },
+                [this](const PtBaseEvents &events) -> bool {
+                    auto paused = static_cast<const Paused *>(&events);
+                    std::string str = paused->ToJson()->Stringify();
+                    std::cout << "JsVariableTestChannel: SendNotification 1:\n" << str << std::endl;
+
+                    ASSERT_EQ(paused->GetName(), "Debugger.paused");
+                    auto frame = paused->GetCallFrames()->at(0).get();
+                    ASSERT_EQ(frame->GetFunctionName(), "foo");
+                    auto scopes = frame->GetScopeChain();
+                    ASSERT_EQ(scopes->size(), 2U);  // 2: contain local and global
+                    for (uint32_t i = 0; i < scopes->size(); i++) {
+                        auto scope = scopes->at(i).get();
+                        if (scope->GetType() != Scope::Type::Local()) {
+                            continue;
+                        }
+                        auto localId = scope->GetObject()->GetObjectId();
+                        GetPropertiesParams params;
+                        params.SetObjectId(localId).SetOwnProperties(true);
+                        std::vector<std::unique_ptr<PropertyDescriptor>> outPropertyDesc;
+                        runtime_->GetProperties(params, &outPropertyDesc, {}, {}, {});
+                        for (const auto &property : outPropertyDesc) {
+                            std::cout << "=====================================" << std::endl;
+                            std::cout << property->GetName() << std::endl;
+                            auto value = property->GetValue();
+                            std::vector<std::string> infos;
+                            PushValueInfo(value, infos);
+                            if (value->GetType() == RemoteObject::TypeName::Object) {
+                                std::vector<std::unique_ptr<PropertyDescriptor>> outPropertyDescInner;
+                                ASSERT_TRUE(value->HasObjectId());
+                                params.SetObjectId(value->GetObjectId()).SetOwnProperties(true);
+                                runtime_->GetProperties(params, &outPropertyDescInner, {}, {}, {});
+                                if (outPropertyDescInner.size() == 0) {
+                                    infos.push_back("none");
+                                }
+                                for (const auto &propertyInner : outPropertyDescInner) {
+                                    std::cout << "###########################################" << std::endl;
+                                    std::cout << propertyInner->GetName() << std::endl;
+                                    infos.push_back(propertyInner->GetName());
+                                    auto innerValue = propertyInner->GetValue();
+                                    PushValueInfo(innerValue, infos);
+                                }
+                            }
+                            ASSERT_EQ(infos.size(), variableMap_.at(property->GetName()).size());
+                            for (uint32_t j = 0; j < infos.size(); j++) {
+                                ASSERT_EQ(infos[j], variableMap_.at(property->GetName())[j]);
+                            }
+                        }
+                    }
+                    return true;
+                }
+            };
+
+            ASSERT_TRUE(eventList[index_](events));
+            index_++;
+        }
+
+    private:
+        NO_COPY_SEMANTIC(JsVariableTestChannel);
+        NO_MOVE_SEMANTIC(JsVariableTestChannel);
+
+        void PushValueInfo(RemoteObject *value, std::vector<std::string> &infos)
+        {
+            std::cout << "type: " << value->GetType() << std::endl;
+            infos.push_back(value->GetType());
+            if (value->HasObjectId()) {
+                std::cout << "id: " << value->GetObjectId() << std::endl;
+            }
+            if (value->HasSubType()) {
+                std::cout << "sub type: " << value->GetSubType() << std::endl;
+                infos.push_back(value->GetSubType());
+            }
+            if (value->HasClassName()) {
+                std::cout << "class name: " << value->GetClassName() << std::endl;
+                infos.push_back(value->GetClassName());
+            }
+            if (value->HasDescription()) {
+                std::cout << "desc: " << value->GetDescription() << std::endl;
+                infos.push_back(value->GetDescription());
+            }
+            if (value->HasValue()) {
+                std::cout << "type: " <<
+                    value->GetValue()->Typeof(vm_)->ToString() << std::endl;
+                std::cout << "tostring: " <<
+                    value->GetValue()->ToString(vm_)->ToString() << std::endl;
+                infos.push_back(value->GetValue()->ToString(vm_)->ToString());
+            }
+        }
+
+        /*
+        * Expect map type: map<name, value list>
+        * value list (optional):
+        *    type
+        *    subType
+        *    className
+        *    description
+        *    value tostring
+        *
+        * if is object value, will push back key and value.
+        *
+        * for example:
+        * var abc = 1
+        *     { "abc", { "number", "1", "1" } }
+        * var obj = { "key": "2" }
+        *     { "obj0", { "object", "Object", "Object", "[object Object]", "key", "string", "2", "2" } }
+        */
+        const std::map<std::string, std::vector<std::string>> variableMap_ = {
+            { "nop", { "undefined" } },
+            { "foo", { "function", "Function", "function foo( { [js code] }",
+                        "Cannot get source code of funtion"} },
+            { "string0", { "string", "helloworld", "helloworld" } },
+            { "string1", { "string", "", "" } },
+            { "number0", { "number", "1", "1" } },
+            { "number1", { "number", "65535", "65535" } },
+            { "obj0", { "object", "Object", "Object", "[object Object]",
+                        "key0", "string", "value0", "value0",
+                        "key1", "number", "100", "100" } },
+            { "obj1", { "object", "Object", "Object", "[object Object]", "none" } },
+        };
+
+        int32_t index_ {0};
+        const EcmaVM *vm_ {nullptr};
+        RuntimeImpl *runtime_ {nullptr};
+    };
+
+    std::string entryPoint_ = "_GLOBAL::func_main_0";
+    JSPtLocation location_ {nullptr, JSPtLocation::EntityId(0), 0};
+    size_t breakpointCounter_ = 0;
+};
+
+std::unique_ptr<TestEvents> GetJsVariableTest()
+{
+    return std::make_unique<JsVariableTest>();
+}
+}  // namespace panda::ecmascript::tooling::test
+
+#endif  // ECMASCRIPT_TOOLING_TEST_UTILS_TESTCASES_JS_VARIABLE_TEST_H
diff --git a/tooling/test/utils/test_channel.h b/tooling/test/utils/test_channel.h
new file mode 100644
index 00000000..e92a1b95
--- /dev/null
+++ b/tooling/test/utils/test_channel.h
@@ -0,0 +1,54 @@
+/*
+ * Copyright (c) 2022 Huawei Device Co., Ltd.
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+#ifndef ECMASCRIPT_TOOLING_TEST_UTILS_TEST_CHANNEL_H
+#define ECMASCRIPT_TOOLING_TEST_UTILS_TEST_CHANNEL_H
+
+#include "tooling/protocol_channel.h"
+
+namespace panda::ecmascript::tooling {
+class TestChannel : public ProtocolChannel {
+public:
+    TestChannel() = default;
+    virtual ~TestChannel() override = default;
+
+    void WaitForDebugger() override
+    {
+        std::cout << "TestChannel: WaitForDebugger" << std::endl;
+    }
+    void RunIfWaitingForDebugger() override
+    {
+        std::cout << "TestChannel: RunIfWaitingForDebugger" << std::endl;
+    }
+    void SendResponse([[maybe_unused]] const DispatchRequest &request,
+                      [[maybe_unused]] const DispatchResponse &response,
+                      const PtBaseReturns &result) override
+    {
+        std::string str = result.ToJson()->Stringify();
+        std::cout << "TestChannel: SendResponse:\n" << str << std::endl;
+    }
+    void SendNotification(const PtBaseEvents &events) override
+    {
+        std::string str = events.ToJson()->Stringify();
+        std::cout << "TestChannel: SendNotification:\n" << str << std::endl;
+    }
+
+private:
+    NO_COPY_SEMANTIC(TestChannel);
+    NO_MOVE_SEMANTIC(TestChannel);
+};
+}  // namespace panda::ecmascript::tooling
+
+#endif  // ECMASCRIPT_TOOLING_TEST_UTILS_TEST_CHANNEL_H
\ No newline at end of file
diff --git a/tooling/test/utils/test_events.h b/tooling/test/utils/test_events.h
index e2eeeeba..a03a1962 100644
--- a/tooling/test/utils/test_events.h
+++ b/tooling/test/utils/test_events.h
@@ -18,7 +18,9 @@
 
 #include <utility>
 
+#include "agent/runtime_impl.h"
 #include "backend/js_pt_hooks.h"
+#include "test/utils/test_channel.h"
 
 namespace panda::ecmascript::tooling::test {
 using BreakpointCallback = std::function<bool(const JSPtLocation &)>;
@@ -57,6 +59,8 @@ struct TestEvents {
     const EcmaVM *vm_ {nullptr};
     JSDebugger *debugInterface_ {nullptr};
     DebuggerImpl *debugger_ {nullptr};
+    RuntimeImpl *runtime_ {nullptr};
+    TestChannel *channel_ {nullptr};
     TestEvents();
     virtual ~TestEvents() = default;
 
diff --git a/tooling/test/utils/test_hooks.h b/tooling/test/utils/test_hooks.h
index f791b7f0..54884743 100644
--- a/tooling/test/utils/test_hooks.h
+++ b/tooling/test/utils/test_hooks.h
@@ -25,12 +25,13 @@ class TestHooks : public PtHooks {
 public:
     TestHooks(const std::string &testName, const EcmaVM *vm) : vm_(vm)
     {
-        runtime_ = std::make_unique<RuntimeImpl>(vm, nullptr);
-        debugger_ = std::make_unique<DebuggerImpl>(vm, nullptr, runtime_.get());
         testName_ = testName;
         test_ = TestUtil::GetTest(testName);
+        runtime_ = std::make_unique<RuntimeImpl>(vm, test_->channel_);
+        debugger_ = std::make_unique<DebuggerImpl>(vm, test_->channel_, runtime_.get());
         test_->vm_ = vm;
         test_->debugger_ = debugger_.get();
+        test_->runtime_ = runtime_.get();
         test_->debugInterface_ = debugger_->jsDebugger_;
         debugInterface_ = debugger_->jsDebugger_;
         TestUtil::Reset();
diff --git a/tooling/test/utils/test_list.cpp b/tooling/test/utils/test_list.cpp
index 0a07229c..954c48e4 100644
--- a/tooling/test/utils/test_list.cpp
+++ b/tooling/test/utils/test_list.cpp
@@ -29,6 +29,7 @@
 #include "test/testcases/js_step_out_test.h"
 #include "test/testcases/js_syntax_exception_test.h"
 #include "test/testcases/js_throw_exception_test.h"
+#include "test/testcases/js_variable_test.h"
 
 namespace panda::ecmascript::tooling::test {
 static std::string g_currentTestName = "";
@@ -47,6 +48,7 @@ static void RegisterTests()
     TestUtil::RegisterTest("JsStepIntoTest", GetJsStepIntoTest());
     TestUtil::RegisterTest("JsStepOverTest", GetJsStepOverTest());
     TestUtil::RegisterTest("JsStepOutTest", GetJsStepOutTest());
+    TestUtil::RegisterTest("JsVariableTest", GetJsVariableTest());
 }
 
 std::vector<const char *> GetTestList()
diff --git a/tooling/test/utils/test_util.h b/tooling/test/utils/test_util.h
index 19955a33..fd755709 100644
--- a/tooling/test/utils/test_util.h
+++ b/tooling/test/utils/test_util.h
@@ -129,7 +129,7 @@ public:
         return lastEvent_ == DebugEvent::VM_DEATH;
     }
 
-    static JSPtLocation GetLocation(int32_t line, int32_t column, const char *pandaFile)
+    static JSPtLocation GetLocation(const char *sourceFile, int32_t line, int32_t column, const char *pandaFile)
     {
         auto jsPandaFile = ::panda::ecmascript::JSPandaFileManager::GetInstance()->FindJSPandaFile(pandaFile);
         if (jsPandaFile == nullptr) {
@@ -138,7 +138,7 @@ public:
         }
         TestExtractor extractor(jsPandaFile);
         auto [id, offset] = extractor.GetBreakpointAddress({jsPandaFile, line, column});
-        return JSPtLocation(jsPandaFile, id, offset);
+        return JSPtLocation(jsPandaFile, id, offset, sourceFile);
     }
 
     static SourceLocation GetSourceLocation(const JSPtLocation &location, const char *pandaFile)
@@ -162,7 +162,7 @@ public:
 
         // Wait for continue
         while (suspended_) {
-            constexpr uint64_t TIMEOUT_MSEC = 10000U;
+            constexpr uint64_t TIMEOUT_MSEC = 120000U;
             bool timeExceeded = suspendCv_.TimedWait(&suspendMutex_, TIMEOUT_MSEC);
             if (timeExceeded) {
                 LOG_DEBUGGER(FATAL) << "Time limit exceeded while suspend";
@@ -190,7 +190,7 @@ private:
             if (lastEvent_ == DebugEvent::VM_DEATH) {
                 return false;
             }
-            constexpr uint64_t TIMEOUT_MSEC = 10000U;
+            constexpr uint64_t TIMEOUT_MSEC = 120000U;
             bool timeExceeded = eventCv_.TimedWait(&eventMutex_, TIMEOUT_MSEC);
             if (timeExceeded) {
                 LOG_DEBUGGER(FATAL) << "Time limit exceeded while waiting " << event;
-- 
Gitee