Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

fix unit test failed #770

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

fix unit test failed #770

wants to merge 3 commits into from

Conversation

xuwenping123
Copy link

Hi, I encountered some problems when building the project.

My current build environment is as follows:

java version:11
ZoneId:Asia/Shanghai
Locale:zh_CN

Execute build command:

mvn clean test

Error message:

// case 1: TimeBasedRollingWithArchiveRemoval_Test.dailySizeBasedRolloverWithSizeCap
[ERROR] Failures: 
[ERROR]   TimeBasedRollingWithArchiveRemoval_Test.dailySizeBasedRolloverWithSizeCap:306->checkFileCount:516 expected: <9> but was: <11>

// case 2: .ConverterTest#testDateConverter_AU_locale
org.opentest4j.AssertionFailedError: 
Expected :21/Oct/2022:21:30:20 +1100
Actual   :21/10月/2022:21:30:20 +1100

ceki and others added 3 commits January 19, 2024 22:01
@ceki @xuwenping123
update license in some old files not part of the ditributed binary
c8cd9aa 
Signed-off-by: Ceki Gulcu <ceki@qos.ch>
Signed-off-by: xuwenping <wenpingxu123@163.com>
@xuwenping123
fix unit test failed
1c749f7 
Signed-off-by: xuwenping <wenpingxu123@163.com>
@xuwenping123
fix unit test failed
af751f6 
Signed-off-by: xuwenping <wenpingxu123@163.com>
@xuwenping123
Copy link
Author

@ceki Sorry, I didn’t sign off the commit I just made. It’s fine now.

What else do I need to do next? Thanks

# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@xuwenping123 @ceki