Skip to content

Conversation

@paulbartell
Copy link
Contributor

@paulbartell paulbartell commented Mar 16, 2021

Run CMock unit tests on each pull request with a github action.

Description

This PR adds a github workflow that runs the kernel CMock unit tests located in the FreeRTOS/FreeRTOS repository on each push to FreeRTOS / FreeRTOS-Kernel.

This workflow also uploads test coverage data to codecov.io so that it can be easily reviewed. Optionally, codecov.io can add a comment to the PR outlining the changes in coverage before and after the changes in the PR.

Test Steps

N/A

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@paulbartell paulbartell force-pushed the pbartell/github-action-unit-tests branch from fac18fc to 6872f24 Compare March 16, 2021 20:33
@paulbartell paulbartell marked this pull request as ready for review March 16, 2021 20:47
@paulbartell paulbartell requested a review from a team as a code owner March 16, 2021 20:47
Include coverage information reported by codecov.io
@paulbartell paulbartell force-pushed the pbartell/github-action-unit-tests branch from 6872f24 to 186d213 Compare March 16, 2021 20:49
@paulbartell paulbartell requested a review from lundinc2 March 16, 2021 21:10
@paulbartell
Copy link
Contributor Author

Here's a link to an example coverage report on codecov.io:
https://codecov.io/github/paulbartell/FreeRTOS-Kernel/commit/186d213c24095f0bb9045dcd7bdb9692afa3aca7

lundinc2
lundinc2 previously approved these changes Mar 16, 2021
nateglims
nateglims previously approved these changes Mar 16, 2021
@paulbartell paulbartell dismissed stale reviews from nateglims and lundinc2 via 300292d March 16, 2021 22:04
@paulbartell paulbartell merged commit 95433d0 into FreeRTOS:main Mar 17, 2021
@paulbartell paulbartell deleted the pbartell/github-action-unit-tests branch March 17, 2021 01:31
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.

4 participants