Refactor tests

Author: lucasssvaz

Diff for: tests/periman/periman.ino

@@ -8,8 +8,6 @@
  * - SDMMC: SDMMC requires a card to be mounted before the pins are attached
  */
 
-#include <unity.h>
-
 #if SOC_I2S_SUPPORTED
 #include "ESP_I2S.h"
 #endif
@@ -22,6 +20,13 @@
 #include "SPI.h"
 #endif
 
+#define ETH_PHY_TYPE        ETH_PHY_LAN8720
+#define ETH_PHY_ADDR         0
+#define ETH_PHY_MDC         23
+#define ETH_PHY_MDIO        18
+#define ETH_PHY_POWER       -1
+#define ETH_CLK_MODE        ETH_CLOCK_GPIO0_IN
+
 #include "ETH.h"
 
 /* Definitions */
@@ -32,9 +37,9 @@
 /* Global variables */
 
 bool test_executed = false;
-String last_test = "";
-int8_t uart1_rx_pin = UART1_RX_DEFAULT;
-int8_t uart1_tx_pin = UART1_TX_DEFAULT;
+String current_test;
+int8_t uart1_rx_pin;
+int8_t uart1_tx_pin;
 
 /* Callback functions */
 
@@ -46,69 +51,81 @@ void onReceive_cb(void) {
   }
 }
 
