refactor(tests): Include stdlib.h where open62541 no longer does it

tests/check_chunking.c

@@ -7,6 +7,7 @@
 #include "ua_securechannel.h"
 #include "ua_types_encoding_binary.h"
 
+#include <stdlib.h>
 #include <check.h>
 
 UA_ByteString *buffers;

tests/check_cj5.c

@@ -6,6 +6,7 @@
 #include <check.h>
 #include "cj5.h"
 
+#include <stdlib.h>
 #include <math.h>
 #include <float.h>
 

tests/check_eventloop.c

@@ -6,6 +6,8 @@
 #include "testing_clock.h"
 #include <time.h>
 #include <stdio.h>
+
+#include <stdlib.h>
 #include <check.h>
 
 #define N_EVENTS 10000

tests/check_eventloop_eth.c

@@ -9,6 +9,7 @@
 
 #include "testing_clock.h"
 #include <time.h>
+#include <stdlib.h>
 #include <check.h>
 
 static UA_EventLoop *el;

tests/check_eventloop_interrupt.c

@@ -10,6 +10,7 @@
 #include "testing_clock.h"
 
 #include <signal.h>
+#include <stdlib.h>
 #include <check.h>
 
 #ifndef _WIN32

tests/check_eventloop_mqtt.c

@@ -11,6 +11,7 @@
 
 #include "testing_clock.h"
 #include <time.h>
+#include <stdlib.h>
 #include <check.h>
 
 unsigned int messageCount = 0;

tests/check_eventloop_tcp.c

@@ -9,6 +9,7 @@
 
 #include "testing_clock.h"
 #include <time.h>
+#include <stdlib.h>
 #include <check.h>
 
 static UA_EventLoop *el;

tests/check_eventloop_udp.c

@@ -9,6 +9,7 @@
 
 #include "testing_clock.h"
 #include <time.h>
+#include <stdlib.h>
 #include <check.h>
 
 static UA_EventLoop *el;

tests/check_securechannel.c

@@ -11,10 +11,12 @@
 #include "ua_types_encoding_binary.h"
 #include "ua_util_internal.h"
 
-#include "check.h"
 #include "testing_networklayers.h"
 #include "testing_policy.h"
 
+#include <stdlib.h>
+#include <check.h>
+
 #define UA_BYTESTRING_STATIC(s) {sizeof(s)-1, (UA_Byte*)(s)}
 
 // Some default testing sizes. Can be overwritten in testing functions.

tests/check_timer.c

@@ -3,8 +3,9 @@
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
 #include "ua_timer.h"
-#include "check.h"
 
+#include <check.h>
+#include <stdlib.h>
 #include <time.h>
 #include <stdio.h>
 

tests/check_types_builtin.c

@@ -9,6 +9,7 @@
 
 #include "ua_util_internal.h"
 
+#include <stdlib.h>
 #include <check.h>
 #include <float.h>
 #include <math.h>

tests/check_types_custom.c

@@ -7,7 +7,8 @@
 
 #include "ua_types_encoding_binary.h"
 
-#include "check.h"
+#include <stdlib.h>
+#include <check.h>
 #include <math.h>
 
 #ifdef __clang__

tests/check_types_parse.c

@@ -6,7 +6,8 @@
 #include <open62541/types_generated_handling.h>
 #include "open62541/util.h"
 
