passenger 4.0.10 → 4.0.13

data/test/integration_tests/standalone_tests.rb

@@ -0,0 +1,275 @@
+source_root = File.expand_path("../..", File.dirname(__FILE__))
+$LOAD_PATH.unshift("#{source_root}/lib")
+require 'phusion_passenger'
+PhusionPassenger.locate_directories
+require 'phusion_passenger/platform_info/binary_compatibility'
+require 'tmpdir'
+require 'fileutils'
+require 'webrick'
+require 'thread'
+require 'open-uri'
+
+ENV['PATH'] = "#{PhusionPassenger.bin_dir}:#{ENV['PATH']}"
+# This environment variable changes Passenger Standalone's behavior,
+# so ensure that it's not set.
+ENV.delete('PASSENGER_DEBUG')
+
+describe "Passenger Standalone" do
+	after :each do
+		ENV.delete('PASSENGER_DEBUG')
+	end
+
+	let(:version) { PhusionPassenger::VERSION_STRING }
+	let(:nginx_version) { PhusionPassenger::PREFERRED_NGINX_VERSION }
+	let(:compat_id) { PhusionPassenger::PlatformInfo.cxx_binary_compatibility_id }
+
+	def use_binaries_from_source_root!
+		ENV['PASSENGER_DEBUG'] = '1'
+	end
+
+	def sh(*command)
+		if !system(*command)
+			abort "Command failed: #{command.join(' ')}"
+		end
+	end
+
+	def capture_output(command)
+		output = `#{command} 2>&1`.strip
+		if $?.exitstatus == 0
+			return output
+		else
+			abort "Command #{command} exited with status #{$?.exitstatus}; output:\n#{output}"
+		end
+	end
+
+	def start_server(document_root)
+		server = WEBrick::HTTPServer.new(:BindAddress => '127.0.0.1',
+			:Port => 0,
+			:DocumentRoot => document_root,
+			:Logger => WEBrick::Log.new("/dev/null"),
+			:AccessLog => [])
+		Thread.new do
+			Thread.current.abort_on_exception = true
+			server.start
+		end
+		[server, "http://127.0.0.1:#{server.config[:Port]}"]
+	end
+
+	def create_tarball(filename, contents = nil)
+		filename = File.expand_path(filename)
+		Dir.mktmpdir("tarball-") do |tarball_dir|
+			Dir.chdir(tarball_dir) do
+				if block_given?
+					yield
+				else
+					contents.each do |content_name|
+						create_file(content_name)
+					end
+				end
+				sh "tar", "-czf", filename, "."
+			end
+		end
+	end
+
+	def create_dummy_support_binaries
+		Dir.mkdir("agents") if !File.exist?("agents")
+		["PassengerWatchdog", "PassengerHelperAgent", "PassengerLoggingAgent"].each do |exe|
+			File.open("agents/#{exe}", "w") do |f|
+				f.puts "#!/bin/bash"
+				f.puts "echo PASS"
+			end
+			File.chmod(0755, "agents/#{exe}")
+		end
+	end
+
+	def create_dummy_nginx_binary
+		File.open("nginx", "w") do |f|
+			f.puts "#!/bin/bash"
+			f.puts "echo nginx version: 1.0.0"
+		end
+		File.chmod(0755, "nginx")
+	end
+
+	def create_file(filename, contents = nil)
+		File.open(filename, "wb") do |f|
+			f.write(contents) if contents
+		end
+	end
+
+	specify "invoking 'passenger' without an argument is equivalent to 'passenger help'" do
+		output = capture_output("passenger")
+		output.should == capture_output("passenger help")
+	end
+
+	specify "'passenger --help' is equivalent to 'passenger help'" do
+		output = capture_output("passenger")
+		output.should == capture_output("passenger help")
+	end
+
+	specify "'passenger --version' displays the version number" do
+		output = capture_output("passenger --version")
+		output.should include("version #{PhusionPassenger::VERSION_STRING}\n")
+	end
+
+	describe "start command" do
+		SUPPORT_BINARIES_DOWNLOAD_MESSAGE = "Downloading Passenger support binaries for your platform, if available"
+		NGINX_BINARY_DOWNLOAD_MESSAGE = "Downloading Nginx binary for your platform, if available"
+		NGINX_SOURCE_DOWNLOAD_MESSAGE = "Downloading Nginx..."
+		COMPILING_MESSAGE = "Installing Phusion Passenger Standalone"
+
+		def test_serving_application(passenger_command)
+			Dir.chdir(@runtime_dir) do
+				File.open("config.ru", "w") do |f|
+					f.write(%Q{
+						app = lambda do |env|
+							[200, { "Content-Type" => "text/plain" }, ["ok"]]
+						end
+						run app
+					})
+				end
+				Dir.mkdir("public")
+				Dir.mkdir("tmp")
+				sh("#{passenger_command} -p 4000 -d >/dev/null")
+				begin
+					open("http://127.0.0.1:4000/") do |f|
+						f.read.should == "ok"
+					end
+				ensure
+					sh("passenger stop -p 4000")
+				end
+			end
+		end
+
+		context "if the runtime is not installed" do
+			before :each do
+				@runtime_dir = Dir.mktmpdir
+				@webroot = Dir.mktmpdir
+				@server, @base_url = start_server(@webroot)
+
+				Dir.mkdir("#{@webroot}/#{version}")
+				Dir.chdir("#{@webroot}/#{version}") do
+					create_tarball("nginx-#{nginx_version}-#{compat_id}.tar.gz") do
+						create_dummy_nginx_binary
+					end
+					create_tarball("support-#{compat_id}.tar.gz") do
+						FileUtils.mkdir_p("agents")
+						FileUtils.mkdir_p("common/libpassenger_common/ApplicationPool2")
+						create_file("common/libboost_oxt.a")
+						create_file("common/libpassenger_common/ApplicationPool2/Implementation.o")
+						create_dummy_support_binaries
+					end
+				end
+
+				create_file("#{PhusionPassenger.resources_dir}/release.txt")
+			end
+
+			after :each do
+				@server.stop
+				File.unlink("#{PhusionPassenger.resources_dir}/release.txt")
+				FileUtils.remove_entry_secure(@webroot)
+				FileUtils.remove_entry_secure(@runtime_dir)
+			end
+
+			context "when originally packaged" do
+				it "downloads binaries from the Internet" do
+					@output = capture_output("passenger start " +
+						"--runtime-dir '#{@runtime_dir}' " +
+						"--runtime-check-only " +
+						"--binaries-url-root '#{@base_url}'")
+					@output.should include(SUPPORT_BINARIES_DOWNLOAD_MESSAGE)
+					@output.should include(NGINX_BINARY_DOWNLOAD_MESSAGE)
+					@output.should_not include(NGINX_SOURCE_DOWNLOAD_MESSAGE)
+					@output.should_not include(COMPILING_MESSAGE)
+				end
+
+				it "builds the runtime if downloading fails" do
+					# Yes, we're testing the entire build system here.
+					command = "passenger start " +
+						"--runtime-dir '#{@runtime_dir}' " +
+						"--binaries-url-root '#{@base_url}/wrong'"
+					@output = capture_output("#{command} --runtime-check-only")
+					@output.should include(SUPPORT_BINARIES_DOWNLOAD_MESSAGE)
+					@output.should include(NGINX_BINARY_DOWNLOAD_MESSAGE)
+					@output.should include(NGINX_SOURCE_DOWNLOAD_MESSAGE)
+					@output.should include(COMPILING_MESSAGE)
+
+					test_serving_application(command)
+				end
+
+				it "starts a server which serves the application" do
+					# The last test already tests this. This empty test here
+					# is merely to show the intent of the tests, and to
+					# speed up the test suite by preventing an unnecessary
+					# compilation.
+				end
+			end
+
+			context "when natively packaged" do
+				before :each do
+					sh "passenger-config --make-locations-ini > '#{@runtime_dir}/locations.ini'"
+					ENV['PASSENGER_LOCATION_CONFIGURATION_FILE'] = "#{@runtime_dir}/locations.ini"
+					create_file("#{PhusionPassenger.lib_dir}/nginx")
+				end
+
+				after :each do
+					ENV.delete('PASSENGER_LOCATION_CONFIGURATION_FILE')
+					File.unlink("#{PhusionPassenger.lib_dir}/nginx")
+				end
+
+				it "downloads only the Nginx binary from the Internet" do
+					File.rename("#{@webroot}/#{version}/nginx-#{nginx_version}-#{compat_id}.tar.gz",
+						"#{@webroot}/#{version}/nginx-0.0.1-#{compat_id}.tar.gz")
+					@output = capture_output("passenger start " +
+						"--runtime-dir '#{@runtime_dir}' " +
+						"--runtime-check-only " +
+						"--binaries-url-root '#{@base_url}' " +
+						"--nginx-version 0.0.1")
+					@output.should_not include(SUPPORT_BINARIES_DOWNLOAD_MESSAGE)
+					@output.should include(NGINX_BINARY_DOWNLOAD_MESSAGE)
+					@output.should_not include(NGINX_SOURCE_DOWNLOAD_MESSAGE)
+					@output.should_not include(COMPILING_MESSAGE)
+				end
+
+				it "only builds Nginx if downloading fails" do
+					# Yes, we're testing the build system here.
+					command = "passenger start " +
+						"--runtime-dir '#{@runtime_dir}' " +
+						"--binaries-url-root '#{@base_url}' " +
+						"--nginx-version 1.4.1"
+					@output = capture_output("#{command} --runtime-check-only")
+					@output.should_not include(SUPPORT_BINARIES_DOWNLOAD_MESSAGE)
+					@output.should include(NGINX_BINARY_DOWNLOAD_MESSAGE)
+					@output.should include(NGINX_SOURCE_DOWNLOAD_MESSAGE)
+					@output.should include(COMPILING_MESSAGE)
+
+					test_serving_application(command)
+				end
+
+				it "starts a server which serves the application" do
+					# The last test already tests this. This empty test here
+					# is merely to show the intent of the tests, and to
+					# speed up the test suite by preventing an unnecessary
+					# compilation.
+				end
+			end
+		end
+
+		context "if the runtime is installed" do
+			it "doesn't download the runtime from the Internet"
+			it "doesn't build the runtime"
+		end
+
+		it "daemonizes if -d is given" do
+			# Earlier tests already test this. This empty test here
+			# is merely to show the intent of the tests, and to
+			# speed up the test suite by preventing an unnecessary
+			# compilation.
+		end
+	end
+
+	describe "help command" do
+		it "displays the available commands" do
+			capture_output("passenger help").should include("Available commands:")
+		end
+	end
+end

