diff --git a/test/message/append.cpp b/test/message/append.cpp
index b0ef54e..20ca441 100644
--- a/test/message/append.cpp
+++ b/test/message/append.cpp
@@ -44,7 +44,7 @@
                                                          nullptr, nullptr,
                                                          nullptr, nullptr))
             .WillRepeatedly(Return(0));
-    };
+    }
 
     sdbusplus::message_t new_message()
     {
diff --git a/test/message/read.cpp b/test/message/read.cpp
index 44253c6..42df2af 100644
--- a/test/message/read.cpp
+++ b/test/message/read.cpp
@@ -41,7 +41,7 @@
                                                          nullptr, nullptr,
                                                          nullptr, nullptr))
             .WillRepeatedly(Return(0));
-    };
+    }
 
     sdbusplus::message_t new_message()
     {
diff --git a/test/unpack_properties.cpp b/test/unpack_properties.cpp
index 1a74c79..bd349e7 100644
--- a/test/unpack_properties.cpp
+++ b/test/unpack_properties.cpp
@@ -21,7 +21,8 @@
     struct UnpackError
     {
         UnpackError(sdbusplus::UnpackErrorReason r, const std::string& p) :
-            reason(r), property(p){};
+            reason(r), property(p)
+        {}
         sdbusplus::UnpackErrorReason reason;
         std::string property;
     };