TongxiJi / shadowsocks-java

shadowsocks server base on netty4 , tcp & udp full support,
MIT License
275 stars 77 forks source link

Add unit tests for cn.wowspeeder.ss.SocksCommonUtils #19

Closed ThomasPerkins1123 closed 5 years ago

ThomasPerkins1123 commented 5 years ago

Hi,

I've analysed your code base and noticed that cn.wowspeeder.ss.SocksCommonUtils in the shadowsocks-netty module is not fully tested.

I've written some tests that cover this class with the help of Diffblue Cover.

Hopefully, these tests should help you detect any regressions caused by future code changes. If you would find it useful to have additional tests written for this repository, I would be more than happy to look at other particular classes that you consider important.