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

Fix #445, Coverage Test typos + bugs #446

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

thnkslprpt
Copy link
Contributor

Checklist

Describe the contribution

  • Fixes Coverage Test typos + bugs #445
    • typos fixed
    • fix correct array index in 2 locations in the coverage tests
    • remove duplicate assert in coveragetest-cfe-psp-memrange.c
    • remove empty statements

Testing performed
GitHub CI actions all passing successfully and local testing confirms coverage unaffected.

Expected behavior changes
Test correct and behaving as expected.

System(s) tested on
Debian 12 using the current main branch of cFS bundle.

Contributor Info
Avi Weiss   @thnkslprpt

@dzbaker dzbaker added CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) CCB:Approved Indicates Approval by CCB and removed CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) labels Apr 8, 2025
@thnkslprpt thnkslprpt force-pushed the fix-445-coverage-test-bugs-and-typos branch from 0811425 to b6147fd Compare April 11, 2025 09:39
@thnkslprpt thnkslprpt force-pushed the fix-445-coverage-test-bugs-and-typos branch from b6147fd to f11be15 Compare April 11, 2025 09:41
@thnkslprpt
Copy link
Contributor Author

Cleared the conflicts and updated to main @dzbaker
Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CCB:Approved Indicates Approval by CCB
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Coverage Test typos + bugs
2 participants