Address final comments

TileDB-Inc · Dec 13, 2023 · 2308151 · 2308151
1 parent de88fcf
commit 2308151
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/test/src/unit-capi-vfs.cc b/test/src/unit-capi-vfs.cc
@@ -36,6 +36,7 @@
 #include "test/support/src/vfs_helpers.h"
 #include "tiledb/api/c_api/vfs/vfs_api_internal.h"
 #include "tiledb/api/c_api_test_support/testsupport_capi_vfs.h"
+#include "tiledb/platform/platform.h"
 #include "tiledb/sm/c_api/tiledb.h"
 #include "tiledb/sm/filesystem/uri.h"
 #include "tiledb/sm/misc/utils.h"
@@ -440,7 +441,7 @@ TEST_CASE("C API: Test virtual filesystem", "[capi][vfs]") {
     }
 
     // Check copy (not yet supported for MemFS or HDFS)
-    if constexpr (!tiledb::sm::filesystem::windows_enabled) {
+    if constexpr (!tiledb::platform::is_os_windows) {
       if (!tiledb::sm::URI::is_memfs(path) && !tiledb::sm::URI::is_hdfs(path)) {
         auto dir = path + "dir/";
         auto file = dir + "file";

diff --git a/test/support/src/vfs_helpers.h b/test/support/src/vfs_helpers.h
@@ -585,6 +585,18 @@ struct vfs_config {
     }
   }
 
+  /** Copy constructor is deleted. */
+  vfs_config(const vfs_config&) = delete;
+
+  /** Move constructor is deleted. */
+  vfs_config(vfs_config&&) = delete;
+
+  /** Copy assignment is deleted. */
+  vfs_config& operator=(const vfs_config&) = delete;
+
+  /** Move assignment is deleted. */
+  vfs_config& operator=(vfs_config&&) = delete;
+
   /** Destructor. */
   ~vfs_config() {
     tiledb_config_free(&config);