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.

Sign up for GitHub

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

Jump to bottom

Add tests for incorrect i2c bus/ address in multi key setup #235

Closed
shawaj opened this issue Mar 28, 2023 · 1 comment · Fixed by #238
Closed

Add tests for incorrect i2c bus/ address in multi key setup #235

shawaj opened this issue Mar 28, 2023 · 1 comment · Fixed by #238

Comments

@shawaj
Copy link
Member

shawaj commented Mar 28, 2023 •
edited
Loading

Add tests for incorrect i2c bus/ address in multi key setup

Related to #234 and #231
@shawaj shawaj mentioned this issue Apr 2, 2023
Merged
4 tasks
@shawaj
Copy link
Member Author

shawaj commented Apr 2, 2023 •
edited
Loading

Need to add case for i2c bus missing and error handling for the same:

Error: Could not open file `/dev/i2c-1' or `/dev/i2c/1': No such file or directory

shawaj added a commit that referenced this issue Apr 7, 2023
@shawaj
feat: add parsing for onboarding key
f6315aa 
- add onboarding key location to all hardware variants
- add get_onboarding_location function to get correct key
- add additional tests for new functions and previous
- add test for non existent i2c bus

Closes: #235
shawaj added a commit that referenced this issue Apr 7, 2023
@shawaj
feat: add parsing for onboarding key
30b14f5 
- add onboarding key location to all hardware variants
- add get_onboarding_location function to get correct key
- add additional tests for new functions and previous
- add test for non existent i2c bus

Closes: #235
@shawaj shawaj mentioned this issue Apr 7, 2023
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: add parsing for onboarding key NebraLtd/hm-pyhelper
1 participant
@shawaj