test(bindings): run full suite with absolute path (second run), add test_seekdb_absolute, remove in-process close+open test

Author: dengfuping

unittest/include/.gitignore

@@ -1,3 +1,4 @@
 seekdb.db
 build
-node_modules
+node_modules
+*.db

unittest/include/CMakeLists.txt

@@ -15,7 +15,12 @@ target_link_libraries(test_seekdb
     libseekdb
 )
 
-# Add test target
+# Add test target: first run with default (relative) path
 add_test(NAME test_seekdb COMMAND test_seekdb)
 set_tests_properties(test_seekdb PROPERTIES TIMEOUT 300)
 
+# Second run with absolute path (same as Rust/Python/Go/Node bindings)
+set(SEEKDB_ABS_DIR "${CMAKE_CURRENT_BINARY_DIR}/seekdb_abs.db")
+add_test(NAME test_seekdb_absolute COMMAND test_seekdb ${SEEKDB_ABS_DIR})
+set_tests_properties(test_seekdb_absolute PROPERTIES TIMEOUT 300 WORKING_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR})
+

unittest/include/go/test.sh

@@ -43,7 +43,26 @@ echo "Running Go tests..."
 echo ""
 # Note: C ABI layer handles SIGSEGV gracefully during static destructors
 # Exit code is 0 and no segfault messages are output
-go run test.go
+go run test.go "${DB_DIR}"
+GO_EXIT=$?
+if [ $GO_EXIT -ne 0 ]; then
+    echo "First run (relative path) failed with exit $GO_EXIT"
+    exit $GO_EXIT
+fi
+
+# Second run: absolute path (same suite, no close+reopen in process)
+DB_DIR_ABS="$(pwd)/seekdb_abs.db"
+rm -rf "${DB_DIR_ABS}"
+echo ""
+echo "Running Go tests with absolute path: $DB_DIR_ABS"
+echo ""
+go run test.go "${DB_DIR_ABS}"
+ABS_EXIT=$?
+rm -rf "${DB_DIR_ABS}" 2>/dev/null || true
+if [ $ABS_EXIT -ne 0 ]; then
+    echo "Second run (absolute path) failed with exit $ABS_EXIT"
+    exit $ABS_EXIT
+fi
 
 echo ""
 echo "Test completed!"

unittest/include/nodejs/test.js

