Skip to content

Conversation

@leshow
Copy link
Collaborator

@leshow leshow commented Dec 30, 2025

  • adds minimal ddns tests
  • fixes an issue with the multiping test
  • remove sudo -E from test runner which was creating an error output in cargo test

@coveralls
Copy link
Collaborator

Coverage Status

coverage: 71.654% (+2.8%) from 68.824%
when pulling d3a1549 on ddns_tests
into 76d4782 on master.

@leshow
Copy link
Collaborator Author

leshow commented Dec 30, 2025

apologies to anyone subscribed to notifications, was struggling with the permissions for test runners. You should be able to run cargo test locally now and in github actions without any errors, at least.

@leshow leshow merged commit ca7c927 into master Dec 30, 2025
10 of 12 checks passed
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