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

Part #17, update code coverage #35

Merged
merged 1 commit into from
Jun 22, 2022
Merged

Part #17, update code coverage #35

merged 1 commit into from
Jun 22, 2022

Conversation

chillfig
Copy link
Contributor

@chillfig chillfig commented Jun 9, 2022

Increases code coverage for fm_child_tests.c and fm_cmds_tests.c

Checklist (Please check before submitting)

Describe the contribution
A clear and concise description of what the contribution is.

Testing performed

  1. lcov --capture --rc lcov_branch_coverage=1 --directory build --output-file coverage_test.info
  2. lcov --rc lcov_branch_coverage=1 --add-tracefile coverage_base.info --add-tracefile coverage_test.info --output-file coverage_total.info
  3. genhtml coverage_total.info --branch-coverage --output-directory lcov

Expected behavior changes
None

System(s) tested on

  • OS: Ubuntu 18.04

Additional context
None

Contributor Info - All information REQUIRED for consideration of pull request
Full name and company/organization/center of all contributors ("Personal" if individual work)

  • Note CLA's apply to software contributions.

unit-test/fm_child_tests.c Outdated Show resolved Hide resolved
unit-test/fm_child_tests.c Outdated Show resolved Hide resolved
unit-test/fm_child_tests.c Outdated Show resolved Hide resolved
unit-test/fm_child_tests.c Outdated Show resolved Hide resolved
@chillfig chillfig force-pushed the lcov branch 3 times, most recently from 2d225fe to 2b30663 Compare June 13, 2022 18:15
@chillfig chillfig changed the title Fix #17, update code coverage Part #17, update code coverage Jun 13, 2022
@astrogeco
Copy link
Contributor

@chillfig looks like the code-format check is failing, probably need some whitespace changes

unit-test/fm_child_tests.c Outdated Show resolved Hide resolved
unit-test/fm_child_tests.c Outdated Show resolved Hide resolved
unit-test/fm_child_tests.c Outdated Show resolved Hide resolved
unit-test/fm_child_tests.c Outdated Show resolved Hide resolved
@chillfig chillfig force-pushed the lcov branch 4 times, most recently from 6714f5f to b3db530 Compare June 16, 2022 17:53
unit-test/fm_child_tests.c Outdated Show resolved Hide resolved
unit-test/fm_child_tests.c Outdated Show resolved Hide resolved
…M_Cmds, FM_ChildDirListFileLoop

Removed UtAsserts for FM_ChildSizeTimeMode
Removed UtAssert for CFE_ES_ExitChildTask
Updated whitespace to clang format
Added Test_FM_ChildDirListFileLoop_BytesWrittenNotEqualWriteLengthInLoop
Added real numbers in place of "any" macros, Added UtAssert_STUB_Count()
Cleans up Test_FM_ChildDeleteDirCmd_StrCmpThisDirectoryZero
Adds UtAssert_VOIDCALL for FM_ChildSleepStat
Clean up Test_FM_ChildDirListFileLoop coverage test
@astrogeco
Copy link
Contributor

astrogeco commented Jun 22, 2022

CCB:2022.06.22 APPROVED

@astrogeco astrogeco merged commit 4336824 into nasa:main Jun 22, 2022
@skliper skliper added this to the Draco milestone Jul 14, 2022
@chillfig chillfig deleted the lcov branch December 16, 2022 20:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants