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

CI: Fix coverage analysis for qemu and native_sim #72707

Closed
wants to merge 1 commit into from

Conversation

ceolin
Copy link
Member

@ceolin ceolin commented May 13, 2024

With changes done in a1cdf27 we were not getting coverage data from qemu and native_sim.

It fixes the issue putting the full simulator name in the simulator list.

With changes done in a1cdf27
we were not getting coverage data from qemu and native_sim.

It fixes the issue putting the full simulator name in the simulator
list.

Signed-off-by: Flavio Ceolin <flavio.ceolin@intel.com>
@gbarkadiusz
Copy link
Collaborator

gbarkadiusz commented May 14, 2024

This list of simulators is not the list of simulators where the coverage data are collected on.
It's only for parsing/counting all testcases per three categories sim_only, hw_only, mixed according to the following lines.

if any(platform in testsuite['platform'] for platform in self.simulators):
    if test_suite['status'] == "":
        test_suite['status'] = 'sim_only'
    if test_suite['status'] == 'hw_only':
        test_suite['status'] = 'mixed'
else:
    if test_suite['status'] == "":
        test_suite['status'] = 'hw_only'
    if test_suite['status'] == 'sim_only':
        test_suite['status'] = 'mixed'

then

if i_testsuit['status'] == "hw_only":
    hw_only_count += 1
elif i_testsuit['status'] == "sim_only":
    sim_only_count += 1
else:
    mixed_count += 1

@gbarkadiusz gbarkadiusz added the DNM This PR should not be merged (Do Not Merge) label May 14, 2024
@ceolin ceolin closed this May 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Continuous Integration DNM This PR should not be merged (Do Not Merge)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants