Mojo: Write some tests for public/system/macros.h.

I really set out to also write no-compile tests, but they're currently totally
disabled. :(

R=sky@chromium.org

Review URL: https://codereview.chromium.org/175343002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@252593 0039d316-1c4b-4281-b951-d872f2087c98

3 files changed, 145 insertions, 0 deletions

diff --git a/mojo/mojo_public.gypi b/mojo/mojo_public.gypi
index d4929fe..21dcd0f 100644
--- a/mojo/mojo_public.gypi
+++ b/mojo/mojo_public.gypi
@@ -145,6 +145,7 @@
         'public/tests/system/core_cpp_unittest.cc',
         'public/tests/system/core_unittest.cc',
         'public/tests/system/core_unittest_pure_c.c',
+        'public/tests/system/macros_unittest.cc',
       ],
     },
     {
diff --git a/mojo/public/system/macros.h b/mojo/public/system/macros.h
index 4bd5e77..b31469f 100644
--- a/mojo/public/system/macros.h
+++ b/mojo/public/system/macros.h
@@ -19,6 +19,7 @@
 // Annotate a function indicating that the caller must examine the return value.
 // Use like:
 //   int foo() MOJO_WARN_UNUSED_RESULT;
+// Note that it can only be used on the prototype, and not the definition.
 #if defined(__GNUC__)
 #define MOJO_WARN_UNUSED_RESULT __attribute__((warn_unused_result))
 #else
diff --git a/mojo/public/tests/system/macros_unittest.cc b/mojo/public/tests/system/macros_unittest.cc
new file mode 100644
index 0000000..edc8a66
--- /dev/null
+++ b/mojo/public/tests/system/macros_unittest.cc
@@ -0,0 +1,143 @@
+// Copyright 2014 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.
+
+// This file consists of "positive" tests, i.e., those verifying that things
+// work (without compile errors, or even warnings if warnings are treated as
+// errors).
+// TODO(vtl): Fix no-compile tests (which are all disabled; crbug.com/105388)
+// and write some "negative" tests.
+
+#include "mojo/public/system/macros.h"
+
+#include <assert.h>
+#include <stdint.h>
+#include <stdlib.h>
+
+#include "mojo/public/system/macros.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace mojo {
+namespace {
+
+TEST(MacrosTest, AllowUnused) {
+  // Test that no warning/error is issued even though |x| is unused.
+  int x MOJO_ALLOW_UNUSED = 123;
+}
+
+int MustUseReturnedResult() MOJO_WARN_UNUSED_RESULT;
+int MustUseReturnedResult() {
+  return 456;
+}
+
+TEST(MacrosTest, WarnUnusedResult) {
+  if (!MustUseReturnedResult())
+    abort();
+}
+
+TEST(MacrosTest, Override) {
+  struct BaseClass {
+    virtual ~BaseClass() {}
+    virtual void ToBeOverridden() {}
+    virtual void AlsoToBeOverridden() = 0;
+  };
+
+  struct Subclass : public BaseClass {
+    virtual ~Subclass() {}
+    virtual void ToBeOverridden() MOJO_OVERRIDE {}
+    virtual void AlsoToBeOverridden() MOJO_OVERRIDE {}
+  };
+
+  Subclass x;
+  x.ToBeOverridden();
+  x.AlsoToBeOverridden();
+}
+
+TEST(MacrosTest, DisallowCopyAndAssign) {
+  class Class {
+   public:
+    Class() {}
+    explicit Class(int) {}
+    void NoOp() {}
+
+   private:
+    MOJO_DISALLOW_COPY_AND_ASSIGN(Class);
+  };
+
+  Class x;
+  x.NoOp();
+  Class y(789);
+  y.NoOp();
+}
+
+// First test |MOJO_COMPILE_ASSERT()| in a global scope.
+MOJO_COMPILE_ASSERT(sizeof(int64_t) == 2 * sizeof(int32_t),
+                    bad_compile_assert_failure_in_global_scope);
+
+TEST(MacrosTest, CompileAssert) {
+  // Then in a local scope.
+  MOJO_COMPILE_ASSERT(sizeof(int32_t) == 2 * sizeof(int16_t),
+                      bad_compile_assert_failure);
+}
+
+// Test that |MOJO_ARRAYSIZE()| works in a |MOJO_COMPILE_ASSERT()|.
+const int kGlobalArray[5] = { 1, 2, 3, 4, 5 };
+MOJO_COMPILE_ASSERT(MOJO_ARRAYSIZE(kGlobalArray) == 5u,
+                    mojo_array_size_failed_in_compile_assert);
+
+TEST(MacrosTest, ArraySize) {
+  double local_array[4] = { 6.7, 7.8, 8.9, 9.0 };
+  EXPECT_EQ(4u, MOJO_ARRAYSIZE(local_array));
+}
+
+TEST(MacrosTest, MoveOnlyTypeForCpp03) {
+  class MoveOnlyInt {
+    MOJO_MOVE_ONLY_TYPE_FOR_CPP_03(MoveOnlyInt, RValue)
+
+   public:
+    MoveOnlyInt() : is_set_(false), value_() {}
+    explicit MoveOnlyInt(int value) : is_set_(true), value_(value) {}
+    ~MoveOnlyInt() {}
+
+    // Move-only constructor and operator=.
+    MoveOnlyInt(RValue other) { *this = other; }
+    MoveOnlyInt& operator=(RValue other) {
+      if (other.object != this) {
+        is_set_ = other.object->is_set_;
+        value_ = other.object->value_;
+        other.object->is_set_ = false;
+      }
+      return *this;
+    }
+
+    int value() const {
+      assert(is_set());
+      return value_;
+    }
+    bool is_set() const { return is_set_; }
+
+   private:
+    bool is_set_;
+    int value_;
+  };
+
+  MoveOnlyInt x(123);
+  EXPECT_TRUE(x.is_set());
+  EXPECT_EQ(123, x.value());
+  MoveOnlyInt y;
+  EXPECT_FALSE(y.is_set());
+  y = x.Pass();
+  EXPECT_FALSE(x.is_set());
+  EXPECT_TRUE(y.is_set());
+  EXPECT_EQ(123, y.value());
+  MoveOnlyInt z(y.Pass());
+  EXPECT_FALSE(y.is_set());
+  EXPECT_TRUE(z.is_set());
+  EXPECT_EQ(123, z.value());
+  z = z.Pass();
+  EXPECT_TRUE(z.is_set());
+  EXPECT_EQ(123, z.value());
+}
+
+}  // namespace
+}  // namespace mojo