buildr 1.3.0 → 1.3.1

data/lib/buildr/java/pom.rb

@@ -71,7 +71,7 @@ module Buildr
 
     def initialize(xml) #:nodoc:
       @project = XmlSimple.xml_in(xml)
-      @parent = POM.load(pom_to_hash(project["parent"].first)) if project["parent"]
+      @parent = POM.load(pom_to_hash(project["parent"].first).merge(:type=>'pom')) if project['parent']
     end
 
     # :call-seq:

data/lib/buildr/java/test_frameworks.rb

@@ -201,6 +201,7 @@ module Buildr
           ant.classpath :path=>dependencies.join(File::PATH_SEPARATOR)
           (options[:properties] || []).each { |key, value| ant.sysproperty :key=>key, :value=>value }
           (options[:environment] || []).each { |key, value| ant.env :key=>key, :value=>value }
+          Array(options[:java_args]).each { |value| ant.jvmarg :value=>value }
           ant.formatter :type=>'plain'
           ant.formatter :type=>'plain', :usefile=>false # log test
           ant.formatter :type=>'xml'

data/lib/buildr/java/version_requirement.rb

@@ -24,8 +24,8 @@ module Buildr
     CMP_REGEX = Gem::Requirement::OP_RE.dup
     CMP_CHARS = CMP_PROCS.keys.join
     BOOL_CHARS = '\|\&\!'
-    VER_CHARS = '\w\.'
-    
+    VER_CHARS = '\w\.\-'
+
     class << self
       # is +str+ a version string?
       def version?(str)

data/lib/buildr/packaging/artifact_namespace.rb

@@ -496,13 +496,25 @@ module Buildr
 
       # Return an artifact spec without the version part.
       def unversioned_spec
-        to_spec[/^([a-zA-Z._-]+(:[a-zA-Z._-]+){2,3})/]
+        str = to_spec
+        return nil if str =~ /^:+/
+        ary = str.split(':')
+        ary = ary[0...-1] if ary.size > 3
+        ary.join(':')
       end
     
       class << self
         # Return an artifact spec without the version part.
         def unversioned_spec(spec)
-          spec.to_s[/^([a-zA-Z._-]+(:[a-zA-Z._-]+){2,3})/] || new(spec).unversioned_spec
+          str = spec.to_s
+          return nil if str =~ /^:+/
+          ary = str.split(':')
+          ary = ary[0...-1] if ary.size > 3
+          if ary.size > 2
+            ary.join(':')
+          else
+            new(spec).unversioned_spec
+          end
         end
       end
     end

data/rakelib/apache.rake

@@ -65,14 +65,39 @@ namespace 'apache' do
     target = args.incubating ? "people.apache.org:/www/www.apache.org/dist/incubator/#{spec.name}/#{spec.version}-incubating" :
       "people.apache.org:/www/www.apache.org/dist/#{spec.name}/#{spec.version}"
     puts 'Uploading packages to Apache distro ...'
-    sh 'rsync', '--progress', 'published/distro/*', target
+    host, remote_dir = target.split(':')
+    sh 'ssh', host, 'rm', '-rf', remote_dir rescue nil
+    sh 'ssh', host, 'mkdir', remote_dir
+    sh 'rsync', '--progress', '--recursive', 'published/distro/', target
     puts 'Done'
   end
 
-
-  task 'distro-links'=>['staged/site', 'apache:sign'] do |task, args|
+  task 'distro-links'=>'staged/distro' do |task, args|
     url = args.incubating ? "http://www.apache.org/dist/incubator/#{spec.name}/#{spec.version}-incubating" :
       "http://www.apache.org/dist/#{spec.name}/#{spec.version}"
