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

Jump to bottom

Update time_UT.c:Test_Print to use the UtAssert_MIR macro #1725

Closed
skliper opened this issue Jul 28, 2021 · 0 comments · Fixed by #1732 or #1759
Closed

Update time_UT.c:Test_Print to use the UtAssert_MIR macro #1725

skliper opened this issue Jul 28, 2021 · 0 comments · Fixed by #1732 or #1759
Assignees
@pepepr08
Labels
unit-test
Milestone
Caelum

Comments

@skliper
Copy link
Contributor

skliper commented Jul 28, 2021

Is your feature request related to a problem? Please describe.
UtAssertEx use isn't great (leads to nonstandard output):

cFE/modules/time/ut-coverage/time_UT.c

Lines 870 to 872 in e298961

UtAssertEx(false, UTASSERT_CASETYPE_MIR, __FILE__, __LINE__,
"Confirm adding seconds = %u, subseconds = %u to configured EPOCH results in time %s", time.Seconds,
time.Subseconds, timeBuf);

Describe the solution you'd like
Once nasa/osal#1118 is implemented, update the MIR tests in Test_Print.

Describe alternatives you've considered
None

Additional context
None

Requester Info
Jacob Hageman - NASA/GSFC
@skliper skliper added this to the 7.0.0 milestone Jul 28, 2021
@skliper skliper mentioned this issue Jul 28, 2021
Merged
pepepr08 added a commit to pepepr08/cFE that referenced this issue Jul 29, 2021
@pepepr08
Fix nasa#1725 Update time_UT.c to use UtAssert_MIR
0db499f
pepepr08 added a commit to pepepr08/cFE that referenced this issue Jul 29, 2021
@pepepr08
Fix nasa#1725 Update time_UT.c to use UtAssert_MIR
35e0269
@pepepr08 pepepr08 mentioned this issue Jul 29, 2021
Merged
pepepr08 added a commit to pepepr08/cFE that referenced this issue Jul 29, 2021
@pepepr08
Fix nasa#1725 Update time_UT.c to use UtAssert_MIR
857ebf3
pepepr08 added a commit to pepepr08/cFE that referenced this issue Jul 30, 2021
@pepepr08
Fix nasa#1725 Update UTs to use UtAssert_MIR
8b77b79
pepepr08 added a commit to pepepr08/cFE that referenced this issue Jul 30, 2021
@pepepr08
Fix nasa#1725 Update UTs to use UtAssert_MIR
a6fc229
pepepr08 added a commit to pepepr08/cFE that referenced this issue Jul 30, 2021
@pepepr08
Fix nasa#1725 Update UTs to use UtAssert_MIR
a5a9556
astrogeco added a commit that referenced this issue Aug 4, 2021
@astrogeco
Merge pull request #1732 from pepepr08/fix1725-use-mir-macro
e0f0f7a 
Fix #1725, Update time_UT.c to use UtAssert_MIR
@astrogeco astrogeco mentioned this issue Aug 4, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pepepr08 pepepr08

Labels
unit-test
Projects
None yet
Milestone
Caelum
2 participants
@pepepr08 @skliper