espressif/esp-modbus
1
0
Fork 1
mirror of https://github.com/espressif/esp-modbus.git synced 2026-08-03 20:24:09 +02:00
Code Issues Packages Projects Releases Wiki Activity

feat(refactor_ci_test): Disabling some of the build/test on the test_app to improve CI runtime

Browse Source
This commit is contained in:
Octavio Godoy
2026-01-26 17:25:34 +01:00
parent 77782773bb
commit 4d64cbd2be
3 changed files with 25 additions and 17 deletions
Download Patch File Download Diff File Expand all files Collapse all files
.gitlab-ci.yml
+1 -1
View File
@@ -25,7 +25,7 @@ variables:
TEST_MARKER: TEST_MARKER:
- "tcp" - "tcp"
- "serial" - "serial"
- "generic" # - "generic" // Disabling during the CI refactor on tests
# - "tcp_p4" # - "tcp_p4"
# - "serial_p4" # - "serial_p4"
test_apps/.build-test-rules.yml
+12 -5
View File
@@ -3,7 +3,8 @@ physical_tests:
- if: IDF_TARGET != "esp32" - if: IDF_TARGET != "esp32"
reason: only manual test is performed for other targets reason: only manual test is performed for other targets
disable: disable:
- if: CONFIG_NAME == "wifi" and SOC_WIFI_SUPPORTED != 1 and IDF_TARGET != "esp32" - if: CONFIG_NAME != "serial"
# - if: CONFIG_NAME == "wifi" and SOC_WIFI_SUPPORTED != 1 and IDF_TARGET != "esp32"
tcp_instances_tests/mb_tcp_master_instances: tcp_instances_tests/mb_tcp_master_instances:
disable_test: disable_test:
@@ -21,10 +22,16 @@ tcp_instances_tests/mb_tcp_slave_instances:
unit_tests: unit_tests:
disable_test: disable_test:
- if: IDF_TARGET != "esp32" - if: "1 == 1" # Disabling for the Ci build and test Refactor
reason: only manual test is performed for other targets # - if: IDF_TARGET != "esp32"
# reason: only manual test is performed for other targets
disable:
- if: "1 == 1" # Disabling for the Ci test Refactor
adapter_tests: adapter_tests:
disable_test: disable_test:
- if: IDF_TARGET != "esp32" - if: "1 == 1" # Disabling for the Ci build and test Refactor
reason: only manual test is performed for other targets # - if: IDF_TARGET != "esp32"
# reason: only manual test is performed for other targets
disable:
- if: "1 == 1" # Disabling for the Ci build and test Refactor
test_apps/physical_tests/pytest_mb_comm_multi_dev.py
+12 -11
View File
@@ -16,15 +16,16 @@ def test_modbus_comm_multi_dev_serial(case_tester) -> None: # type: ignore
print(f"Test case: {case.name}") print(f"Test case: {case.name}")
case_tester.run_multi_dev_case(case=case, reset=True) case_tester.run_multi_dev_case(case=case, reset=True)
## Disabling for the Ci build and test Refactor
@pytest.mark.parametrize("count, config", [(2, "ethernet")], indirect=True) # @pytest.mark.parametrize("count, config", [(2, "ethernet")], indirect=True)
@pytest.mark.parametrize("target", ["esp32"], indirect=True) # @pytest.mark.parametrize("target", ["esp32"], indirect=True)
@pytest.mark.multi_dut_modbus_tcp # @pytest.mark.multi_dut_modbus_tcp
def test_modbus_comm_multi_dev_tcp(case_tester) -> None: # type: ignore # def test_modbus_comm_multi_dev_tcp(case_tester) -> None: # type: ignore
for case in case_tester.test_menu: # for case in case_tester.test_menu:
if ( # if (
case.attributes.get("test_env", "multi_dut_modbus_tcp") # case.attributes.get("test_env", "multi_dut_modbus_tcp")
== "multi_dut_modbus_tcp" # == "multi_dut_modbus_tcp"
): # ):
print(f"Test case: {case.name}") # print(f"Test case: {case.name}")
case_tester.run_multi_dev_case(case=case, reset=True) # case_tester.run_multi_dev_case(case=case, reset=True)