[ 
https://issues.apache.org/jira/browse/DIRKRB-251?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14543280#comment-14543280
 ] 

Xu Yaning commented on DIRKRB-251:
----------------------------------

Hi, Lin,
Thx for your review, maybe it's better to rename the two classes. The two 
classes looks similar. However, they override {{allowUdp()}} and {{allowTcp()}} 
separately. I don't think we can simplify it more. 

> Create test for udp & tcp separatedly using NettyKdcNetwork
> -----------------------------------------------------------
>
>                 Key: DIRKRB-251
>                 URL: https://issues.apache.org/jira/browse/DIRKRB-251
>             Project: Directory Kerberos
>          Issue Type: Test
>            Reporter: Xu Yaning
>            Assignee: Xu Yaning
>         Attachments: DIRKRB-251-v1.patch
>
>
> Tcp test and udp test for NettyKdcNetwork are both in 
> {{JsonBackendKdcTest.java}}, which makes it confusing whether tcp or udp 
> works. So it is natural to separate the test for them.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to