mac/rlz: Remove an incorrect check

rlz/lib/rlz_lib_test.cc

Index: rlz/lib/rlz_lib_test.cc
diff --git a/rlz/lib/rlz_lib_test.cc b/rlz/lib/rlz_lib_test.cc
index 118f48cb8cdc1292e12b5e0b4400fef1f6a9f599..4cfb7b9239a457bf49bdeb7936df1e64266f88cb 100644
--- a/rlz/lib/rlz_lib_test.cc
+++ b/rlz/lib/rlz_lib_test.cc
@@ -25,6 +25,12 @@
 #if defined(OS_WIN)
 #include <Windows.h>
 #include "rlz/win/lib/machine_deal.h"
+#elif defined(OS_MACOSX)
+namespace rlz_lib {
+namespace testing {
+std::string RlzPlistFilenameStr();
+}
+}

[Nico, 2012/08/21 20:43:44]

This is a bit gross, but rlz_value_store_mac.h can't be included here since it
relies on ObjC code.

[Mark Mentovai, 2012/08/21 20:59:29]

Nico wrote:
It doesn’t have to. There’s nothing Objective about that header except for the
pair of classes. Use this standard trick in the header:

#ifdef __OBJC__
@class NSDictionary;
@class NSMutableDictionary;
#else
class NSDictionary;
class NSMutableDictionary;
#endif

Then this file gets far less gross.

I notice that SetRlzStoreDirectory is declared in rlz/lib/rlz_value_store.h.
Maybe you should move it to rlz/mac/lib/rlz_value_store_mac.h too.

[Nico, 2012/08/21 21:05:03]

The included of scoped_nsobject.h needs to be conditional too, which in turn
requires the definition of the RlzValueStore class to be made conditional. If
you prefer that, I can do that, but that seemed pretty gross too.
Maybe I should move this new functions to rlz/lib/rlz_value_store.h instead. Did
that.

 #endif
 
 #if defined(RLZ_NETWORK_IMPLEMENTATION_CHROME_NET)
@@ -871,4 +877,22 @@ TEST_F(RlzLibTest, ConcurrentStoreAccessWithProcessExitsWhileLockHeld) {
   EXPECT_TRUE(rlz_lib::RecordProductEvent(rlz_lib::TOOLBAR_NOTIFIER,
       rlz_lib::IE_DEFAULT_SEARCH, rlz_lib::INSTALL));
 }
+
+TEST_F(RlzLibTest, LockAcquistionSucceedsButPlistCannotBeCreated) {
+  // See the comment at the top of WriteFails.
+  if (!rlz_lib::SupplementaryBranding::GetBrand().empty())
+    return;
+
+  // Create a directory where the rlz file is supposed to appear. This way,
+  // the lock file can be created successfully, but creation of the rlz file
+  // itself will fail.
+  int mkdir_result = mkdir(rlz_lib::testing::RlzPlistFilenameStr().c_str(),
+                     0500);

[Mark Mentovai, 2012/08/21 20:59:29]

Bump this over to line up with the other argument.

+  ASSERT_EQ(0, mkdir_result);
+
+  rlz_lib::SupplementaryBranding branding("TEST");
+  EXPECT_FALSE(rlz_lib::RecordProductEvent(rlz_lib::TOOLBAR_NOTIFIER,
+      rlz_lib::IE_DEFAULT_SEARCH, rlz_lib::INSTALL));
+}
+
 #endif