@@ -832,6 +832,7 @@ if (require.main === module) {
 
 module.exports = {
     testOpen,
+    testAbsolutePathOpen,
     testConnection,
     testErrorHandling,
     testResultOperations,

unittest/include/nodejs/test.sh

@@ -49,7 +49,26 @@ echo "Running Node.js tests..."
 echo ""
 # Note: C ABI layer handles SIGSEGV gracefully during static destructors
 # Exit code is 0 and no segfault messages are output
-node test.js
+node test.js "${DB_DIR}" "test"
+NODE_EXIT=$?
+if [ $NODE_EXIT -ne 0 ]; then
+    echo "First run (relative path) failed with exit $NODE_EXIT"
+    exit $NODE_EXIT
+fi
+
+# Second run: absolute path (same suite, no close+reopen in process)
+DB_DIR_ABS="$(pwd)/seekdb_abs.db"
+rm -rf "${DB_DIR_ABS}"
+echo ""
+echo "Running Node.js tests with absolute path: $DB_DIR_ABS"
+echo ""
+node test.js "${DB_DIR_ABS}" "test"
+ABS_EXIT=$?
+rm -rf "${DB_DIR_ABS}" 2>/dev/null || true
+if [ $ABS_EXIT -ne 0 ]; then
+    echo "Second run (absolute path) failed with exit $ABS_EXIT"
+    exit $ABS_EXIT
+fi
 
 echo ""
 echo "Test completed!"

unittest/include/nodejs_napi/test.sh

@@ -49,7 +49,26 @@ echo "Running Node.js N-API tests..."
 echo ""
 # Note: C ABI layer handles SIGSEGV gracefully during static destructors
 # Exit code is 0 and no segfault messages are output
-node test.js "$@"
+node test.js "${DB_DIR}" "test"
+NODE_EXIT=$?
+if [ $NODE_EXIT -ne 0 ]; then
+    echo "First run (relative path) failed with exit $NODE_EXIT"
+    exit $NODE_EXIT
+fi
+
+# Second run: absolute path (same suite, no close+reopen in process)
+DB_DIR_ABS="${SCRIPT_DIR}/seekdb_abs.db"
+rm -rf "${DB_DIR_ABS}"
+echo ""
+echo "Running Node.js N-API tests with absolute path: $DB_DIR_ABS"
+echo ""
+node test.js "${DB_DIR_ABS}" "test"
+ABS_EXIT=$?
+rm -rf "${DB_DIR_ABS}" 2>/dev/null || true
+if [ $ABS_EXIT -ne 0 ]; then
+    echo "Second run (absolute path) failed with exit $ABS_EXIT"
+    exit $ABS_EXIT
+fi
 
 echo ""
 echo "Test completed!"

unittest/include/python/test.sh

@@ -53,6 +53,25 @@ echo ""
 # Python also uses os._exit() to avoid cleanup issues
 # Exit code is 0 and no segfault messages are output
 $PYTHON_CMD -u test.py "${DB_DIR}" "test"
+PY_EXIT=$?
+if [ $PY_EXIT -ne 0 ]; then
+    echo "First run (relative path) failed with exit $PY_EXIT"
+    exit $PY_EXIT
+fi
+
+# Second run: absolute path (same suite, no close+reopen in process)
+DB_DIR_ABS="$(pwd)/seekdb_abs.db"
+rm -rf "${DB_DIR_ABS}"
+echo ""
+echo "Running Python tests with absolute path: $DB_DIR_ABS"
+echo ""
+$PYTHON_CMD -u test.py "${DB_DIR_ABS}" "test"
+ABS_EXIT=$?
+rm -rf "${DB_DIR_ABS}" 2>/dev/null || true
+if [ $ABS_EXIT -ne 0 ]; then
+    echo "Second run (absolute path) failed with exit $ABS_EXIT"
+    exit $ABS_EXIT
+fi
 
 echo ""
 echo "Test completed!"

unittest/include/rust/test.sh

@@ -65,6 +65,25 @@ if [ ! -f "$TEST_BIN" ]; then
 fi
 
 "$TEST_BIN" "${DB_DIR}" "test"
+RUST_EXIT=$?
+if [ $RUST_EXIT -ne 0 ]; then
+    echo "First run (relative path) failed with exit $RUST_EXIT"
+    exit $RUST_EXIT
+fi
+
+# Second run: absolute path (same suite, no close+reopen in process)
+DB_DIR_ABS="$(pwd)/seekdb_abs.db"
+rm -rf "${DB_DIR_ABS}"
+echo ""
+echo "Running Rust tests with absolute path: $DB_DIR_ABS"
+echo ""
+"$TEST_BIN" "${DB_DIR_ABS}" "test"
+ABS_EXIT=$?
+rm -rf "${DB_DIR_ABS}" 2>/dev/null || true
+if [ $ABS_EXIT -ne 0 ]; then
+    echo "Second run (absolute path) failed with exit $ABS_EXIT"
+    exit $ABS_EXIT
+fi
 
 echo ""
 echo "Test completed!"

unittest/include/test_seekdb.cpp

@@ -25,6 +25,9 @@ struct TestResult {
     std::string message;
 };
 
+// Database path used by main(); set in main() so same-path-reuse tests use it
+static const char* g_db_path = "./seekdb.db";
+
 // Test database open (database is already open in main, just verify it's open)
 TestResult test_open() {
     // Database is already open in main(), so just verify we can create a connection
@@ -2713,13 +2716,13 @@ TestResult test_empty_json_metadata() {
     return {true, ""};
 }
 
-// Same-process same-path reuse: second seekdb_open() with same path (relative) must return success (no "db opened by other process").
+// Same-process same-path reuse: second seekdb_open() with same path must return success (no "db opened by other process").
 TestResult test_embedded_same_path_reuse_relative() {
-    // Database already opened in main() with "./seekdb.db"; open again with same path must succeed
-    int ret = seekdb_open("./seekdb.db");
+    // Re-open with the same path used in main() (relative or absolute)
+    int ret = seekdb_open(g_db_path);
     if (ret != SEEKDB_SUCCESS) {
         const char* err = seekdb_last_error();
-        return {false, std::string("second seekdb_open(\"./seekdb.db\") should succeed, got: ") + (err ? err : "unknown")};
+        return {false, std::string("second seekdb_open(same path) should succeed, got: ") + (err ? err : "unknown")};
     }
     return {true, ""};
 }
@@ -2730,7 +2733,7 @@ TestResult test_embedded_same_path_reuse_absolute() {
     if (getcwd(cwd, sizeof(cwd)) == nullptr) {
         return {false, "getcwd failed"};
     }
-    std::string abs_path = std::string(cwd) + "/seekdb.db";
+    std::string abs_path = (g_db_path[0] == '/') ? g_db_path : (std::string(cwd) + "/" + g_db_path);
     int ret = seekdb_open(abs_path.c_str());
     if (ret != SEEKDB_SUCCESS) {
         const char* err = seekdb_last_error();
@@ -4304,14 +4307,17 @@ TestResult test_stmt_store_result() {
     return {true, ""};
 }
 
-int main() {
+int main(int argc, char* argv[]) {
+    // Database path: argv[1] if provided, else default relative path (same as other bindings)
+    g_db_path = (argc >= 2 && argv[1] && argv[1][0] != '\0') ? argv[1] : "./seekdb.db";
+
     std::cout << std::string(70, '=') << std::endl;
     std::cout << "SeekDB C++ Binding Test Suite" << std::endl;
     std::cout << std::string(70, '=') << std::endl;
     std::cout << std::endl;
-    
+
     // Open database once at the beginning
-    int ret = seekdb_open("./seekdb.db");
+    int ret = seekdb_open(g_db_path);
     if (ret != SEEKDB_SUCCESS) {
         std::cerr << "Failed to open database: " << ret << std::endl;
         return 1;