feedc0de/esp-idf
1
0
Fork 0
forked from espressif/esp-idf
Code Pull Requests Activity

Merge branch 'bugfix/eth_icmp_dest_localhost_v4.2' into 'release/v4.2'

Browse Source 
ci/esp_eth: Make the ICMP destination localhost (v4.2)

See merge request espressif/esp-idf!18753
This commit is contained in:
David Čermák
2022-07-13 16:06:53 +08:00
parent 7a1df44319 c9bc1943fa
commit bef5f431d9
1 changed files with 9 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
components/esp_eth/test/test_emac.c
View File

@@ -35,6 +35,13 @@ static const char *TAG = "esp32_eth_test";
#define ETH_PING_DURATION_MS (5000)
#define ETH_PING_END_TIMEOUT_MS (ETH_PING_DURATION_MS * 2)
/** To make the test stable in our CI environment we ping only internal localhost endpoint,
* so the packet doesn't really leave the chip boundary, let alone via ethernet interface.
* The ICMP test doesn't logically belong to esp-eth component, but is historically kept here.
* Note the test was moved to lwip component in the further release branches
*/
#define TEST_ICMP_DESTINATION_DOMAIN_NAME "127.0.0.1"
// compute md5 of download file
static struct MD5Context md5_context;
static uint8_t digest[16];
@@ -342,8 +349,8 @@ TEST_CASE("esp32 ethernet icmp test", "[ethernet][test_env=UT_T2_Ethernet]")
struct addrinfo *res = NULL;
memset(&hint, 0, sizeof(hint));
memset(&target_addr, 0, sizeof(target_addr));
/* convert URL to IP */
TEST_ASSERT(getaddrinfo("www.espressif.com", NULL, &hint, &res) == 0);
/* convert ICMP destination string to lwip's representation of an IP address */
TEST_ASSERT(getaddrinfo(TEST_ICMP_DESTINATION_DOMAIN_NAME, NULL, &hint, &res) == 0);
struct in_addr addr4 = ((struct sockaddr_in *)(res->ai_addr))->sin_addr;
inet_addr_to_ip4addr(ip_2_ip4(&target_addr), &addr4);
freeaddrinfo(res);