[SKIP CI] github actions: run unit tests#4543
Merged
Merged
Conversation
marc-hb
changed the title
marc-hb
force-pushed
the
gh-unit-tests
branch
4 times, most recently
from
81641ba to
bc41f51
Compare
Collaborator
Author
|
The "Misplaced SPDX-License-Identifier" is a checkpatch bug: it gets confused when the diff is at a certain distance from the top of the file. |
Fixes 'no previous prototype for trace_log_filtered' -Werr and redefinition of ‘trace_flush_dma_to_mbox’ error. Signed-off-by: Marc Herbert <[email protected]>
This has been finding several pre-processing bugs already. Short of randconfig some variety is the best we have. Signed-off-by: Marc Herbert <[email protected]>
Link to the CMake section in sof-docs which documents a lot of what this script does and is probably out of date now. It would be less out of date if this link had been there and more people had been aware that this documentation even existed. Signed-off-by: Marc Herbert <[email protected]>
Before this it was required to find and run the executables one by one Signed-off-by: Marc Herbert <[email protected]>
Adds --error-exitcode=1 to valgrind options (otherwise what's the point of using valgrind?) Skip alloc test that does not pass on HOST (passes with xt-run) Add help message. Runnable from anywhere. Use shell functions thesofproject/sof-test#740 Fix all quoting issues and other shellcheck warnings. Add comments. Signed-off-by: Marc Herbert <[email protected]>
It is failing and skipped only when compiled on the host, runs fine with xt-run. This is temporarily needed to add host-based unit tests to CI now and catch any regression in any other test now. Signed-off-by: Marc Herbert <[email protected]>
Because we can. Signed-off-by: Marc Herbert <[email protected]>
Collaborator
Author
marc-hb
requested review from
cujomalainey,
dbaluta,
lbetlej,
lgirdwood,
mmaka1,
plbossart and
zrombel
as code owners
marc-hb
requested review from
aiChaoSONG,
akloniex,
fredoh9,
keyonjie,
ktrzcinx,
lyakh and
paulstelian97
Collaborator
Author
lgirdwood
approved these changes
Jul 26, 2021
marc-hb
added a commit
to marc-hb/sof-docs
that referenced
this pull request
Jul 26, 2021
... and now even CI does it since thesofproject/sof#4543 Signed-off-by: Marc Herbert <[email protected]>
marc-hb
added a commit
to marc-hb/sof-docs
that referenced
this pull request
Jul 26, 2021
... and now even CI does it since thesofproject/sof#4543 Also mention run-cmocks.sh Signed-off-by: Marc Herbert <[email protected]>
lgirdwood
pushed a commit
to thesofproject/sof-docs
that referenced
this pull request
Jul 27, 2021
... and now even CI does it since thesofproject/sof#4543 Also mention run-cmocks.sh Signed-off-by: Marc Herbert <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Skip the "alloc" test for now as it fails when built on the host (it passes and it is run with xt-run)