-
Notifications
You must be signed in to change notification settings - Fork 27
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
ci: add ci for qemu tests #253
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Worth to mention that, after #255 got merged, one ARM-based runner will be enabled as well.
please rebase after that one got merged, and download qemu only on X64
one.
087b594
to
8eaa06c
Compare
rebased to run the tests |
I fixed the related dependencies, and now it works. However, there are problems with some tests, specifically with the expectation the menu |
75656a8
to
678b5cb
Compare
91216c4
to
f7d0582
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let's pick this PR back. I've left a new comment and rebased the branch.
# here we can't use confirm_write because the sub cases won't print anything | ||
# can't rely on time.sleep either cause it will break some time-related child cases | ||
self.write(str(sub_case['index'])) | ||
self.confirm_write(str(sub_case['index']), expect_str=f"Running stage {sub_case['index']}...") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let's revert this line and test with normal single dut test cases.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let's pick this PR back. I've left a new comment and rebased the branch.
ac8fdf6
to
fe89f5d
Compare
This pull request adds support for QEMU tests to the project's CI pipeline: installs necessary dependencies, and runs tests using QEMU.