passenger-jmazzi 2.2.9

data/test/ruby/spawn_server_spec.rb

@@ -0,0 +1,26 @@
+require 'support/config'
+
+require 'ruby/abstract_server_spec'
+
+shared_examples_for "a spawn server" do
+	it_should_behave_like "an AbstractServer"
+	
+	it "raises an AbstractServer::ServerError if the server was killed" do
+		Process.kill('SIGABRT', @spawner.server_pid)
+		spawning = lambda { spawn_arbitrary_application }
+		spawning.should raise_error(AbstractServer::ServerError)
+	end
+	
+	it "works correctly after a restart, if something went wrong" do
+		Process.kill('SIGABRT', @spawner.server_pid)
+		spawning = lambda { spawn_arbitrary_application }
+		spawning.should raise_error(AbstractServer::ServerError)
+		
+		@spawner.stop
+		@spawner.start
+		app = spawn_arbitrary_application
+		app.pid.should_not == 0
+		app.app_root.should_not be_nil
+		app.close
+	end
+end

data/test/ruby/utils_spec.rb

@@ -0,0 +1,335 @@
+require 'support/config'
+
+require 'tmpdir'
+require 'fileutils'
+require 'stringio'
+require 'phusion_passenger/message_channel'
+require 'phusion_passenger/utils'
+
+include PhusionPassenger
+
+shared_examples_for "a pseudo stderr created by #report_app_init_status" do
+	before :each do
+		@sink = StringIO.new
+		@temp_channel = MessageChannel.new(StringIO.new)
+	end
+	
+	after :each do
+		File.unlink("output.tmp") rescue nil
+	end
+	
+	it "redirects everything written to the pseudo STDERR/$stderr to the sink" do
+		report_app_init_status(@temp_channel, @sink) do
+			STDERR.puts "Something went wrong!"
+			$stderr.puts "Something went wrong again!"
+			raise StandardError, ":-(" if @raise_error
+		end
+		@sink.string.should =~ /Something went wrong!/
+		@sink.string.should =~ /Something went wrong again!/
+	end
+	
+	it "redirects reopen operations on the pseudo stderr to the sink" do
+		@sink.should_receive(:reopen).with("output.tmp", "w")
+		report_app_init_status(@temp_channel, @sink) do
+			STDERR.reopen("output.tmp", "w")
+			raise StandardError, ":-(" if @raise_error
+		end
+	end
+	
+	specify "after the function has finished, every operation on the old pseudo stderr object will still be redirected to the sink" do
+		pseudo_stderr = nil
+		report_app_init_status(@temp_channel, @sink) do
+			pseudo_stderr = STDERR
+			raise StandardError, ":-(" if @raise_error
+		end
+		
+		pseudo_stderr.puts "hello world"
+		@sink.string.should =~ /hello world/
+		
+		@sink.should_receive(:reopen).with("output.tmp", "w")
+		pseudo_stderr.reopen("output.tmp", "w")
+	end
+	
+	specify "after the function has finished, every output operation on the old pseudo stderr object will not be buffered" do
+		pseudo_stderr = nil
+		report_app_init_status(@temp_channel, @sink) do
+			pseudo_stderr = STDERR
+			pseudo_stderr.instance_variable_get(:@buffer).should_not be_nil
+			raise StandardError, ":-(" if @raise_error
+		end
+		pseudo_stderr.instance_variable_get(:@buffer).should be_nil
+	end
+end
+
+describe Utils do
+	include Utils
+	
+	specify "#close_all_io_objects_for_fds closes all IO objects that are associated with the given file descriptors" do
+		filename = "#{Dir.tmpdir}/passenger_test.#{Process.pid}.txt"
+		begin
+			pid = safe_fork('utils_spec') do
+				a, b = IO.pipe
+				close_all_io_objects_for_fds([0, 1, 2])
+				File.open(filename, "w") do |f|
+					f.write("#{a.closed?}, #{b.closed?}")
+				end
+			end
+			Process.waitpid(pid) rescue nil
+			File.read(filename).should == "true, true"
+		ensure
+			File.unlink(filename) rescue nil
+		end
+	end
+	
+	describe "#report_app_init_status" do
+		it "reports normal errors, which #unmarshal_and_raise_errors raises" do
+			a, b = IO.pipe
+			begin
+				pid = safe_fork('utils_spec') do
+					a.close
+					report_app_init_status(MessageChannel.new(b)) do
+						raise RuntimeError, "hello world"
+					end
+				end
+				b.close
+				lambda { unmarshal_and_raise_errors(MessageChannel.new(a)) }.should raise_error(/hello world/)
+			ensure
+				a.close rescue nil
+				b.close rescue nil
+			end
+		end
+		
+		it "reports SystemExit errors, which #unmarshal_and_raise_errors raises" do
+			a, b = IO.pipe
+			begin
+				pid = safe_fork('utils_spec') do
+					a.close
+					report_app_init_status(MessageChannel.new(b)) do
+						exit
+					end
+				end
+				b.close
+				lambda { unmarshal_and_raise_errors(MessageChannel.new(a)) }.should raise_error(/exited during startup/)
+			ensure
+				a.close rescue nil
+				b.close rescue nil
+			end
+		end
+		
+		it "returns whether the block succeeded" do
+			channel = MessageChannel.new(StringIO.new)
+			success = report_app_init_status(channel) do
+				false
+			end
+			success.should be_true
+			
+			success = report_app_init_status(channel) do
+				raise StandardError, "hi"
+			end
+			success.should be_false
+		end
+		
+		it "reports all data written to STDERR and $stderr" do
+			a, b = IO.pipe
+			begin
+				pid = safe_fork('utils_spec') do
+					a.close
+					report_app_init_status(MessageChannel.new(b), nil) do
+						STDERR.puts "Something went wrong!"
+						$stderr.puts "Something went wrong again!"
+						exit
+					end
+				end
+				b.close
+				
+				begin
+					unmarshal_and_raise_errors(MessageChannel.new(a))
+					violated "No exception raised"
+				rescue AppInitError => e
+					e.stderr.should =~ /Something went wrong!/
+					e.stderr.should =~ /Something went wrong again!/
+				end
+			ensure
+				a.close rescue nil
+				b.close rescue nil
+			end
+		end
+		
+		it "reports all data written to STDERR and $stderr even if it was reopened" do
+			a, b = IO.pipe
+			begin
+				pid = safe_fork('utils_spec') do
+					a.close
+					report_app_init_status(MessageChannel.new(b), nil) do
+						STDERR.puts "Something went wrong!"
+						STDERR.reopen("output.tmp", "w")
+						STDERR.puts "Something went wrong again!"
+						STDERR.flush
+						$stderr.puts "Something went wrong yet again!"
+						$stderr.flush
+						exit
+					end
+				end
+				b.close
+				
+				begin
+					unmarshal_and_raise_errors(MessageChannel.new(a))
+					violated "No exception raised"
+				rescue AppInitError => e
+					e.stderr.should =~ /Something went wrong!/
+					e.stderr.should =~ /Something went wrong again!/
+					e.stderr.should =~ /Something went wrong yet again!/
+				end
+				
+				file_contents = File.read("output.tmp")
+				file_contents.should =~ /Something went wrong again!/
+				file_contents.should =~ /Something went wrong yet again!/
+			ensure
+				a.close rescue nil
+				b.close rescue nil
+				File.unlink("output.tmp") rescue nil
+			end
+		end
+		
+		describe "if the block failed" do
+			before :each do
+				@raise_error = true
+			end
+			
+			it_should_behave_like "a pseudo stderr created by #report_app_init_status"
+		end
+		
+		describe "if the block succeeded" do
+			it_should_behave_like "a pseudo stderr created by #report_app_init_status"
+		end
+	end
+	
+	specify "#safe_fork with double_fork == false reseeds the pseudo-random number generator" do
+		a, b = IO.pipe
+		begin
+			pid = safe_fork do
+				b.puts(rand)
+			end
+			Process.waitpid(pid) rescue nil
+			pid = safe_fork do
+				b.puts(rand)
+			end
+			Process.waitpid(pid) rescue nil
+			
+			first_num = a.readline
+			second_num = a.readline
+			first_num.should_not == second_num
+		ensure
+			a.close rescue nil
+			b.close rescue nil
+		end
+	end
+	
+	specify "#safe_fork with double_fork == true reseeds the pseudo-random number generator" do
+		a, b = IO.pipe
+		begin
+			# Seed the pseudo-random number generator here
+			# so that it doesn't happen in the child processes.
+			srand
+			
+			safe_fork(self.class, true) do
+				b.puts(rand)
+			end
+			safe_fork(self.class, true) do
+				b.puts(rand)
+			end
+			
+			first_num = a.readline
+			second_num = a.readline
+			first_num.should_not == second_num
+		ensure
+			a.close rescue nil
+			b.close rescue nil
+		end
+	end
+	
+	describe "#unmarshal_and_raise_errors" do
+		before :each do
+			@a, @b = IO.pipe
+			@report_channel = MessageChannel.new(@a)
+			report_app_init_status(MessageChannel.new(@b)) do
+				raise StandardError, "Something went wrong!"
+			end
+		end
+		
+		after :each do
+			@a.close rescue nil
+			@b.close rescue nil
+		end
+		
+		it "prints the exception information to the 'print_exception' argument using #puts, if 'print_exception' responds to that" do
+			buffer = StringIO.new
+			lambda { unmarshal_and_raise_errors(@report_channel, buffer) }.should raise_error(AppInitError)
+			buffer.string.should =~ /Something went wrong!/
+			buffer.string.should =~ /utils\.rb/
+			buffer.string.should =~ /utils_spec\.rb/
+		end
+		
+		it "appends the exception information to the file pointed to by 'print_exception', if 'print_exception' responds to #to_str" do
+			begin
+				lambda { unmarshal_and_raise_errors(@report_channel, "exception.txt") }.should raise_error(AppInitError)
+				data = File.read('exception.txt')
+				data.should =~ /Something went wrong!/
+				data.should =~ /utils\.rb/
+				data.should =~ /utils_spec\.rb/
+			ensure
+				File.unlink('exception.txt') rescue nil
+			end
+		end
+	end
+	
+	specify "#to_boolean works" do
+		to_boolean(nil).should be_false
+		to_boolean(false).should be_false
+		to_boolean(true).should be_true
+		to_boolean(1).should be_true
+		to_boolean(0).should be_true
+		to_boolean("").should be_true
+		to_boolean("true").should be_true
+		to_boolean("false").should be_false
+		to_boolean("bla bla").should be_true
+	end
+	
+	describe "#passenger_tmpdir" do
+		before :each do
+			@old_passenger_tmpdir = Utils.passenger_tmpdir
+			Utils.passenger_tmpdir = nil
+		end
+		
+		after :each do
+			Utils.passenger_tmpdir = @old_passenger_tmpdir
+		end
+		
+		it "returns a directory under Dir.tmpdir if Utils.passenger_tmpdir is nil" do
+			File.dirname(passenger_tmpdir(false)).should == Dir.tmpdir
+		end
+		
+		it "returns a directory under Dir.tmpdir if Utils.passenger_tmpdir is an empty string" do
+			Utils.passenger_tmpdir = ''
+			File.dirname(passenger_tmpdir(false)).should == Dir.tmpdir
+		end
+		
+		it "returns Utils.passenger_tmpdir if it's set" do
+			Utils.passenger_tmpdir = '/foo'
+			passenger_tmpdir(false).should == '/foo'
+		end
+		
+		it "creates the directory if it doesn't exist, if the 'create' argument is true" do
+			Utils.passenger_tmpdir = 'utils_spec.tmp'
+			passenger_tmpdir
+			begin
+				File.directory?('utils_spec.tmp').should be_true
+			ensure
+				FileUtils.chmod_R(0777, 'utils_spec.tmp')
+				FileUtils.rm_rf('utils_spec.tmp')
+			end
+		end
+	end
+	
+	######################
+end

