Skip to content

Conversation

@igrr
Copy link
Member

@igrr igrr commented Oct 10, 2024

This PR adds pytest scripts for spi_nand_flash component and extends the test matrix to run these tests on hardware.

CI scripts are updated to allow the test app inside the component to be in test_app/ subdirectory, previously only test_apps/ were considered.

@igrr igrr self-assigned this Oct 10, 2024
@igrr igrr changed the title ci(spi_nand_flash): run example and test on hardware ci(spi_nand_flash): run example and test on hardware (IEC-180) Oct 10, 2024
@github-actions
Copy link

github-actions bot commented Oct 10, 2024

Test Results

15 files  15 suites   2m 31s ⏱️
 3 tests  3 ✅ 0 💤 0 ❌
15 runs  15 ✅ 0 💤 0 ❌

Results for commit c8fb264.

♻️ This comment has been updated with latest results.

@igrr igrr force-pushed the ci/spi_nand_flash branch from 99e3fa4 to 3cee2fc Compare October 11, 2024 11:18
@igrr igrr force-pushed the ci/spi_nand_flash branch from 3cee2fc to c8fb264 Compare October 11, 2024 11:48
@igrr igrr requested a review from RathiSonika October 11, 2024 11:49
@igrr
Copy link
Member Author

igrr commented Oct 11, 2024

Looks like the runner is working now, tests passed e.g. in https://github.com/espressif/idf-extra-components/actions/runs/11291400912/job/31405895686.

@igrr igrr marked this pull request as ready for review October 11, 2024 11:50
Copy link
Collaborator

@RathiSonika RathiSonika left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thank you!

@igrr igrr merged commit 333e1ef into master Oct 11, 2024
49 checks passed
@igrr igrr deleted the ci/spi_nand_flash branch October 11, 2024 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants