Skip to content

Add reusable InT test scripts #3923

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

Open
wants to merge 21 commits into
base: devel
Choose a base branch
from

Conversation

chuynh4duarte
Copy link
Collaborator

Related Issue(s)
Has Unit Tests (y/n)
Documentation Included (y/n)

Change Description

A description of the changes contained in the PR.

Rationale

A rationale for this change. e.g. fixes bug, or most projects need XYZ feature.

Testing/Review Recommendations

Fill in testing procedures, specific items to focus on for review, or other info to help the team verify these changes are flight-quality.

Future Work

Note any additional work that will be done relating to this issue.

Ref/config.json Outdated
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

could we rename this file to something more descriptive such as int_config.json or something?

Ref/config.json Outdated
"Svc.FileManager" : "Ref.fileManager",
"Svc.FileUplink" : "Ref.fileUplink",
"Ref.PingReceiver" : "Ref.pingRcvr",
"Svc.ActiveLogger" : "CDHCore.events",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Svc.ActiveLogger is now Svc.EventManager

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also I think CDHCore should be CdhCore ?

Tests command send_and_assert command events level and state
"""
# Expected True for (WARNING_HI, WARNING_LO, COMMAND, ACTIVITY_HI, ACTIVITY_LO & False for DIAGNOSTIC)
fprime_test_api.send_and_assert_command(fprime_test_api.get_mnemonic('Svc.ActiveLogger') + '.' + 'DUMP_FILTER_STATE', max_delay=1)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ActiveLogger is now EventManager (also in other places in this file)

@thomas-bc thomas-bc changed the title fixed spelling fileDownlink,health,systemResources and add config.json file Add reusable InT test scripts Jul 23, 2025
@LeStarch LeStarch added the Deferred To Next Release This PR is marked as deferred until after the upcoming release. label Aug 4, 2025
@LeStarch LeStarch removed the Deferred To Next Release This PR is marked as deferred until after the upcoming release. label Aug 7, 2025
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.

3 participants