data/test/ruby/wsgi/application_spawner_spec.rb

@@ -0,0 +1,54 @@
+require 'support/config'
+require 'support/test_helper'
+require 'phusion_passenger/wsgi/application_spawner'
+require 'phusion_passenger/utils'
+require 'fileutils'
+require 'tempfile'
+
+include PhusionPassenger
+
+describe PhusionPassenger::WSGI::ApplicationSpawner do
+	include TestHelper
+	include PhusionPassenger::Utils
+	
+	before :each do
+		@old_passenger_tmpdir = Utils.passenger_tmpdir
+		Utils.passenger_tmpdir = "#{Dir.tmpdir}/wsgi_test.tmp"
+		@stub = setup_stub('wsgi')
+		File.unlink("#{@stub.app_root}/passenger_wsgi.pyc") rescue nil
+	end
+	
+	after :each do
+		@stub.destroy
+		FileUtils.chmod_R(0700, Utils.passenger_tmpdir)
+		FileUtils.rm_rf(Utils.passenger_tmpdir)
+		Utils.passenger_tmpdir = @old_passenger_tmpdir
+	end
+	
+	it "can spawn our stub application" do
+		spawn(@stub.app_root).close
+	end
+	
+	it "creates a socket in Phusion Passenger's temp directory" do
+		begin
+			app = spawn(@stub.app_root)
+			File.chmod(0700, "#{passenger_tmpdir}/backends")
+			Dir["#{passenger_tmpdir}/backends/wsgi_backend.*"].should have(1).item
+		ensure
+			app.close rescue nil
+		end
+	end
+	
+	specify "the backend process deletes its socket upon termination" do
+		spawn(@stub.app_root).close
+		sleep 0.25 # Give it some time to terminate.
+		File.chmod(0700, "#{passenger_tmpdir}/backends")
+		Dir["#{passenger_tmpdir}/backends/wsgi_backend.*"].should be_empty
+	end
+	
+	def spawn(app_root)
+		PhusionPassenger::WSGI::ApplicationSpawner.spawn_application(app_root,
+			true, CONFIG['lowest_user'])
+	end
+end
+

