nasa / FM

The Core Flight System (cFS) File Manager (FM) application.
Apache License 2.0
42 stars 24 forks source link

Fix #17, update code coverage for fm_cmd_utils.c #33

Closed havencarlson closed 2 years ago

havencarlson commented 2 years ago

Checklist (Please check before submitting)

Describe the contribution

Testing performed Steps taken to test the contribution:

  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 no impact to behavior

System(s) tested on

Additional context Add any other context about the contribution here.

Third party code If included, identify any third party code and provide text file of license

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

astrogeco commented 2 years ago

@havencarlson this needs a run through clang format to fix white space. You can also fix it manually by looking at the output of the format check workflow

skliper commented 2 years ago

@astrogeco - should be ready to merge... we'll need to update the workflow allowed missed branches/lines once these are all in.

astrogeco commented 2 years ago

@chillfig Still need some format fixes

astrogeco commented 2 years ago

CCB:2022-06-15 APPROVED