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

Mock FRAM code #286

Merged
merged 12 commits into from
May 12, 2024
Merged

Mock FRAM code #286

merged 12 commits into from
May 12, 2024

Conversation

jeromehue
Copy link
Contributor

@jeromehue jeromehue commented May 1, 2024

Description

FRAM code is mocked according to #200. Apart from empty default do functions, we also provide do functions that simulate the FRAM in RAM. The empty and the RAM do functions are tested in UnitTests/Fram.test.cpp.

Fixes #207

@jeromehue jeromehue force-pushed the 207-mock-fram-code branch from 8dd1f4c to ec3dd36 Compare May 1, 2024 20:15
@codecov-commenter
Copy link

codecov-commenter commented May 1, 2024

Codecov Report

Attention: Patch coverage is 87.91209% with 11 lines in your changes are missing coverage. Please review.

Project coverage is 87.11%. Comparing base (614c299) to head (a2f9416).

Files Patch % Lines
Sts1CobcSw/Periphery/FramMock.cpp 87.75% 6 Missing ⚠️
Sts1CobcSw/Utility/Time.cpp 0.00% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #286      +/-   ##
==========================================
+ Coverage   86.54%   87.11%   +0.56%     
==========================================
  Files          21       25       +4     
  Lines         513      613     +100     
  Branches        7        7              
==========================================
+ Hits          444      534      +90     
- Misses         69       79      +10     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jeromehue jeromehue force-pushed the 207-mock-fram-code branch 3 times, most recently from c97f86b to 472cdd5 Compare May 1, 2024 20:50
@jeromehue jeromehue changed the title Mock Fram Code WIP: Mock Fram Code May 1, 2024
@jeromehue jeromehue force-pushed the 207-mock-fram-code branch 2 times, most recently from 7c28670 to fa52476 Compare May 3, 2024 14:19
@jeromehue jeromehue marked this pull request as ready for review May 3, 2024 14:42
@jeromehue jeromehue changed the title WIP: Mock Fram Code Mock Fram code May 5, 2024
@PatrickKa PatrickKa force-pushed the 207-mock-fram-code branch from 75cca8c to 97dd611 Compare May 10, 2024 09:38
@PatrickKa PatrickKa changed the title Mock Fram code Mock FRAM code May 11, 2024
@PatrickKa PatrickKa force-pushed the 207-mock-fram-code branch from 3f0c01e to a2f9416 Compare May 12, 2024 15:13
@PatrickKa PatrickKa merged commit 7372280 into master May 12, 2024
6 checks passed
@PatrickKa PatrickKa deleted the 207-mock-fram-code branch May 12, 2024 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Mock FRAM code
3 participants