-
Notifications
You must be signed in to change notification settings - Fork 16
To_proc_time cmd added #455
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
base: main
Are you sure you want to change the base?
Changes from all commits
989a4f5
d92e6cf
45aec08
e5b955e
284495e
2240804
8386485
e4a68d7
69eaaf2
61edc63
f7f6fa6
55aedcf
dc98708
e754191
1909b0b
2f673c8
a97296e
0e0589d
779e77a
3d5dd0f
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -436,3 +436,20 @@ def test_10_double_set_test(fprime_test_api: IntegrationTestAPI, start_gds): | |
| fprime_test_api.send_command(f"{rtcManager}.ALARM_SET", [alarm_time_data_str]) | ||
| # Assert that we receive an AlarmNotSet event within 10 seconds | ||
| fprime_test_api.await_event(f"{rtcManager}.AlarmNotSet", timeout=10) | ||
|
|
||
|
|
||
| @pytest.mark.uart_only(reason="Test functionality of to_proc_time") | ||
| def test_11_proc_toggle(fprime_test_api: IntegrationTestAPI, start_gds): | ||
| """Test for events emitted by proc time toggle""" | ||
|
|
||
| # Set time to Curiosity landing on Mars (7 minutes of terror! https://youtu.be/Ki_Af_o9Q9s) | ||
|
ineskhou marked this conversation as resolved.
|
||
| curiosity_landing = datetime(2012, 8, 6, 5, 17, 57, tzinfo=timezone.utc) | ||
| set_time(fprime_test_api, curiosity_landing) | ||
|
|
||
| # Assert that we receive a TimeBase event within 10 seconds | ||
| fprime_test_api.await_event(f"{rtcManager}.TimeBase", timeout=10) | ||
|
|
||
| fprime_test_api.send_command(f"{rtcManager}.TO_PROC_TIME") | ||
|
|
||
| # Assert that we receive a TimeBase event within 10 seconds | ||
| fprime_test_api.await_event(f"{rtcManager}.TimeBase", timeout=10) | ||
|
coderabbitai[bot] marked this conversation as resolved.
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Could be good to have this test assert what the timebase is that is being set (PROC vs RTC) and/or actually check the physcial timebase. Checking the event doesn't let us know if the system of the sat is on PROC time. At minimum before merge, woudl want manual evidence on this happening on a board That being said, test updates will not block merge. |
||
Uh oh!
There was an error while loading. Please reload this page.