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

NF-613: Fix card.binary HIL tests. #160

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

haydenroche5
Copy link
Collaborator

  • Hardcode Notestation to los-angeles-notestation-1. This one's known to be set up for running these tests. I have work in progress to make it so this doesn't need to be hardcoded (i.e. the Notestation client can discover which stations can run these tests), but it's not finished, yet.
  • Make sure to kill the Notestation client in the GitHub Actions workflow when done with the Notestation.
  • Move the file .pio/build/debug/firmware.elf gdb command earlier in the custom upload command for PlatformIO. This just gets rid of some warnings I was seeing in the log. No functional difference.
  • Change test_port to /tmp/ns_mcu_usb from /tmp/mcu_usb. The former is the new default path for the MCU USB device with the latest Notestation client.

@haydenroche5 haydenroche5 requested a review from zfields November 18, 2024 21:33
@haydenroche5 haydenroche5 self-assigned this Nov 18, 2024
- Hardcode Notestation to los-angeles-notestation-1. This one's known to be set
up for running these tests. I have work in progress to make it so this doesn't
need to be hardcoded (i.e. the Notestation client can discover which stations
can run these tests), but it's not finished, yet.
- Make sure to kill the Notestation client in the GitHub Actions workflow when
done with the Notestation.
- Move the `file .pio/build/debug/firmware.elf` gdb command earlier in the
custom upload command for PlatformIO. This just gets rid of some warnings I was
seeing in the log. No functional difference.
- Change test_port to /tmp/ns_mcu_usb from /tmp/mcu_usb. The former is the new
default path for the MCU USB device with the latest Notestation client.
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.

1 participant