Refactor dentry

1 files changed, 3 insertions, 12 deletions

diff --git a/kernel/src/filesystem/dentry.tests.cpp b/kernel/src/filesystem/dentry.tests.cpp
index c42c405..200262a 100644
--- a/kernel/src/filesystem/dentry.tests.cpp
+++ b/kernel/src/filesystem/dentry.tests.cpp
@@ -13,7 +13,7 @@ SCENARIO("Dentry construction", "[filesystem][dentry]")
   GIVEN("A parent dentry and inode")
   {
     auto inode = kstd::make_shared<kernel::tests::filesystem::inode>();
-    auto parent_dentry = kstd::make_shared<kernel::filesystem::dentry>(nullptr, inode);
+    auto parent_dentry = kstd::make_shared<kernel::filesystem::dentry>(nullptr, inode, "parent");
 
     WHEN("constructing a dentry")
     {
@@ -34,18 +34,9 @@ SCENARIO("Dentry construction", "[filesystem][dentry]")
 
     WHEN("constructing a dentry with an empty name")
     {
-      auto child_dentry = kernel::filesystem::dentry{parent_dentry, inode};
-
       THEN("the dentry has the correct parent and inode, and an empty name")
       {
-        REQUIRE(child_dentry.get_parent() == parent_dentry);
-        REQUIRE(child_dentry.get_inode() == inode);
-        REQUIRE(child_dentry.get_name().empty());
-      }
-
-      THEN("no flag is set")
-      {
-        REQUIRE_FALSE(child_dentry.has_flag(kernel::filesystem::dentry::dentry_flags::mounted));
+        REQUIRE_THROWS_AS((kernel::filesystem::dentry{parent_dentry, inode, ""}), kernel::tests::cpu::halt);
       }
     }
 
@@ -81,7 +72,7 @@ SCENARIO("Dentry child logic", "[filesystem][dentry]")
   GIVEN("A parent dentry and inode")
   {
     auto inode = kstd::make_shared<kernel::tests::filesystem::inode>();
-    auto parent_dentry = kstd::make_shared<kernel::filesystem::dentry>(nullptr, inode);
+    auto parent_dentry = kstd::make_shared<kernel::filesystem::dentry>(nullptr, inode, "parent");
 
     WHEN("adding child dentries")
     {