data/test/ruby/admin_tools_spec.rb

@@ -57,7 +57,7 @@ describe AdminTools::ServerInstance do
 	describe ".list" do
 		before :each do
 			AdminTools.should_receive(:tmpdir).and_return(passenger_tmpdir)
-			AdminTools::ServerInstance.stub!(:current_time).
+			AdminTools::ServerInstance.stub(:current_time).
 				and_return(Time.now + AdminTools::ServerInstance::STALE_TIME_THRESHOLD + 1)
 		end
 		

data/test/ruby/rails4.0/loader_spec.rb

@@ -0,0 +1,28 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+require 'ruby/shared/loader_sharedspec'
+require 'ruby/shared/rails/analytics_logging_extensions_sharedspec'
+
+if RUBY_VERSION_INT >= 190
+module PhusionPassenger
+
+describe "Rack loader with Rails 4.0" do
+	include LoaderSpecHelper
+
+	before :each do
+		@stub = register_stub(RackStub.new("rails4.0"))
+	end
+
+	def start(options = {})
+		@loader = Loader.new(["ruby", "#{PhusionPassenger.helper_scripts_dir}/rack-loader.rb"], @stub.app_root)
+		return @loader.start(options)
+	end
+
+	def rails_version
+		return "4.0"
+	end
+
+	include_examples "analytics logging extensions for Rails"
+end
+
+end # module PhusionPassenger
+end