-/* Unity functions */
-
-// This function is automatically called by unity before each test is run
-void setUp(void) {
+// This function is called by before each test is run
+void setup_test(String test_name, int8_t rx_pin = UART1_RX_DEFAULT, int8_t tx_pin = UART1_TX_DEFAULT) {
   log_v("Setting up next test");
+
+  current_test = test_name;
+  uart1_rx_pin = rx_pin;
+  uart1_tx_pin = tx_pin;
+  test_executed = false;
+
+  pinMode(uart1_rx_pin, INPUT_PULLUP);
+  pinMode(uart1_tx_pin, OUTPUT);
   Serial1.setPins(uart1_rx_pin, uart1_tx_pin);
   uart_internal_loopback(1, uart1_rx_pin);
   delay(100);
   log_v("Running test");
 }
 
-// This function is automatically called by unity after each test is run
-void tearDown(void) {
+// This function is called after each test is run
+void teardown_test(void) {
   log_v("Tearing down last test");
   if (test_executed) {
-    Serial1.print(last_test);
+    pinMode(uart1_rx_pin, INPUT_PULLUP);
+    pinMode(uart1_tx_pin, OUTPUT);
+    Serial1.print(current_test);
     Serial1.println(" test: This should not be printed");
     Serial1.flush();
 
     Serial1.setPins(uart1_rx_pin, uart1_tx_pin);
     uart_internal_loopback(1, uart1_rx_pin);
     delay(100);
-    test_executed = false;
   }
 
-  Serial1.print(last_test);
+  Serial1.print(current_test);
   Serial1.println(" test: This should be printed");
   Serial1.flush();
 }
 
 /* Test functions */
-/* These functions must only init the peripheral on the same pins and update "last_test" */
+/* These functions must only init the peripheral on the same pins and update "current_test" */
 
 void gpio_test(void) {
-  last_test = "GPIO";
+  setup_test("GPIO");
   test_executed = true;
   pinMode(uart1_rx_pin, INPUT);
   pinMode(uart1_tx_pin, OUTPUT);
+  teardown_test();
 }
 
 void sigmadelta_test(void) {
-  last_test = "SigmaDelta";
+  setup_test("SigmaDelta");
 #if SOC_SDM_SUPPORTED
   test_executed = true;
   if (!sigmaDeltaAttach(uart1_rx_pin, 312500)) {
-    TEST_FAIL_MESSAGE("SigmaDelta init failed");
+    Serial.println("SigmaDelta init failed");
   }
   if (!sigmaDeltaAttach(uart1_tx_pin, 312500)) {
-    TEST_FAIL_MESSAGE("SigmaDelta init failed");
+    Serial.println("SigmaDelta init failed");
   }
 #endif
+  teardown_test();
 }
 
 void adc_oneshot_test(void) {
-  last_test = "ADC Oneshot";
-#if SOC_ADC_SUPPORTED
+#if !SOC_ADC_SUPPORTED
+  setup_test("ADC Oneshot");
+#else
+  setup_test("ADC Oneshot", A3, A4);
   test_executed = true;
   analogReadResolution(12);
   pinMode(A3, INPUT);
   pinMode(A4, INPUT);
   analogRead(A3);
   analogRead(A4);
 #endif
+  teardown_test();
 }
 
 #if SOC_ADC_SUPPORTED
@@ -119,8 +136,10 @@ void ARDUINO_ISR_ATTR adcComplete() {
 #endif
 
 void adc_continuous_test(void) {
-  last_test = "ADC Continuous";
-#if SOC_ADC_SUPPORTED
+#if !SOC_ADC_SUPPORTED
+  setup_test("ADC Continuous");
+#else
+  setup_test("ADC Continuous", A3, A4);
   test_executed = true;
   uint8_t adc_pins[] = {A3, A4};
   uint8_t adc_pins_count = 2;
@@ -137,93 +156,106 @@ void adc_continuous_test(void) {
   }
 
   if (!analogContinuousRead(&result, 0)) {
-    TEST_FAIL_MESSAGE("ADC continuous read failed");
+    Serial.println("ADC continuous read failed");
   }
 
   analogContinuousStop();
 #endif
+  teardown_test();
 }
 
 void dac_test(void) {
-  last_test = "DAC";
-#if SOC_DAC_SUPPORTED
+#if !SOC_DAC_SUPPORTED
+  setup_test("DAC");
+#else
+  setup_test("DAC", DAC1, DAC2);
   test_executed = true;
   dacWrite(DAC1, 255);
   dacWrite(DAC2, 255);
 #endif
+  teardown_test();
 }
 
 void ledc_test(void) {
-  last_test = "LEDC";
+  setup_test("LEDC");
 #if SOC_LEDC_SUPPORTED
   test_executed = true;
   if (!ledcAttach(uart1_rx_pin, 5000, 12)) {
-    TEST_FAIL_MESSAGE("LEDC init failed");
+    Serial.println("LEDC init failed");
   }
   if (!ledcAttach(uart1_tx_pin, 5000, 12)) {
-    TEST_FAIL_MESSAGE("LEDC init failed");
+    Serial.println("LEDC init failed");
   }
 #endif
+  teardown_test();
 }
 
 void rmt_test(void) {
-  last_test = "RMT";
+  setup_test("RMT");
 #if SOC_RMT_SUPPORTED
   test_executed = true;
   if (!rmtInit(uart1_rx_pin, RMT_TX_MODE, RMT_MEM_NUM_BLOCKS_1, 10000000)) {
-    TEST_FAIL_MESSAGE("RMT init failed");
+    Serial.println("RMT init failed");
   }
   if (!rmtInit(uart1_tx_pin, RMT_RX_MODE, RMT_MEM_NUM_BLOCKS_1, 10000000)) {
-    TEST_FAIL_MESSAGE("RMT init failed");
+    Serial.println("RMT init failed");
   }
 #endif
+  teardown_test();
 }
 
 void i2s_test(void) {
-  last_test = "I2S";
+  setup_test("I2S");
 #if SOC_I2S_SUPPORTED
   test_executed = true;
   I2SClass i2s;
 
   i2s.setPins(uart1_rx_pin, uart1_tx_pin, -1);
   i2s.setTimeout(1000);
   if (!i2s.begin(I2S_MODE_STD, 16000, I2S_DATA_BIT_WIDTH_16BIT, I2S_SLOT_MODE_STEREO)) {
-    TEST_FAIL_MESSAGE("I2S init failed");
+    Serial.println("I2S init failed");
   }
 #endif
+  teardown_test();
 }
 
 void i2c_test(void) {
-  last_test = "I2C";
+  setup_test("I2C");
 #if SOC_I2C_SUPPORTED
   test_executed = true;
   if (!Wire.begin(uart1_rx_pin, uart1_tx_pin)) {
-    TEST_FAIL_MESSAGE("I2C init failed");
+    Serial.println("I2C init failed");
   }
 #endif
+  teardown_test();
 }
 
 void spi_test(void) {
-  last_test = "SPI";
+  setup_test("SPI");
 #if SOC_GPSPI_SUPPORTED
   test_executed = true;
   SPI.begin(uart1_rx_pin, uart1_tx_pin, -1, -1);
 #endif
+  teardown_test();
 }
 
 void touch_test(void) {
-  last_test = "Touch";
-#if SOC_TOUCH_SENSOR_SUPPORTED
+#if !SOC_TOUCH_SENSOR_SUPPORTED
+  setup_test("Touch");
+#else
+  setup_test("Touch", T1, T2);
   test_executed = true;
   touchRead(T1);
   touchRead(T2);
 #endif
+  teardown_test();
 }
 
 void eth_test(void) {
-  last_test = "ETH";
+  setup_test("ETH");
   test_executed = true;
   ETH.begin();
+  teardown_test();
 }
 
 /* Main functions */
@@ -238,28 +270,18 @@ void setup() {
   Serial1.onReceive(onReceive_cb);
   uart_internal_loopback(1, uart1_rx_pin);
 
-  UNITY_BEGIN();
-
-  RUN_TEST(gpio_test);
-  RUN_TEST(sigmadelta_test);
-  RUN_TEST(ledc_test);
-  RUN_TEST(rmt_test);
-  RUN_TEST(i2s_test);
-  RUN_TEST(i2c_test);
-  RUN_TEST(spi_test);
-  uart1_tx_pin = A3;
-  uart1_rx_pin = A4;
-  RUN_TEST(adc_oneshot_test);
-  RUN_TEST(adc_continuous_test);
-  uart1_tx_pin = DAC1;
-  uart1_rx_pin = DAC2;
-  RUN_TEST(dac_test);
-  uart1_tx_pin = T1;
-  uart1_rx_pin = T2;
-  RUN_TEST(touch_test);
-  RUN_TEST(eth_test);
-
-  UNITY_END();
+  gpio_test();
+  sigmadelta_test();
+  ledc_test();
+  rmt_test();
+  i2s_test();
+  i2c_test();
+  spi_test();
+  adc_oneshot_test();
+  adc_continuous_test();
+  dac_test();
+  touch_test();
+  eth_test();
 }
 
 void loop() {}