LoRaMac unittested

Antti Kauppila · adbridge · commit 18a1e66ebc8b · 2018-10-08T15:29:18.000+01:00
diff --git a/UNITTESTS/features/lorawan/loramac/Test_LoRaMac.cpp b/UNITTESTS/features/lorawan/loramac/Test_LoRaMac.cpp
diff --git a/UNITTESTS/features/lorawan/loraphy/unittest.cmake b/UNITTESTS/features/lorawan/loraphy/unittest.cmake
@@ -33,7 +33,7 @@ set(unittest-includes ${unittest-includes}
 set(unittest-test-sources
   ../features/lorawan/loraphy/Test_LoRaPHY.cpp
   stubs/LoRaWANTimer_stub.cpp
-
+  stubs/mbed_assert_stub.c
 )
 
 
diff --git a/UNITTESTS/features/lorawan/loraphyas923/unittest.cmake b/UNITTESTS/features/lorawan/loraphyas923/unittest.cmake
@@ -34,6 +34,7 @@ set(unittest-test-sources
   ../features/lorawan/loraphyas923/Test_LoRaPHYAS923.cpp
   stubs/LoRaPHY_stub.cpp
   stubs/LoRaWANTimer_stub.cpp
+  stubs/mbed_assert_stub.c
 
 )
 
diff --git a/UNITTESTS/features/lorawan/loraphyau915/unittest.cmake b/UNITTESTS/features/lorawan/loraphyau915/unittest.cmake
@@ -34,6 +34,7 @@ set(unittest-test-sources
   ../features/lorawan/loraphyau915/Test_LoRaPHYAU915.cpp
   stubs/LoRaPHY_stub.cpp
   stubs/LoRaWANTimer_stub.cpp
+  stubs/mbed_assert_stub.c
 
 )
 
diff --git a/UNITTESTS/features/lorawan/loraphycn470/unittest.cmake b/UNITTESTS/features/lorawan/loraphycn470/unittest.cmake
@@ -34,6 +34,7 @@ set(unittest-test-sources
   ../features/lorawan/loraphycn470/Test_LoRaPHYCN470.cpp
   stubs/LoRaPHY_stub.cpp
   stubs/LoRaWANTimer_stub.cpp
+  stubs/mbed_assert_stub.c
 
 )
 
diff --git a/UNITTESTS/features/lorawan/loraphycn779/unittest.cmake b/UNITTESTS/features/lorawan/loraphycn779/unittest.cmake
@@ -33,6 +33,7 @@ set(unittest-includes ${unittest-includes}
 set(unittest-test-sources
   ../features/lorawan/loraphycn779/Test_LoRaPHYCN779.cpp
   stubs/LoRaPHY_stub.cpp
+  stubs/mbed_assert_stub.c
 
 )
 
diff --git a/UNITTESTS/features/lorawan/loraphyeu433/unittest.cmake b/UNITTESTS/features/lorawan/loraphyeu433/unittest.cmake
@@ -33,6 +33,7 @@ set(unittest-includes ${unittest-includes}
 set(unittest-test-sources
   ../features/lorawan/loraphyeu433/Test_LoRaPHYEU433.cpp
   stubs/LoRaPHY_stub.cpp
+  stubs/mbed_assert_stub.c
 
 )
 
diff --git a/UNITTESTS/features/lorawan/loraphyeu868/unittest.cmake b/UNITTESTS/features/lorawan/loraphyeu868/unittest.cmake
@@ -33,6 +33,7 @@ set(unittest-includes ${unittest-includes}
 set(unittest-test-sources
   ../features/lorawan/loraphyeu868/Test_LoRaPHYEU868.cpp
   stubs/LoRaPHY_stub.cpp
+  stubs/mbed_assert_stub.c
 
 )
 
diff --git a/UNITTESTS/features/lorawan/loraphyin865/unittest.cmake b/UNITTESTS/features/lorawan/loraphyin865/unittest.cmake
@@ -33,6 +33,7 @@ set(unittest-includes ${unittest-includes}
 set(unittest-test-sources
   ../features/lorawan/loraphyin865/Test_LoRaPHYIN865.cpp
   stubs/LoRaPHY_stub.cpp
+  stubs/mbed_assert_stub.c
 
 )
 
diff --git a/UNITTESTS/features/lorawan/loraphykr920/unittest.cmake b/UNITTESTS/features/lorawan/loraphykr920/unittest.cmake
@@ -34,6 +34,7 @@ set(unittest-test-sources
   ../features/lorawan/loraphykr920/Test_LoRaPHYKR920.cpp
   stubs/LoRaPHY_stub.cpp
   stubs/LoRaWANTimer_stub.cpp
+  stubs/mbed_assert_stub.c
 
 )
 
diff --git a/UNITTESTS/features/lorawan/loraphyus915/unittest.cmake b/UNITTESTS/features/lorawan/loraphyus915/unittest.cmake
@@ -34,6 +34,7 @@ set(unittest-test-sources
   ../features/lorawan/loraphyus915/Test_LoRaPHYUS915.cpp
   stubs/LoRaPHY_stub.cpp
   stubs/LoRaWANTimer_stub.cpp
+  stubs/mbed_assert_stub.c
 
 )
 
diff --git a/UNITTESTS/stubs/EventQueue_stub.cpp b/UNITTESTS/stubs/EventQueue_stub.cpp
@@ -17,9 +17,13 @@
 
 #include "EventQueue.h"
 #include "Callback.h"
+#include "EventQueue_stub.h"
 
 using namespace mbed;
 
+int EventQueue_stub::int_value = 0;
+unsigned EventQueue_stub::unsigned_value = 0;
+
 namespace events {
 
 EventQueue::EventQueue(unsigned event_size, unsigned char *event_pointer)
@@ -40,7 +44,7 @@ void EventQueue::break_dispatch()
 
 unsigned EventQueue::tick()
 {
-    return 0;
+    return EventQueue_stub::unsigned_value;
 }
 
 void EventQueue::cancel(int id)
@@ -49,7 +53,7 @@ void EventQueue::cancel(int id)
 
 int EventQueue::time_left(int id)
 {
-    return 0;
+    return EventQueue_stub::int_value;
 }
 
 void EventQueue::background(Callback<void(int)> update)
diff --git a/UNITTESTS/stubs/EventQueue_stub.h b/UNITTESTS/stubs/EventQueue_stub.h
@@ -0,0 +1,25 @@
+/*
+ * Copyright (c) , Arm Limited and affiliates.
+ * SPDX-License-Identifier: Apache-2.0
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+#include <inttypes.h>
+
+namespace EventQueue_stub
+{
+extern int int_value;
+extern unsigned unsigned_value;
+}
+
diff --git a/UNITTESTS/stubs/LoRaMacCommand_stub.cpp b/UNITTESTS/stubs/LoRaMacCommand_stub.cpp
@@ -18,6 +18,12 @@
 #include "LoRaMacCommand.h"
 #include "LoRaMac.h"
 
+#include "LoRaMacCommand_stub.h"
+
+lorawan_status_t LoRaMacCommand_stub::status_value = LORAWAN_STATUS_OK;
+bool LoRaMacCommand_stub::bool_value = false;
+uint8_t LoRaMacCommand_stub::uint8_value = 0;
+int32_t LoRaMacCommand_stub::int32_value = 0;
 
 LoRaMacCommand::LoRaMacCommand()
 {
@@ -29,12 +35,12 @@ void LoRaMacCommand::clear_command_buffer()
 
 uint8_t LoRaMacCommand::get_mac_cmd_length() const
 {
-    return 0;
+    return LoRaMacCommand_stub::uint8_value;
 }
 
 uint8_t *LoRaMacCommand::get_mac_commands_buffer()
 {
-    return NULL;
+    return &LoRaMacCommand_stub::uint8_value;
 }
 
 void LoRaMacCommand::parse_mac_commands_to_repeat()
@@ -52,7 +58,7 @@ void LoRaMacCommand::copy_repeat_commands_to_buffer()
 
 uint8_t LoRaMacCommand::get_repeat_commands_length() const
 {
-    return 0;
+    return LoRaMacCommand_stub::uint8_value;
 }
 
 void LoRaMacCommand::clear_sticky_mac_cmd()
@@ -61,7 +67,7 @@ void LoRaMacCommand::clear_sticky_mac_cmd()
 
 bool LoRaMacCommand::has_sticky_mac_cmd() const
 {
-    return false;
+    return LoRaMacCommand_stub::bool_value;
 }
 
 lorawan_status_t LoRaMacCommand::process_mac_commands(const uint8_t *payload, uint8_t mac_index,
@@ -70,12 +76,12 @@ lorawan_status_t LoRaMacCommand::process_mac_commands(const uint8_t *payload, ui
                                                       lora_mac_system_params_t &mac_sys_params,
                                                       LoRaPHY &lora_phy)
 {
-    return LORAWAN_STATUS_OK;
+    return LoRaMacCommand_stub::status_value;
 }
 
 int32_t LoRaMacCommand::cmd_buffer_remaining() const
 {
-    return 0;
+    return LoRaMacCommand_stub::int32_value;
 }
 
 void LoRaMacCommand::set_batterylevel_callback(mbed::Callback<uint8_t(void)> battery_level)
@@ -84,45 +90,45 @@ void LoRaMacCommand::set_batterylevel_callback(mbed::Callback<uint8_t(void)> bat
 
 lorawan_status_t LoRaMacCommand::add_link_check_req()
 {
-    return LORAWAN_STATUS_OK;
+    return LoRaMacCommand_stub::status_value;
 }
 
 lorawan_status_t LoRaMacCommand::add_link_adr_ans(uint8_t status)
 {
-    return LORAWAN_STATUS_OK;
+    return LoRaMacCommand_stub::status_value;
 }
 
 lorawan_status_t LoRaMacCommand::add_duty_cycle_ans()
 {
-    return LORAWAN_STATUS_OK;
+    return LoRaMacCommand_stub::status_value;
 }
 
 lorawan_status_t LoRaMacCommand::add_rx_param_setup_ans(uint8_t status)
 {
-    return LORAWAN_STATUS_OK;
+    return LoRaMacCommand_stub::status_value;
 }
 
 lorawan_status_t LoRaMacCommand::add_dev_status_ans(uint8_t battery, uint8_t margin)
 {
-    return LORAWAN_STATUS_OK;
+    return LoRaMacCommand_stub::status_value;
 }
 
 lorawan_status_t LoRaMacCommand::add_new_channel_ans(uint8_t status)
 {
-    return LORAWAN_STATUS_OK;
+    return LoRaMacCommand_stub::status_value;
 }
 
 lorawan_status_t LoRaMacCommand::add_rx_timing_setup_ans()
 {
-    return LORAWAN_STATUS_OK;
+    return LoRaMacCommand_stub::status_value;
 }
 
 lorawan_status_t LoRaMacCommand::add_tx_param_setup_ans()
 {
-    return LORAWAN_STATUS_OK;
+    return LoRaMacCommand_stub::status_value;
 }
 
 lorawan_status_t LoRaMacCommand::add_dl_channel_ans(uint8_t status)
 {
-    return LORAWAN_STATUS_OK;
+    return LoRaMacCommand_stub::status_value;
 }
diff --git a/UNITTESTS/stubs/LoRaMacCommand_stub.h b/UNITTESTS/stubs/LoRaMacCommand_stub.h
@@ -0,0 +1,28 @@
+/*
+ * Copyright (c) , Arm Limited and affiliates.
+ * SPDX-License-Identifier: Apache-2.0
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+
+#include <inttypes.h>
+#include "lorawan_types.h"
+
+namespace LoRaMacCommand_stub
+{
+extern lorawan_status_t status_value;
+extern bool bool_value;
+extern uint8_t uint8_value;
+extern int32_t int32_value;
+}
diff --git a/UNITTESTS/stubs/LoRaMacCrypto_stub.cpp b/UNITTESTS/stubs/LoRaMacCrypto_stub.cpp
@@ -21,6 +21,10 @@
 #include "LoRaMacCrypto.h"
 #include "system/lorawan_data_structures.h"
 
+#include "LoRaMacCrypto_stub.h"
+
+int LoRaMacCrypto_stub::int_table_idx_value = 0;
+int LoRaMacCrypto_stub::int_table[20] = {};
 
 LoRaMacCrypto::LoRaMacCrypto()
 {
@@ -29,34 +33,34 @@ LoRaMacCrypto::LoRaMacCrypto()
 int LoRaMacCrypto::compute_mic(const uint8_t *, uint16_t , const uint8_t *, uint32_t, uint32_t,
                                uint8_t dir, uint32_t, uint32_t *)
 {
-    return 0;
+    return LoRaMacCrypto_stub::int_table[LoRaMacCrypto_stub::int_table_idx_value++];
 }
 
 int LoRaMacCrypto::encrypt_payload(const uint8_t *, uint16_t , const uint8_t *, uint32_t, uint32_t,
                                    uint8_t , uint32_t , uint8_t *)
 {
-    return 0;
+    return LoRaMacCrypto_stub::int_table[LoRaMacCrypto_stub::int_table_idx_value++];
 }
 
 int LoRaMacCrypto::decrypt_payload(const uint8_t *, uint16_t , const uint8_t *, uint32_t, uint32_t,
                                    uint8_t , uint32_t , uint8_t *)
 {
-    return 0;
+    return LoRaMacCrypto_stub::int_table[LoRaMacCrypto_stub::int_table_idx_value++];
 }
 
 int LoRaMacCrypto::compute_join_frame_mic(const uint8_t *, uint16_t , const uint8_t *, uint32_t, uint32_t *)
 {
-    return 0;
+    return LoRaMacCrypto_stub::int_table[LoRaMacCrypto_stub::int_table_idx_value++];
 }
 
-int LoRaMacCrypto::decrypt_join_frame(const uint8_t *, uint16_t , const uint8_t *, uint32_t, uint8_t *)
+int LoRaMacCrypto::decrypt_join_frame(const uint8_t *in, uint16_t size, const uint8_t *, uint32_t, uint8_t *out)
 {
-    return 0;
+    memcpy(out, in, size);
+    return LoRaMacCrypto_stub::int_table[LoRaMacCrypto_stub::int_table_idx_value++];
 }
 
 int LoRaMacCrypto::compute_skeys_for_join_frame(const uint8_t *, uint32_t, const uint8_t *, uint16_t ,
                                                 uint8_t *, uint8_t *)
 {
-    return 0;
+    return LoRaMacCrypto_stub::int_table[LoRaMacCrypto_stub::int_table_idx_value++];
 }
-
diff --git a/UNITTESTS/stubs/LoRaMacCrypto_stub.h b/UNITTESTS/stubs/LoRaMacCrypto_stub.h
@@ -0,0 +1,25 @@
+/*
+ * Copyright (c) , Arm Limited and affiliates.
+ * SPDX-License-Identifier: Apache-2.0
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+
+#include <inttypes.h>
+
+namespace LoRaMacCrypto_stub
+{
+extern int int_table_idx_value;
+extern int int_table[20];
+}
diff --git a/UNITTESTS/stubs/LoRaWANTimer_stub.cpp b/UNITTESTS/stubs/LoRaWANTimer_stub.cpp
@@ -15,11 +15,13 @@
  * limitations under the License.
  */
 
+#include "mbed_assert.h"
 #include "LoRaWANTimer.h"
 
 #include "LoRaWANTimer_stub.h"
 
 lorawan_time_t LoRaWANTimer_stub::time_value = 0;
+bool LoRaWANTimer_stub::call_cb_immediately = false;
 
 LoRaWANTimeHandler::LoRaWANTimeHandler()
     : _queue(NULL)
@@ -46,6 +48,10 @@ lorawan_time_t LoRaWANTimeHandler::get_elapsed_time(lorawan_time_t saved_time)
 
 void LoRaWANTimeHandler::init(timer_event_t &obj, mbed::Callback<void()> callback)
 {
+    if (callback && LoRaWANTimer_stub::call_cb_immediately) {
+        callback();
+    }
+    obj.timer_id = 1;
 }
 
 void LoRaWANTimeHandler::start(timer_event_t &obj, const uint32_t timeout)
@@ -54,4 +60,5 @@ void LoRaWANTimeHandler::start(timer_event_t &obj, const uint32_t timeout)
 
 void LoRaWANTimeHandler::stop(timer_event_t &obj)
 {
+    obj.timer_id = 0;
 }
diff --git a/UNITTESTS/stubs/LoRaWANTimer_stub.h b/UNITTESTS/stubs/LoRaWANTimer_stub.h
diff --git a/features/lorawan/lorastack/mac/LoRaMac.cpp b/features/lorawan/lorastack/mac/LoRaMac.cpp

Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,7 @@ set(unittest-includes ${unittest-includes}`
`33`	`33`	`set(unittest-test-sources`
`34`	`34`	`../features/lorawan/loraphy/Test_LoRaPHY.cpp`
`35`	`35`	`stubs/LoRaWANTimer_stub.cpp`
`36`		`-`
	`36`	`+ stubs/mbed_assert_stub.c`
`37`	`37`	`)`
`38`	`38`
`39`	`39`
Original file line number	Diff line number	Diff line change
`@@ -34,6 +34,7 @@ set(unittest-test-sources`
`34`	`34`	`../features/lorawan/loraphyas923/Test_LoRaPHYAS923.cpp`
`35`	`35`	`stubs/LoRaPHY_stub.cpp`
`36`	`36`	`stubs/LoRaWANTimer_stub.cpp`
	`37`	`+ stubs/mbed_assert_stub.c`
`37`	`38`
`38`	`39`	`)`
`39`	`40`
Original file line number	Diff line number	Diff line change
`@@ -33,6 +33,7 @@ set(unittest-includes ${unittest-includes}`
`33`	`33`	`set(unittest-test-sources`
`34`	`34`	`../features/lorawan/loraphycn779/Test_LoRaPHYCN779.cpp`
`35`	`35`	`stubs/LoRaPHY_stub.cpp`
	`36`	`+ stubs/mbed_assert_stub.c`
`36`	`37`
`37`	`38`	`)`
`38`	`39`