Tests: enable the v5-clean reference double-free conformance test

Adds test_reference_double_free (js-native-api) to the enabled suite -- a real v5 reference/double-free
test, green on macOS (system JSC, including ASan) and Android (V8). Its test_wrap.js is quarantined via
a known-failing allow-list in test_main.cpp: JSC napi_remove_wrap on an unwrapped object returns
napi_invalid_arg (the consecutive remove_wrap+delete_reference path itself does not crash); tracked as a
separate JSC fix.

The rest of the reference/finalizer/wrap suite (test_reference, test_finalizer, 6_object_wrap) targets
newer Node-API -- node_api_symbol_for (v9), napi_get_instance_data (v6),
node_api_basic_env / node_api_post_finalizer (v9) -- and is documented for the staged NAPI_VERSION bump.

Author: matthargett

Tests/NodeApi/CMakeLists.txt

@@ -7,6 +7,14 @@ set(JSR_NODE_API_NATIVE_TEST_DIRS
     3_callbacks
     4_object_factory
     5_function_factory
+    # v5-clean reference coverage. The rest of the reference/finalizer/wrap suite targets newer
+    # Node-API and is staged for the NAPI_VERSION bump:
+    #   test_reference      -> node_api_symbol_for (v9)
+    #   test_finalizer/     -> napi_get_instance_data (v6), node_api_basic_env / node_api_post_finalizer (v9)
+    #   6_object_wrap       -> napi_get_instance_data (v6), node_api_basic_env (v9)
+    # (test_reference_double_free/test_wrap.js is quarantined in test_main.cpp -- a JSC remove_wrap
+    #  edge case that returns napi_invalid_arg; tracked as a separate fix, not a v5 blocker.)
+    test_reference_double_free
 )
 
 function(node_api_copy_test_sources TARGET_NAME)

Tests/NodeApi/test_main.cpp

@@ -60,6 +60,21 @@ class NodeApiTestFixture : public TestFixtureBase {
     ASSERT_TRUE(static_cast<bool>(config.run_script))
         << "Node-API test runner is not configured.";
 
+    // Quarantined conformance cases: known-failing against the current v5 surface. Kept registered
+    // (and skipped, so they stay visible) rather than silently dropped. Each needs a separate fix or
+    // a NAPI_VERSION bump and must not block the v5 suite.
+    static constexpr const char* kQuarantined[] = {
+        // JSC napi_remove_wrap on an object that was never wrapped returns napi_invalid_arg, which the
+        // runner surfaces as a failure. The consecutive remove_wrap+delete_reference path itself does
+        // not crash. Tracked separately as a JSC remove_wrap fix.
+        "test_reference_double_free/test_wrap.js",
+    };
+    for (const char* quarantined : kQuarantined) {
+      if (m_jsFilePath.generic_string().find(quarantined) != std::string::npos) {
+        GTEST_SKIP() << "Quarantined (known-failing on the v5 surface): " << quarantined;
+      }
+    }
+
     ProcessResult result = config.run_script(m_jsFilePath);
     if (result.status == 0) {
       return;

Tests/UnitTests/Android/app/src/main/cpp/CMakeLists.txt

@@ -74,7 +74,7 @@ if(ANDROID)
     target_link_libraries(UnitTestsJNI PRIVATE dl)
 endif()
 target_compile_definitions(UnitTestsJNI
-    PRIVATE NODE_API_AVAILABLE_NATIVE_TESTS="2_function_arguments,3_callbacks,4_object_factory,5_function_factory")
+    PRIVATE NODE_API_AVAILABLE_NATIVE_TESTS="2_function_arguments,3_callbacks,4_object_factory,5_function_factory,test_reference_double_free")
 
 # The conformance addons are built as standalone SHARED lib<name>.so by Tests/NodeApi/CMakeLists.txt,
 # packaged by AGP into nativeLibraryDir, and dlopen'd in-process by node_lite_android. Both they and