-
-
Notifications
You must be signed in to change notification settings - Fork 75
Make ns module public #545
-
|
Is your feature request related to a problem? Please describe. Describe the solution you'd like I need this so i can write unit tests. |
Beta Was this translation helpful? Give feedback.
All reactions
Replies: 2 comments 1 reply
-
|
Out of curiosity, why do you need |
Beta Was this translation helpful? Give feedback.
All reactions
-
|
So any function I write that has an argument like I understand I can use axum to test at a higher level but that doesn't count for the coverage report and unfortunately that's what I'm graded on. My only alternative that i can think of at least is transmute but that raises flags too lol |
Beta Was this translation helpful? Give feedback.
All reactions
-
|
I understand that this lib needs better testability. However I won't make the If you really want to make your handler unit testable you can:
Feel free to ask more details on this discussion or on the tokio discord in the socketioxide channel. |
Beta Was this translation helpful? Give feedback.
All reactions
-
1 -
1