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

Desmakers 3803 new hil setup #309

Merged

Conversation

OlafFilies
Copy link
Member

Description
This PR includes:

  • HIL setup for unity tests including compile and flash on multiple devices with interconnection of devices
  • generic GH action yaml
  • including yaml file for controlling available unity tests
  • reporting of console output for single and multiple devices
  • collecting of builds and reports for downloading as artifact
  • cleanup and extension of the main make file

Context
This HIL setup runs on Infineon self hosted runner with certain devices and interconnected devices for testing XMC-for-Arduino software

Commits will be squashed.

Copy link
Member

@jaenrig-ifx jaenrig-ifx left a comment

Choose a reason for hiding this comment

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

Good progress 👍
In next iteration it would be a good idea to encapsulate some of those lengthy bash into some .bash files and functions?
Eventually they could be also run locally, and the workflow file gets shorten and only handling runner matrix and stages control.

Copy link
Collaborator

@LinjingZhang LinjingZhang left a comment

Choose a reason for hiding this comment

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

Great 😍 I don't have much to comment on. Just like juan said, maybe have a local bash file would be nice

@LinjingZhang
Copy link
Collaborator

Maybe a open topic for next iteration
Now the connected tests on the XMC4700 fails. It still pass in CICD. How do we check the report.json and returns the test results?

@OlafFilies OlafFilies merged commit d362b9d into feature/Unity_codecheck_setup Nov 6, 2024
11 checks passed
@OlafFilies OlafFilies deleted the DESMAKERS-3803-new-hil-setup branch November 6, 2024 09:06
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.

3 participants