data/test/stub/apache2/httpd.conf.erb

@@ -0,0 +1,81 @@
+ServerRoot "<%= @server_root %>"
+Listen 127.0.0.1:<%= @port %>
+
+<% if @www_user %>
+	User <%= @www_user %>
+<% end %>
+<% if @www_group %>
+	Group <%= @www_group %>
+<% end %>
+
+<% if !has_builtin_module?('mod_log_config.c') %>
+	LoadModule log_config_module "<%= modules_dir %>/mod_log_config.so"
+<% end %>
+<% if !has_builtin_module?('mod_mime.c') %>
+	LoadModule mime_module "<%= modules_dir %>/mod_mime.so"
+<% end %>
+<% if !has_builtin_module?('mod_dir.c') %>
+	LoadModule dir_module "<%= modules_dir %>/mod_dir.so"
+<% end %>
+<% if !has_builtin_module?('mod_alias.c') %>
+	LoadModule alias_module "<%= modules_dir %>/mod_alias.so"
+<% end %>
+<% if !has_builtin_module?('mod_rewrite.c') && has_module?('mod_rewrite.so') %>
+	LoadModule rewrite_module "<%= modules_dir %>/mod_rewrite.so"
+<% end %>
+<% if !has_builtin_module?('mod_env.c') %>
+	LoadModule env_module "<%= modules_dir %>/mod_env.so"
+<% end %>
+LoadModule passenger_module "<%= @mod_passenger %>"
+
+PassengerRoot "<%= @passenger_root %>"
+PassengerRuby "<%= PlatformInfo::RUBY %>"
+PassengerDefaultUser <%= CONFIG['lowest_user'] %>
+RailsEnv production
+RackEnv production
+PassengerLogLevel 1
+<% for line in @extra %>
+	<%= line %>
+<% end %>
+
+TypesConfig "<%= @server_root %>/mime.types"
+StartServers          1
+ServerLimit          50
+MaxRequestsPerChild  50
+MaxClients           50
+<IfModule mpm_prefork_module>
+	MinSpareServers       1
+	MaxSpareServers       1
+</IfModule>
+<IfModule mpm_worker_module>
+	MinSpareThreads       1
+	MaxSpareThreads       1
+	ThreadsPerChild       2
+</IfModule>
+
+<Directory />
+	AllowOverride all
+</Directory>
+
+ServerAdmin admin@passenger.test
+ServerName passenger.test
+DocumentRoot "<%= @server_root %>"
+
+LockFile  <%= @server_root %>/httpd.lock
+PidFile   <%= @server_root %>/httpd.pid
+ErrorLog  <%= @passenger_root %>/test/apache2.log
+CustomLog <%= @server_root %>/access.log combined
+
+<% if !vhosts.empty? %>
+	NameVirtualHost *:<%= @port %>
+<% end %>
+<% for vhost in vhosts %>
+	<VirtualHost *:<%= @port %>>
+		ServerName <%= vhost.domain %>
+		DocumentRoot "<%= vhost.document_root %>"
+		AllowEncodedSlashes On
+		<% for line in vhost.additional_configs %>
+			<%= line %>
+		<% end %>
+	</VirtualHost>
+<% end %>