🐛 Fix for unused drop value warning

Author: kris-jusiak

include/GUnit/GAssert.h

@@ -152,7 +152,7 @@ class op {
   explicit op(const info& info) : info_{info} {}
 
   template <class TLhs>
-  comp<TLhs> operator%(const TLhs& lhs) const {
+  comp<TLhs> operator<<(const TLhs& lhs) const {
     return comp<TLhs>{info_, lhs};
   }
 
@@ -176,21 +176,21 @@ struct ret_void {
 }  // namespace v1
 }  // namespace testing
 
-#define EXPECT(...)                                                            \
-  (::testing::detail::op<std::true_type>{                                      \
-       ::testing::detail::info{__FILE__, __LINE__, #__VA_ARGS__,               \
-                               ::testing::TestPartResult::kNonFatalFailure}} % \
-   __VA_ARGS__)
-
-#define ASSERT(...)                                                            \
-  if (::testing::detail::op<std::false_type>{                                  \
-          ::testing::detail::info{__FILE__, __LINE__, #__VA_ARGS__,            \
-                                  ::testing::TestPartResult::kFatalFailure}} % \
-      __VA_ARGS__)                                                             \
-    ::testing::detail::drop{};                                                 \
-  else                                                                         \
-    return ::testing::detail::ret_void{} ==                                    \
-           (::testing::detail::op<std::true_type>{::testing::detail::info{     \
-                __FILE__, __LINE__, #__VA_ARGS__,                              \
-                ::testing::TestPartResult::kFatalFailure}} %                   \
-            __VA_ARGS__)
+#define EXPECT(...)                                                          \
+  (::testing::detail::op<std::true_type>{                                    \
+       ::testing::detail::info{__FILE__, __LINE__, #__VA_ARGS__,             \
+                               ::testing::TestPartResult::kNonFatalFailure}} \
+   << __VA_ARGS__)
+
+#define ASSERT(...)                                                          \
+  if (::testing::detail::op<std::false_type>{                                \
+          ::testing::detail::info{__FILE__, __LINE__, #__VA_ARGS__,          \
+                                  ::testing::TestPartResult::kFatalFailure}} \
+      << __VA_ARGS__)                                                        \
+    void(::testing::detail::drop{});                                         \
+  else                                                                       \
+    return ::testing::detail::ret_void{} ==                                  \
+           (::testing::detail::op<std::true_type>{::testing::detail::info{   \
+                __FILE__, __LINE__, #__VA_ARGS__,                            \
+                ::testing::TestPartResult::kFatalFailure}}                   \
+            << __VA_ARGS__)

include/GUnit/GSteps.h

@@ -388,7 +388,7 @@ class Steps : public ::testing::EmptyTestEventListener {
     return Step<-1, true>{*this, {"Then", File::c_str(), line}, pattern};
   }
 
-  // clang-format off
+// clang-format off
   #if defined(__clang__)
   #pragma clang diagnostic ignored "-Wdollar-in-identifier-extension"
   #endif

test/GAssert.cpp

@@ -29,6 +29,7 @@ TEST(GAssert, ShouldSupportExpect) {
   EXPECT(i < 100) << "message";
 
   EXPECT(42.0 == 42.);
+  EXPECT(2*21 == i);
 }
 
 TEST(GAssert, ShouldSupportASSERT) {
@@ -48,4 +49,5 @@ TEST(GAssert, ShouldSupportASSERT) {
   ASSERT(i < 100) << "message";
 
   ASSERT(42.0 == 42.);
+  ASSERT(2*21 == i);
 }