Run clang-format

kuzudb · Nov 21, 2024 · 2cf9207 · 2cf9207
1 parent f6e7168
commit 2cf9207
Show file tree

Hide file tree

Showing 3 changed files with 26 additions and 17 deletions.
diff --git a/src/common/file_system/local_file_system.cpp b/src/common/file_system/local_file_system.cpp
@@ -222,7 +222,7 @@ void LocalFileSystem::createDir(const std::string& dir) const {
         auto directoryToCreate = dir;
         if (directoryToCreate.ends_with('/')
 #if defined(_WIN32)
-        || directoryToCreate.ends_with('\\')
+            || directoryToCreate.ends_with('\\')
 #endif
         ) {
             // This is a known issue with std::filesystem::create_directories. (link:
@@ -261,7 +261,7 @@ bool isSubdirectory(const std::filesystem::path& base, const std::filesystem::pa
         // Size check for a "." result.
         // If the path starts with "..", it's not a subdirectory.
         return !relative.empty() && !(relative.starts_with(".."));
-        
+
     } catch (const std::filesystem::filesystem_error& e) {
         // Handle errors, e.g., if paths don't exist
         std::cerr << "Filesystem error: " << e.what() << std::endl;
@@ -271,7 +271,6 @@ bool isSubdirectory(const std::filesystem::path& base, const std::filesystem::pa
     return false;
 }
 
-
 void LocalFileSystem::removeFileIfExists(const std::string& path) {
     if (!fileOrPathExists(path)) {
         return;

diff --git a/src/main/database.cpp b/src/main/database.cpp
@@ -101,7 +101,6 @@ std::string expandPath(main::ClientContext* context, const std::string& path) {
     // Normalize the path to resolve '.' and '..'
     std::filesystem::path normalizedPath = std::filesystem::absolute(fullPath).lexically_normal();
     return normalizedPath.string();
-
 }
 
 void Database::initMembers(std::string_view dbPath, construct_bm_func_t initBmFunc) {

diff --git a/test/c_api/database_test.cpp b/test/c_api/database_test.cpp
@@ -1,9 +1,10 @@
+#include <fstream>
+
 #include "c_api/kuzu.h"
+#include "common/exception/io.h"
+#include "common/file_system/virtual_file_system.h"
 #include "graph_test/api_graph_test.h"
 #include "gtest/gtest.h"
-#include "common/file_system/virtual_file_system.h"
-#include "common/exception/io.h"
-#include <fstream>
 
 using namespace kuzu::main;
 using namespace kuzu::testing;
@@ -115,7 +116,8 @@ TEST_F(CApiDatabaseTest, VirtualFileSystemDeleteFiles) {
         vfs.removeFileIfExists("/tmp/test1");
     } catch (const kuzu::common::IOException& e) {
         // Expected behavior
-        EXPECT_STREQ(e.what(), "IO exception: Error: Path /tmp/test1 is not within the allowed home directory /tmp/dbHome/");
+        EXPECT_STREQ(e.what(), "IO exception: Error: Path /tmp/test1 is not within the allowed "
+                               "home directory /tmp/dbHome/");
     }
 
     vfs.removeFileIfExists("/tmp/dbHome/test1");
@@ -140,15 +142,17 @@ TEST_F(CApiDatabaseTest, VirtualFileSystemDeleteFilesWithHome) {
         vfs.removeFileIfExists("~/tmp/test1");
     } catch (const kuzu::common::IOException& e) {
         // Expected behavior
-        EXPECT_STREQ(e.what(), "IO exception: Error: Path ~/tmp/test1 is not within the allowed home directory ~/tmp/dbHome/");
+        EXPECT_STREQ(e.what(), "IO exception: Error: Path ~/tmp/test1 is not within the allowed "
+                               "home directory ~/tmp/dbHome/");
     }
 
     // Attempt to delete files outside the home directory (should error)
     try {
         vfs.removeFileIfExists("~");
     } catch (const kuzu::common::IOException& e) {
         // Expected behavior
-        EXPECT_STREQ(e.what(), "IO exception: Error: Path ~ is not within the allowed home directory ~/tmp/dbHome/");
+        EXPECT_STREQ(e.what(),
+            "IO exception: Error: Path ~ is not within the allowed home directory ~/tmp/dbHome/");
     }
 
     vfs.removeFileIfExists("~/tmp/dbHome/test1");
@@ -174,42 +178,48 @@ TEST_F(CApiDatabaseTest, VirtualFileSystemDeleteFilesEdgeCases) {
         vfs.removeFileIfExists("/tmp/dbHome/../test2");
     } catch (const kuzu::common::IOException& e) {
         // Expected behavior
-        EXPECT_STREQ(e.what(), "IO exception: Error: Path /tmp/dbHome/../test2 is not within the allowed home directory /tmp/dbHome/");
+        EXPECT_STREQ(e.what(), "IO exception: Error: Path /tmp/dbHome/../test2 is not within the "
+                               "allowed home directory /tmp/dbHome/");
     }
 
     try {
         vfs.removeFileIfExists("/tmp");
     } catch (const kuzu::common::IOException& e) {
         // Expected behavior
-        EXPECT_STREQ(e.what(), "IO exception: Error: Path /tmp is not within the allowed home directory /tmp/dbHome/");
+        EXPECT_STREQ(e.what(),
+            "IO exception: Error: Path /tmp is not within the allowed home directory /tmp/dbHome/");
     }
 
     try {
         vfs.removeFileIfExists("/tmp/");
     } catch (const kuzu::common::IOException& e) {
         // Expected behavior
-        EXPECT_STREQ(e.what(), "IO exception: Error: Path /tmp/ is not within the allowed home directory /tmp/dbHome/");
+        EXPECT_STREQ(e.what(), "IO exception: Error: Path /tmp/ is not within the allowed home "
+                               "directory /tmp/dbHome/");
     }
 
     try {
         vfs.removeFileIfExists("/tmp//////////////////");
     } catch (const kuzu::common::IOException& e) {
         // Expected behavior
-        EXPECT_STREQ(e.what(), "IO exception: Error: Path /tmp////////////////// is not within the allowed home directory /tmp/dbHome/");
+        EXPECT_STREQ(e.what(), "IO exception: Error: Path /tmp////////////////// is not within the "
+                               "allowed home directory /tmp/dbHome/");
     }
 
     try {
         vfs.removeFileIfExists("/tmp/./.././");
     } catch (const kuzu::common::IOException& e) {
         // Expected behavior
-        EXPECT_STREQ(e.what(), "IO exception: Error: Path /tmp/./.././ is not within the allowed home directory /tmp/dbHome/");
+        EXPECT_STREQ(e.what(), "IO exception: Error: Path /tmp/./.././ is not within the allowed "
+                               "home directory /tmp/dbHome/");
     }
 
     try {
         vfs.removeFileIfExists("/");
     } catch (const kuzu::common::IOException& e) {
         // Expected behavior
-        EXPECT_STREQ(e.what(), "IO exception: Error: Path / is not within the allowed home directory /tmp/dbHome/");
+        EXPECT_STREQ(e.what(),
+            "IO exception: Error: Path / is not within the allowed home directory /tmp/dbHome/");
     }
 
     vfs.removeFileIfExists("/tmp/dbHome/test2");
@@ -240,7 +250,8 @@ TEST_F(CApiDatabaseTest, VirtualFileSystemDeleteFilesWindowsPaths) {
         vfs.removeFileIfExists("C:\\test1");
         FAIL() << "Expected exception for path outside home directory.";
     } catch (const kuzu::common::IOException& e) {
-        EXPECT_STREQ(e.what(), "IO exception: Error: Path C:\\test1 is not within the allowed home directory C:\\Desktop\\dir");
+        EXPECT_STREQ(e.what(), "IO exception: Error: Path C:\\test1 is not within the allowed home "
+                               "directory C:\\Desktop\\dir");
     }
 
     // Attempt to delete file inside the home directory with mixed separators (should succeed)