[GitHub] drill pull request #904: DRILL-5717: change some date time test cases with s...

[vvysotskyi]

Github user vvysotskyi commented on a diff in the pull request:

    https://github.com/apache/drill/pull/904#discussion_r148000393
  
    --- Diff: 
exec/java-exec/src/test/java/org/apache/drill/exec/fn/impl/testing/TestDateConversions.java
 ---
    @@ -104,6 +110,13 @@ public void testPostgresDate() throws Exception {
     
       @Test
       public void testJodaTime() throws Exception {
    +    new MockUp<DateTimeUtils>() {
    --- End diff --
    
    Since the same block of code is used in many places, we should move it to 
the separate method (and add Javadoc). 


---