data/test/ruby/rails4.0/preloader_spec.rb

@@ -0,0 +1,34 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+require 'ruby/shared/loader_sharedspec'
+require 'ruby/shared/rails/analytics_logging_extensions_sharedspec'
+
+if RUBY_VERSION_INT >= 190
+module PhusionPassenger
+
+describe "Rack loader with Rails 4.0" do
+	include LoaderSpecHelper
+
+	before :each do
+		@stub = register_stub(RackStub.new("rails4.0"))
+	end
+
+	def start(options = {})
+		@preloader = Preloader.new(["ruby", "#{PhusionPassenger.helper_scripts_dir}/rack-preloader.rb"], @stub.app_root)
+		result = @preloader.start(options)
+		if result[:status] == "Ready"
+			@loader = @preloader.spawn(options)
+			return @loader.start(options)
+		else
+			return result
+		end
+	end
+
+	def rails_version
+		return "4.0"
+	end
+
+	include_examples "analytics logging extensions for Rails"
+end
+
+end # module PhusionPassenger
+end

data/test/ruby/spec_helper.rb

@@ -2,6 +2,8 @@ if GC.respond_to?(:copy_on_write_friendly?) && !GC.copy_on_write_friendly?
 	GC.copy_on_write_friendly = true
 end
 
+RUBY_VERSION_INT = RUBY_VERSION.split('.')[0..2].join.to_i
+
 source_root = File.expand_path(File.dirname(__FILE__) + "/../..")
 Dir.chdir("#{source_root}/test")
 
