forked from espressif/esp-idf
fix(test_apps): do not check line endings in dut.expect
The default newlib read/write syscalls automatically convert newline endings from LF to CRLF on stdout. Therefore, the test should not specifically check for LF in the expected output, as the standard conversion for stdout is CRLF. While it is possible to change the expected line ending in the test from LF to CRLF, it seems more dependable to just avoid verifying line endings entirely. Signed-off-by: Frantisek Hrbata <frantisek.hrbata@espressif.com>
This commit is contained in:
@ -27,7 +27,7 @@ def test_app_mmu_page_size_32k_and_bootloader_mmu_page_size_64k(dut: Dut, app_do
|
||||
dut.serial.bootloader_flash(path_to_mmu_page_size_64k_build)
|
||||
dut.expect('MMU page size mismatch')
|
||||
dut.expect('App is running')
|
||||
dut.expect('Partition test done\n')
|
||||
dut.expect('Partition test done')
|
||||
|
||||
|
||||
@pytest.mark.esp32c6
|
||||
@ -49,4 +49,4 @@ def test_app_mmu_page_size_64k_and_bootloader_mmu_page_size_32k(dut: Dut, app_do
|
||||
dut.serial.bootloader_flash(path_to_mmu_page_size_32k_build)
|
||||
dut.expect('MMU page size mismatch')
|
||||
dut.expect('App is running')
|
||||
dut.expect('Partition test done\n')
|
||||
dut.expect('Partition test done')
|
||||
|
@ -34,7 +34,7 @@ def test_multicore_app_and_unicore_bootloader(dut: Dut, app_downloader, config)
|
||||
if 'psram' in config:
|
||||
dut.expect(re.compile(r'Adding pool of \d+K of PSRAM memory to heap allocator'))
|
||||
dut.expect('App is running')
|
||||
dut.expect('NVS test done\n')
|
||||
dut.expect('NVS test done')
|
||||
|
||||
|
||||
@pytest.mark.esp32
|
||||
@ -62,4 +62,4 @@ def test_unicore_app_and_multicore_bootloader(dut: Dut, app_downloader, config)
|
||||
if 'psram' in config:
|
||||
dut.expect(re.compile(r'Adding pool of \d+K of PSRAM memory to heap allocator'))
|
||||
dut.expect('App is running')
|
||||
dut.expect('NVS test done\n')
|
||||
dut.expect('NVS test done')
|
||||
|
Reference in New Issue
Block a user