photocopier 0.0.10 → 1.0.0.pre

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 9653d2b9d9c48cead649c2a9ce05db2b9620ed2f
-  data.tar.gz: 3bed95de37b30e22b05bd67eea32bb4d39032930
+  metadata.gz: 3ba20dc26aeefaef97481f03d7195f203b74800c
+  data.tar.gz: 02e7e5bb7a24ad4052a39c75cb8d86f2e6125374
 SHA512:
-  metadata.gz: f3e37d9509d326bf57b4ec75d62a9e2ca918ae6218c0d8634f9dec892aca1e69e0464df7037f942e50eb35a3d7d18581162191a746618fde47af17a64fc20ee0
-  data.tar.gz: 54526e7532807c40184236d575338c03fa4b691910bc691d8285b319646f8128a76be3d0b24d2713beb58103b7d7785dfe8f03410937fb5dd8486fe5fca6b953
+  metadata.gz: d568ac3259e1609af3efa5c0609385ef255a5479ce8d7800a7e068eab727ce1ad02dbabd3e805ac351ed4e2740f783dcdcb874aaa1c96964ecaeb63aea93474a
+  data.tar.gz: 9f5b43f82fbed2e7e78d459001a2a84aeb650fd56c494f1a1df2b03b8394955faf9334d4b9903a3530bd730ba57b8afbb8b1591568f43d128df8122690b89529

data/.gitignore

@@ -1 +1,17 @@
+*.gem
+*.rbc
+.bundle
+.config
+.yardoc
+Gemfile.lock
+InstalledFiles
+_yardoc
+coverage
+doc/
+lib/bundler/man
 pkg
+rdoc
+spec/reports
+test/tmp
+test/version_tmp
+tmp

data/.rspec

@@ -1 +1,2 @@
---colour
+--color
+--require spec_helper

data/.ruby-gemset

@@ -0,0 +1 @@
+photocopier

data/.ruby-version

@@ -1 +1 @@
-2.0.0-p247
+2.1.2

data/.travis.yml

@@ -1,4 +1,4 @@
 language: ruby
 rvm:
-  - 2.0.0
-  - 1.9.3
+  - 2.1.2
+  - 2.2.1

data/CHANGELOG.md

@@ -0,0 +1,8 @@
+# 1.0.0.pre (2015-03-21)
+- semantic versioning
+- use ruby version 2.1.2
+- allow scheme (protocol) for ftp in options
+- fixed ftp password escaping
+- replace `escape` gem with `Shellwords`
+- public interface with less methods
+

data/README.md

@@ -44,7 +44,7 @@ ssh.put('./local_file.txt', 'remote_file.txt')
 # deletes a file
 ssh.delete('remote_file.txt')
 
-# mirros the remote directory into the local machine (needs rsync on the local machine)
+# mirros the remote directory content into the local machine (needs rsync on the local machine)
 ssh.get_directory('remote_dir', './local_dir')
 
 # and viceversa
@@ -57,14 +57,14 @@ The very same commands are valid for the `Photocopier::FTP` adapter.
 
 ## FTP
 
-`Photocopier::FTP.new` accepts the following parameters (you need to pass them
-all).
+`Photocopier::FTP.new` accepts the following parameters 
 
 ```ruby
 {
-  host: '',
-  user: '',
-  password: ''
+  host: '', #mandatory
+  user: '', #mandatory
+  password: '', #mandatory
+  scheme: 'ftp' #default, could be any scheme compatibile with your lftp version
 }
 ```
 For performance reasons, the `.get_directory` and `.put_directory` commands make
