From 1465c3c0eb9760cbad5427e46463db9ea4278223 Mon Sep 17 00:00:00 2001
From: yangkun27 <yangkun27@xiaomi.com>
Date: Fri, 14 Feb 2025 16:13:00 +0800
Subject: [PATCH] Unit test:type matching issue and code redundancy (#4079)

---
 tests/unit/interpreter/interpreter_test.cc |  2 +-
 tests/unit/wasm-vm/wasm_vm.cc              | 12 +++++-------
 2 files changed, 6 insertions(+), 8 deletions(-)

diff --git a/tests/unit/interpreter/interpreter_test.cc b/tests/unit/interpreter/interpreter_test.cc
index e9fa8f9e..af00020c 100644
--- a/tests/unit/interpreter/interpreter_test.cc
+++ b/tests/unit/interpreter/interpreter_test.cc
@@ -45,6 +45,6 @@ TEST_F(InterpreterTest, wasm_runtime_is_built_in_module)
     bool ret = wasm_runtime_is_built_in_module("env");
     ASSERT_TRUE(ret);
 
-    ret = ret = wasm_runtime_is_built_in_module("env1");
+    ret = wasm_runtime_is_built_in_module("env1");
     ASSERT_FALSE(ret);
 }
\ No newline at end of file
diff --git a/tests/unit/wasm-vm/wasm_vm.cc b/tests/unit/wasm-vm/wasm_vm.cc
index f4f5a834..73c2c761 100644
--- a/tests/unit/wasm-vm/wasm_vm.cc
+++ b/tests/unit/wasm-vm/wasm_vm.cc
@@ -65,7 +65,7 @@ class WasmVMTest : public testing::Test
 
 TEST_F(WasmVMTest, Test_app1)
 {
-    unsigned argv[10];
+    uint32 argv[10];
 
     ASSERT_TRUE(app1_wasm != NULL);
 
@@ -151,7 +151,7 @@ TEST_F(WasmVMTest, Test_app1)
 
 TEST_F(WasmVMTest, Test_app2)
 {
-    unsigned argv[10];
+    uint32 argv[10];
 
     /* Load module */
     module = wasm_runtime_load(app2_wasm, sizeof(app2_wasm), error_buf,
@@ -416,7 +416,7 @@ TEST_F(WasmVMTest, Test_app2)
 
 TEST_F(WasmVMTest, Test_app3)
 {
-    unsigned argv[10];
+    uint32 argv[10];
 
     /* Load module */
     module = wasm_runtime_load(app3_wasm, sizeof(app3_wasm), error_buf,
@@ -465,7 +465,6 @@ TEST_F(WasmVMTest, Test_app3)
     ASSERT_TRUE(buf != NULL);
 
     ASSERT_EQ(wasm_runtime_addr_native_to_app(module_inst, buf), argv[0]);
-    int32 buf_offset = argv[0];
 
     /* call my_malloc */
     func_inst = wasm_runtime_lookup_function(module_inst, "my_malloc");
@@ -482,7 +481,6 @@ TEST_F(WasmVMTest, Test_app3)
     ASSERT_TRUE(buf1 != NULL);
 
     ASSERT_EQ(wasm_runtime_addr_native_to_app(module_inst, buf1), argv[0]);
-    int32 buf_offset1 = argv[0];
 
     wasm_runtime_deinstantiate(module_inst);
     wasm_runtime_unload(module);
@@ -526,7 +524,7 @@ TEST_F(WasmVMTest, Test_app4_single)
     uint8 *buffer = NULL;
     uint32 buffer_size = 0;
     bool ret = false;
-    unsigned argv[10];
+    uint32 argv[10];
 
     wasm_runtime_set_module_reader(&module_reader_callback,
                                    &module_destroyer_callback);
@@ -584,7 +582,7 @@ TEST_F(WasmVMTest, Test_app4_plus_one)
     uint8 *buffer = NULL;
     uint32 buffer_size = 0;
     bool ret = false;
-    uint32_t argv[10] = { 0 };
+    uint32 argv[10] = { 0 };
 
     wasm_runtime_set_module_reader(&module_reader_callback,
                                    &module_destroyer_callback);