diff options
author | Gil <mcg@google.com> | 2018-06-12 09:36:08 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-06-12 09:36:08 -0700 |
commit | 9e14b80e0716c2be71c6100cad7aa7c61ac46c6e (patch) | |
tree | 09d4d4583571b56feb89f809a24626cef14a3356 /Firestore/core/test/firebase/firestore/util/status_test_util.h | |
parent | f5bf0a37a7dd40e7538a1aed77af05471b7fe713 (diff) |
Create Status objects from errno (#1374)
* Add a portable interface to strerror
* Add Status::FromErrno
* Add strerror_test.cc to the Xcode project
* Use glibc feature selection macros instead of return-type overloads
* Fix tensorflow references
Diffstat (limited to 'Firestore/core/test/firebase/firestore/util/status_test_util.h')
-rw-r--r-- | Firestore/core/test/firebase/firestore/util/status_test_util.h | 32 |
1 files changed, 26 insertions, 6 deletions
diff --git a/Firestore/core/test/firebase/firestore/util/status_test_util.h b/Firestore/core/test/firebase/firestore/util/status_test_util.h index 745f3aa..fcf9135 100644 --- a/Firestore/core/test/firebase/firestore/util/status_test_util.h +++ b/Firestore/core/test/firebase/firestore/util/status_test_util.h @@ -20,16 +20,36 @@ #include "Firestore/core/src/firebase/firestore/util/status.h" #include "gtest/gtest.h" -// Macros for testing the results of functions that return tensorflow::Status. -#define EXPECT_OK(statement) \ - EXPECT_EQ(::firebase::firestore::util::Status::OK(), (statement)) -#define ASSERT_OK(statement) \ - ASSERT_EQ(::firebase::firestore::util::Status::OK(), (statement)) +namespace firebase { +namespace firestore { +namespace util { + +inline testing::AssertionResult Equal(Status expected, Status actual) { + if (expected != actual) { + return testing::AssertionFailure() + << "Should have seen status " << expected.ToString() << " but got " + << actual.ToString(); + } + + return testing::AssertionSuccess(); +} + +} // namespace util +} // namespace firestore +} // namespace firebase + +// Macros for testing the results of functions that return util::Status. +#define EXPECT_OK(statement) \ + EXPECT_TRUE(::firebase::firestore::util::Equal( \ + ::firebase::firestore::util::Status::OK(), (statement))); +#define ASSERT_OK(statement) \ + ASSERT_TRUE(::firebase::firestore::util::Equal( \ + ::firebase::firestore::util::Status::OK(), (statement))); // There are no EXPECT_NOT_OK/ASSERT_NOT_OK macros since they would not // provide much value (when they fail, they would just print the OK status // which conveys no more information than EXPECT_FALSE(status.ok()); // If you want to check for particular errors, a better alternative is: -// EXPECT_EQ(..expected tensorflow::error::Code..., status.code()); +// EXPECT_EQ(..expected FirestoreErrorCode..., status.code()); #endif // FIRESTORE_CORE_TEST_FIREBASE_FIRESTORE_UTIL_STATUS_TEST_UTIL_H_ |