@@ -107,7 +107,7 @@ On Linux, you can install it with your standard package manager. On Mac, you can
 have it via [`brew`](https://github.com/mxcl/homebrew):
 
 ```
-sudo brew install https://raw.github.com/gist/1513663/3e98bf9e03feb7e31eeddcd08f89ca86163a376d/sshpass.rb
+brew install https://raw.github.com/eugeneoden/homebrew/eca9de1/Library/Formula/sshpass.rb
 ```
 
 **Please note that on Ubuntu 11.10 `sshpass` is at version 1.04, which has a

data/bin/console

@@ -0,0 +1,7 @@
+#!/usr/bin/env ruby
+
+require "bundler/setup"
+require "photocopier"
+require "pry-byebug"
+
+Pry.start

data/bin/rspec

@@ -0,0 +1,16 @@
+#!/usr/bin/env ruby
+#
+# This file was generated by Bundler.
+#
+# The application 'rspec' is installed as part of a gem, and
+# this file is here to facilitate running it.
+#
+
+require 'pathname'
+ENV['BUNDLE_GEMFILE'] ||= File.expand_path("../../Gemfile",
+  Pathname.new(__FILE__).realpath)
+
+require 'rubygems'
+require 'bundler/setup'
+
+load Gem.bin_path('rspec-core', 'rspec')

data/bin/setup

@@ -0,0 +1,7 @@
+#!/bin/bash
+set -euo pipefail
+IFS=$'\n\t'
+
+bundle install
+
+# Do any other automated setup that you need to do here

data/lib/photocopier/adapter.rb

@@ -1,8 +1,3 @@
-require 'escape'
-require 'tempfile'
-require 'logger'
-require 'active_support/all'
-
 module Photocopier
 
   class Adapter
@@ -30,8 +25,7 @@ module Photocopier
 
     protected
 
-    def run(*args)
-      command = Escape.shell_command(args)
+    def run(command)
       if logger.present?
         logger.info command
       end

data/lib/photocopier/ftp.rb

@@ -1,7 +1,3 @@
-require 'net/ftp'
-require 'photocopier/adapter'
-require 'fileutils'
-
 module Photocopier
   class FTP < Adapter
 
@@ -34,6 +30,8 @@ module Photocopier
       lftp(local_path, remote_path, true, exclude)
     end
 
+    private
+
     def session
       if @session.nil?
         @session = Net::FTP.open(options[:host], options[:user], options[:password])
@@ -42,26 +40,25 @@ module Photocopier
       @session
     end
 
-    private
-
     def lftp(local, remote, reverse, exclude)
-      run "lftp",
-          "-c",
-          [
-            "set ftp:list-options -a",
-            "open #{remote_ftp_url}",
-            "mkdir -p #{remote}",
-            "cd #{remote}",
-            "lcd #{local}",
-            lftp_mirror_arguments(reverse, exclude)
-          ].join("; ")
+      command = [
+          "set ftp:list-options -a",
+          "open #{remote_ftp_url}",
+          "mkdir -p #{Shellwords.escape(remote)}",
+          "cd #{Shellwords.escape(remote)}",
+          "lcd #{Shellwords.escape(local)}",
+          lftp_mirror_arguments(reverse, exclude)
+      ].join("; ")
+
+      run "lftp -c '#{command}'"
     end
 
     def remote_ftp_url
-      url = "ftp://"
+      url = options[:scheme].presence || "ftp"
+      url << "://"
       if options[:user].present?
-        url << options[:user]
-        url << ":#{options[:password]}" if options[:password].present?
+        url << CGI.escape(options[:user])
+        url << ":#{CGI.escape(options[:password])}" if options[:password].present?
         url << "@"
       end
       url << options[:host]

data/lib/photocopier/ssh.rb

@@ -1,10 +1,3 @@
-require 'net/ssh'
-require 'net/ssh/gateway'
-require 'net/scp'
-require 'fileutils'
-
-require 'photocopier/adapter'
-
 module Photocopier
   class SSH < Adapter
 
@@ -29,28 +22,18 @@ module Photocopier
     end
 
     def delete(remote_path)
-      exec!("rm -rf #{remote_path}")
+      exec!("rm -rf #{Shellwords.escape(remote_path)}")
     end
 
     def get_directory(remote_path, local_path, exclude = [])
       FileUtils.mkdir_p(local_path)
+      remote_path << "/" unless remote_path.end_with?("/")
       rsync ":#{remote_path}", local_path, exclude
     end
 
     def put_directory(local_path, remote_path, exclude = [])
-      rsync local_path, ":#{remote_path}", exclude
-    end
-
-    def session
-      opts = options
-      host = opts.delete(:host)
-      user = opts.delete(:user)
-
-      @session ||= if gateway_options
-                     gateway.ssh(host, user, opts)
-                   else
-                     Net::SSH.start(host, user, opts)
-                   end
+      local_path << "/" unless local_path.end_with?("/")
+      rsync "#{local_path}", ":#{remote_path}", exclude
     end
 
     def exec!(cmd)
@@ -74,31 +57,47 @@ module Photocopier
       [ stdout, stderr, exit_code ]
     end
 
+    private
+
+    def session
+      opts = options
+      host = opts.delete(:host)
+      user = opts.delete(:user)
+
+      @session ||= if gateway_options
+                     gateway.ssh(host, user, opts)
+                   else
+                     Net::SSH.start(host, user, opts)
+                   end
+    end
+
     def rsync_command
-      [
+      command = [
         "rsync",
         "--progress",
         "-e",
-        rsh_arguments,
+        "'#{rsh_arguments}'",
         "-rlpt",
         "--compress",
         "--omit-dir-times",
         "--delete",
-        rsync_options
-      ].compact
+      ]
+      command.concat Shellwords.split(rsync_options) if rsync_options
+      command.compact
     end
 
     def rsync(source, destination, exclude = [])
       command = rsync_command
 
-      exclude.map do |glob|
+      exclude.each do |glob|
         command << "--exclude"
         command << glob
       end
 
-      command << "#{source}/" << destination
+      command << Shellwords.escape(source)
+      command << Shellwords.escape(destination)
 
-      run *command
+      run command.join(" ")
     end
 
     def rsh_arguments
@@ -119,8 +118,6 @@ module Photocopier
       command
     end
 
-    private
-
     def gateway
       return unless gateway_options
 
@@ -130,6 +127,5 @@ module Photocopier
 
       @gateway ||= Net::SSH::Gateway.new(host, user, opts)
     end
-
   end
 end

data/lib/photocopier/version.rb

@@ -1,3 +1,3 @@
 module Photocopier
-  VERSION = "0.0.10"
+  VERSION = "1.0.0.pre"
 end

data/lib/photocopier.rb

@@ -1,4 +1,14 @@
-require "active_support/core_ext"
+require 'tempfile'
+require 'logger'
+require 'fileutils'
+require 'net/ftp'
+require 'net/ssh'
+require 'net/ssh/gateway'
+require 'net/scp'
+require 'fileutils'
+require 'shellwords'
+require 'active_support/all'
 
+require 'photocopier/adapter'
 require "photocopier/ssh"
 require "photocopier/ftp"

data/photocopier.gemspec

@@ -1,27 +1,34 @@
-# -*- encoding: utf-8 -*-
-require File.expand_path('../lib/photocopier/version', __FILE__)
+# coding: utf-8
+lib = File.expand_path('../lib', __FILE__)
+$LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
+require "photocopier/version"
 
-Gem::Specification.new do |gem|
-  gem.authors       = ["Stefano Verna", "Ju Liu"]
-  gem.email         = ["stefano.verna@welaika.com", "ju.liu@welaika.com"]
-  gem.description   = %q{Photocopier provides FTP/SSH adapters to abstract away file and directory copying.}
-  gem.summary       = %q{Photocopier provides FTP/SSH adapters to abstract away file and directory copying.}
-  gem.homepage      = "https://github.com/welaika/photocopier"
+Gem::Specification.new do |spec|
+  spec.name          = "photocopier"
+  spec.version       = Photocopier::VERSION
+  spec.authors       = ["Stefano Verna", "Ju Liu", "Fabrizio Monti"]
+  spec.email         = ["stefano.verna@welaika.com", "ju.liu@welaika.com", "fabrizio.monti@welaika.com"]
 
-  gem.files         = `git ls-files`.split($\)
-  gem.executables   = gem.files.grep(%r{^bin/}).map{ |f| File.basename(f) }
-  gem.test_files    = gem.files.grep(%r{^(test|spec|features)/})
-  gem.name          = "photocopier"
-  gem.require_paths = ["lib"]
-  gem.version       = Photocopier::VERSION
+  spec.summary       = %q{Photocopier provides FTP/SSH adapters to abstract away file and directory copying.}
+  spec.description   = %q{Photocopier provides FTP/SSH adapters to abstract away file and directory copying.}
+  spec.homepage      = "https://github.com/welaika/photocopier"
+  spec.license       = "MIT"
 
-  gem.add_dependency "activesupport"
-  gem.add_dependency "i18n"
-  gem.add_dependency "net-ssh"
-  gem.add_dependency "net-scp"
-  gem.add_dependency "net-ssh-gateway"
-  gem.add_dependency "escape"
+  spec.files         = `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
+  spec.bindir        = "exe"
+  spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
+  spec.require_paths = ["lib"]
 
-  gem.add_development_dependency "rake"
-  gem.add_development_dependency "rspec"
+  spec.required_ruby_version = ">= 2.1.2"
+
+  spec.add_dependency "activesupport", "~> 4.2.0"
+  spec.add_dependency "net-ssh", "~> 2.9.2"
+  spec.add_dependency "net-scp", "~> 1.2.1"
+  spec.add_dependency "net-ssh-gateway", "~> 1.2.0"
+  spec.add_dependency "escape", "~> 0.0.4"
+
+  spec.add_development_dependency "bundler", ">= 1.6.2"
+  spec.add_development_dependency "rake", "~> 10.0"
+  spec.add_development_dependency "rspec", "~> 3.2"
+  spec.add_development_dependency "pry-byebug", "~> 3.0.1"
 end

metadata

@@ -1,159 +1,175 @@
 --- !ruby/object:Gem::Specification
 name: photocopier
 version: !ruby/object:Gem::Version
-  version: 0.0.10
+  version: 1.0.0.pre
 platform: ruby
 authors:
 - Stefano Verna
 - Ju Liu
+- Fabrizio Monti
 autorequire: 
-bindir: bin
+bindir: exe
 cert_chain: []
-date: 2013-08-30 00:00:00.000000000 Z
+date: 2015-03-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 4.2.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 4.2.0
 - !ruby/object:Gem::Dependency
-  name: i18n
+  name: net-ssh
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 2.9.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 2.9.2
 - !ruby/object:Gem::Dependency
-  name: net-ssh
+  name: net-scp
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 1.2.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 1.2.1
 - !ruby/object:Gem::Dependency
-  name: net-scp
+  name: net-ssh-gateway
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 1.2.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 1.2.0
 - !ruby/object:Gem::Dependency
-  name: net-ssh-gateway
+  name: escape
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 0.0.4
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 0.0.4
 - !ruby/object:Gem::Dependency
-  name: escape
+  name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
-  type: :runtime
+        version: 1.6.2
+  type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 1.6.2
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '10.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '10.0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.2'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.2'
+- !ruby/object:Gem::Dependency
+  name: pry-byebug
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 3.0.1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 3.0.1
 description: Photocopier provides FTP/SSH adapters to abstract away file and directory
   copying.
 email:
 - stefano.verna@welaika.com
 - ju.liu@welaika.com
+- fabrizio.monti@welaika.com
 executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- .gitignore
-- .rspec
-- .ruby-version
-- .rvmrc
-- .travis.yml
+- ".gitignore"
+- ".rspec"
+- ".ruby-gemset"
+- ".ruby-version"
+- ".travis.yml"
+- CHANGELOG.md
 - Gemfile
-- Gemfile.lock
 - LICENSE
 - README.md
 - Rakefile
+- bin/console
+- bin/rspec
+- bin/setup
 - lib/photocopier.rb
 - lib/photocopier/adapter.rb
 - lib/photocopier/ftp.rb
 - lib/photocopier/ssh.rb
 - lib/photocopier/version.rb
 - photocopier.gemspec
-- spec/adapter_spec.rb
-- spec/ftp_spec.rb
-- spec/shared_examples_for_adapter.rb
-- spec/ssh_spec.rb
 homepage: https://github.com/welaika/photocopier
-licenses: []
+licenses:
+- MIT
 metadata: {}
 post_install_message: 
 rdoc_options: []
@@ -161,23 +177,19 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 2.1.2
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">"
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.3.1
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.0.3
+rubygems_version: 2.4.5
 signing_key: 
 specification_version: 4
 summary: Photocopier provides FTP/SSH adapters to abstract away file and directory
   copying.
-test_files:
-- spec/adapter_spec.rb
-- spec/ftp_spec.rb
-- spec/shared_examples_for_adapter.rb
-- spec/ssh_spec.rb
+test_files: []

data/.rvmrc

@@ -1 +0,0 @@
-rvm use 1.9.3@photocopier

data/Gemfile.lock

@@ -1,51 +0,0 @@
-PATH
-  remote: .
-  specs:
-    photocopier (0.0.10)
-      activesupport
-      escape
-      i18n
-      net-scp
-      net-ssh
-      net-ssh-gateway
-
-GEM
-  remote: https://rubygems.org/
-  specs:
-    activesupport (4.0.0)
-      i18n (~> 0.6, >= 0.6.4)
-      minitest (~> 4.2)
-      multi_json (~> 1.3)
-      thread_safe (~> 0.1)
-      tzinfo (~> 0.3.37)
-    atomic (1.1.13)
-    diff-lcs (1.1.3)
-    escape (0.0.4)
-    i18n (0.6.5)
-    minitest (4.7.5)
-    multi_json (1.7.9)
-    net-scp (1.1.2)
-      net-ssh (>= 2.6.5)
-    net-ssh (2.6.8)
-    net-ssh-gateway (1.2.0)
-      net-ssh (>= 2.6.5)
-    rake (10.1.0)
-    rspec (2.11.0)
-      rspec-core (~> 2.11.0)
-      rspec-expectations (~> 2.11.0)
-      rspec-mocks (~> 2.11.0)
-    rspec-core (2.11.1)
-    rspec-expectations (2.11.3)
-      diff-lcs (~> 1.1.3)
-    rspec-mocks (2.11.3)
-    thread_safe (0.1.2)
-      atomic
-    tzinfo (0.3.37)
-
-PLATFORMS
-  ruby
-
-DEPENDENCIES
-  photocopier!
-  rake
-  rspec

data/spec/adapter_spec.rb

@@ -1,6 +0,0 @@
-require 'photocopier/adapter'
-require 'shared_examples_for_adapter'
-
-describe Photocopier::Adapter do
-  it_behaves_like "a Photocopier adapter"
-end

data/spec/ftp_spec.rb

@@ -1,163 +0,0 @@
-require 'photocopier/ftp'
-require 'shared_examples_for_adapter'
-
-describe Photocopier::FTP do
-  it_behaves_like "a Photocopier adapter"
-
-  let(:ftp) { Photocopier::FTP.new(options) }
-  let(:options) do { host: "host", user: "user", password: "password" } end
-
-  context "#session" do
-    it "retrieves an FTP session" do
-      Net::FTP.should_receive(:open).with("host", "user", "password")
-      ftp.session
-    end
-
-    context "passive mode" do
-      let(:options) do { host: "host", passive: true } end
-      let(:ftp) { double('ftp').as_null_object }
-
-      it "should enable passive mode" do
-        Net::FTP.stub(:open).and_return(ftp)
-        ftp.session.should be_passive
-      end
-    end
-  end
-
-  context "#remote_ftp_url" do
-    let(:options) do { host: "host" } end
-
-    it "should build an ftp url" do
-      ftp.send(:remote_ftp_url).should == "ftp://host"
-    end
-
-    context "given an username" do
-      let(:options) do { host: "host", user: "user" } end
-
-      it "should add it to the url" do
-        ftp.send(:remote_ftp_url).should == "ftp://user@host"
-      end
-
-      context "given a password" do
-        let(:options) do { host: "host", user: "user", password: "password" } end
-
-        it "should add it to the url" do
-          ftp.send(:remote_ftp_url).should == "ftp://user:password@host"
-        end
-      end
-    end
-  end
-
-  context "#lftp_mirror_arguments" do
-    let(:lftp_arguments) do
-      %w(
-        mirror
-        --delete
-        --use-cache
-        --verbose
-        --allow-chown
-        --allow-suid
-        --no-umask
-        --parallel=2
-      )
-    end
-
-    it "should build arguments for lftp" do
-      ftp.send(:lftp_mirror_arguments, false, []).should == lftp_arguments.join(" ")
-    end
-
-    it "should build args for reverse mirroring" do
-      lftp_arguments << "--reverse"
-      ftp.send(:lftp_mirror_arguments, true, []).should == lftp_arguments.join(" ")
-    end
-
-    it "should exclude files" do
-      lftp_arguments << "--exclude-glob .git"
-      ftp.send(:lftp_mirror_arguments, false, [".git"]).should == lftp_arguments.join(" ")
-    end
-  end
-
-  context "#lftp" do
-
-    let(:remote) { "remote" }
-    let(:local) { "local" }
-
-    before(:each) do
-      ftp.stub(:remote_ftp_url).and_return("remote_ftp_url")
-      ftp.stub(:lftp_mirror_arguments).and_return("lftp_mirror_arguments")
-    end
-
-    let(:lftp_command) do
-      [
-        "lftp",
-        "-c",
-        [
-          "set ftp:list-options -a",
-          "open remote_ftp_url",
-          "mkdir -p #{remote}",
-          "cd #{remote}",
-          "lcd #{local}",
-          "lftp_mirror_arguments"
-        ].join("; ")
-      ]
-    end
-
-    it "should build a lftp command" do
-      ftp.should_receive(:run).with(*lftp_command)
-      ftp.send(:lftp, local, remote, false, [])
-    end
-  end
-
-  context "adapter interface" do
-
-    let(:remote_path) { double }
-    let(:local_path)  { double }
-    let(:file_path)   { double }
-    let(:session)     { double }
-
-    before(:each) do
-      ftp.stub(:session).and_return(session)
-    end
-
-    context "#get" do
-      it "should get a remote path" do
-        session.should_receive(:get).with(remote_path, file_path)
-        ftp.get(remote_path, file_path)
-      end
-    end
-
-    context "#put_file" do
-      it "should send a file to remote" do
-        session.should_receive(:put).with(file_path, remote_path)
-        ftp.put_file(file_path, remote_path)
-      end
-    end
-
-    context "#delete" do
-      it "should delete a remote path" do
-        session.should_receive(:delete).with(remote_path)
-        ftp.delete(remote_path)
-      end
-    end
-
-    context "directories management" do
-      let(:remote_path) { "remote_path" }
-      let(:exclude_list) { [] }
-
-      context "#get_directory" do
-        it "should get a remote directory" do
-          FileUtils.should_receive(:mkdir_p).with(local_path)
-          ftp.should_receive(:lftp).with(local_path, remote_path, false, exclude_list)
-          ftp.get_directory(remote_path, local_path, exclude_list)
-        end
-      end
-
-      context "#put_directory" do
-        it "should send a directory to remote" do
-          ftp.should_receive(:lftp).with(local_path, remote_path, true, exclude_list)
-          ftp.put_directory(local_path, remote_path, exclude_list)
-        end
-      end
-    end
-  end
-end

data/spec/shared_examples_for_adapter.rb

@@ -1,71 +0,0 @@
-require 'photocopier/adapter'
-
-shared_examples_for "a Photocopier adapter" do
-  let(:adapter) { described_class.new }
-
-  context "adapter interface" do
-    it "should expose all adapter methods" do
-      [
-        :put,
-        :put_file,
-        :put_directory,
-        :get,
-        :get_directory,
-        :delete
-      ].each do |sym|
-        adapter.should respond_to(sym)
-      end
-    end
-  end
-
-  context "#put" do
-    let(:file_path) { Tempfile.new("tmp").path }
-    let(:remote_path) { double }
-
-    context "given a real file path" do
-      it "should put a file" do
-        adapter.should_receive(:put_file).with(file_path, remote_path)
-        adapter.put(file_path, remote_path)
-      end
-    end
-
-    context "given a string" do
-      let(:string) { "foobar" }
-      let(:file) { double(path: "path") }
-
-      it "should write it to file, put it and remove the file" do
-        Tempfile.stub(:new).and_return(file)
-
-        file.should_receive(:write).with(string)
-        file.should_receive(:close)
-        adapter.should_receive(:put_file).with("path", remote_path)
-        file.should_receive(:unlink)
-
-        adapter.put(string, remote_path)
-      end
-    end
-
-    context "#run" do
-      let(:arg) { double }
-      let(:command) { double }
-      before(:each) {
-        Escape.stub(:shell_command).with([arg]).and_return(command)
-      }
-
-      it "should delegate to Kernel system" do
-        adapter.should_receive(:system).with(command)
-        adapter.send(:run, arg)
-      end
-
-      context "given a logger" do
-        let(:logger) { double }
-
-        it "should send the command to the logger" do
-          adapter.logger = logger
-          logger.should_receive(:info).with(command)
-          adapter.send(:run, arg)
-        end
-      end
-    end
-  end
-end

data/spec/ssh_spec.rb

@@ -1,161 +0,0 @@
-require 'photocopier/ssh'
-require 'shared_examples_for_adapter'
-
-describe Photocopier::SSH do
-  it_behaves_like "a Photocopier adapter"
-
-  let(:ssh) { Photocopier::SSH.new(options) }
-  let(:options) do { host: "host", user: "user" } end
-  let(:gateway_config) do { host: "gate_host", user: "gate_user" } end
-  let(:options_with_gateway) do
-    {
-      host: "host",
-      user: "user",
-      gateway: gateway_config
-    }
-  end
-
-  context "#session" do
-    it "retrieves an SSH session" do
-      Net::SSH.should_receive(:start).with("host", "user", {})
-      ssh.session
-    end
-
-    context "given a gateway " do
-      let(:options) { options_with_gateway }
-      let(:gateway) { double }
-
-      it "goes through it to retrieve a session" do
-        Net::SSH::Gateway.stub(:new).with("gate_host", "gate_user", {}).and_return(gateway)
-        gateway.should_receive(:ssh).with("host", "user", {})
-        ssh.session
-      end
-    end
-  end
-
-  context "#ssh_command" do
-    let(:options) do { host: "host" } end
-
-    it "should build an ssh command" do
-      ssh.ssh_command(options).should == "ssh host"
-    end
-
-    context "given a port" do
-      let(:options) do { host: "host", port: "port" } end
-      it "should be added to the command" do
-        ssh.ssh_command(options).should == "ssh -p port host"
-      end
-    end
-
-    context "given a user" do
-      let(:options) do { host: "host", user: "user" } end
-      it "should be added to the command" do
-        ssh.ssh_command(options).should == "ssh user@host"
-      end
-    end
-
-    context "given a password" do
-      let(:options) do { host: "host", password: "password" } end
-
-      it "sshpass should be added to the command" do
-        ssh.ssh_command(options).should == "sshpass -p password ssh host"
-      end
-    end
-  end
-
-  context "#rsh_arguments" do
-    it "should build arguments for rsync" do
-      ssh.should_receive(:ssh_command).with(options)
-      ssh.rsh_arguments
-    end
-
-    context "given a gateway" do
-      let(:options) { options_with_gateway }
-
-      it "should include gateway options" do
-        ssh.should_receive(:ssh_command).with(gateway_config)
-        ssh.should_receive(:ssh_command).with(options)
-        ssh.rsh_arguments
-      end
-    end
-  end
-
-  context "#rsync" do
-    before(:each) do
-      ssh.stub(:rsh_arguments).and_return("rsh_arguments")
-      ssh.stub(:rsync_options).and_return("rsync_options")
-    end
-
-    let(:rsync_command) { ssh.rsync_command }
-
-    it "should build an rsync command" do
-      rsync_command << "source/" << "destination"
-      ssh.should_receive(:run).with(*rsync_command)
-      ssh.rsync("source", "destination")
-    end
-
-    context "given an exclude list" do
-      it "should skip excluded paths" do
-        rsync_command << "--exclude" << ".git"
-        rsync_command << "source/" << "destination"
-        ssh.should_receive(:run).with(*rsync_command)
-        ssh.rsync("source", "destination", [".git"])
-      end
-    end
-  end
-
-  context "adapter interface" do
-
-    let(:remote_path) { double }
-    let(:local_path)  { double }
-    let(:file_path)   { double }
-    let(:scp)         { double }
-    let(:session)     { double(scp: scp) }
-
-    before(:each) do
-      ssh.stub(:session).and_return(session)
-    end
-
-    context "#get" do
-      it "should get a remote path" do
-        scp.should_receive(:download!).with(remote_path, file_path)
-        ssh.get(remote_path, file_path)
-      end
-    end
-
-    context "#put_file" do
-      it "should send a file to remote" do
-        scp.should_receive(:upload!).with(file_path, remote_path)
-        ssh.put_file(file_path, remote_path)
-      end
-    end
-
-    context "#delete" do
-      it "should delete a remote path" do
-        ssh.should_receive(:exec!).with("rm -rf foo")
-        ssh.delete("foo")
-      end
-    end
-
-    context "directories management" do
-      let(:remote_path) { "remote_path" }
-      let(:exclude_list) { [] }
-
-      context "#get_directory" do
-        it "should get a remote directory" do
-          FileUtils.should_receive(:mkdir_p).with(local_path)
-          ssh.should_receive(:rsync).with(":remote_path", local_path, exclude_list)
-          ssh.get_directory(remote_path, local_path, exclude_list)
-        end
-      end
-
-      context "#put_directory" do
-        it "should send a directory to remote" do
-          ssh.should_receive(:rsync).with(local_path, ":remote_path", exclude_list)
-          ssh.put_directory(local_path, remote_path, exclude_list)
-        end
-      end
-    end
-  end
-
-end