diff --git a/test/unit/Filelists.cmake b/test/unit/Filelists.cmake index c5c8afb7a..50843e060 100644 --- a/test/unit/Filelists.cmake +++ b/test/unit/Filelists.cmake @@ -29,4 +29,5 @@ set(LWIP_TESTFILES ${LWIP_TESTDIR}/tcp/test_tcp_state.c ${LWIP_TESTDIR}/tcp/test_tcp.c ${LWIP_TESTDIR}/udp/test_udp.c + ${LWIP_TESTDIR}/ppp/test_pppos.c ) diff --git a/test/unit/Filelists.mk b/test/unit/Filelists.mk index 327625594..a51b6f874 100644 --- a/test/unit/Filelists.mk +++ b/test/unit/Filelists.mk @@ -49,5 +49,6 @@ TESTFILES=$(TESTDIR)/lwip_unittests.c \ $(TESTDIR)/tcp/test_tcp_oos.c \ $(TESTDIR)/tcp/test_tcp_state.c \ $(TESTDIR)/tcp/test_tcp.c \ - $(TESTDIR)/udp/test_udp.c + $(TESTDIR)/udp/test_udp.c \ + $(TESTDIR)/ppp/test_pppos.c diff --git a/test/unit/lwip_unittests.c b/test/unit/lwip_unittests.c index cd6b5851e..8c45e7e3e 100644 --- a/test/unit/lwip_unittests.c +++ b/test/unit/lwip_unittests.c @@ -17,6 +17,7 @@ #include "mdns/test_mdns.h" #include "mqtt/test_mqtt.h" #include "api/test_sockets.h" +#include "ppp/test_pppos.h" #include "lwip/init.h" #if !NO_SYS @@ -81,6 +82,9 @@ int main(void) mdns_suite, mqtt_suite, sockets_suite, +#if PPP_SUPPORT && PPPOS_SUPPORT + pppos_suite, +#endif /* PPP_SUPPORT && PPPOS_SUPPORT */ #endif ip4route_suite }; diff --git a/test/unit/lwipopts.h b/test/unit/lwipopts.h index ee419ad40..b330197fc 100644 --- a/test/unit/lwipopts.h +++ b/test/unit/lwipopts.h @@ -68,6 +68,10 @@ #define LWIP_MDNS_RESPONDER 1 #define LWIP_NUM_NETIF_CLIENT_DATA (LWIP_MDNS_RESPONDER) +/* Enable PPP and PPPOS support for PPPOS test suites */ +#define PPP_SUPPORT 1 +#define PPPOS_SUPPORT 1 + /* Minimal changes to opt.h required for etharp unit tests: */ #define ETHARP_SUPPORT_STATIC_ENTRIES 1 diff --git a/test/unit/ppp/test_pppos.c b/test/unit/ppp/test_pppos.c new file mode 100644 index 000000000..a6fbb9343 --- /dev/null +++ b/test/unit/ppp/test_pppos.c @@ -0,0 +1,64 @@ +#include "test_pppos.h" + +#include "lwip/netif.h" +#include "netif/ppp/pppos.h" +#include "netif/ppp/ppp.h" + +#if PPP_SUPPORT && PPPOS_SUPPORT +static struct netif pppos_netif; +static ppp_pcb *ppp; + +static u32_t ppp_output_cb(struct ppp_pcb_s *pcb, u8_t *data, u32_t len, void *ctx) +{ + LWIP_UNUSED_ARG(pcb); + LWIP_UNUSED_ARG(data); + LWIP_UNUSED_ARG(len); + LWIP_UNUSED_ARG(ctx); + + return 0; +} + +static void ppp_link_status_cb(ppp_pcb *pcb, int err_code, void *ctx) +{ + LWIP_UNUSED_ARG(pcb); + LWIP_UNUSED_ARG(err_code); + LWIP_UNUSED_ARG(ctx); +} + +static void pppos_setup(void) +{ + ppp = pppos_create(&pppos_netif, ppp_output_cb, ppp_link_status_cb, NULL); + fail_if(ppp == NULL); + ppp_connect(ppp, 0); +} + +static void pppos_teardown(void) +{ +} + +START_TEST(test_pppos_empty_packet_with_valid_fcs) +{ + u8_t two_breaks[] = { 0x7e, 0, 0, 0x7e }; + u8_t other_packet[] = { 0x7e, 0x7d, 0x20, 0x00, 0x7e }; + /* Set internal states of the underlying pcb */ + pppos_pcb *pppos = (pppos_pcb *)ppp->link_ctx_cb; + pppos->open = 1; /* Pretend the connection is open already */ + pppos->in_accm[0] = 0xf0; /* Make sure 0x0's are not escaped chars */ + + pppos_input(ppp, two_breaks, sizeof(two_breaks)); + pppos_input(ppp, other_packet, sizeof(other_packet)); + +} +END_TEST + +/** Create the suite including all tests for this module */ +Suite * +pppos_suite(void) +{ + testfunc tests[] = { + TESTFUNC(test_pppos_empty_packet_with_valid_fcs) + }; + return create_suite("PPPOS", tests, sizeof(tests)/sizeof(testfunc), pppos_setup, pppos_teardown); +} + +#endif /* PPP_SUPPORT && PPPOS_SUPPORT */ diff --git a/test/unit/ppp/test_pppos.h b/test/unit/ppp/test_pppos.h new file mode 100644 index 000000000..56b3b0cbb --- /dev/null +++ b/test/unit/ppp/test_pppos.h @@ -0,0 +1,13 @@ +#ifndef LWIP_HDR_TEST_PPPOS_H +#define LWIP_HDR_TEST_PPPOS_H + +#include "../lwip_check.h" +#include "netif/ppp/ppp.h" + +#if PPP_SUPPORT && PPPOS_SUPPORT + +Suite* pppos_suite(void); + +#endif /* PPP_SUPPORT && PPPOS_SUPPORT */ + +#endif /* LWIP_HDR_TEST_PPPOS_H */