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

Add support to write DIDs from command line #118

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

alexdetrano
Copy link
Contributor

See #117. In summary, the testing harness is failing and I need help debugging!

@kasperkarlsson
Copy link
Contributor

Hi there, I just wanted to say that I have had a first look and cleaned up the code a bit (branches where uninitialized variables were called etc), but finding the root cause of the error remains 😅 I will have another look!

@kasperkarlsson
Copy link
Contributor

kasperkarlsson commented Jul 28, 2024

Update: My current gut feeling says your code is not at fault. It might be caused by a bug in the testing harness, where the MockEcuIso14229 implementation is broken which affects its ability to keep state within the UDS test suite.

Digging into that class and its parents has caused me to make a few double takes so far, I will have to keep looking 😅 Might be threading related.

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.

2 participants