Skip to content

Commit 978b441

Browse files
lucasssvazpre-commit-ci-lite[bot]
lucasssvaz
and
pre-commit-ci-lite[bot]
authored
test(wokwi): Add I2C Master test and enable GPIO and PSRAM tests (espressif#10848)
* test(wokwi): Enable PSRAM test * fix(tests): Add missing diagram for ESP32-P4 * test(wokwi): Enable GPIO test * test(wokwi): Add I2C master test * fix(tests): Add missing requirement and improve logging * ci(pre-commit): Apply automatic fixes --------- Co-authored-by: pre-commit-ci-lite[bot] <117423508+pre-commit-ci-lite[bot]@users.noreply.github.com>
1 parent eec2af3 commit 978b441

15 files changed

+488
-8
lines changed

‎.github/scripts/tests_run.sh‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -48,10 +48,10 @@ function run_test {
4848
return 0
4949
fi
5050

51-
local right_target
52-
right_target=$(grep -E "^CONFIG_IDF_TARGET=\"$target\"$" "$sdkconfig_path")
53-
if [ -z "$right_target" ]; then
54-
printf "\033[91mError: Sketch %s compiled for different target\n\033[0m\n" "$sketchname"
51+
local compiled_target
52+
compiled_target=$(grep -E "CONFIG_IDF_TARGET=" "$sdkconfig_path" | cut -d'"' -f2)
53+
if [ "$compiled_target" != "$target" ]; then
54+
printf "\033[91mError: Sketch %s compiled for %s, expected %s\033[0m\n" "$sketchname" "$compiled_target" "$target"
5555
printf "\n\n\n"
5656
return 1
5757
fi

‎tests/requirements.txt‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,3 +5,4 @@ pytest-embedded-serial-esp==1.12.0
55
pytest-embedded-arduino==1.12.0
66
pytest-embedded-wokwi==1.12.0
77
pytest-embedded-qemu==1.12.0
8+
esptool==4.8.1

‎tests/validation/gpio/ci.json‎

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
{
22
"platforms": {
33
"hardware": false,
4-
"qemu": false,
5-
"wokwi": false
4+
"qemu": false
65
}
76
}

‎tests/validation/gpio/diagram.esp32p4.json‎

Lines changed: 28 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,28 @@
1+
{
2+
"version": 1,
3+
"author": "lucasssvaz",
4+
"editor": "wokwi",
5+
"parts": [
6+
{
7+
"type": "board-esp32-p4-function-ev",
8+
"id": "esp32",
9+
"top": -66.32,
10+
"left": -277.63,
11+
"attrs": {}
12+
},
13+
{
14+
"type": "wokwi-pushbutton",
15+
"id": "btn1",
16+
"top": -128.2,
17+
"left": -19.2,
18+
"attrs": { "color": "green", "bounce": "1" }
19+
}
20+
],
21+
"connections": [
22+
[ "esp32:38", "$serialMonitor:TX", "", [] ],
23+
[ "esp32:37", "$serialMonitor:RX", "", [] ],
24+
[ "btn1:2.r", "esp32:GND.3", "black", [ "h19.4", "v29" ] ],
25+
[ "esp32:0", "btn1:1.l", "blue", [ "h-48", "v-67.2" ] ]
26+
],
27+
"dependencies": {}
28+
}

‎tests/validation/i2c_master/ci.json‎

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
{
2+
"platforms": {
3+
"hardware": false,
4+
"qemu": false
5+
},
6+
"requires": [
7+
"CONFIG_SOC_I2C_SUPPORTED=y"
8+
]
9+
}

‎tests/validation/i2c_master/diagram.esp32.json‎

Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,24 @@
1+
{
2+
"version": 1,
3+
"author": "lucasssvaz",
4+
"editor": "wokwi",
5+
"parts": [
6+
{
7+
"type": "board-esp32-devkit-c-v4",
8+
"id": "esp32",
9+
"top": -57.6,
10+
"left": -177.56,
11+
"attrs": {}
12+
},
13+
{ "type": "wokwi-ds1307", "id": "rtc1", "top": -43.8, "left": -19.1, "attrs": {} }
14+
],
15+
"connections": [
16+
[ "esp32:RX", "$serialMonitor:TX", "", [] ],
17+
[ "esp32:TX", "$serialMonitor:RX", "", [] ],
18+
[ "esp32:22", "rtc1:SCL", "green", [ "h38.4", "v-9.6" ] ],
19+
[ "esp32:21", "rtc1:SDA", "blue", [ "h48", "v-28.8", "h19.2" ] ],
20+
[ "esp32:GND.2", "rtc1:GND", "black", [ "v0" ] ],
21+
[ "rtc1:5V", "esp32:5V", "red", [ "h-28.8", "v-67.6", "h-172.8", "v230.4" ] ]
22+
],
23+
"dependencies": {}
24+
}

‎tests/validation/i2c_master/diagram.esp32c3.json‎

Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,25 @@
1+
{
2+
"version": 1,
3+
"author": "lucasssvaz",
4+
"editor": "wokwi",
5+
"parts": [
6+
{
7+
"type": "board-esp32-c3-devkitm-1",
8+
"id": "esp32",
9+
"top": -57.6,
10+
"left": -177.56,
11+
"attrs": {}
12+
},
13+
{ "type": "wokwi-ds1307", "id": "rtc1", "top": -43.8, "left": -19.1, "attrs": {} }
14+
],
15+
"connections": [
16+
[ "esp32:RX", "$serialMonitor:TX", "", [] ],
17+
[ "esp32:TX", "$serialMonitor:RX", "", [] ],
18+
[ "rtc1:5V", "esp32:5V", "red", [ "h-28.8", "v-67.6", "h-172.8", "v230.4" ] ],
19+
[ "esp32:5V.1", "rtc1:5V", "red", [ "h-18.22", "v-144.3", "h153.6", "v48" ] ],
20+
[ "esp32:GND.10", "rtc1:GND", "black", [ "h56.16", "v9.3" ] ],
21+
[ "esp32:8", "rtc1:SDA", "green", [ "h17.76", "v-19.5" ] ],
22+
[ "esp32:9", "rtc1:SCL", "blue", [ "h0" ] ]
23+
],
24+
"dependencies": {}
25+
}

‎tests/validation/i2c_master/diagram.esp32c6.json‎

Lines changed: 26 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,26 @@
1+
{
2+
"version": 1,
3+
"author": "lucasssvaz",
4+
"editor": "wokwi",
5+
"parts": [
6+
{
7+
"type": "board-esp32-c6-devkitc-1",
8+
"id": "esp32",
9+
"top": -57.6,
10+
"left": -177.56,
11+
"attrs": {}
12+
},
13+
{ "type": "wokwi-ds1307", "id": "rtc1", "top": -43.8, "left": -19.1, "attrs": {} }
14+
],
15+
"connections": [
16+
[ "esp32:RX", "$serialMonitor:TX", "", [] ],
17+
[ "esp32:TX", "$serialMonitor:RX", "", [] ],
18+
[ "rtc1:5V", "esp32:5V", "red", [ "h-28.8", "v-67.6", "h-172.8", "v230.4" ] ],
19+
[ "esp32:5V.1", "rtc1:5V", "red", [ "h-18.22", "v-144.3", "h153.6", "v48" ] ],
20+
[ "esp32:GND.10", "rtc1:GND", "black", [ "h56.16", "v9.3" ] ],
21+
[ "esp32:23", "rtc1:SDA", "green", [ "h17.38", "v-23.51" ] ],
22+
[ "esp32:GND.4", "rtc1:GND", "black", [ "h55.78", "v-4.31" ] ],
23+
[ "esp32:22", "rtc1:SCL", "blue", [ "h26.98", "v-23.51" ] ]
24+
],
25+
"dependencies": {}
26+
}

‎tests/validation/i2c_master/diagram.esp32h2.json‎

Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,24 @@
1+
{
2+
"version": 1,
3+
"author": "lucasssvaz",
4+
"editor": "wokwi",
5+
"parts": [
6+
{
7+
"type": "board-esp32-h2-devkitm-1",
8+
"id": "esp32",
9+
"top": -57.6,
10+
"left": -177.56,
11+
"attrs": {}
12+
},
13+
{ "type": "wokwi-ds1307", "id": "rtc1", "top": -43.8, "left": -19.1, "attrs": {} }
14+
],
15+
"connections": [
16+
[ "esp32:RX", "$serialMonitor:TX", "", [] ],
17+
[ "esp32:TX", "$serialMonitor:RX", "", [] ],
18+
[ "esp32:GND.6", "rtc1:GND", "black", [ "h0" ] ],
19+
[ "esp32:5V", "rtc1:5V", "red", [ "h-29.14", "v-160.97", "h172.8", "v48" ] ],
20+
[ "esp32:12", "rtc1:SDA", "green", [ "h36.58", "v-36.17" ] ],
21+
[ "esp32:22", "rtc1:SCL", "blue", [ "v-7.37", "h46.18", "v-38.4" ] ]
22+
],
23+
"dependencies": {}
24+
}

‎tests/validation/i2c_master/diagram.esp32p4.json‎

Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,24 @@
1+
{
2+
"version": 1,
3+
"author": "lucasssvaz",
4+
"editor": "wokwi",
5+
"parts": [
6+
{
7+
"type": "board-esp32-p4-function-ev",
8+
"id": "esp32",
9+
"top": -57.6,
10+
"left": -177.56,
11+
"attrs": {}
12+
},
13+
{ "type": "wokwi-ds1307", "id": "rtc1", "top": -197.4, "left": 57.7, "attrs": {} }
14+
],
15+
"connections": [
16+
[ "esp32:38", "$serialMonitor:TX", "", [] ],
17+
[ "esp32:37", "$serialMonitor:RX", "", [] ],
18+
[ "esp32:5V.1", "rtc1:5V", "red", [ "v0" ] ],
19+
[ "esp32:GND.1", "rtc1:GND", "black", [ "v-133.52", "h5.53" ] ],
20+
[ "esp32:7", "rtc1:SDA", "green", [ "v0" ] ],
21+
[ "esp32:8", "rtc1:SCL", "blue", [ "h15.13", "v-114.12" ] ]
22+
],
23+
"dependencies": {}
24+
}

0 commit comments

Comments
 (0)