-#include "check.h"
+#include <stdlib.h>
+#include <check.h>
 
 START_TEST(base64) {
     UA_ByteString test1 = UA_BYTESTRING("abc123\nopen62541");

tests/check_types_range.c

@@ -7,7 +7,8 @@
 
 #include "ua_server_internal.h"
 
-#include "check.h"
+#include <stdlib.h>
+#include <check.h>
 
 START_TEST(parseRange) {
     UA_NumericRange range;

tests/client/check_client_highlevel.c

@@ -9,6 +9,7 @@
 #include <open62541/server_config_default.h>
 
 #include <check.h>
+#include <stdlib.h>
 
 #include "thread_wrapper.h"
 

tests/client/check_client_historical_data.c

@@ -18,13 +18,14 @@
 #include "client/ua_client_internal.h"
 
 #include <check.h>
+#include <stdlib.h>
+#include <stddef.h>
+#include <stdio.h>
 
 #include "testing_clock.h"
 #include "testing_networklayers.h"
 #include "thread_wrapper.h"
 #include "historical_read_test_data.h"
-#include <stddef.h>
-#include <stdio.h>
 
 static UA_Server *server;
 static UA_HistoryDataGathering *gathering;

tests/client/check_client_securechannel.c

@@ -10,7 +10,9 @@
 
 #include "client/ua_client_internal.h"
 
-#include "check.h"
+#include <check.h>
+#include <stdlib.h>
+
 #include "testing_clock.h"
 #include "thread_wrapper.h"
 

tests/multithreading/check_mt_addDeleteObject.c

@@ -7,6 +7,7 @@
 #include <open62541/client_config_default.h>
 #include <open62541/client_highlevel.h>
 #include <check.h>
+#include <stdlib.h>
 #include <stdio.h>
 #include "thread_wrapper.h"
 #include "deviceObjectType.h"

tests/multithreading/check_mt_addObjectNode.c

@@ -7,6 +7,7 @@
 #include <open62541/client_highlevel.h>
 #include <check.h>
 #include <stdio.h>
+#include <stdlib.h>
 #include "thread_wrapper.h"
 #include "deviceObjectType.h"
 

tests/multithreading/check_mt_addVariableNode.c

@@ -5,8 +5,11 @@
 #include <open62541/plugin/log_stdout.h>
 #include <open62541/client_config_default.h>
 #include <open62541/client_highlevel.h>
+
 #include <check.h>
 #include <stdio.h>
+#include <stdlib.h>
+
 #include "thread_wrapper.h"
 #include "mt_testing.h"
 

tests/multithreading/check_mt_addVariableTypeNode.c

@@ -7,6 +7,7 @@
 #include <open62541/client_config_default.h>
 #include <open62541/client_highlevel.h>
 #include <check.h>
+#include <stdlib.h>
 #include <stdio.h>
 #include "thread_wrapper.h"
 #include "mt_testing.h"

tests/multithreading/check_mt_readValueAttribute.c

@@ -6,6 +6,7 @@
 #include <open62541/client_config_default.h>
 #include <open62541/client_highlevel.h>
 #include <check.h>
+#include <stdlib.h>
 #include "thread_wrapper.h"
 #include "mt_testing.h"
 

tests/multithreading/check_mt_readWriteDelete.c

@@ -6,6 +6,7 @@
 #include <open62541/client_config_default.h>
 #include <open62541/client_highlevel.h>
 #include <check.h>
+#include <stdlib.h>
 #include <testing_clock.h>
 #include "thread_wrapper.h"
 #include "mt_testing.h"

tests/multithreading/check_mt_readWriteDeleteCallback.c

@@ -6,6 +6,7 @@
 #include <open62541/client_config_default.h>
 #include <open62541/client_highlevel.h>
 #include <check.h>
+#include <stdlib.h>
 #include <testing_clock.h>
 #include "thread_wrapper.h"
 #include "mt_testing.h"

tests/multithreading/check_mt_writeValueAttribute.c

@@ -7,6 +7,7 @@
 #include <open62541/client_config_default.h>
 #include <open62541/client_highlevel.h>
 #include <check.h>
+#include <stdlib.h>
 #include "thread_wrapper.h"
 #include "mt_testing.h"
 

tests/nodeset-loader/add_node_integration_test/client.c

@@ -4,6 +4,8 @@
 
 #include "ua_types_encoding_xml.h"
 
+#include <signal.h>
+#include <stdlib.h>
 #include <stdio.h>
 
 #define MIN_ARGUMENTS_NUM 3

tests/nodeset-loader/add_node_integration_test/server.c

@@ -6,6 +6,10 @@
 #include <open62541/server_config_default.h>
 #include <open62541/plugin/nodesetloader.h>
 
+#include <signal.h>
+#include <stdlib.h>
+#include <stdio.h>
+
 UA_Boolean running = true;
 static void stopHandler(int sign) {
     UA_LOG_INFO(UA_Log_Stdout, UA_LOGCATEGORY_SERVER, "received ctrl-c");

tests/nodeset-loader/check_nodeset_loader_autoid.c

@@ -6,7 +6,9 @@
 #include <open62541/server_config_default.h>
 #include <open62541/plugin/nodesetloader.h>
 
-#include "check.h"
+#include <check.h>
+#include <stdlib.h>
+
 #include "testing_clock.h"
 
 UA_Server *server = NULL;

tests/nodeset-loader/check_nodeset_loader_di.c

@@ -6,7 +6,9 @@
 #include <open62541/server_config_default.h>
 #include <open62541/plugin/nodesetloader.h>
 
-#include "check.h"
+#include <check.h>
+#include <stdlib.h>
+
 #include "testing_clock.h"
 
 UA_Server *server = NULL;

tests/nodeset-loader/check_nodeset_loader_input.c

@@ -6,7 +6,9 @@
 #include <open62541/server_config_default.h>
 #include <open62541/plugin/nodesetloader.h>
 
-#include "check.h"
+#include <check.h>
+#include <stdlib.h>
+
 #include "testing_clock.h"
 
 UA_Server *server = NULL;

tests/nodeset-loader/check_nodeset_loader_plc.c

@@ -6,7 +6,9 @@
 #include <open62541/server_config_default.h>
 #include <open62541/plugin/nodesetloader.h>
 
-#include "check.h"
+#include <check.h>
+#include <stdlib.h>
+
 #include "testing_clock.h"
 
 UA_Server *server = NULL;

tests/server/check_accesscontrol.c

@@ -8,6 +8,7 @@
 #include <open62541/server_config_default.h>
 #include <open62541/types.h>
 
+#include <stdlib.h>
 #include <check.h>
 
 #include "thread_wrapper.h"

tests/server/check_discovery.c

@@ -16,7 +16,9 @@
 #ifndef WIN32
 #include <sys/stat.h>
 #endif
+
 #include <check.h>
+#include <stdlib.h>
 
 #ifndef _WIN32
 #include <sys/stat.h>

tests/server/check_local_monitored_item.c

@@ -10,9 +10,9 @@
 #include <open62541/server_config_default.h>
 #include <open62541/types.h>
 
-#include <stddef.h>
+#include <check.h>
+#include <stdlib.h>
 
-#include "check.h"
 #include "testing_clock.h"
 #include "testing_networklayers.h"
 

tests/server/check_monitoreditem_filter.c

@@ -14,6 +14,7 @@
 #include "client/ua_client_internal.h"
 
 #include <check.h>
+#include <stdlib.h>
 
 #include "testing_clock.h"
 #include "thread_wrapper.h"

tests/server/check_node_inheritance.c

@@ -7,6 +7,7 @@
 #include "server/ua_server_internal.h"
 
 #include <check.h>
+#include <stdlib.h>
 
 UA_Server *server = NULL;
 UA_UInt32 valueToBeInherited = 42;

tests/server/check_server_asyncop.c

@@ -14,6 +14,7 @@
 #include "thread_wrapper.h"
 
 #include <check.h>
+#include <stdlib.h>
 
 UA_Boolean running;
 THREAD_HANDLE server_thread;
@@ -22,11 +23,11 @@ static size_t clientCounter;
 
 static UA_StatusCode
 methodCallback(UA_Server *serverArg,
-         const UA_NodeId *sessionId, void *sessionHandle,
-         const UA_NodeId *methodId, void *methodContext,
-         const UA_NodeId *objectId, void *objectContext,
-         size_t inputSize, const UA_Variant *input,
-         size_t outputSize, UA_Variant *output) {
+               const UA_NodeId *sessionId, void *sessionHandle,
+               const UA_NodeId *methodId, void *methodContext,
+               const UA_NodeId *objectId, void *objectContext,
+               size_t inputSize, const UA_Variant *input,
+               size_t outputSize, UA_Variant *output) {
     return UA_STATUSCODE_GOOD;
 }
 

tests/server/check_server_callbacks.c

@@ -6,9 +6,11 @@
 #include <open62541/plugin/log_stdout.h>
 #include <open62541/client_config_default.h>
 #include <open62541/client_highlevel.h>
-#include <check.h>
 #include "thread_wrapper.h"
 
+#include <stdlib.h>
+#include <check.h>
+
 /* While server initialization, value callbacks are called twice.
  * This counter is used to ensure that the deletion of the variable is triggered by the client (not while the server initialization)*/
 int counter  = 0;

tests/server/check_server_historical_data.c

@@ -19,6 +19,7 @@
 #include "server/ua_server_internal.h"
 
 #include <check.h>
+#include <stdlib.h>
 #include <stdio.h>
 
 #include "testing_clock.h"

tests/server/check_server_historical_data_circular.c

@@ -17,6 +17,7 @@
 #include <open62541/server_config_default.h>
 
 #include <check.h>
+#include <stdlib.h>
 #include <stdio.h>
 
 #include "server/ua_server_internal.h"

tests/server/check_server_jobs.c

@@ -7,6 +7,7 @@
 #include "server/ua_server_internal.h"
 
 #include <check.h>
+#include <stdlib.h>
 
 #include "testing_clock.h"
 

tests/server/check_server_monitoringspeed.c

@@ -10,6 +10,7 @@
 #include "ua_server_internal.h"
 
 #include <check.h>
+#include <stdlib.h>
 #include <stdio.h>
 #include <time.h>
 

tests/server/check_server_readspeed.c

@@ -12,6 +12,7 @@
 #include "ua_types_encoding_binary.h"
 
 #include <check.h>
+#include <stdlib.h>
 #include <time.h>
 #include <stdio.h>
 

tests/server/check_server_speed_addnodes.c

@@ -5,6 +5,7 @@
 #include <open62541/server_config_default.h>
 
 #include <check.h>
+#include <stdlib.h>
 #include <time.h>
 #include <stdio.h>
 

tests/server/check_server_userspace.c

@@ -6,6 +6,7 @@
 #include <open62541/types.h>
 
 #include <check.h>
+#include <stdlib.h>
 
 #ifdef __clang__
 //required for ck_assert_ptr_eq and const casting

tests/server/check_services_subscriptions.c

@@ -10,6 +10,7 @@
 #include "server/ua_subscription.h"
 
 #include <check.h>
+#include <stdlib.h>
 
 #include "testing_clock.h"
 

tests/server/check_session.c

@@ -12,6 +12,7 @@
 #include "client/ua_client_internal.h"
 
 #include <check.h>
+#include <stdlib.h>
 
 #include "thread_wrapper.h"
 

tests/server/check_subscription_event_filter.c

@@ -14,6 +14,7 @@
 #include <open62541/client_subscriptions.h>
 
 #include <check.h>
+#include <stdlib.h>
 #include <stdio.h>
 
 #include "testing_clock.h"

tests/server/check_subscription_events.c

@@ -16,6 +16,7 @@
 #include "server/ua_subscription.h"
 
 #include <check.h>
+#include <stdlib.h>
 #include <stdio.h>
 
 #include "testing_clock.h"