Merge pull request #312 from mikz/printer-test

`SSHKit::Backend::Printer#test` now always returns true
capistrano · Dec 29, 2015 · 1f6fb49 · 1f6fb49
2 parents fb37ebc + bb2b005
commit 1f6fb49
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -15,6 +15,8 @@ appear at the top.
     @steved
   * `SSHKit::Formatter::Abstract` now accepts an optional Hash of options
     [PR #308](https://github.com/capistrano/sshkit/pull/308) @mattbrictson
+  * `SSHKit::Backend::Printer#test` now always returns true
+    [PR #312](https://github.com/capistrano/sshkit/pull/312) @mikz
 
 ## 1.8.1
 

diff --git a/lib/sshkit/backends/printer.rb b/lib/sshkit/backends/printer.rb
@@ -10,6 +10,11 @@ def execute_command(cmd)
 
       alias :upload! :execute
       alias :download! :execute
+
+      def test(*)
+        super
+        true
+      end
     end
   end
 end
diff --git a/test/unit/backends/test_printer.rb b/test/unit/backends/test_printer.rb
@@ -28,7 +28,7 @@ def test_execute
       end
 
       def test_test_method
-        printer.test '[ -d /some/file ]'
+        assert printer.test('[ -d /some/file ]'), 'test should return true'
 
         assert_output_lines(
           ' DEBUG [aaaaaa] Running [ -d /some/file ] on example.com',