Merge pull request capistrano#1573 from marcovtwout/svn-revision-new

Added option to set specific revision when using Subversion as SCM

CHANGELOG.md

@@ -31,6 +31,7 @@ https://github.com/capistrano/capistrano/compare/v3.4.0...HEAD
   * Deduplicate list of linked directories
   * Allow dot in :application name (@marcovtwout)
   * Fixed git-ssh permission error (@spight)
+  * Added option to set specific revision when using Subversion as SCM (@marcovtwout)
 
 ## `3.4.0`
 

lib/capistrano/svn.rb

@@ -9,6 +9,7 @@ def svn(*args)
     args.unshift(:svn)
     args.push "--username #{fetch(:svn_username)}" if fetch(:svn_username)
     args.push "--password #{fetch(:svn_password)}" if fetch(:svn_password)
+    args.push "--revision #{fetch(:svn_revision)}" if fetch(:svn_revision)
     context.execute(*args)
   end
 

spec/lib/capistrano/svn_spec.rb

@@ -12,6 +12,7 @@ module Capistrano
         context.expects(:execute).with(:svn, :init, '--username someuser', '--password somepassword')
         context.expects(:fetch).twice.with(:svn_username).returns('someuser')
         context.expects(:fetch).twice.with(:svn_password).returns('somepassword')
+        context.expects(:fetch).once.with(:svn_revision).returns(nil)
         subject.svn(:init)
       end
     end
@@ -47,6 +48,7 @@ module Capistrano
         context.expects(:repo_path).returns(:path)
         context.expects(:fetch).twice.with(:svn_username).returns('someuser')
         context.expects(:fetch).twice.with(:svn_password).returns('somepassword')
+        context.expects(:fetch).once.with(:svn_revision).returns(nil)
 
         context.expects(:execute).with(:svn, :checkout, :url, :path, '--username someuser', '--password somepassword')
 
@@ -59,6 +61,18 @@ module Capistrano
         context.expects(:execute).with(:svn, :update, '--username someuser', '--password somepassword')
         context.expects(:fetch).twice.with(:svn_username).returns('someuser')
         context.expects(:fetch).twice.with(:svn_password).returns('somepassword')
+        context.expects(:fetch).once.with(:svn_revision).returns(nil)
+
+        subject.update
+      end
+    end
+
+    describe "#update_specific_revision" do
+      it "should run svn update and update to a specific revision" do
+        context.expects(:execute).with(:svn, :update, '--username someuser', '--password somepassword', '--revision 12345')
+        context.expects(:fetch).twice.with(:svn_username).returns('someuser')
+        context.expects(:fetch).twice.with(:svn_password).returns('somepassword')
+        context.expects(:fetch).twice.with(:svn_revision).returns('12345')
 
         subject.update
       end
@@ -69,6 +83,7 @@ module Capistrano
         context.expects(:release_path).returns(:path)
         context.expects(:fetch).twice.with(:svn_username).returns('someuser')
         context.expects(:fetch).twice.with(:svn_password).returns('somepassword')
+        context.expects(:fetch).once.with(:svn_revision).returns(nil)
 
         context.expects(:execute).with(:svn, :export, '--force', '.', :path, '--username someuser', '--password somepassword')