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

Update unit test cases to support cmd handler lookup table #2504

Closed
jphickey opened this issue Jan 25, 2024 · 0 comments · Fixed by #2505
Closed

Update unit test cases to support cmd handler lookup table #2504

jphickey opened this issue Jan 25, 2024 · 0 comments · Fixed by #2505
Assignees

Comments

@jphickey
Copy link
Contributor

Is your feature request related to a problem? Please describe.
The CFE test cases currently invoke the command handler by setting up the MsgID + CC and relying on the FSW "switch" statement to call the handler.

An alternative implementation is to use a (generic) command handler lookup table, but the unit tests don't easily adapt to this model currently.

Describe the solution you'd like
Add the logic to allow setting up the stubs for lookup table dispatching.

Additional context
EDS uses generic lookup tables for message dispatching, and thus the stubs need to be configured accordingly. This can be accomplished by changing how the UT dispatch setup is configured.

Requester Info
Joseph Hickey, Vantage Systems, Inc.

@jphickey jphickey self-assigned this Jan 25, 2024
jphickey added a commit to jphickey/cFE that referenced this issue Jan 25, 2024
Update UT dispatch objects to allow invoking handlers based on
a table lookup rather than configuring the MsgID + CC in the stubs.
jphickey added a commit to jphickey/cFE that referenced this issue Jan 25, 2024
Update UT dispatch objects to allow invoking handlers based on
a table lookup rather than configuring the MsgID + CC in the stubs.
jphickey added a commit to jphickey/cFE that referenced this issue Jan 25, 2024
Update UT dispatch objects to allow invoking handlers based on
a table lookup rather than configuring the MsgID + CC in the stubs.
jphickey added a commit to jphickey/cFE that referenced this issue Jan 25, 2024
Update UT dispatch objects to allow invoking handlers based on
a table lookup rather than configuring the MsgID + CC in the stubs.
jphickey added a commit to jphickey/cFE that referenced this issue Jan 29, 2024
Update UT dispatch objects to allow invoking handlers based on
a table lookup rather than configuring the MsgID + CC in the stubs.
jphickey added a commit to jphickey/cFE that referenced this issue Jan 29, 2024
Update UT dispatch objects to allow invoking handlers based on
a table lookup rather than configuring the MsgID + CC in the stubs.
jphickey added a commit to jphickey/cFE that referenced this issue Jan 29, 2024
Update UT dispatch objects to allow invoking handlers based on
a table lookup rather than configuring the MsgID + CC in the stubs.
dzbaker added a commit that referenced this issue Jan 31, 2024
Fix #2502, #2504, CFE updates needed for generated header compatibility (EDS)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant