Merge pull request #14708 from Patater/fix-test-function-return

test: Fix function does not return a value warnings

Author: 0xc0170

UNITTESTS/stubs/FileHandle_stub.h

@@ -67,6 +67,7 @@ class FileHandle_stub : public FileHandle {
 
     virtual int close()
     {
+        return 0;
     }
 
     virtual short poll(short events) const

UNITTESTS/stubs/connectivity/AT_CellularDevice_stub.cpp

@@ -56,6 +56,7 @@ ATHandler *AT_CellularDevice::get_at_handler()
 
 CellularContext *create_context(const char *apn)
 {
+    return nullptr;
 }
 
 void delete_context(CellularContext *context)

UNITTESTS/stubs/connectivity/LoRaMac_stub.cpp

@@ -94,6 +94,7 @@ void LoRaMac::post_process_mlme_ind()
 
 lorawan_time_t LoRaMac::get_current_time(void)
 {
+    return 0;
 }
 
 rx_slot_t LoRaMac::get_current_slot(void)

UNITTESTS/stubs/drivers/BufferedSerial_stub.cpp

@@ -85,6 +85,7 @@ ssize_t BufferedSerial::write_unbuffered(const char *buf_ptr, size_t length)
 
 bool BufferedSerial::hup() const
 {
+    return false;
 }
 
 void BufferedSerial::wake()

UNITTESTS/stubs/events/equeue_stub.c

@@ -105,7 +105,7 @@ void equeue_background(equeue_t *queue,
 
 int equeue_chain(equeue_t *queue, equeue_t *target)
 {
-
+    return 0;
 }
 
 int equeue_call_in(equeue_t *q, int ms, void (*cb)(void *), void *data)

UNITTESTS/stubs/storage/EmulatedSD.cpp

@@ -62,6 +62,7 @@ int EmulatedSD::deinit()
 {
     fclose(_p->fs);
     _p->fs = nullptr;
+    return 0;
 }
 
 int EmulatedSD::read(void *buffer, bd_addr_t addr, bd_size_t size)

connectivity/cellular/tests/UNITTESTS/framework/AT/at_cellularcontext/at_cellularcontexttest.cpp

@@ -58,10 +58,8 @@ class TestAT_CellularContext : public testing::Test {
         ATHandler_stub::call_immediately = false;
         ATHandler_stub::resp_info_true_counter = false;
         ATHandler_stub::info_elem_true_counter = false;
-        ATHandler_stub::int_valid_count_table[kRead_int_table_size];
         ATHandler_stub::int_count = kRead_int_table_size;
         ATHandler_stub::read_string_index = kRead_string_table_size;
-        ATHandler_stub::read_string_table[kRead_string_table_size];
         ATHandler_stub::resp_stop_success_count = kResp_stop_count_default;
         CellularDevice_stub::connect_counter = 2;
 
@@ -153,6 +151,8 @@ class my_AT_CTXIPV6 : public AT_CellularContext {
         if (!_stack) {
             _stack = new my_stack(_at, *get_device());
         }
+
+        return _stack;
     }
     virtual uint32_t get_timeout_for_operation(ContextOperation op) const
     {

connectivity/lorawan/tests/UNITTESTS/features/lorawan/loramac/Test_LoRaMac.cpp

@@ -537,7 +537,7 @@ TEST_F(Test_LoRaMac, post_process_mlme_ind)
 
 uint8_t batt_cb()
 {
-
+    return 100;
 }
 
 TEST_F(Test_LoRaMac, set_batterylevel_callback)

connectivity/lorawan/tests/UNITTESTS/features/lorawan/loraphy/Test_LoRaPHY.cpp

@@ -88,6 +88,7 @@ class my_radio : public LoRaRadio {
 
     virtual uint32_t random(void)
     {
+        return 4;
     };
 
     virtual uint8_t get_status(void)
@@ -105,6 +106,7 @@ class my_radio : public LoRaRadio {
 
     virtual uint32_t time_on_air(radio_modems_t modem, uint8_t pkt_len)
     {
+        return 0;
     };
 
     virtual bool perform_carrier_sense(radio_modems_t modem,

connectivity/lorawan/tests/UNITTESTS/features/lorawan/loraphyas923/Test_LoRaPHYAS923.cpp

@@ -71,6 +71,7 @@ class my_radio : public LoRaRadio {
 
     virtual uint32_t random(void)
     {
+        return 4;
     };
 
     virtual uint8_t get_status(void)
@@ -88,6 +89,7 @@ class my_radio : public LoRaRadio {
 
     virtual uint32_t time_on_air(radio_modems_t modem, uint8_t pkt_len)
     {
+        return 0;
     };
 
     virtual bool perform_carrier_sense(radio_modems_t modem,