Skip to content
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.

Sign up for GitHub

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

Windows: Fix a bug that the wrong log file is reopened with log rotate setting when flushing or graceful reloading #4054

Merged
merged 9 commits into from
Feb 16, 2023
Merged

Windows: Fix a bug that the wrong log file is reopened with log rotate setting when flushing or graceful reloading #4054

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
7b285a0
Windows: Apply the modified log path for rotation to reopen
daipom Feb 13, 2023
560c1bb
Test: Make sure to close LogDevice
daipom Feb 14, 2023
abbcc64
Test: Reset tmp files at setup
daipom Feb 14, 2023
1305157
Test: Make sure LoggerInitializerTest doesn't change the global logger
daipom Feb 14, 2023
83041ee
Test: Add tests for rotate and reopening
daipom Feb 15, 2023
6107d00
Test: Refactor LoggerInitializerTest
daipom Feb 15, 2023
0d9581c
Test: Remove needless code
daipom Feb 15, 2023
ade4857
Test: Make sure not to leave tmp files
daipom Feb 15, 2023
60c94ad
Test: Refactor logic of recovering global logger
daipom Feb 16, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 2 additions & 5 deletions test/test_logger_initializer.rb
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,11 +5,8 @@
class LoggerInitializerTest < ::Test::Unit::TestCase
TMP_DIR = File.expand_path(File.dirname(__FILE__) + "/tmp/logger_initializer#{ENV['TEST_ENV_NUMBER']}")

teardown do
begin
FileUtils.rm_rf(TMP_DIR)
rescue => _
end
setup do
FileUtils.rm_rf(TMP_DIR) rescue nil
ashie marked this conversation as resolved.
Show resolved Hide resolved
end

test 'when path is given' do
Expand Down