Fix windows issue

TileDB-Inc · Jul 11, 2024 · fa37615 · fa37615
1 parent 9bb1de7
commit fa37615
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 7 deletions.
diff --git a/tiledb/api/c_api/attribute/test/unit_capi_attribute.cc b/tiledb/api/c_api/attribute/test/unit_capi_attribute.cc
@@ -34,6 +34,7 @@
 #include "../../../c_api_test_support/testsupport_capi_datatype.h"
 #include "../../filter_list/filter_list_api_internal.h"
 #include "../attribute_api_external.h"
+
 using namespace tiledb::api::test_support;
 
 TEST_CASE(
@@ -366,14 +367,12 @@ TEST_CASE(
     tiledb_string_t* tdb_string;
     capi_return_t rc =
         tiledb_attribute_dump_str(nullptr, attr.attribute, &tdb_string);
-    tiledb_string_free(&tdb_string);
     REQUIRE(tiledb_status(rc) == TILEDB_INVALID_CONTEXT);
   }
   SECTION("null dimension") {
     tiledb_string_t* tdb_string;
     capi_return_t rc =
         tiledb_attribute_dump_str(attr.context(), nullptr, &tdb_string);
-    tiledb_string_free(&tdb_string);
     REQUIRE(tiledb_status(rc) == TILEDB_ERR);
   }
   // SECTION("null file pointer") `nullptr` is allowed; it's mapped to `stdout`

diff --git a/tiledb/api/c_api/dimension/test/unit_capi_dimension.cc b/tiledb/api/c_api/dimension/test/unit_capi_dimension.cc
@@ -34,6 +34,7 @@
 #include "../../../c_api_test_support/testsupport_capi_datatype.h"
 #include "../../filter_list/filter_list_api_internal.h"
 #include "../dimension_api_internal.h"
+
 using namespace tiledb::api::test_support;
 
 TEST_CASE(
@@ -344,14 +345,12 @@ TEST_CASE(
     tiledb_string_t* tdb_string;
     capi_return_t rc =
         tiledb_dimension_dump_str(nullptr, dim.dimension, &tdb_string);
-    tiledb_string_free(&tdb_string);
     REQUIRE(tiledb_status(rc) == TILEDB_INVALID_CONTEXT);
   }
   SECTION("null dimension") {
     tiledb_string_t* tdb_string;
     capi_return_t rc =
         tiledb_dimension_dump_str(dim.ctx.context, nullptr, &tdb_string);
-    tiledb_string_free(&tdb_string);
     REQUIRE(tiledb_status(rc) == TILEDB_ERR);
   }
   // SECTION("null file pointer") `nullptr` is allowed; it's mapped to `stdout`

diff --git a/tiledb/api/c_api/domain/test/unit_capi_domain.cc b/tiledb/api/c_api/domain/test/unit_capi_domain.cc
@@ -29,7 +29,6 @@
  */
 #define CATCH_CONFIG_MAIN
 #include <test/support/tdb_catch.h>
-
 #include "../../../c_api_test_support/testsupport_capi_context.h"
 #include "../domain_api_external.h"
 #include "../domain_api_internal.h"
@@ -249,13 +248,11 @@ TEST_CASE("C API: tiledb_domain_dump argument validation", "[capi][domain]") {
   SECTION("null context") {
     tiledb_string_t* tdb_string;
     auto rc{tiledb_domain_dump_str(nullptr, dom.domain, &tdb_string)};
-    tiledb_string_free(&tdb_string);
     CHECK(rc == TILEDB_INVALID_CONTEXT);
   }
   SECTION("null domain") {
     tiledb_string_t* tdb_string;
     auto rc{tiledb_domain_dump_str(ctx, nullptr, &tdb_string)};
-    tiledb_string_free(&tdb_string);
     CHECK(rc == TILEDB_ERR);
   }
   // SECTION("null FILE") is omitted. Null FILE* defaults to stderr