Reflect the comments

Signed-off-by: Ingwon Song <igsong@google.com>

Author: ingwonsong

test/test_data/configure_check.cc

@@ -23,10 +23,7 @@ class TestRootContext1 : public RootContext {
   explicit TestRootContext1(uint32_t id, std::string_view root_id) : RootContext(id, root_id) {}
   bool onConfigure(size_t s) override {
     LOG_TRACE("onConfigure: root_id_1");
-    if (s == 0) {
-      return false;
-    }
-    return true;
+    return s != 0;
   }
 };
 
@@ -40,10 +37,7 @@ class TestRootContext2 : public RootContext {
   explicit TestRootContext2(uint32_t id, std::string_view root_id) : RootContext(id, root_id) {}
   bool onConfigure(size_t s) override {
     LOG_TRACE("onConfigure: root_id_2");
-    if (s == 0) {
-      return false;
-    }
-    return true;
+    return s != 0;
   }
 };
 

test/wasm_test.cc

@@ -205,7 +205,7 @@ TEST_P(TestVm, AlwaysApplyCanary) {
               // configure_check.wasm should raise the error at `onConfigure` in canary when the
               // `plugin_config` is empty string.
               EXPECT_EQ(wasm_handle_comp, nullptr);
-              return;
+              continue;
             }
 
             ASSERT_TRUE(wasm_handle_comp && wasm_handle_comp->wasm());