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

Add feature to CFE assert to write output to log file #1625

Closed
jphickey opened this issue Jun 16, 2021 · 0 comments · Fixed by #1633 or #1632
Closed

Add feature to CFE assert to write output to log file #1625

jphickey opened this issue Jun 16, 2021 · 0 comments · Fixed by #1633 or #1632
Assignees
@jphickey
Labels
enhancement
Milestone
Caelum

Comments

@jphickey
Copy link
Contributor

jphickey commented Jun 16, 2021 •
edited
Loading

Is your feature request related to a problem? Please describe.
The CFE functional tests executed via the cfe_assert helper app currently report test cases as events. Problem is:

  • Events are (by default) limited to 122 chars, and assert strings tend to be long-ish, so these are likely to get truncated
  • Makes it difficult to write test analyzing tools - either need to somehow subscribe to those event IDs (non-trivial w/current script capabilities), or pipe the entire terminal log to a file (OS dependent). And in both cases, still subject to text truncation.

Describe the solution you'd like
The cfe_assert app should have a feature to internally "tee" the results to a log file. This would alleviate the need (and OS dependency) on trying to "tee" the output externally, and the output would contain ONLY test logs i.e. not mixed with other syslog messages.

Additional context
See #1608, this would make CI tests more reliable

Requester Info
Joseph Hickey, Vantage Systems, Inc.
@skliper skliper added this to the 7.0.0 milestone Jun 16, 2021
jphickey added a commit to jphickey/cFE that referenced this issue Jun 22, 2021
@jphickey
Fix nasa#1625, add test log file
7de2314 
Add cfe_assert capability to "tee" all test log output to a file
in addition to the regular output (console/event).

This aids in scripting and automation, by creating a file containing
only test results, not intermixed with other info, and not subject
to the length limitations of events.
jphickey added a commit to jphickey/cFE that referenced this issue Jun 22, 2021
@jphickey
Fix nasa#1625, add test log file
0cd4cbe 
Add cfe_assert capability to "tee" all test log output to a file
in addition to the regular output (console/event).

This aids in scripting and automation, by creating a file containing
only test results, not intermixed with other info, and not subject
to the length limitations of events.
@jphickey jphickey mentioned this issue Jun 22, 2021
Merged
astrogeco added a commit that referenced this issue Jun 23, 2021
@astrogeco
Merge pull request #1633 from jphickey/fix-1625-cfe-assert-logfile
2527452 
Fix #1625, add test log file
@astrogeco astrogeco mentioned this issue Jun 23, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jphickey jphickey

Labels
enhancement
Projects
None yet
Milestone
Caelum
Development

Successfully merging a pull request may close this issue.

Fix #1625, add test log file jphickey/cFE
cFE Integration candidate: 2021-06-22 nasa/cFE
2 participants
@jphickey @skliper