Skip to content

Commit

Permalink
Removing Dir.mktmpdir since windows 2.2/2.3 doesnt like it
Browse files Browse the repository at this point in the history
  • Loading branch information
sthirugn committed May 1, 2020
1 parent 8b38312 commit 58287b2
Showing 1 changed file with 34 additions and 31 deletions.
65 changes: 34 additions & 31 deletions test/test_launcher.rb
Expand Up @@ -80,51 +80,54 @@ def test_pid_file
end

def test_state_permission_0640
Dir.mktmpdir do |dir|
tmp_path = File.join(dir, "puma-state")
Tempfile.new(tmp_path)
tmp_permission = 0640

conf = Puma::Configuration.new do |c|
c.state_path tmp_path
c.state_permission tmp_permission
end

launcher(conf).write_state
tmp_file = Tempfile.new("puma-test")
tmp_path = tmp_file.path
tmp_file.close!
tmp_permission = 0640

assert File.stat(tmp_path).mode.to_s(8)[-4..-1], tmp_permission
conf = Puma::Configuration.new do |c|
c.state_path tmp_path
c.state_permission tmp_permission
end

launcher(conf).write_state

assert File.stat(tmp_path).mode.to_s(8)[-4..-1], tmp_permission

File.unlink tmp_path
end

def test_state_permission_nil
Dir.mktmpdir do |dir|
tmp_path = File.join(dir, "puma-state")
Tempfile.new(tmp_path)
tmp_file = Tempfile.new("puma-test")
tmp_path = tmp_file.path
tmp_file.close!

conf = Puma::Configuration.new do |c|
c.state_path tmp_path
c.state_permission nil
end
conf = Puma::Configuration.new do |c|
c.state_path tmp_path
c.state_permission nil
end

launcher(conf).write_state

launcher(conf).write_state
assert File.exist?(tmp_path)

assert File.exist?(tmp_path)
end
File.unlink tmp_path
end

def test_no_state_permission
Dir.mktmpdir do |dir|
tmp_path = File.join(dir, "puma-state")
Tempfile.new(tmp_path)
tmp_file = Tempfile.new("puma-test")
tmp_path = tmp_file.path
tmp_file.close!

conf = Puma::Configuration.new do |c|
c.state_path tmp_path
end
conf = Puma::Configuration.new do |c|
c.state_path tmp_path
end

launcher(conf).write_state
launcher(conf).write_state

assert File.exist?(tmp_path)
end
assert File.exist?(tmp_path)

File.unlink tmp_path
end

def test_puma_stats
Expand Down

0 comments on commit 58287b2

Please sign in to comment.