drill-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From vvysotskyi <...@git.apache.org>
Subject [GitHub] drill pull request #904: DRILL-5717: change some date time test cases with s...
Date Wed, 08 Nov 2017 09:44:47 GMT
Github user vvysotskyi commented on a diff in the pull request:

    https://github.com/apache/drill/pull/904#discussion_r149616688
  
    --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/ExecTest.java ---
    @@ -120,4 +124,27 @@ protected LogicalExpression parseExpr(String expr) throws RecognitionException
{
         return ret.e;
       }
     
    +  /**
    +   * mock current locale to US
    +   */
    +  public static void mockUSLocale() {
    +    new MockUp<DateTimeUtils>() {
    +      @Mock
    +      public DateFormatSymbols getDateFormatSymbols(Locale locale) {
    +        return new DateFormatSymbols(Locale.US);
    +      }
    +    };
    +  }
    +
    +  /**
    +   * mock current timezone to UTC
    --- End diff --
    
    Please start the sentence with a capital letter and add a description about the method
which is mocked.


---

Mime
View raw message