Remove unused TempDirectoryPolicy

rikyoz · Dec 4, 2023 · a1242dd · a1242dd
1 parent 78999dc
commit a1242dd
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 15 deletions.
diff --git a/tests/src/utils/filesystem.cpp b/tests/src/utils/filesystem.cpp
@@ -288,11 +288,11 @@ auto create_temp_directory( const std::string& name ) -> fs::path {
     return tempDir;
 }
 
-TempDirectory::TempDirectory( const std::string& dirName, TempDirectoryPolicy policy )
-    : mDirectory{ create_temp_directory( dirName + "_" + random_test_id() ) }, mPolicy{ policy } {}
+TempDirectory::TempDirectory( const std::string& dirName )
+    : mDirectory{ create_temp_directory( dirName + "_" + random_test_id() ) } {}
 
 TempDirectory::~TempDirectory() {
-    if ( mPolicy == TempDirectoryPolicy::CleanupOnExit && fs::is_empty( mDirectory ) ) {
+    if ( fs::is_empty( mDirectory ) ) {
         fs::remove( mDirectory );
     }
 }
@@ -301,8 +301,8 @@ auto TempDirectory::path() -> const fs::path& {
     return mDirectory;
 }
 
-TempTestDirectory::TempTestDirectory( const std::string& dirName, TempDirectoryPolicy policy )
-    : TempDirectory{ dirName, policy }, TestDirectory{ path() } {}
+TempTestDirectory::TempTestDirectory( const std::string& dirName )
+    : TempDirectory{ dirName }, TestDirectory{ path() } {}
 
 } // namespace filesystem
 } // namespace test

diff --git a/tests/src/utils/filesystem.hpp b/tests/src/utils/filesystem.hpp
@@ -159,15 +159,9 @@ class TestDirectory {
         fs::path mOldCurrentDirectory;
 };
 
-enum struct TempDirectoryPolicy : std::uint8_t {
-    CleanupOnExit,
-    KeepOnExit
-};
-
 class TempDirectory {
     public:
-        explicit TempDirectory( const std::string& dirName,
-                                TempDirectoryPolicy policy = TempDirectoryPolicy::CleanupOnExit );
+        explicit TempDirectory( const std::string& dirName );
 
         explicit TempDirectory( const TempDirectory& ) = delete;
 
@@ -183,12 +177,10 @@ class TempDirectory {
 
     private:
         fs::path mDirectory;
-        TempDirectoryPolicy mPolicy;
 };
 
 struct TempTestDirectory : TempDirectory, TestDirectory {
-    explicit TempTestDirectory( const std::string& dirName,
-                                TempDirectoryPolicy policy = TempDirectoryPolicy::CleanupOnExit );
+    explicit TempTestDirectory( const std::string& dirName );
 };
 
 #endif