Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add initial HIL testing #596

Merged
merged 18 commits into from
Feb 23, 2024
Merged

Add initial HIL testing #596

merged 18 commits into from
Feb 23, 2024

Conversation

SergioGasquez
Copy link
Member

@SergioGasquez SergioGasquez commented Feb 22, 2024

Add HIL CI that:

  • In Ubuntu runners, build esp-hal hello_world example for every target, upload artifacts to actions
  • In Ubuntu runner, build espflash, upload artifacts to actions
  • In self-hosted runners, run the HIL tests:
    • Run board-info, checks that "esp32" is printed
    • Run flash, checks that "Flashing has completed!" is printed
    • Run monitor for 5secs, checks that "Hello world!" is printed
    • Run erase-flash, checks that "Flash has been erased!" is printed
    • Run read-flash, checks that "Flash content successfully read and written to" is printed
    • Use hexdump to verify that read flash is empty (its been erased, so it should be)
    • Run save-image, checks that "Image successfully saved!" is printed
    • Run write-bin, checks that "Binary successfully written to flash!" is printed
    • Run monitor for 5secs, checks that "Hello world!" is printed
  • Add info! logs for when the command ends successfully
  • Add a non-interactive flag to monitoring so it can be used in CI
  • Add an app binary for every target under resources folder. Apps where generated with esp-template (rev: 905de42)default settings

HIL CI errors:

  • ESP32 is failing when trying to flash the board, not sure what is happening there.
  • ESP32-C2 seems to be offline.
  • ESP32-C3 fails as its a rust board and it cant read the uart.
  • ESP32-C2 cant monitor. Its a 26MHz C2 and it only reads noise

We would also need to decide when this action runs

Thanks to @achxkloel for the investigation on the monitoring issue

@SergioGasquez SergioGasquez linked an issue Feb 22, 2024 that may be closed by this pull request
3 tasks
@SergioGasquez
Copy link
Member Author

HIL CI is now green! Only CI on macos is failing due to: #597. The only thing left would be to decide when the HIL CI should run.

@SergioGasquez SergioGasquez marked this pull request as ready for review February 23, 2024 14:34
Copy link
Member

@MabezDev MabezDev left a comment

Choose a reason for hiding this comment

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

Really nice job! Very excited to see this merged!

Thank you @SergioGasquez and @achxkloel for taking care of this!

@MabezDev MabezDev merged commit f21829a into main Feb 23, 2024
27 of 28 checks passed
@SergioGasquez SergioGasquez deleted the feat/hil branch February 23, 2024 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Set up automated testing on hardware
2 participants