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

Make the shared table used in the cFE functional tests configurable #2352

Closed
dmknutsen opened this issue May 25, 2023 · 0 comments · Fixed by #2354
Closed

Make the shared table used in the cFE functional tests configurable #2352

dmknutsen opened this issue May 25, 2023 · 0 comments · Fixed by #2354

Comments

@dmknutsen
Copy link
Contributor

Is your feature request related to a problem? Please describe.
Projects typically will remove sample_app from their build. This creates an issue because the cFE functional tests are dependent on it such that tests will fail after removal. Need a mechanism to easily swap out the sample_app table used in the functional tests such that projects can still run the tests as part of their CI/CD once sample_app is removed.

Describe the solution you'd like
Make the name of the table used in the cFE functional tests configurable such that projects can easily swap out the sample_app table with a table in their project specific build.

Requester Info
Dan Knutsen
NASA Goddard

@dmknutsen dmknutsen self-assigned this May 25, 2023
@dmknutsen dmknutsen added this to the Equuleus milestone May 25, 2023
dmknutsen added a commit to dmknutsen/cFE that referenced this issue May 25, 2023
dmknutsen added a commit to dmknutsen/cFE that referenced this issue May 25, 2023
dmknutsen added a commit to dmknutsen/cFE that referenced this issue May 25, 2023
dzbaker added a commit that referenced this issue May 26, 2023
Fix #2352, make shared tbl configurable
dzbaker added a commit that referenced this issue Jun 1, 2023
Fix #2352, make shared tbl configurable
@dzbaker dzbaker closed this as completed in aa9da9a Jun 1, 2023
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