feat(eppp): Add simple CI check

Author: david-cermak

.pre-commit-config.yaml

@@ -61,8 +61,8 @@ repos:
   - repo: local
     hooks:
       - id: commit message scopes
-        name: "commit message must be scoped with: mdns, modem, websocket, asio, mqtt_cxx, console, common"
-        entry: '\A(?!(feat|fix|ci|bump|test|docs)\((mdns|modem|common|console|websocket|asio|mqtt_cxx|examples|eppp_link)\)\:)'
+        name: "commit message must be scoped with: mdns, modem, websocket, asio, mqtt_cxx, console, common, eppp"
+        entry: '\A(?!(feat|fix|ci|bump|test|docs)\((mdns|modem|common|console|websocket|asio|mqtt_cxx|examples|eppp)\)\:)'
         language: pygrep
         args: [--multiline]
         stages: [commit-msg]

components/eppp_link/.cz.yaml

@@ -1,6 +1,6 @@
 ---
 commitizen:
-  bump_message: 'bump(eppp_link): $current_version -> $new_version'
+  bump_message: 'bump(eppp): $current_version -> $new_version'
   pre_bump_hooks: python ../../ci/changelog.py eppp_link
   tag_format: epp_link-v$version
   version: 0.0.1

components/eppp_link/eppp_link.c

@@ -58,6 +58,14 @@ enum blocked_status {
     SLAVE_WANTS_WRITE,
 };
 
+static void IRAM_ATTR timer_callback(void *arg)
+{
+    struct eppp_handle *h = arg;
+    if (h->blocked == SLAVE_WANTS_WRITE) {
+        gpio_set_level(h->gpio_intr, 0);
+    }
+}
+
 #endif // CONFIG_EPPP_LINK_DEVICE_SPI
 
 struct eppp_handle {
@@ -128,14 +136,6 @@ static esp_err_t transmit(void *h, void *buffer, size_t len)
     return ESP_OK;
 }
 
-static void IRAM_ATTR timer_callback(void *arg)
-{
-    struct eppp_handle *h = arg;
-    if (h->blocked == SLAVE_WANTS_WRITE) {
-        gpio_set_level(h->gpio_intr, 0);
-    }
-}
-
 static void netif_deinit(esp_netif_t *netif)
 {
     if (netif == NULL) {

components/eppp_link/test/test_app/main/app_main.c

@@ -171,7 +171,7 @@ TEST(eppp_test, open_close)
     TEST_ASSERT_NOT_NULL(client.netif);
 
     // Now that we're connected, let's try to ping clients address
-    bits = ping_test(config.ppp.their_ip4_addr, eppp_server, client.event);
+    bits = ping_test(config.ppp.their_ip4_addr.addr, eppp_server, client.event);
     TEST_ASSERT_EQUAL(bits & (PING_SUCCEEDED | PING_FAILED), PING_SUCCEEDED);
 
     // Trigger client disconnection and close the server
@@ -237,7 +237,7 @@ TEST(eppp_test, open_close_nonblocking)
     TEST_ASSERT_EQUAL(bits & wait_bits, wait_bits);
 
     // Now that we're connected, let's try to ping clients address
-    bits = ping_test(server_config.ppp.their_ip4_addr, eppp_server, event);
+    bits = ping_test(server_config.ppp.their_ip4_addr.addr, eppp_server, event);
     TEST_ASSERT_EQUAL(bits & (PING_SUCCEEDED | PING_FAILED), PING_SUCCEEDED);
 
     // stop network for both client and server
@@ -304,7 +304,7 @@ TEST(eppp_test, open_close_taskless)
     xEventGroupClearBits(info.event, wait_bits);
 
     // Now that we're connected, let's try to ping clients address
-    bits = ping_test(server_config.ppp.their_ip4_addr, info.eppp_server, info.event);
+    bits = ping_test(server_config.ppp.their_ip4_addr.addr, info.eppp_server, info.event);
     TEST_ASSERT_EQUAL(bits & (PING_SUCCEEDED | PING_FAILED), PING_SUCCEEDED);
 
     // stop network for both client and server, we won't wait for completion so expecting ESP_FAIL