+    rows = FileList['staged/distro/*.{gem,tgz,zip}'].map { |pkg|
+      name, md5 = File.basename(pkg), MD5.file(pkg).to_s
+      %{| "#{name}":#{url}/#{name} | "#{md5}":#{url}/#{name}.md5 | "Sig":#{url}/#{name}.asc |}
+    }
+    textile = <<-TEXTILE
+h3. #{spec.name} #{spec.version}#{args.incubating && "-incubating"} (#{Time.now.strftime('%Y-%m-%d')})
+
+|_. Package |_. MD5 Checksum |_. PGP |
+#{rows.join("\n")}
+
+p>. ("Release signing keys":#{url}/KEYS)
+    TEXTILE
+    file_name = 'doc/pages/download.textile'
+    print "Adding download links to #{file_name} ... "
+    modified = File.read(file_name).sub(/h2.*binaries.*source.*/i) { |header| "#{header}\n\n#{textile}" }
+    File.open file_name, 'w' do |file|
+      file.write modified
+    end
+    puts 'Done'
+  end
+
+=begin
+  task 'distro-links'=>['staged/site', 'apache:sign'] do |task, args|
     rows = FileList['staged/distro/*.{gem,tgz,zip}'].map { |pkg|
       name, md5 = File.basename(pkg), File.read("#{pkg}.md5").split.first
       <<-HTML
@@ -97,9 +122,9 @@ namespace 'apache' do
       file.write modified
     end
   end
+=end
 
-  file 'staged/site'=>'site' do
-    mkpath 'staged'
+  file 'staged/site'=>['distro-links', 'staged', 'site'] do
     rm_rf 'staged/site'
     cp_r 'site', 'staged'
   end
@@ -109,7 +134,7 @@ namespace 'apache' do
     target = args.incubating ? "people.apache.org:/www/incubator.apache.org/#{spec.name}" :
       "people.apache.org:/www/#{spec.name}.apache.org"
     puts 'Uploading Apache Web site ...'
-    sh 'rsync', '--progress', '--recursive', '--delete', 'published/distro/site/', target
+    sh 'rsync', '--progress', '--recursive', '--delete', 'published/site/', target
     puts 'Done'
   end
 
@@ -117,7 +142,7 @@ end
 
 
 task 'stage:check'=>['apache:license', 'apache:check']
-task 'stage:prepare'=>['staged/distro', 'staged/site', 'apache:distro-links'] do |task|
+task 'stage:prepare'=>['staged/distro', 'staged/site'] do |task|
   # Since this requires input (passphrase), do it at the very end.
   task.enhance do
     task('apache:sign').invoke

data/rakelib/changelog.rake

@@ -40,7 +40,8 @@ namespace 'changelog' do
   end
 
   task 'wrapup'=>'CHANGELOG' do
-    next_version = spec.version.to_ints.zip([0, 0, 1]).map { |a| a.inject(0) { |t,i| t + i } }.join('.')
+    next_version = spec.version.to_s.split('.').map { |v| v.to_i }.
+      zip([0, 0, 1]).map { |a| a.inject(0) { |t,i| t + i } }.join('.')
     print 'Adding new entry to CHANGELOG ... '
     modified = "#{next_version} (Pending)\n\n" + File.read('CHANGELOG')
     File.open 'CHANGELOG', 'w' do |file|

data/rakelib/release.rake

@@ -38,9 +38,11 @@ task 'release'=>['release:prepare', 'release:publish', 'release:wrapup']
 
 
 task 'next_version' do
+  next_version = spec.version.to_s.split('.').map { |v| v.to_i }.
+    zip([0, 0, 1]).map { |a| a.inject(0) { |t,i| t + i } }.join('.')
+
   ver_file = "lib/#{spec.name}.rb"
   if File.exist?(ver_file)
-    next_version = spec.version.to_ints.zip([0, 0, 1]).map { |a| a.inject(0) { |t,i| t + i } }.join('.')
     print "Updating #{ver_file} to next version number (#{next_version}) ... "
     modified = File.read(ver_file).sub(/(VERSION\s*=\s*)(['"])(.*)\2/) { |line| "#{$1}#{$2}#{next_version}#{$2}" } 
     File.open ver_file, 'w' do |file|
@@ -48,6 +50,16 @@ task 'next_version' do
     end
     puts 'Done'
   end
+
+  spec_file = "#{spec.name}.gemspec"
+  if File.exist?(spec_file)
+    print "Updating #{spec_file} to next version number (#{next_version}) ... "
+    modified = File.read(spec_file).sub(/(s(?:pec)?\.version\s*=\s*)(['"])(.*)\2/) { |line| "#{$1}#{$2}#{next_version}#{$2}" } 
+    File.open spec_file, 'w' do |file|
+      file.write modified
+    end
+    puts 'Done'
+  end
 end
 
 task 'release:wrapup'=>'next_version'

data/rakelib/rubyforge.rake

@@ -29,14 +29,21 @@ end
 
 namespace 'rubyforge' do
 
-  task 'release'=>'published' do |task|
+  file 'published/rubyforge'=>'published' do
+    mkdir 'published/rubyforge'
+    FileList['published/distro/*.{gem,tgz,zip}'].each do |pkg|
+      cp pkg, 'published/rubyforge/' + File.basename(pkg).sub(/-incubating/, '')
+    end
+  end
+
+  task 'release'=>'published/rubyforge' do |task|
     changes = FileList['published/CHANGES'].first
-    files = FileList['published/*.{gem,tgz,zip}'].exclude(changes).existing
+    files = FileList['published/rubyforge/*.{gem,tgz,zip}'].exclude(changes).existing
     print "Uploading #{spec.version} to RubyForge ... "
     rubyforge = RubyForge.new
     rubyforge.login 
     rubyforge.userconfig.merge!('release_changes'=>changes,  'preformatted' => true) if changes
-    rubyforge.add_release spec.rubyforge_project.downcase, spec.name.downcase, spec.version, *files
+    rubyforge.add_release spec.rubyforge_project.downcase, spec.name.downcase, spec.version.to_s, *files
     puts 'Done'
   end
 

data/rakelib/setup.rake

@@ -17,7 +17,7 @@
 
 
 require 'rubygems/source_info_cache'
-
+require 'stringio' # for Gem::RemoteFetcher
 
 def windows?
   Config::CONFIG['host_os'] =~ /windows|cygwin|bccwin|cygwin|djgpp|mingw|mswin|wince/i
@@ -38,11 +38,12 @@ end
 
 def install_gem(name, ver_requirement = nil)
   dep = Gem::Dependency.new(name, ver_requirement)
+  rb_bin = File.join(Config::CONFIG['bindir'], Config::CONFIG['ruby_install_name'])
   if Gem::SourceIndex.from_installed_gems.search(dep).empty?
-    spec = Gem::SourceInfoCache.search(dep).last
+    spec = Gem::SourceInfoCache.search(dep, true, true).last
     fail "#{dep} not found in local or remote repository!" unless spec
     puts "Installing #{spec} ..."
-    args = [Config::CONFIG['ruby_install_name'], '-S', 'gem', 'install', spec.name, '-v', spec.version.to_s]
+    args = [rb_bin, '-S', 'gem', 'install', spec.name, '-v', spec.version.to_s]
     args.unshift('sudo', 'env', 'JAVA_HOME=' + ENV['JAVA_HOME']) unless windows?
     sh *args
   end

data/spec/application_spec.rb

@@ -210,6 +210,11 @@ describe Buildr, 'settings' do
       Buildr.settings.user.should == { 'foo'=>'bar' }
     end
 
+    it 'should return loaded settings.yml file' do
+      write 'home/.buildr/settings.yml', 'foo: bar'
+      Buildr.settings.user.should == { 'foo'=>'bar' }
+    end
+
     it 'should fail if settings.yaml file is not a hash' do
       write 'home/.buildr/settings.yaml', 'foo bar'
       lambda { Buildr.settings.user }.should raise_error(RuntimeError, /expecting.*settings.yaml/i)
@@ -231,6 +236,11 @@ describe Buildr, 'settings' do
       Buildr.settings.build.should == { 'foo'=>'bar' }
     end
 
+    it 'should return loaded build.yml file' do
+      write 'build.yml', 'foo: bar'
+      Buildr.settings.build.should == { 'foo'=>'bar' }
+    end
+
     it 'should fail if build.yaml file is not a hash' do
       write 'build.yaml', 'foo bar'
       lambda { Buildr.settings.build }.should raise_error(RuntimeError, /expecting.*build.yaml/i)
@@ -255,6 +265,14 @@ describe Buildr, 'settings' do
       Buildr.settings.profiles.should == { 'development'=> { 'foo'=>'bar' } }
     end
 
+    it 'should return loaded profiles.yml file' do
+      write 'profiles.yml', <<-YAML
+        development:
+          foo: bar
+      YAML
+      Buildr.settings.profiles.should == { 'development'=> { 'foo'=>'bar' } }
+    end
+
     it 'should fail if profiles.yaml file is not a hash' do
       write 'profiles.yaml', 'foo bar'
       lambda { Buildr.settings.profiles }.should raise_error(RuntimeError, /expecting.*profiles.yaml/i)

data/spec/archive_spec.rb

@@ -42,7 +42,6 @@ describe 'ArchiveTask', :shared=>true do
 
   def create_for_merge
     zip(@archive + '.src').include(@files).tap do |task|
-      task.invoke
       yield task
     end
   end

data/spec/artifact_spec.rb

@@ -424,6 +424,15 @@ describe Buildr, '#artifact' do
     artifact = artifact('group:id:jar:1.0').from('test.jar')
     lambda { artifact.invoke }.should change { File.exist?(artifact.to_s) }.to(true)
   end
+
+  it 'should reference artifacts defined on build.yaml by using ruby symbols' do
+    write 'build.yaml', <<-YAML
+      artifacts: 
+        j2ee: geronimo-spec:geronimo-spec-j2ee:jar:1.4-rc4
+    YAML
+    Buildr.application.load_artifacts
+    artifact(:j2ee).to_s.pathmap('%f').should == 'geronimo-spec-j2ee-1.4-rc4.jar'
+  end
 end
 
 

data/spec/common_spec.rb

@@ -411,10 +411,6 @@ describe Buildr::Filter do
     @filter.from('src').into('target').run.should be(false)
   end
 
-  it 'should fail is source directory not set' do
-    lambda { Filter.new.into('target').run }.should raise_error(RuntimeError, /No source directory/)
-  end
-
   it 'should fail if source directory doesn\'t exist' do
     lambda { Filter.new.from('srced').into('target').run }.should raise_error(RuntimeError, /doesn't exist/)
   end

data/spec/java_spec.rb

@@ -0,0 +1,38 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license agreements.  See the NOTICE file distributed with this
+# work for additional information regarding copyright ownership.  The ASF
+# licenses this file to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#    http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
+# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.  See the
+# License for the specific language governing permissions and limitations under
+# the License.
+
+require File.join(File.dirname(__FILE__), 'spec_helpers')
+
+describe ENV, 'JAVA_HOME on OS X' do
+  before do
+    @old_home, ENV['JAVA_HOME'] = ENV['JAVA_HOME'], nil
+    Config::CONFIG.should_receive(:[]).with('host_os').and_return('darwin0.9')
+  end
+
+  it 'should point to default JVM' do
+    load File.expand_path('../lib/buildr/java.rb')
+    ENV['JAVA_HOME'].should == '/System/Library/Frameworks/JavaVM.framework/Home'
+  end
+
+  it 'should use value of environment variable if specified' do
+    ENV['JAVA_HOME'] = '/System/Library/Frameworks/JavaVM.specified'
+    load File.expand_path('../lib/buildr/java.rb')
+    ENV['JAVA_HOME'].should == '/System/Library/Frameworks/JavaVM.specified'
+  end
+
+  after do
+    ENV['JAVA_HOME'] = @old_home
+  end
+end

data/spec/java_test_frameworks_spec.rb

@@ -179,6 +179,18 @@ describe Buildr::JUnit do
     project('foo').test.invoke
   end
 
+  it 'should pass environment to JVM' do
+    write 'src/test/java/EnvironmentTest.java', <<-JAVA
+      public class EnvironmentTest extends junit.framework.TestCase {
+        public void testEnvironment() {
+          assertEquals("value", System.getenv("NAME"));
+        }
+      }
+    JAVA
+    define('foo').test.using :environment=>{ 'NAME'=>'value' }
+    project('foo').test.invoke
+  end
+
   it 'should set current directory' do
     mkpath 'baz'
     expected = File.expand_path('baz')

data/spec/test_spec.rb

@@ -374,11 +374,13 @@ describe Buildr::Project, '#test' do
 
   it 'should inherit options from parent project' do
     define 'foo' do
-      test.using :fail_on_failure=>false, :fork=>:each, :properties=>{ :foo=>'bar' }
+      test.using :fail_on_failure=>false, :fork=>:each, :properties=>{ :foo=>'bar' }, :environment=>{ 'config'=>'config.yaml' }
       define 'bar' do
+        test.using :junit
         test.options[:fail_on_failure].should be_false
         test.options[:fork].should == :each
         test.options[:properties][:foo].should == 'bar'
+        test.options[:environment]['config'].should == 'config.yaml'
       end
     end
   end
@@ -386,11 +388,13 @@ describe Buildr::Project, '#test' do
   it 'should clone options from parent project' do
     define 'foo' do
       define 'bar' do
-        test.using :fail_on_failure=>false, :fork=>:each, :properties=>{ :foo=>'bar' }
+        test.using :fail_on_failure=>false, :fork=>:each, :properties=>{ :foo=>'bar' }, :environment=>{ 'config'=>'config.yaml' }
+        test.using :junit
       end.invoke
       test.options[:fail_on_failure].should be_true
       test.options[:fork].should == :once
-      test.options[:other].should be_nil
+      test.options[:properties].should be_empty
+      test.options[:environment].should be_empty
     end
   end
 end
@@ -454,6 +458,7 @@ end
 describe Buildr::Project, 'test:resources' do
   it 'should ignore resources unless they exist' do
     define('foo').test.resources.sources.should be_empty
+    project('foo').test.resources.target.should be_nil
   end
 
   it 'should pick resources from src/test/resources if found' do
@@ -467,6 +472,11 @@ describe Buildr::Project, 'test:resources' do
     file('targeted/test/resources/foo').should contain('Foo')
   end
 
+  it 'should create target directory even if no files to copy' do
+    define('foo').test.resources.filter.into('resources')
+    lambda { file(File.expand_path('resources')).invoke }.should change { File.exist?('resources') }.to(true)
+  end
+
   it 'should execute alongside compile task' do
     task 'action'
     define('foo') { test.resources { task('action').invoke } }

data/spec/transport_spec.rb

@@ -221,7 +221,9 @@ describe URI::HTTP, '#read' do
     @proxy = 'http://john:smith@myproxy:8080'
     @domain = 'domain'
     @host_domain = "host.#{@domain}"
-    @uri = URI("http://#{@host_domain}")
+    @path = "/foo/bar/baz"
+    @query = "?query"
+    @uri = URI("http://#{@host_domain}#{@path}#{@query}")
     @no_proxy_args = [@host_domain, 80]
     @proxy_args = @no_proxy_args + ['myproxy', 8080, 'john', 'smith']
     @http = mock('http')
@@ -297,4 +299,190 @@ describe URI::HTTP, '#read' do
     request.should_receive(:basic_auth).with('john', 'secret')
     URI("http://john:secret@#{@host_domain}").read
   end
+
+  it 'should include the query part when performing HTTP GET' do
+    # should this test be generalized or shared with any other URI subtypes?
+    Net::HTTP.stub!(:new).and_return(@http)
+    Net::HTTP::Get.should_receive(:new).with(/#{Regexp.escape(@query)}$/, nil)
+    @uri.read
+  end
+
+end
+
+
+describe URI::HTTP, '#write' do
+  before do
+    @content = 'Readme. Please!'
+    @uri = URI('http://john:secret@host.domain/foo/bar/baz.jar')
+    @http = mock('Net::HTTP')
+    @http.stub!(:request).and_return(Net::HTTPOK.new(nil, nil, nil))
+    Net::HTTP.stub!(:new).and_return(@http)
+  end
+
+  it 'should open connection to HTTP server' do
+    Net::HTTP.should_receive(:new).with('host.domain', 80).and_return(@http)
+    @uri.write @content
+  end
+
+  it 'should use HTTP basic authentication' do
+    @http.should_receive(:request) do |request|
+      request['authorization'].should == ('Basic ' + ['john:secret'].pack('m').delete("\r\n"))
+      Net::HTTPOK.new(nil, nil, nil)
+    end
+    @uri.write @content
+  end
+
+  it 'should use HTTPS if applicable' do
+    Net::HTTP.should_receive(:new).with('host.domain', 443).and_return(@http)
+    @http.should_receive(:use_ssl=).with(true)
+    URI(@uri.to_s.sub(/http/, 'https')).write @content
+  end
+
+  it 'should upload file with PUT request' do
+    @http.should_receive(:request) do |request|
+      request.should be_kind_of(Net::HTTP::Put)
+      Net::HTTPOK.new(nil, nil, nil)
+    end
+    @uri.write @content
+  end
+
+  it 'should set Content-Length header' do
+    @http.should_receive(:request) do |request|
+      request.content_length.should == @content.size
+      Net::HTTPOK.new(nil, nil, nil)
+    end
+    @uri.write @content
+  end
+
+  it 'should set Content-MD5 header' do
+    @http.should_receive(:request) do |request|
+      request['Content-MD5'].should == Digest::MD5.hexdigest(@content)
+      Net::HTTPOK.new(nil, nil, nil)
+    end
+    @uri.write @content
+  end
+
+  it 'should send entire content' do
+    @http.should_receive(:request) do |request|
+      body_stream = request.body_stream
+      body_stream.read(1024).should == @content
+      body_stream.read(1024).should be_nil
+      Net::HTTPOK.new(nil, nil, nil)
+    end
+    @uri.write @content
+  end
+
+  it 'should fail on 4xx response' do
+    @http.should_receive(:request) do |request|
+      Net::HTTPBadRequest.new(nil, nil, nil)
+    end
+    lambda { @uri.write @content }.should raise_error(RuntimeError, /failed to upload/i)
+  end
+
+  it 'should fail on 5xx response' do
+    @http.should_receive(:request) do |request|
+      Net::HTTPServiceUnavailable.new(nil, nil, nil)
+    end
+    lambda { @uri.write @content }.should raise_error(RuntimeError, /failed to upload/i)
+  end
+
+end
+
+
+describe URI::SFTP, '#read' do
+  before do
+    @uri = URI('sftp://john:secret@localhost/path/readme')
+    @content = 'Readme. Please!'
+
+    @ssh_session = mock('Net::SSH::Session')
+    @sftp_session = mock('Net::SFTP::Session')
+    @file_factory = mock('Net::SFTP::Operations::FileFactory')
+    Net::SSH.stub!(:start).with('localhost', 'john', :password=>'secret', :port=>22) do
+      Net::SFTP::Session.should_receive(:new).with(@ssh_session).and_yield(@sftp_session).and_return(@sftp_session)
+      @sftp_session.should_receive(:connect!).and_return(@sftp_session)
+      @sftp_session.should_receive(:loop)
+      @sftp_session.should_receive(:file).with.and_return(@file_factory)
+      @file_factory.stub!(:open)
+      @ssh_session.should_receive(:close)
+      @ssh_session
+    end
+  end
+
+  it 'should open connection to SFTP server' do
+    @uri.read
+  end
+
+  it 'should open file for reading' do
+    @file_factory.should_receive(:open).with('/path/readme', 'r')
+    @uri.read
+  end
+
+  it 'should read contents of file and return it' do
+    file = mock('Net::SFTP::Operations::File')
+    file.should_receive(:read).with(an_instance_of(Numeric)).once.and_return(@content, nil)
+    @file_factory.should_receive(:open).with('/path/readme', 'r').and_yield(file)
+    @uri.read.should eql(@content)
+  end
+
+  it 'should read contents of file and pass it to block' do
+    file = mock('Net::SFTP::Operations::File')
+    file.should_receive(:read).with(an_instance_of(Numeric)).once.and_return(@content, nil)
+    @file_factory.should_receive(:open).with('/path/readme', 'r').and_yield(file)
+    content = ''
+    @uri.read do |chunk|
+      content << chunk
+    end
+    content.should eql(@content)
+  end
+end
+
+
+describe URI::SFTP, '#write' do
+  before do
+    @uri = URI('sftp://john:secret@localhost/path/readme')
+    @content = 'Readme. Please!'
+
+    @ssh_session = mock('Net::SSH::Session')
+    @sftp_session = mock('Net::SFTP::Session')
+    @file_factory = mock('Net::SFTP::Operations::FileFactory')
+    Net::SSH.stub!(:start).with('localhost', 'john', :password=>'secret', :port=>22) do
+      Net::SFTP::Session.should_receive(:new).with(@ssh_session).and_yield(@sftp_session).and_return(@sftp_session)
+      @sftp_session.should_receive(:connect!).and_return(@sftp_session)
+      @sftp_session.should_receive(:loop)
+      @sftp_session.stub!(:mkdir)
+      @sftp_session.should_receive(:file).with.and_return(@file_factory)
+      @file_factory.stub!(:open)
+      @ssh_session.should_receive(:close)
+      @ssh_session
+    end
+  end
+
+  it 'should open connection to SFTP server' do
+    @uri.write @content
+  end
+
+  it 'should create path recursively on SFTP server' do
+    @sftp_session.should_receive(:mkdir).ordered.with('', {})
+    @sftp_session.should_receive(:mkdir).ordered.with('/path', {})
+    @uri.write @content
+  end
+
+  it 'should only create paths that don\'t exist' do
+    @sftp_session.should_receive(:realpath).any_number_of_times
+    @sftp_session.should_not_receive(:mkdir)
+    @uri.write @content
+  end
+
+  it 'should open file for writing' do
+    @file_factory.should_receive(:open).with('/path/readme', 'w')
+    @uri.write @content
+  end
+
+  it 'should write contents to file' do
+    file = mock('Net::SFTP::Operations::File')
+    file.should_receive(:write).with(@content)
+    @file_factory.should_receive(:open).with('/path/readme', 'w').and_yield(file)
+    @uri.write @content
+  end
+
 end