Expand tests for move semantics

bugdea1er · Sep 24, 2024 · ac4dfc2 · ac4dfc2
1 parent 2312f2e
commit ac4dfc2
Show file tree

Hide file tree

Showing 2 changed files with 42 additions and 28 deletions.
diff --git a/tests/directory.cpp b/tests/directory.cpp
@@ -134,34 +134,41 @@ TEST(directory, destructor) {
 /// Tests directory move constructor
 TEST(directory, move_constructor) {
   directory fst = directory();
-  directory snd = std::move(fst);
+  directory snd = directory(std::move(fst));
 
-  EXPECT_TRUE(fst.path().empty());
+  fst.~directory();
+
+  EXPECT_FALSE(snd.path().empty());
   EXPECT_TRUE(fs::exists(snd));
   EXPECT_TRUE(native_handle_is_valid(snd.native_handle()));
 }
 
 /// Tests directory move assignment operator
 TEST(directory, move_assignment) {
   directory fst = directory();
-  directory snd = directory();
+  {
+    directory snd = directory();
 
-  fs::path path1 = fst;
-  fs::path path2 = snd;
+    fs::path path1 = fst;
+    fs::path path2 = snd;
 
-  entry::native_handle_type fst_handle = fst.native_handle();
-  entry::native_handle_type snd_handle = snd.native_handle();
+    entry::native_handle_type fst_handle = fst.native_handle();
+    entry::native_handle_type snd_handle = snd.native_handle();
+
+    fst = std::move(snd);
 
-  fst = std::move(snd);
+    EXPECT_FALSE(fs::exists(path1));
+    EXPECT_TRUE(fs::exists(path2));
 
-  EXPECT_FALSE(fs::exists(path1));
-  EXPECT_TRUE(fs::exists(path2));
+    EXPECT_TRUE(fs::exists(fst));
+    EXPECT_TRUE(fs::equivalent(fst, path2));
 
-  EXPECT_TRUE(fs::exists(fst));
-  EXPECT_TRUE(fs::equivalent(fst, path2));
+    EXPECT_FALSE(native_handle_is_valid(fst_handle));
+    EXPECT_TRUE(native_handle_is_valid(snd_handle));
+  }
 
-  EXPECT_FALSE(native_handle_is_valid(fst_handle));
-  EXPECT_TRUE(native_handle_is_valid(snd_handle));
+  EXPECT_FALSE(fst.path().empty());
+  EXPECT_TRUE(native_handle_is_valid(fst.native_handle()));
 }
 
 /// Tests directory moving

diff --git a/tests/file.cpp b/tests/file.cpp
@@ -358,34 +358,41 @@ TEST(file, destructor) {
 /// Tests file move constructor
 TEST(file, move_constructor) {
   file fst = file();
-  file snd = std::move(fst);
+  file snd = file(std::move(fst));
 
-  EXPECT_TRUE(fst.path().empty());
+  fst.~file();
+
+  EXPECT_FALSE(snd.path().empty());
   EXPECT_TRUE(fs::exists(snd));
   EXPECT_TRUE(native_handle_is_valid(snd.native_handle()));
 }
 
 /// Tests file move assignment operator
 TEST(file, move_assignment) {
   file fst = file();
-  file snd = file();
+  {
+    file snd = file();
 
-  fs::path path1 = fst;
-  fs::path path2 = snd;
+    fs::path path1 = fst;
+    fs::path path2 = snd;
 
-  entry::native_handle_type fst_handle = fst.native_handle();
-  entry::native_handle_type snd_handle = snd.native_handle();
+    entry::native_handle_type fst_handle = fst.native_handle();
+    entry::native_handle_type snd_handle = snd.native_handle();
+
+    fst = std::move(snd);
 
-  fst = std::move(snd);
+    EXPECT_FALSE(fs::exists(path1));
+    EXPECT_TRUE(fs::exists(path2));
 
-  EXPECT_FALSE(fs::exists(path1));
-  EXPECT_TRUE(fs::exists(path2));
+    EXPECT_TRUE(fs::exists(fst));
+    EXPECT_TRUE(fs::equivalent(fst, path2));
 
-  EXPECT_TRUE(fs::exists(fst));
-  EXPECT_TRUE(fs::equivalent(fst, path2));
+    EXPECT_FALSE(native_handle_is_valid(fst_handle));
+    EXPECT_TRUE(native_handle_is_valid(snd_handle));
+  }
 
-  EXPECT_FALSE(native_handle_is_valid(fst_handle));
-  EXPECT_TRUE(native_handle_is_valid(snd_handle));
+  EXPECT_FALSE(fst.path().empty());
+  EXPECT_TRUE(native_handle_is_valid(fst.native_handle()));
 }
 
 /// Tests file moving