Add wrappers around getxattr() and setxattr().

util/mac/xattr_test.cc

Index: util/mac/xattr_test.cc
diff --git a/util/mac/xattr_test.cc b/util/mac/xattr_test.cc
new file mode 100644
index 0000000000000000000000000000000000000000..0865ef7742de3f3918c0fa4facc7a5c4b27c558f
--- /dev/null
+++ b/util/mac/xattr_test.cc
@@ -0,0 +1,122 @@
+// Copyright 2014 The Crashpad Authors. All rights reserved.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+#include "util/mac/xattr.h"
+
+#include <fcntl.h>
+#include <sys/stat.h>
+#include <unistd.h>
+
+#include <limits>
+
+#include "base/files/scoped_file.h"
+#include "base/posix/eintr_wrapper.h"
+#include "base/strings/stringprintf.h"
+#include "gtest/gtest.h"
+#include "util/test/errors.h"
+
+namespace crashpad {
+namespace test {
+namespace {
+
+class Xattr : public testing::Test {
+ public:
+  // testing::Test:
+
+  void SetUp() override {
+    path_ = base::FilePath(
+        base::StringPrintf("/tmp/com.googlecode.crashpad.test.xattr.%p", this));

[Mark Mentovai, 2014/12/30 19:21:51]

You didn’t use O_EXCL, but I can see why you might not want to with this using
%p/this.

OK, so stick with O_TRUNC instead of O_EXCL for now, but at least make things a
little less likely to collide by using %d/getpid().

[Robert Sesek, 2014/12/30 21:59:04]

Done.

+    // TODO(rsesek): This should use something like ScopedTempDir.
+    base::ScopedFD tmp(HANDLE_EINTR(
+          open(path_.value().c_str(), O_CREAT | O_TRUNC, 0644)));
+    EXPECT_GE(tmp.get(), 0) << ErrnoMessage("open");
+  }
+
+  void TearDown() override {
+    EXPECT_EQ(0, unlink(path_.value().c_str())) << ErrnoMessage("unlink");
+  }
+
+  base::FilePath path() { return path_; }
+
+ private:
+  base::FilePath path_;
+};
+
+const char kKey[] = "com.google.crashpad.test";
+
+TEST_F(Xattr, ReadNonExistentXattr) {
+  std::string value;
+  EXPECT_FALSE(ReadXattr(path(), kKey, &value));
+}
+
+TEST_F(Xattr, WriteAndReadString) {
+  std::string value = "hello world";
+  EXPECT_TRUE(WriteXattr(path(), kKey, value));
+
+  std::string actual;
+  EXPECT_TRUE(ReadXattr(path(), kKey, &actual));
+  EXPECT_EQ(value, actual);
+}
+
+TEST_F(Xattr, WriteAndReadVeryLongString) {
+  std::string value(533, 'A');
+  EXPECT_TRUE(WriteXattr(path(), kKey, value));
+
+  std::string actual;
+  EXPECT_TRUE(ReadXattr(path(), kKey, &actual));
+  EXPECT_EQ(value, actual);
+}
+
+TEST_F(Xattr, WriteAndReadBool) {
+  EXPECT_TRUE(WriteXattrBool(path(), kKey, true));
+  bool actual = false;
+  EXPECT_TRUE(ReadXattrBool(path(), kKey, &actual));
+  EXPECT_TRUE(actual);
+
+  EXPECT_TRUE(WriteXattrBool(path(), kKey, false));
+  EXPECT_TRUE(ReadXattrBool(path(), kKey, &actual));
+  EXPECT_FALSE(actual);
+}
+
+TEST_F(Xattr, WriteAndReadInt) {
+  int expected = 42;
+  int actual;
+
+  EXPECT_TRUE(WriteXattrInt(path(), kKey, expected));
+  EXPECT_TRUE(ReadXattrInt(path(), kKey, &actual));
+  EXPECT_EQ(expected, actual);
+
+  expected = std::numeric_limits<int>::max();
+  EXPECT_TRUE(WriteXattrInt(path(), kKey, expected));
+  EXPECT_TRUE(ReadXattrInt(path(), kKey, &actual));
+  EXPECT_EQ(expected, actual);
+}
+
+TEST_F(Xattr, WriteAndReadTimeT) {
+  time_t expected = time(nullptr);
+  time_t actual;
+
+  EXPECT_TRUE(WriteXattrTimeT(path(), kKey, expected));
+  EXPECT_TRUE(ReadXattrTimeT(path(), kKey, &actual));
+  EXPECT_EQ(expected, actual);
+
+  expected = std::numeric_limits<time_t>::max();
+  EXPECT_TRUE(WriteXattrTimeT(path(), kKey, expected));
+  EXPECT_TRUE(ReadXattrTimeT(path(), kKey, &actual));
+  EXPECT_EQ(expected, actual);
+}
+
+}  // namespace
+}  // namespace test
+}  // namespace crashpad