Display alll warning during build

 Issue #18

Author: suikan4github

CHANGELOG.md

@@ -8,6 +8,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 ### Added
 - [Issue #17](https://github.com/suikan4github/template_application/issues/17) :  Add gcovr support. 
 ### Changed
+- [Issue #18](https://github.com/suikan4github/template_application/issues/18) :   Display alll warning during build
 ### Fixed
 
 ## [1.2.0] - 2022-01-02

src/CMakeLists.txt

@@ -10,9 +10,9 @@ file(GLOB ALL_SRCS "*.cpp")
 
 add_executable(${EXECUTABLE_NAME} ${ALL_SRCS})
 
-# Add pthread library to support std::thread
-# In the case of MSVC, this is not required.
-if (NOT(MSVC))
-    target_link_libraries(${EXECUTABLE_NAME} "pthread")
-endif(NOT(MSVC))
+    # Add pthread library to support std::thread
+    # In the case of MSVC, this is not required.
+    if (NOT(MSVC))
+        target_link_libraries(${EXECUTABLE_NAME} "pthread")
+    endif(NOT(MSVC))
 

test/CMakeLists.txt

@@ -19,15 +19,22 @@ set(TEST_EXECUTABLE_NAME test_${PROJECT_NAME})
 # Add the executable for the testcase which is using googletest
 add_executable(${TEST_EXECUTABLE_NAME} ${TEST_SRC} ${LIB_SRC})
 
-# Add the library under test.
-target_link_libraries(${TEST_EXECUTABLE_NAME}  
-                        "gtest_main"
-                         )
-# Add the include directory for test executable.
-target_include_directories(${TEST_EXECUTABLE_NAME}
-                            PUBLIC
-                            "${CMAKE_CURRENT_SOURCE_DIR}/../src"
+    # Add the library under test.
+    target_link_libraries(${TEST_EXECUTABLE_NAME}  
+                            "gtest_main"
                             )
+    # Add the include directory for test executable.
+    target_include_directories(${TEST_EXECUTABLE_NAME}
+                                PUBLIC
+                                "${CMAKE_CURRENT_SOURCE_DIR}/../src"
+                                )
+    if(MSVC)
+        target_compile_options(${TEST_EXECUTABLE_NAME} PRIVATE /W4 )
+    else()
+        target_compile_options(${TEST_EXECUTABLE_NAME} PRIVATE -Wall -Wextra -pedantic )
+    endif()
+                        
+
 
 # Find and integrate the Google Test package
 

test/test_calc_death.cpp

@@ -7,9 +7,8 @@
 // Read GoogleTest document https://github.com/google/googletest/blob/main/docs/advanced.md#death-test-naming
 TEST(CalcDeathTest, SqrtWithMinusParam)
 {
-    Calc calc;
-
 #ifndef NDEBUG
+    Calc calc;
     // We test only when assert() works.
     ASSERT_DEATH(calc.sqrt(-1), "param >= 0");
 #endif