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 a CI action to build and execute the cFE coverage tests #1483

Closed
skliper opened this issue May 4, 2021 · 4 comments · Fixed by #1590 or #1619
Closed

Add a CI action to build and execute the cFE coverage tests #1483

skliper opened this issue May 4, 2021 · 4 comments · Fixed by #1590 or #1619

Comments

@skliper
Copy link
Contributor

skliper commented May 4, 2021

Is your feature request related to a problem? Please describe.
CI doesn't build and execute the cFE coverage tests

Describe the solution you'd like
Add

Describe alternatives you've considered
None

Additional context
None

Requester Info
Jacob Hageman - NASA/GSFC

@skliper
Copy link
Contributor Author

skliper commented May 21, 2021

@pepepr08 - want to take this one? Trigger on push or pull_request and include the duplicate check. Should be able to just make -C pathtomodule and then make -C pathtomodule test for each module.

@pepepr08
Copy link
Contributor

Yes I can work on this.

@astrogeco
Copy link
Contributor

@skliper aren't we building these at the bundle level?

@skliper
Copy link
Contributor Author

skliper commented May 25, 2021

@skliper aren't we building these at the bundle level?

@astrogeco Yes, but that means they don't get tested for cFE PR's at the time of submittal, so errors sneak in and slow the merge process. Also will make coverage verification easier at the level of cFE since it'll report just the cFE coverage vs the entire bundle.

pepepr08 added a commit to pepepr08/cFE that referenced this issue May 28, 2021
pepepr08 added a commit to pepepr08/cFE that referenced this issue May 28, 2021
pepepr08 added a commit to pepepr08/cFE that referenced this issue May 28, 2021
pepepr08 added a commit to pepepr08/cFE that referenced this issue May 28, 2021
pepepr08 added a commit to pepepr08/cFE that referenced this issue May 28, 2021
pepepr08 added a commit to pepepr08/cFE that referenced this issue May 28, 2021
pepepr08 added a commit to pepepr08/cFE that referenced this issue May 28, 2021
pepepr08 added a commit to pepepr08/cFE that referenced this issue May 28, 2021
pepepr08 added a commit to pepepr08/cFE that referenced this issue May 28, 2021
pepepr08 added a commit to pepepr08/cFE that referenced this issue Jun 2, 2021
astrogeco added a commit that referenced this issue Jun 10, 2021
Fix #1483, Add CI workflow to run cFE coverage tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment