Non-path Time Machine Exclusions

base/mac/mac_util_unittest.mm

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #import <Cocoa/Cocoa.h>
 
 #include "base/mac/mac_util.h"
 
 #include "base/file_path.h"
 #include "base/file_util.h"
@@ skipping 77 matching lines @@
   };
   for (size_t i = 0; i < ARRAYSIZE_UNSAFE(valid_inputs); i++) {
     out = GetAppBundlePath(FilePath(valid_inputs[i].in));
     EXPECT_FALSE(out.empty()) << "loop: " << i;
     EXPECT_STREQ(valid_inputs[i].expected_out,
         out.value().c_str()) << "loop: " << i;
   }
 }
 
 TEST_F(MacUtilTest, TestExcludeFileFromBackups) {
-  NSString* homeDirectory = NSHomeDirectory();
-  NSString* dummyFilePath =
-      [homeDirectory stringByAppendingPathComponent:@"DummyFile"];
-  const char* dummy_file_path = [dummyFilePath fileSystemRepresentation];
-  ASSERT_TRUE(dummy_file_path);
-  FilePath file_path(dummy_file_path);
-  // It is not actually necessary to have a physical file in order to
-  // set its exclusion property.
-  NSURL* fileURL = [NSURL URLWithString:dummyFilePath];
-  // Reset the exclusion in case it was set previously.
-  SetFileBackupExclusion(file_path, false);
-  Boolean excludeByPath;
+  FilePath temp_dir;
+  ASSERT_TRUE(file_util::CreateNewTempDirectory(FilePath::StringType(),
+                                                &temp_dir));

[Mark Mentovai, 2011/05/25 19:19:06]

Who’s responsible for cleaning up temp_dir? Maybe use ScopedTempDir from
"base/scoped_temp_dir.h" here.

[mrossetti, 2011/05/25 21:12:31]

Done.

+  EXPECT_TRUE(file_util::PathExists(temp_dir));
+  FilePath dummy_file_path = temp_dir.Append("DummyFile");
+  FILE* dummy_file = file_util::OpenFile(dummy_file_path, "w");

[Mark Mentovai, 2011/05/25 19:19:06]

Put this in a file_util::ScopedFILE to ensure it closes? ("base/file_util.h")

[mrossetti, 2011/05/25 21:12:31]

Eliminated in favor of just using WriteFile.

+  EXPECT_TRUE(dummy_file);
+  const char dummy_data[] = "All your base are belong to us!";

[Mark Mentovai, 2011/05/25 19:19:06]

:)

+  EXPECT_TRUE(fwrite(dummy_data, ARRAYSIZE_UNSAFE(dummy_data), 1, dummy_file));

[Mark Mentovai, 2011/05/25 19:19:06]

Instead of file_util::OpenFile + fwrite + file_util::CloseFile (or
file_util::ScopedFILE), how about just using file_util::WriteFile?

[Mark Mentovai, 2011/05/25 19:19:06]

The “safe” arraysize didn’t work here?

[mrossetti, 2011/05/25 21:12:31]

Done.

[mrossetti, 2011/05/25 21:12:31]

Done.

+  // The file must already exist in order to set its exclusion property.
+  NSString* fileURLString =
+      [NSString stringWithUTF8String:dummy_file_path.value().c_str()];
+  NSURL* fileURL = [NSURL URLWithString:fileURLString];
   // Initial state should be non-excluded.
-  EXPECT_FALSE(CSBackupIsItemExcluded((CFURLRef)fileURL, &excludeByPath));
+  EXPECT_FALSE(CSBackupIsItemExcluded((CFURLRef)fileURL, NULL));

[Mark Mentovai, 2011/05/25 19:19:06]

I wonder if this is always true in the temp directory you wind up in.

[mrossetti, 2011/05/25 21:12:31]

It might be different under 10.7 but it comes up non-excluded here and then
excluded below.

   // Exclude the file.
-  EXPECT_TRUE(SetFileBackupExclusion(file_path, true));
-  EXPECT_TRUE(CSBackupIsItemExcluded((CFURLRef)fileURL, &excludeByPath));
-  // Un-exclude the file.
-  EXPECT_TRUE(SetFileBackupExclusion(file_path, false));
-  EXPECT_FALSE(CSBackupIsItemExcluded((CFURLRef)fileURL, &excludeByPath));
+  EXPECT_TRUE(SetFileBackupExclusion(dummy_file_path));
+  // SetFileBackupExclusion never excludes by path.
+  Boolean excluded_by_path = false;

[Mark Mentovai, 2011/05/25 19:19:06]

FALSE

[mrossetti, 2011/05/25 21:12:31]

Yeah, yeah — I'm getting it.  ;^)

+  Boolean excluded =
+      CSBackupIsItemExcluded((CFURLRef)fileURL, &excluded_by_path);
+  EXPECT_TRUE(excluded);
+  EXPECT_FALSE(excluded_by_path);
+  EXPECT_TRUE(file_util::CloseFile(dummy_file));
 }
 
 TEST_F(MacUtilTest, TestGetValueFromDictionary) {
   ScopedCFTypeRef<CFMutableDictionaryRef> dict(
       CFDictionaryCreateMutable(0, 0,
                                 &kCFTypeDictionaryKeyCallBacks,
                                 &kCFTypeDictionaryValueCallBacks));
   CFDictionarySetValue(dict.get(), CFSTR("key"), CFSTR("value"));
 
   EXPECT_TRUE(CFEqual(CFSTR("value"),
@@ skipping 26 matching lines @@
   EXPECT_EQ(2U, [array retainCount]);
 
   NSObjectRelease(array);
   EXPECT_EQ(1U, [array retainCount]);
 }
 
 }  // namespace
 
 }  // namespace mac
 }  // namespace base