feat: add graph error handling tests and update build configurations (#712)

Author: halajohn

.vscode/launch.json

@@ -137,7 +137,7 @@
       "request": "launch",
       "program": "${workspaceFolder}/out/linux/x64/tests/standalone/ten_runtime_smoke_test",
       "args": [
-        "--gtest_filter=PropertyTest.InGraphUseEnv3"
+        "--gtest_filter=GraphTest.FailedToCreateExtension"
       ],
       "cwd": "${workspaceFolder}/out/linux/x64/tests/standalone/",
       "env": {

core/src/ten_runtime/extension/ten_env/on_xxx.c

@@ -209,6 +209,8 @@ bool ten_extension_on_init_done(ten_env_t *self) {
              "Should not happen.");
 
   if (extension_thread->is_close_triggered) {
+    // Do not proceed with the subsequent start flow, as the extension thread is
+    // about to shut down.
     TEN_LOGD(
         "[%s] Since the close process has already been triggered, no further "
         "steps will be carried out after `on_init_done`. Enter `on_stop`"

core/src/ten_runtime/extension_thread/extension_thread.c

@@ -441,6 +441,8 @@ static void ten_extension_thread_start_life_cycle_of_all_extensions_task(
   TEN_ASSERT(ten_extension_thread_check_integrity(self, true),
              "Should not happen.");
 
+  // The extension system is about to be shut down, so do not proceed with
+  // initialization anymore.
   if (self->is_close_triggered) {
     return;
   }

tests/ten_runtime/smoke/BUILD.gn

@@ -47,6 +47,7 @@ ten_executable("ten_runtime_smoke_test") {
     "extension_group",
     "failed_to_transfer_msg",
     "graph",
+    "graph_error",
     "graph_name",
     "graph_test",
     "lock",

tests/ten_runtime/smoke/graph_error/BUILD.gn

@@ -0,0 +1,24 @@
+#
+# Copyright © 2025 Agora
+# This file is part of TEN Framework, an open source project.
+# Licensed under the Apache License, Version 2.0, with certain conditions.
+# Refer to the "LICENSE" file in the root directory for more information.
+#
+import("//build/ten_runtime/glob.gni")
+import("//build/ten_runtime/ten.gni")
+
+ten_runtime_glob("graph_error") {
+  file_list = all_native_files
+  deps = [
+    "//third_party/msgpack:msgpackc",
+    "//third_party/nlohmann_json",
+  ]
+  include_dirs = [
+    "//packages",
+    "//tests/ten_runtime",
+  ]
+  public_deps = [
+    "//third_party/googlemock",
+    "//third_party/googletest",
+  ]
+}

tests/ten_runtime/smoke/graph_test/failed_to_create_extension.cc renamed to tests/ten_runtime/smoke/graph_error/failed_to_create_extension.cc

@@ -100,7 +100,7 @@ TEN_CONSTRUCTOR(
 
 }  // namespace
 
-TEST(GraphTest, FailedToCreateExtension) {  // NOLINT
+TEST(GraphErrorTest, FailedToCreateExtension) {  // NOLINT
   auto *app_thread =
       ten_thread_create("app thread", test_app_thread_main, nullptr);
 

tests/ten_runtime/smoke/predefined_graph/predefined_graph_incorrect_1.cc renamed to tests/ten_runtime/smoke/graph_error/predefined_graph_incorrect_1.cc

@@ -79,7 +79,7 @@ TEN_CPP_REGISTER_ADDON_AS_EXTENSION(
 
 }  // namespace
 
-TEST(PredefinedGraphTest, PredefinedGraphIncorrect1) {  // NOLINT
+TEST(GraphErrorTest, PredefinedGraphIncorrect1) {  // NOLINT
   auto *app_thread = ten_thread_create("app thread", app_thread_main, nullptr);
 
   ten_thread_join(app_thread, -1);

tests/ten_runtime/smoke/predefined_graph/predefined_graph_incorrect_2.cc renamed to tests/ten_runtime/smoke/graph_error/predefined_graph_incorrect_2.cc

@@ -84,7 +84,7 @@ TEN_CPP_REGISTER_ADDON_AS_EXTENSION(
 
 }  // namespace
 
-TEST(PredefinedGraphTest, PredefinedGraphIncorrect2) {  // NOLINT
+TEST(GraphErrorTest, PredefinedGraphIncorrect2) {  // NOLINT
   auto *app_thread = ten_thread_create("app thread", app_thread_main, nullptr);
 
   ten_thread_join(app_thread, -1);

tests/ten_runtime/smoke/predefined_graph/predefined_graph_incorrect_3.cc renamed to tests/ten_runtime/smoke/graph_error/predefined_graph_incorrect_3.cc

@@ -84,7 +84,7 @@ TEN_CPP_REGISTER_ADDON_AS_EXTENSION(
 
 }  // namespace
 
-TEST(PredefinedGraphTest, PredefinedGraphIncorrect3) {  // NOLINT
+TEST(GraphErrorTest, PredefinedGraphIncorrect3) {  // NOLINT
   auto *app_thread = ten_thread_create("app thread", app_thread_main, nullptr);
 
   ten_thread_join(app_thread, -1);

tests/ten_runtime/smoke/property/property_in_graph_use_env_3.cc renamed to tests/ten_runtime/smoke/graph_error/property_in_graph_use_env_3.cc

@@ -98,7 +98,7 @@ TEN_CPP_REGISTER_ADDON_AS_EXTENSION(property_in_graph_use_env_3__extension,
 
 }  // namespace
 
-TEST(PropertyTest, DISABLED_InGraphUseEnv3) {  // NOLINT
+TEST(GraphErrorTest, DISABLED_InGraphUseEnv3) {  // NOLINT
   // Start app.
   auto *app_thread =
       ten_thread_create("app thread", test_app_thread_main, nullptr);