Add skeleton tests for receiving and sending ISO-TP messages.

4 files changed, 226 insertions, 30 deletions

diff --git a/tests/common.c b/tests/common.c
new file mode 100644
index 00000000..fb26c4af
--- /dev/null
+++ b/tests/common.c
@@ -0,0 +1,86 @@
+#include <isotp/isotp.h>
+#include <stdint.h>
+#include <stdio.h>
+#include <stdlib.h>
+#include <stdarg.h>
+
+IsoTpShims SHIMS;
+IsoTpHandler ISOTP_HANDLER;
+
+uint16_t last_can_frame_sent_arb_id;
+uint8_t last_can_payload_sent;
+uint8_t last_can_payload_size;
+bool can_frame_was_sent;
+
+bool message_was_received;
+uint16_t last_message_received_arb_id;
+uint8_t* last_message_received_payload;
+uint8_t last_message_received_payload_size;
+
+uint16_t last_message_sent_arb_id;
+bool last_message_sent_status;
+uint8_t* last_message_sent_payload;
+uint8_t last_message_sent_payload_size;
+
+void debug(const char* format, ...) {
+    va_list args;
+    va_start(args, format);
+    vprintf(format, args);
+    va_end(args);
+}
+
+void mock_send_can(const uint16_t arbitration_id, const uint8_t* data,
+        const uint8_t size) {
+}
+
+void mock_set_timer(uint16_t time_ms, void (*callback)) {
+}
+
+void message_received(const uint16_t arbitration_id, const uint8_t* payload,
+        const uint16_t size) {
+    debug("Received ISO-TP message:");
+    log_isotp_message(arbitration_id, payload, size);
+    last_message_received_arb_id = arbitration_id;
+    last_message_received_payload_size = size;
+    memcpy(last_message_received_payload, payload, size);
+}
+
+void message_sent(const bool success, const uint16_t arbitration_id,
+        const uint8_t* payload, const uint16_t size) {
+    if(success) {
+        debug("Sent ISO-TP message:");
+    } else {
+        debug("Unable to send ISO-TP message:");
+    }
+    log_isotp_message(arbitration_id, payload, size);
+
+    message_was_received = true;
+    last_message_sent_arb_id = arbitration_id;
+    last_message_sent_payload_size = size;
+    memcpy(last_message_sent_payload, payload, size);
+}
+
+void can_frame_sent(const uint16_t arbitration_id,
+        const uint8_t* payload, const uint8_t size) {
+    debug("Sent CAN Frame:");
+    log_isotp_message(arbitration_id, payload, size);
+    for(int i = 0; i < size; i++) {
+        debug("0x%x", payload[i]);
+    }
+
+    can_frame_was_sent = true;
+    last_can_frame_sent_arb_id = arbitration_id;
+    last_can_payload_sent = size;
+    memcpy(last_can_payload_sent, payload, size);
+}
+
+void setup() {
+    SHIMS = isotp_init_shims(debug, mock_send_can, mock_set_timer);
+    ISOTP_HANDLER = isotp_init(&SHIMS, 0x2a, message_received, message_sent,
+            can_frame_sent);
+    last_message_received_payload = malloc(MAX_ISO_TP_MESSAGE_SIZE);
+    last_message_sent_status = false;
+    message_was_received = false;
+    can_frame_was_sent = false;
+}
+
diff --git a/tests/test_receive.c b/tests/test_receive.c
new file mode 100644
index 00000000..4134cfd6
--- /dev/null
+++ b/tests/test_receive.c
@@ -0,0 +1,79 @@
+#include <isotp/isotp.h>
+#include <check.h>
+#include <stdint.h>
+#include <stdio.h>
+#include <stdlib.h>
+#include <stdarg.h>
+
+extern IsoTpShims SHIMS;
+extern IsoTpHandler ISOTP_HANDLER;
+
+extern uint16_t last_can_frame_sent_arb_id;
+extern uint8_t last_can_payload_sent;
+extern uint8_t last_can_payload_size;
+extern bool can_frame_was_sent;
+
+extern bool message_was_received;
+extern uint16_t last_message_received_arb_id;
+extern uint8_t* last_message_received_payload;
+extern uint8_t last_message_received_payload_size;
+
+extern uint16_t last_message_sent_arb_id;
+extern bool last_message_sent_status;
+extern uint8_t* last_message_sent_payload;
+extern uint8_t last_message_sent_payload_size;
+
+extern void setup();
+
+START_TEST (test_receive_wrong_id)
+{
+    const uint8_t data[8] = {0};
+    isotp_receive_can_frame(0x100, data, sizeof(data));
+    fail_if(message_was_received);
+}
+END_TEST
+
+START_TEST (test_receive_bad_pci)
+{
+    // 4 is a reserved number for the PCI field - only 0-3 are allowed
+    const uint8_t data[8] = {0x40};
+    isotp_receive_can_frame(0x2a, data, sizeof(data));
+    fail_if(message_was_received);
+}
+END_TEST
+
+START_TEST (test_receive_single_frame)
+{
+    const uint8_t data[8] = {0x0, 0x12, 0x34};
+    isotp_receive_can_frame(0x2a, data, sizeof(data));
+    fail_unless(message_was_received);
+    ck_assert_int_eq(last_message_received_arb_id, 0x2a);
+    ck_assert_int_eq(last_message_received_payload_size, 2);
+    ck_assert_int_eq(last_message_received_payload[0], 0x12);
+    ck_assert_int_eq(last_message_received_payload[0], 0x34);
+}
+END_TEST
+
+Suite* testSuite(void) {
+    Suite* s = suite_create("iso15765");
+    TCase *tc_core = tcase_create("receive");
+    tcase_add_checked_fixture(tc_core, setup, NULL);
+    tcase_add_test(tc_core, test_receive_wrong_id);
+    tcase_add_test(tc_core, test_receive_bad_pci);
+    tcase_add_test(tc_core, test_receive_single_frame);
+    suite_add_tcase(s, tc_core);
+
+    return s;
+}
+
+int main(void) {
+    int numberFailed;
+    Suite* s = testSuite();
+    SRunner *sr = srunner_create(s);
+    // Don't fork so we can actually use gdb
+    srunner_set_fork_status(sr, CK_NOFORK);
+    srunner_run_all(sr, CK_NORMAL);
+    numberFailed = srunner_ntests_failed(sr);
+    srunner_free(sr);
+    return (numberFailed == 0) ? 0 : 1;
+}
diff --git a/tests/test_send.c b/tests/test_send.c
new file mode 100644
index 00000000..144e040f
--- /dev/null
+++ b/tests/test_send.c
@@ -0,0 +1,61 @@
+#include <isotp/isotp.h>
+#include <check.h>
+#include <stdint.h>
+#include <stdio.h>
+#include <stdlib.h>
+#include <stdarg.h>
+
+extern IsoTpShims SHIMS;
+extern IsoTpHandler ISOTP_HANDLER;
+
+extern uint16_t last_can_frame_sent_arb_id;
+extern uint8_t last_can_payload_sent;
+extern uint8_t last_can_payload_size;
+extern bool can_frame_was_sent;
+
+extern bool message_was_received;
+extern uint16_t last_message_received_arb_id;
+extern uint8_t* last_message_received_payload;
+extern uint8_t last_message_received_payload_size;
+
+extern uint16_t last_message_sent_arb_id;
+extern bool last_message_sent_status;
+extern uint8_t* last_message_sent_payload;
+extern uint8_t last_message_sent_payload_size;
+
+extern void setup();
+
+START_TEST (test_send_single_frame)
+{
+    fail_if(true);
+}
+END_TEST
+
+START_TEST (test_send_multi_frame)
+{
+    fail_if(true);
+}
+END_TEST
+
+Suite* testSuite(void) {
+    Suite* s = suite_create("iso15765");
+    TCase *tc_core = tcase_create("send");
+    tcase_add_checked_fixture(tc_core, setup, NULL);
+    tcase_add_test(tc_core, test_send_single_frame);
+    tcase_add_test(tc_core, test_send_multi_frame);
+    suite_add_tcase(s, tc_core);
+
+    return s;
+}
+
+int main(void) {
+    int numberFailed;
+    Suite* s = testSuite();
+    SRunner *sr = srunner_create(s);
+    // Don't fork so we can actually use gdb
+    srunner_set_fork_status(sr, CK_NOFORK);
+    srunner_run_all(sr, CK_NORMAL);
+    numberFailed = srunner_ntests_failed(sr);
+    srunner_free(sr);
+    return (numberFailed == 0) ? 0 : 1;
+}
diff --git a/tests/tests.c b/tests/tests.c
deleted file mode 100644
index 62b3df8d..00000000
--- a/tests/tests.c
+++ /dev/null
@@ -1,30 +0,0 @@
-#include <check.h>
-#include <stdint.h>
-#include <bitfield/bitfield.h>
-
-START_TEST (test_fail)
-{
-    fail_if(true);
-}
-END_TEST
-
-Suite* bitfieldSuite(void) {
-    Suite* s = suite_create("iso15765");
-    TCase *tc_core = tcase_create("core");
-    tcase_add_test(tc_core, test_fail);
-    suite_add_tcase(s, tc_core);
-
-    return s;
-}
-
-int main(void) {
-    int numberFailed;
-    Suite* s = bitfieldSuite();
-    SRunner *sr = srunner_create(s);
-    // Don't fork so we can actually use gdb
-    srunner_set_fork_status(sr, CK_NOFORK);
-    srunner_run_all(sr, CK_NORMAL);
-    numberFailed = srunner_ntests_failed(sr);
-    srunner_free(sr);
-    return (numberFailed == 0) ? 0 : 1;
-}