Unittest: Enable & ban specific warnings

This commit also fixes 4 of these warnings, that occured once each,
namely -Werror={switch,memset-elt-size,comment,unused-value}.

Author: anordal

test/unit-test/CMakeLists.txt

@@ -18,12 +18,51 @@ if( ${PROJECT_SOURCE_DIR} STREQUAL ${PROJECT_BINARY_DIR} )
     message( FATAL_ERROR "In-source build is not allowed. Please build in a separate directory, such as ${PROJECT_SOURCE_DIR}/build." )
 endif()
 
-set(SANITIZE "" CACHE STRING "Comma-separated list of compiler sanitizers to enable; empty string disables.")
-if(NOT ${SANITIZE} STREQUAL "")
+set( SANITIZE "" CACHE STRING "Comma-separated list of compiler sanitizers to enable; empty string disables." )
+if( NOT ${SANITIZE} STREQUAL "" )
     add_compile_options(-fsanitize=${SANITIZE} -fno-sanitize-recover)
     add_link_options(-fsanitize=${SANITIZE} -fno-sanitize-recover)
 endif()
 
+# Align with: test/build-combination/CMakeLists.txt
+add_compile_options(
+    $<$<COMPILE_LANG_AND_ID:C,GNU>:-fdiagnostics-color=always>
+    $<$<COMPILE_LANG_AND_ID:C,Clang>:-fcolor-diagnostics>
+
+    ### On the road to -Wall: Forbid warnings that need not be tolerated.
+    $<$<COMPILE_LANG_AND_ID:C,GNU,Clang>:-Werror=uninitialized> # definite use of uninitialized variable
+    $<$<COMPILE_LANG_AND_ID:C,GNU,Clang>:-Werror=array-bounds> # indexing outside sized array
+    $<$<COMPILE_LANG_AND_ID:C,GNU,Clang>:-Werror=switch> # enum switch with unhandled value
+    $<$<COMPILE_LANG_AND_ID:C,GNU,Clang>:-Werror=format> # format string wrong
+    $<$<COMPILE_LANG_AND_ID:C,GNU,Clang>:-Werror=format-security> # format string missing
+    $<$<COMPILE_LANG_AND_ID:C,GNU,Clang>:-Werror=comment> # comment within a comment
+    $<$<COMPILE_LANG_AND_ID:C,GNU,Clang>:-Werror=unused-value> # "statement has no effect"
+    $<$<COMPILE_LANG_AND_ID:C,GNU>:-Werror=maybe-uninitialized> # possible use of uninitialized variable
+    $<$<COMPILE_LANG_AND_ID:C,GNU>:-Werror=memset-elt-size> # memsetting array by length instead of size
+    # FIXME (part of -Wall):
+    # -Werror=return-type # "control reaches end of non-void function"
+    # -Werror=array-parameter # name clash
+    # -Werror=implicit-function-declaration # need header
+    # -Werror=unused-but-set-variable
+    # -Werror=unused-variable
+    # -Werror=unused-function
+    # -Werror=unused-parameter
+    # -Werror=missing-braces
+    # -Werror=pointer-sign
+    # -Werror=dangling-pointer
+    # -Werror=vla
+
+    ### Currently needed to build cleanly with Clang 19:
+    $<$<COMPILE_LANG_AND_ID:C,Clang>:-Werror>
+    $<$<COMPILE_LANG_AND_ID:C,Clang>:-Wno-pragma-pack>
+    $<$<COMPILE_LANG_AND_ID:C,Clang>:-Wno-unused-parameter>
+    $<$<COMPILE_LANG_AND_ID:C,Clang>:-Wno-return-type>
+    $<$<COMPILE_LANG_AND_ID:C,Clang>:-Wno-pointer-sign>
+    $<$<COMPILE_LANG_AND_ID:C,Clang>:-Wno-typedef-redefinition>
+    $<$<COMPILE_LANG_AND_ID:C,Clang>:-Wno-strict-prototypes>
+    $<$<COMPILE_LANG_AND_ID:C,Clang>:-Wno-sometimes-uninitialized>
+)
+
 # Set global path variables.
 get_filename_component( __MODULE_ROOT_DIR "${CMAKE_CURRENT_LIST_DIR}/../.." ABSOLUTE )
 set( MODULE_ROOT_DIR ${__MODULE_ROOT_DIR} CACHE INTERNAL "FreeRTOS-Plus-TCP repository root." )

test/unit-test/FreeRTOS_DHCPv6/FreeRTOS_DHCPv6_utest.c

@@ -127,6 +127,11 @@ static void vAddBitOperation( eTestDHCPv6BitOperationType_t eType,
             TEST_ASSERT_LESS_THAN( TEST_DHCPv6_BIT_OPERATION_MAX_SIZE, ulSize );
             memcpy( xTestDHCPv6BitOperation[ ulTestDHCPv6BitOperationWriteIndex ].val.ucValCustom, pvVal, ulSize );
             break;
+
+        case eTestDHCPv6BitOperationNone:
+        case eTestDHCPv6BitOperationSetError:
+        case eTestDHCPv6BitOperationReturnFalse:
+            break;
     }
 
     TEST_ASSERT_LESS_THAN_size_t( TEST_DHCPv6_BIT_OPERATION_DEBUG_MSG_MAX_SIZE, strlen( pucDebugMsg ) );

test/unit-test/FreeRTOS_DNS_Cache/FreeRTOS_DNS_Cache_utest.c

@@ -331,8 +331,6 @@ void test_processDNS_CACHE_exceed_IP_entry_limit( void )
     pxIP_2.xIs_IPv6 = 0;
     pxIP_2.xIPAddress.ulIP_IPv4 = pulIP;
 
-    memset( pulIP_arr, 123, ipconfigDNS_CACHE_ADDRESSES_PER_ENTRY );
-
     for( i = 0; i < ipconfigDNS_CACHE_ADDRESSES_PER_ENTRY; i++ )
     {
         pxIP[ i ].xIPAddress.ulIP_IPv4 = pulIP_arr[ i ];

test/unit-test/FreeRTOS_DNS_Callback/FreeRTOS_DNS_Callback_utest.c

@@ -66,7 +66,7 @@ typedef void (* FOnDNSEvent ) ( const char * /* pcName */,
 static int callback_called = 0;
 
 /* The second element is for the flexible array member
- * /* when pvPortMalloc is mocked to return this object.
+ * when pvPortMalloc is mocked to return this object.
  */
 static DNSCallback_t dnsCallback[ 2 ];
 

test/unit-test/FreeRTOS_Sockets/FreeRTOS_Sockets_TCP_API_utest.c

@@ -1888,7 +1888,7 @@ void test_FreeRTOS_get_tx_base_InvalidParams( void )
     pucReturn = FreeRTOS_get_tx_base( &xSocket );
     TEST_ASSERT_EQUAL( NULL, pucReturn );
 
-    xSocket.u.xTCP.bits.bMallocError == pdTRUE_UNSIGNED;
+    xSocket.u.xTCP.bits.bMallocError = pdTRUE_UNSIGNED;
     pucReturn = FreeRTOS_get_tx_base( &xSocket );
     TEST_ASSERT_EQUAL( NULL, pucReturn );
 }