New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix timezone issue on testing. #1770
Conversation
@@ -8,7 +8,9 @@ | |||
<property name="fileExtensions" value="java, properties, xml"/> | |||
|
|||
<!-- maximum file length 2000 lines --> | |||
<module name="FileLength"/> | |||
<module name="FileLength"> | |||
<property name="max" value="2500"/> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is only overridden on a per-file basis.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rollbacked that.
I misunderstood it.
@songsungkyun Which tests failed? |
This is a part of failed.
|
Other than maybe squashing some of the commits, I think this change is fine. |
@@ -13,6 +14,9 @@ | |||
|
|||
@Before | |||
public void setUp() { | |||
System.setProperty("user.timezone", "UTC"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what failures do you have with this test?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@timowest
weird.
I remembered it was failed with testing on querydls-sql module.
but it's gone now.
Could you squash the commits? Then it would be fine to merge. |
moved to #1775 |
you can also force push the result of the rebase. |
Yes, as long as you move the tip of the branch the pull request points to ( |
some tests with datetime are failed at South Korea.
so add some code with timezone and it's resolved.