You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In an effort to improve the test coverage for serverimpl, I started to add some tests with TLS enabled. I noticed we already have some useful code for generating and reading certs in internal/examples/..., and it seems like a good idea to use for tests. I was thinking of moving them to /internal or maybe internal/testhelper and use in both serverimpl tests and examples.
@tigrannajaryan do you see any issues with this? If not, I can send a PR for this along with the new serverimpl TLS tests.
The text was updated successfully, but these errors were encountered:
In an effort to improve the test coverage for serverimpl, I started to add some tests with TLS enabled. I noticed we already have some useful code for generating and reading certs in
internal/examples/...
, and it seems like a good idea to use for tests. I was thinking of moving them to/internal
or maybeinternal/testhelper
and use in both serverimpl tests and examples.@tigrannajaryan do you see any issues with this? If not, I can send a PR for this along with the new serverimpl TLS tests.
The text was updated successfully, but these errors were encountered: