summary refs log tree commit
diff options
context:
space:
mode:
authorRafael França <rafael@franca.dev>2019-09-09 15:58:28 -0400
committerGitHub <noreply@github.com>2019-09-09 15:58:28 -0400
commitf7d0fdebf57024e6d8c674380ce7777927b3b1fe (patch)
tree314b59ab5e2acd58b8a17dab995edea7e164fcaa
parente44c6873ce21de9a6d4aac70b9a684e07aa11d4d (diff)
parenta0ae289a8961837906a7091ffa0cfdbf3d7d7303 (diff)
downloadrack-f7d0fdebf57024e6d8c674380ce7777927b3b1fe.tar.gz
Merge pull request #1389 from felixbuenemann/skip-process-owner-spec-in-docker
Skip failing server spec that can't work in Docker
-rw-r--r--test/spec_server.rb2
1 files changed, 2 insertions, 0 deletions
diff --git a/test/spec_server.rb b/test/spec_server.rb
index 7a60a61e..3ec88f16 100644
--- a/test/spec_server.rb
+++ b/test/spec_server.rb
@@ -161,6 +161,8 @@ describe Rack::Server do
   end
 
   it "check pid file presence and not owned process" do
+    owns_pid_1 = (Process.kill(0, 1) rescue nil) == 1
+    skip "cannot test if pid 1 owner matches current process (eg. docker/lxc)" if owns_pid_1
     pidfile = Tempfile.open('pidfile') { |f| f.write(1); break f }.path
     server = Rack::Server.new(pid: pidfile)
     server.send(:pidfile_process_status).must_equal :not_owned