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.

Sign up for GitHub

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

Just for test: Improve unit test printout #114

Closed
wants to merge 1 commit into from
Closed

Just for test: Improve unit test printout #114

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 6 additions & 1 deletion .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,10 @@ jobs:
run: ./build.sh -r

- name: Run Unit Tests
run: build/bin/app_utests > test-results.md
shell: bash
run: |
set -o pipefail
build/bin/app_utests | tee test-results.md

- name: Generate coverage report
run: cd build && gcovr -r ..
Expand All @@ -90,7 +93,9 @@ jobs:

- name: Add test results to job summary
run: |
echo -e "Unit test results: \n\`\`\` " >> $GITHUB_STEP_SUMMARY
cat test-results.md >> $GITHUB_STEP_SUMMARY
echo -e "\n\`\`\`" >> $GITHUB_STEP_SUMMARY
cat code-coverage-results.md >> $GITHUB_STEP_SUMMARY

- name: Run Linters
Expand Down
14 changes: 7 additions & 7 deletions NOTICE-3RD-PARTY-CONTENT.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
| Dependency | Version | License |
|:-----------|:-------:|--------:|
|bottle|0.12.25|MIT|
|certifi|2024.7.4|Mozilla Public License 2.0|
|certifi|2024.8.30|Mozilla Public License 2.0|
|cfgv|3.4.0|MIT|
|charset-normalizer|3.3.2|MIT|
|colorama|0.4.6|BSD|
Expand All @@ -16,9 +16,9 @@
|filelock|3.15.4|The Unlicense (Unlicense)|
|gcovr|5.2|BSD|
|identify|2.6.0|MIT|
|idna|3.7|BSD|
|idna|3.8|BSD|
|jinja2|3.1.4|BSD|
|lxml|5.2.2|New BSD|
|lxml|5.3.0|New BSD|
|MarkupSafe|2.1.5|New BSD|
|node-semver|0.6.1|MIT|
|nodeenv|1.9.1|BSD|
Expand All @@ -27,13 +27,13 @@
|pluginbase|1.0.1|BSD|
|pre-commit|3.5.0|MIT|
|pygments|2.18.0|Simplified BSD|
|PyJWT|2.8.0|MIT|
|PyJWT|2.9.0|MIT|
|python-dateutil|2.9.0.post0|Apache 2.0<br/>BSD|
|PyYAML|6.0.1|MIT|
|PyYAML|6.0.2|MIT|
|requests|2.32.3|Apache 2.0|
|six|1.16.0|MIT|
|tqdm|4.66.4|MIT<br/>Mozilla Public License 2.0 (MPL 2.0)|
|urllib3|1.26.19|MIT|
|tqdm|4.66.5|MIT<br/>Mozilla Public License 2.0 (MPL 2.0)|
|urllib3|1.26.20|MIT|
|virtualenv|20.26.3|MIT|
## Workflows
| Dependency | Version | License |
Expand Down
2 changes: 1 addition & 1 deletion app/tests/utests/SampleApp_test.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,5 +20,5 @@

TEST(SampleAppTest, skeleton_test) {
// TODO: Fill me
EXPECT_TRUE(true);
EXPECT_TRUE(false);
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mikehaller - ith the change in this PR you can the output below



Run set -o pipefail
Running main() from /__w/vehicle-app-cpp-template/vehicle-app-cpp-template/build/_deps/googletest-src/googletest/src/gtest_main.cc
[==========] Running 1 test from 1 test suite.
[----------] Global test environment set-up.
[----------] 1 test from SampleAppTest
[ RUN      ] SampleAppTest.skeleton_test
/__w/vehicle-app-cpp-template/vehicle-app-cpp-template/app/tests/utests/SampleApp_test.cpp:23: Failure
Value of: false
  Actual: false
Expected: true
[  FAILED  ] SampleAppTest.skeleton_test (0 ms)
[----------] 1 test from SampleAppTest (0 ms total)

[----------] Global test environment tear-down
[==========] 1 test from 1 test suite ran. (0 ms total)
[  PASSED  ] 0 tests.
[  FAILED  ] 1 test, listed below:
[  FAILED  ] SampleAppTest.skeleton_test

 1 FAILED TEST

}
Loading