@@ -26,7 +28,9 @@ def boolean_option(name, default_value = false)
 end
 
 DEBUG = boolean_option('DEBUG')
-TEST_CLASSIC_RAILS = boolean_option('TEST_CLASSIC_RAILS', true)
+TEST_CLASSIC_RAILS = boolean_option('TEST_CLASSIC_RAILS', Gem::VERSION <= '1.9')
+
+ENV.delete('PASSENGER_DEBUG')
 
 $LOAD_PATH.unshift("#{source_root}/lib")
 $LOAD_PATH.unshift("#{source_root}/test")

data/test/ruby/standalone/runtime_installer_spec.rb

@@ -0,0 +1,399 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+require 'phusion_passenger/standalone/runtime_installer'
+require 'tmpdir'
+require 'fileutils'
+require 'stringio'
+
+module PhusionPassenger
+module Standalone
+
+describe RuntimeInstaller do
+	before :each do
+		@temp_dir = Dir.mktmpdir
+		Dir.mkdir("#{@temp_dir}/support")
+		Dir.mkdir("#{@temp_dir}/nginx")
+		@logs = StringIO.new
+		PhusionPassenger.stub(:installed_from_release_package?).and_return(true)
+	end
+
+	after :each do
+		FileUtils.remove_entry_secure(@temp_dir)
+	end
+
+	let(:binaries_url_root) { "http://somewhere" }
+	let(:version) { PhusionPassenger::VERSION_STRING }
+	let(:nginx_version) { PhusionPassenger::PREFERRED_NGINX_VERSION }
+	let(:cxx_compat_id) { PlatformInfo.cxx_binary_compatibility_id }
+	let(:support_binaries_url) { "#{binaries_url_root}/#{version}/support-#{cxx_compat_id}.tar.gz" }
+	let(:nginx_binary_url) { "#{binaries_url_root}/#{version}/nginx-#{nginx_version}-#{cxx_compat_id}.tar.gz" }
+	let(:nginx_source_url) { "http://nginx.org/download/nginx-#{nginx_version}.tar.gz" }
+
+	def create_installer(options = {})
+		options = {
+			:binaries_url_root => binaries_url_root,
+			:stdout => @logs,
+			:stderr => @logs
+		}.merge(options)
+		@installer = RuntimeInstaller.new(options)
+	end
+
+	def create_tarball(filename, contents = nil)
+		Dir.mktmpdir("tarball-", @temp_dir) do |tarball_dir|
+			Dir.chdir(tarball_dir) do
+				if block_given?
+					yield
+				else
+					contents.each do |content_name|
+						File.open(content_name, "w").close
+					end
+				end
+				sh "tar", "-czf", filename, "."
+			end
+		end
+	end
+
+	def create_dummy_support_binaries
+		Dir.mkdir("agents")
+		["PassengerWatchdog", "PassengerHelperAgent", "PassengerLoggingAgent"].each do |exe|
+			File.open("agents/#{exe}", "w") do |f|
+				f.puts "#!/bin/bash"
+				f.puts "echo PASS"
+			end
+			File.chmod(0755, "agents/#{exe}")
+		end
+	end
+
+	def create_dummy_nginx_binary
+		File.open("nginx", "w") do |f|
+			f.puts "#!/bin/bash"
+			f.puts "echo nginx version: 1.0.0"
+		end
+		File.chmod(0755, "nginx")
+	end
+
+	def create_dummy_nginx_source
+		Dir.mkdir("nginx-#{nginx_version}")
+		File.open("nginx-#{nginx_version}/configure", "w") do |f|
+			f.puts %Q{echo "$@" > '#{@temp_dir}/configure.txt'}
+		end
+		File.chmod(0700, "nginx-#{nginx_version}/configure")
+		File.open("nginx-#{nginx_version}/Makefile", "w") do |f|
+			f.puts("all:")
+			f.puts("	mkdir objs")
+			f.puts("	echo ok > objs/nginx")
+		end
+	end
+
+	def create_file(filename)
+		File.open(filename, "w").close
+	end
+
+	def sh(*command)
+		if !system(*command)
+			raise "Command failed: #{command.join(' ')}"
+		end
+	end
+
+	def test_download_nginx_binary
+		create_installer(:targets => [:nginx],
+				:nginx_dir => "#{@temp_dir}/nginx",
+				:lib_dir => PhusionPassenger.lib_dir)
+
+			@installer.should_receive(:download).
+				and_return do |url, output, options|
+					url.should == nginx_binary_url
+					options[:use_cache].should be_true
+					create_tarball(output) do
+						create_dummy_nginx_binary
+					end
+					true
+				end
+
+			@installer.should_receive(:check_for_download_tool)
+			@installer.should_not_receive(:check_depdendencies)
+			@installer.should_not_receive(:compile_support_binaries)
+			@installer.should_not_receive(:download_and_extract_nginx_sources)
+			@installer.should_not_receive(:compile_nginx)
+			@installer.run
+
+			File.exist?("#{@temp_dir}/nginx/nginx").should be_true
+	end
+
+	def test_building_nginx_binary
+		create_installer(:targets => [:nginx],
+				:nginx_dir => "#{@temp_dir}/nginx",
+				:lib_dir   => PhusionPassenger.lib_dir)
+
+			@installer.should_receive(:download).twice.and_return do |url, output|
+				if url == nginx_binary_url
+					false
+				elsif url == nginx_source_url
+					create_tarball(output) do
+						create_dummy_nginx_source
+					end
+					true
+				else
+					raise "Unexpected download URL: #{url}"
+				end
+			end
+
+			@installer.should_receive(:check_for_download_tool)
+			@installer.should_receive(:check_dependencies).and_return(true)
+			@installer.should_not_receive(:compile_support_binaries)
+			@installer.should_receive(:strip_binary).
+				with(an_instance_of(String)).
+				and_return(true)
+			@installer.run
+
+			File.read("#{@temp_dir}/nginx/nginx").should == "ok\n"
+			File.read("#{@temp_dir}/configure.txt").should include(
+				"--add-module=#{PhusionPassenger.nginx_module_source_dir}")
+	end
+
+	context "when originally packaged" do
+		before :each do
+			PhusionPassenger.stub(:originally_packaged?).and_return(true)
+			PhusionPassenger.stub(:natively_packaged?).and_return(false)
+		end
+
+		it "downloads the support binaries from the Internet if :support_binaries is specified as target" do
+			create_installer(:targets => [:support_binaries],
+				:support_dir => "#{@temp_dir}/support")
+
+			@installer.should_receive(:download).
+				and_return do |url, output, options|
+					url.should == "#{binaries_url_root}/#{version}/support-#{cxx_compat_id}.tar.gz"
+					options[:use_cache].should be_true
+					create_tarball(output) do
+						create_dummy_support_binaries
+					end
+					true
+				end
+			
+			@installer.should_receive(:check_for_download_tool)
+			@installer.should_not_receive(:check_depdendencies)
+			@installer.should_not_receive(:compile_support_binaries)
+			@installer.should_not_receive(:download_and_extract_nginx_sources)
+			@installer.should_not_receive(:compile_nginx)
+			@installer.run
+
+			File.exist?("#{@temp_dir}/support/agents/PassengerWatchdog").should be_true
+		end
+
+		it "downloads the Nginx binary from the Internet if :nginx is specified as target" do
+			test_download_nginx_binary
+		end
+
+		it "downloads everything if :support_binaries and :nginx are both specified as target" do
+			create_installer(:targets => [:support_binaries, :nginx],
+				:support_dir => "#{@temp_dir}/support",
+				:nginx_dir => "#{@temp_dir}/nginx",
+				:lib_dir => PhusionPassenger.lib_dir)
+
+			@installer.should_receive(:download).
+				twice.
+				and_return do |url, output, options|
+					if url == support_binaries_url
+						create_tarball(output) do
+							create_dummy_support_binaries
+						end
+					elsif url == nginx_binary_url
+						create_tarball(output) do
+							create_dummy_nginx_binary
+						end
+					else
+						raise "Unexpected download URL: #{url}"
+					end
+					options[:use_cache].should be_true
+					true
+				end
+
+			@installer.should_receive(:check_for_download_tool)
+			@installer.should_not_receive(:check_depdendencies)
+			@installer.should_not_receive(:compile_support_binaries)
+			@installer.should_not_receive(:download_and_extract_nginx_sources)
+			@installer.should_not_receive(:compile_nginx)
+			@installer.run
+
+			File.exist?("#{@temp_dir}/support/agents/PassengerWatchdog").should be_true
+			File.exist?("#{@temp_dir}/nginx/nginx").should be_true
+		end
+
+		it "builds the support binaries if it cannot be downloaded" do
+			create_installer(:targets => [:support_binaries],
+				:support_dir => "#{@temp_dir}/support")
+			nginx_libs = COMMON_LIBRARY.
+				only(*NGINX_LIBS_SELECTOR).
+				set_output_dir("#{@temp_dir}/support/libpassenger_common").
+				link_objects
+			built_files = nil
+
+			@installer.should_receive(:run_rake_task!).with(
+				"nginx_without_native_support CACHING=false OUTPUT_DIR='#{@temp_dir}/support'").
+				and_return do
+					FileUtils.mkdir_p("#{@temp_dir}/agents")
+					create_file("#{@temp_dir}/agents/PassengerWatchdog")
+
+					nginx_libs.each do |object_filename|
+						dir = File.dirname(object_filename)
+						FileUtils.mkdir_p(dir)
+						create_file(object_filename)
+					end
+
+					built_files = `find '#{@temp_dir}/support'`
+				end
+
+			@installer.should_receive(:check_for_download_tool)
+			@installer.should_receive(:download).and_return(false)
+			@installer.should_receive(:check_dependencies).and_return(true)
+			@installer.should_not_receive(:download_and_extract_nginx_sources)
+			@installer.should_not_receive(:compile_nginx)
+			@installer.run
+			`find '#{@temp_dir}/support'`.should == built_files
+		end
+
+		it "builds the Nginx binary if it cannot be downloaded" do
+			test_building_nginx_binary
+		end
+
+		it "aborts if the support binaries cannot be built" do
+			create_installer(:targets => [:support_binaries],
+				:support_dir => "#{@temp_dir}/support")
+
+			@installer.should_receive(:run_rake_task!).with(
+				"nginx_without_native_support CACHING=false OUTPUT_DIR='#{@temp_dir}/support'").
+				and_raise(RuntimeError, "Rake failed")
+
+			@installer.should_receive(:check_for_download_tool)
+			@installer.should_receive(:download).and_return(false)
+			@installer.should_receive(:check_dependencies).and_return(true)
+			@installer.should_not_receive(:download_and_extract_nginx_sources)
+			@installer.should_not_receive(:compile_nginx)
+			lambda { @installer.run }.should raise_error("Rake failed")
+		end
+	end
+
+	context "when natively packaged" do
+		before :each do
+			PhusionPassenger.stub(:source_root).and_return("/locations.ini")
+			PhusionPassenger.stub(:originally_packaged?).and_return(false)
+			PhusionPassenger.stub(:natively_packaged?).and_return(true)
+		end
+
+		it "refuses to accept :support_binaries as target" do
+			block = lambda do
+				create_installer(:targets => [:support_binaries],
+					:support_dir => "#{@temp_dir}/support")
+			end
+			block.should raise_error(ArgumentError, /You cannot specify :support_binaries/)
+		end
+
+		it "downloads the Nginx binary from the Internet if :nginx is specified as target" do
+			test_download_nginx_binary
+		end
+
+		it "builds the Nginx binary if it cannot be downloaded" do
+			test_building_nginx_binary
+		end
+	end
+
+	it "commits downloaded binaries after checking whether they're usable" do
+		create_installer(:targets => [:support_binaries, :nginx],
+				:support_dir => "#{@temp_dir}/support",
+				:nginx_dir => "#{@temp_dir}/nginx",
+				:lib_dir => PhusionPassenger.lib_dir)
+
+			@installer.should_receive(:download).
+				exactly(3).times.
+				and_return do |url, output, options|
+					if url == support_binaries_url
+						options[:use_cache].should be_true
+						create_tarball(output) do
+							create_dummy_support_binaries
+						end
+					elsif url == nginx_binary_url
+						options[:use_cache].should be_true
+						create_tarball(output) do
+							create_dummy_nginx_binary
+						end
+					elsif url == nginx_source_url
+						create_tarball(output) do
+							create_dummy_nginx_source
+						end
+					else
+						raise "Unexpected download URL: #{url}"
+					end
+					true
+				end
+
+			@installer.should_receive(:check_for_download_tool)
+			@installer.should_receive(:check_support_binaries).and_return(false)
+			@installer.should_receive(:check_nginx_binary).and_return(false)
+			@installer.should_receive(:check_dependencies).and_return(true)
+			@installer.should_receive(:compile_support_binaries)
+			@installer.should_receive(:compile_nginx)
+			@installer.run
+
+			Dir["#{@temp_dir}/nginx/*"].should be_empty
+			Dir["#{@temp_dir}/support/*"].should be_empty
+	end
+
+	it "aborts if the Nginx source tarball cannot be extracted" do
+		create_installer(:targets => [:nginx],
+			:nginx_dir => "#{@temp_dir}/nginx",
+			:lib_dir   => PhusionPassenger.lib_dir)
+
+		@installer.should_receive(:download).twice.and_return do |url, output, options|
+			if url == nginx_binary_url
+				false
+			elsif url == nginx_source_url
+				File.open(output, "w") do |f|
+					f.write("garbage")
+				end
+				true
+			else
+				raise "Unexpected download URL: #{url}"
+			end
+		end
+
+		@installer.should_receive(:check_for_download_tool)
+		@installer.should_receive(:check_dependencies).and_return(true)
+		@installer.should_not_receive(:compile_support_binaries)
+		lambda { @installer.run }.should raise_error(SystemExit)
+		@logs.string.should =~ %r{Unable to download or extract Nginx source tarball}
+	end
+	
+	it "aborts if the Nginx binary cannot be built" do
+		create_installer(:targets => [:nginx],
+			:nginx_dir => "#{@temp_dir}/nginx",
+			:lib_dir   => PhusionPassenger.lib_dir)
+
+		@installer.should_receive(:download).twice.and_return do |url, output, options|
+			if url == nginx_binary_url
+				false
+			elsif url == nginx_source_url
+				create_tarball(output) do
+					Dir.mkdir("nginx-#{nginx_version}")
+					File.open("nginx-#{nginx_version}/configure", "w") do |f|
+						f.puts("#!/bin/bash")
+						f.puts("echo error")
+						f.puts("exit 1")
+					end
+					File.chmod(0700, "nginx-#{nginx_version}/configure")
+				end
+				true
+			else
+				raise "Unexpected download URL: #{url}"
+			end
+		end
+
+		@installer.should_receive(:check_for_download_tool)
+		@installer.should_receive(:check_dependencies).and_return(true)
+		@installer.should_not_receive(:compile_support_binaries)
+		lambda { @installer.run }.should raise_error(SystemExit)
+		@logs.string.should =~ %r{command failed:.*./configure}
+	end
+end
+
+end # module Standalone
+end # module PhusionPassenger