fake_ftp 0.0.9 → 0.1.0

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA1:
+  metadata.gz: ef1324d66ffeb222bc35a3d1fe172bf649fefa9e
+  data.tar.gz: 285616254bf9b2f5ad441bfc41d17943a1609681
+SHA512:
+  metadata.gz: b20a024c9e598db59fac9751517b08cc70c5325f0ba55b96352d719a0227827a18a956ef2e30e85f9c544009c4214f7285bf3d69a0139e66bdd3e0bed207c711
+  data.tar.gz: ecd74f559bf7d48318846671268c5b3d96468c93a3ad6fff487f310014b47e413244e55f700e8881c54565c4dd066d2d0c0573322aba70eb57d75d33f3dc1d16

data/.travis.yml

@@ -0,0 +1,5 @@
+language: ruby
+rvm:
+  - 1.8.7
+gemfile:
+  - Gemfile.travis

data/Gemfile

@@ -2,3 +2,11 @@ source "http://rubygems.org"
 
 # Specify your gem's dependencies in fake_ftp.gemspec
 gemspec
+
+group :test do
+  gem "rspec", '>2'
+  gem "rake", '>=0.8.7'
+
+  gem "guard-rspec"
+  gem 'pry-nav'
+end

data/Gemfile.lock

@@ -1,27 +1,44 @@
 PATH
   remote: .
   specs:
-    fake_ftp (0.0.8)
+    fake_ftp (0.1.0)
 
 GEM
   remote: http://rubygems.org/
   specs:
-    diff-lcs (1.1.3)
+    coderay (1.0.9)
+    diff-lcs (1.2.4)
+    ffi (1.0.11)
+    guard (1.0.1)
+      ffi (>= 0.5.0)
+      thor (~> 0.14.6)
+    guard-rspec (0.6.0)
+      guard (>= 0.10.0)
+    method_source (0.8.2)
+    pry (0.9.12.2)
+      coderay (~> 1.0.5)
+      method_source (~> 0.8)
+      slop (~> 3.4)
+    pry-nav (0.2.3)
+      pry (~> 0.9.10)
     rake (0.9.2.2)
-    rspec (2.7.0)
-      rspec-core (~> 2.7.0)
-      rspec-expectations (~> 2.7.0)
-      rspec-mocks (~> 2.7.0)
-    rspec-core (2.7.1)
-    rspec-expectations (2.7.0)
-      diff-lcs (~> 1.1.2)
-    rspec-mocks (2.7.0)
+    rspec (2.14.1)
+      rspec-core (~> 2.14.0)
+      rspec-expectations (~> 2.14.0)
+      rspec-mocks (~> 2.14.0)
+    rspec-core (2.14.5)
+    rspec-expectations (2.14.3)
+      diff-lcs (>= 1.1.3, < 2.0)
+    rspec-mocks (2.14.3)
+    slop (3.4.6)
+    thor (0.14.6)
 
 PLATFORMS
   ruby
 
 DEPENDENCIES
-  bundler (>= 1.0.10)
   fake_ftp!
+  guard-rspec
+  pry-nav
   rake (>= 0.8.7)
   rspec (> 2)

data/Gemfile.travis

@@ -0,0 +1,5 @@
+source "http://rubygems.org"
+
+# Specify your gem's dependencies in fake_ftp.gemspec
+gemspec
+

data/Guardfile

@@ -0,0 +1,10 @@
+# A sample Guardfile
+# More info at https://github.com/guard/guard#readme
+
+guard 'rspec', :version => 2 do
+  watch(%r{^spec/.+_spec\.rb$})
+  watch(%r{^lib/(.+)\.rb$})     { |m| "spec/lib/#{m[1]}_spec.rb" }
+  watch('spec/spec_helper.rb')  { "spec" }
+  watch(%r{^spec/support/(.+)\.rb$})                  { "spec" }
+end
+

data/README.markdown

@@ -1,85 +1,83 @@
 FakeFtp
 =======
 
+[![Build status](https://secure.travis-ci.org/livinginthepast/fake_ftp.png)](http://travis-ci.org/livinginthepast/fake_ftp)
+
 This is a gem that allows you to test FTP implementations in ruby. It is a minimal single-client FTP server
 that can be bound to any arbitrary port on localhost.
 
-Why?
-----
+## Why?
 
 We want to ensure that our code works, in a way that is agnostic to the implementation used (unlike with stubs or mocks).
 
-How
----
+## How
 
 FakeFtp is a simple FTP server that fakes out enough of the protocol to get us by, allowing us to test that files get to
 their intended destination rather than testing how our code does so.
 
-Usage
------
+## Usage
 
 To test passive upload:
-
-    require 'fake_ftp'
-    require 'net/ftp'
-
-    server = FakeFtp::Server.new(21212, 21213)
-    ## 21212 is the control port, which is used by FTP for the primary connection
-    ## 21213 is the data port, used in FTP passive mode to send file contents
-    server.start
-
-    ftp = Net::FTP.new
-    ftp.connect('127.0.0.1', 21212)
-    ftp.login('user', 'password')
-    ftp.passive = true
-    ftp.put('some_file.txt')
-    ftp.close
-
-    server.files.should include('some_file.txt')
-    server.file('some_file.txt').bytes.should == 25
-    server.file('some_file.txt').should be_passive
-    server.file('some_file.txt').should_not be_active
-
-    server.stop
+``` ruby
+require 'fake_ftp'
+require 'net/ftp'
+
+server = FakeFtp::Server.new(21212, 21213)
+## 21212 is the control port, which is used by FTP for the primary connection
+## 21213 is the data port, used in FTP passive mode to send file contents
+server.start
+
+ftp = Net::FTP.new
+ftp.connect('127.0.0.1', 21212)
+ftp.login('user', 'password')
+ftp.passive = true
+ftp.put('some_file.txt')
+ftp.close
+
+server.files.should include('some_file.txt')
+server.file('some_file.txt').bytes.should == 25
+server.file('some_file.txt').should be_passive
+server.file('some_file.txt').should_not be_active
+
+server.stop
+```
 
 To test active upload:
-
-    server = FakeFtp::Server.new(21212)
-    ## 21212 is the control port, which is used by FTP for the primary connection
-    ## 21213 is the data port, used in FTP passive mode to send file contents
-    server.start
-
-    ftp = Net::FTP.new
-    ftp.connect('127.0.0.1', 21212)
-    ftp.login('user', 'password')
-    ftp.passive = false
-    ftp.put('some_file.txt')
-    ftp.close
-
-    server.files.should include('some_file.txt')
-    server.file('some_file.txt').bytes.should == 25
-    server.file('some_file.txt').should be_active
-    server.file('some_file.txt').should_not be_passive
-
-    server.stop
+``` ruby
+server = FakeFtp::Server.new(21212)
+## 21212 is the control port, which is used by FTP for the primary connection
+## 21213 is the data port, used in FTP passive mode to send file contents
+server.start
+
+ftp = Net::FTP.new
+ftp.connect('127.0.0.1', 21212)
+ftp.login('user', 'password')
+ftp.passive = false
+ftp.put('some_file.txt')
+ftp.close
+
+server.files.should include('some_file.txt')
+server.file('some_file.txt').bytes.should == 25
+server.file('some_file.txt').should be_active
+server.file('some_file.txt').should_not be_passive
+
+server.stop
+```
 
 Note that many FTP clients default to active, unless specified otherwise.
 
-References
-----------
+## References
 
 * http://rubyforge.org/projects/ftpd/ - a simple ftp daemon written by Chris Wanstrath
 * http://ruby-doc.org/stdlib/libdoc/gserver/rdoc/index.html - a generic server in the Ruby standard library, by John W Small
 
-Contributors
-------------
+## Contributors
 
 * Eric Saxby
 * Colin Shield
 * liehann (https://github.com/liehann)
 
-License
--------
+## License
 
 The MIT License
 

data/fake_ftp.gemspec

@@ -13,12 +13,7 @@ Gem::Specification.new do |s|
   s.description = %q{Testing FTP? Use this!}
 
   s.required_rubygems_version = ">= 1.3.6"
-  s.rubyforge_project = "fake_ftp"
   
-  s.add_development_dependency "bundler", '>=1.0.10'
-  s.add_development_dependency "rspec", '>2'
-  s.add_development_dependency "rake", '>=0.8.7'
-
   s.files         = `git ls-files`.split("\n")
   s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
   s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }

data/lib/fake_ftp/file.rb

@@ -1,18 +1,21 @@
 module FakeFtp
   class File
-    attr_accessor :bytes, :name
+    attr_accessor :bytes, :name, :last_modified_time
     attr_writer :type
     attr_accessor :data
     attr_reader :created
 
-    def initialize(name = nil, data = nil, type = nil)
+    def initialize(name = nil, data = nil, type = nil, last_modified_time = Time.now)
       @created = Time.now
       @name = name
       @data = data
+      # FIXME this is far too ambiguous. args should not mean different
+      # things in different contexts.
       data_is_bytes = (data.nil? || Integer === data)
-      @bytes = data_is_bytes ? data : data.length
+      @bytes = data_is_bytes ? data : data.to_s.length
       @data = data_is_bytes ? nil : data
       @type = type
+      @last_modified_time = last_modified_time.utc
     end
 
     def data=(data)

data/lib/fake_ftp/server.rb

@@ -6,9 +6,29 @@ module FakeFtp
   class Server
 
     attr_accessor :port, :passive_port
-    attr_reader :mode
-
-    CMDS = %w[acct cwd cdup list nlst pass pasv port pwd quit stor retr type user]
+    attr_reader :mode, :path
+
+    CMDS = %w(
+      acct
+      cwd
+      cdup
+      dele
+      list
+      mdtm
+      mkd
+      nlst
+      pass
+      pasv
+      port
+      pwd
+      quit
+      stor
+      retr
+      rnfr
+      rnto
+      type
+      user
+    )
     LNBK = "\r\n"
 
     def initialize(control_port = 21, data_port = nil, options = {})
@@ -20,6 +40,7 @@ module FakeFtp
       @options = options
       @files = []
       @mode = :active
+      @path = "/pub"
     end
 
     def files
@@ -34,8 +55,8 @@ module FakeFtp
       @files.clear
     end
 
-    def add_file(filename, data)
-      @files << FakeFtp::File.new(::File.basename(filename.to_s), data, @mode)
+    def add_file(filename, data, last_modified_time = Time.now)
+      @files << FakeFtp::File.new(::File.basename(filename.to_s), data, @mode, last_modified_time)
     end
 
     def start
@@ -43,18 +64,25 @@ module FakeFtp
       @server = ::TCPServer.new('127.0.0.1', port)
       @thread = Thread.new do
         while @started
-          @client = @server.accept
-          respond_with('220 Can has FTP?')
-          @connection = Thread.new(@client) do |socket|
-            while @started && !socket.nil? && !socket.closed?
-              respond_with parse(socket.gets)
+          @client = @server.accept rescue nil
+          if @client
+            respond_with('220 Can has FTP?')
+            @connection = Thread.new(@client) do |socket|
+              while @started && !socket.nil? && !socket.closed?
+                input = socket.gets rescue nil
+                respond_with parse(input) if input
+              end
+              unless @client.nil?
+                @client.close unless @client.closed?
+                @client = nil
+              end
             end
-            @client.close
-            @client = nil
           end
         end
-        @server.close
-        @server = nil
+        unless @server.nil?
+          @server.close unless @server.closed?
+          @server = nil
+        end
       end
 
       if passive_port
@@ -88,13 +116,14 @@ module FakeFtp
       contents = request.split
       message = contents[1..contents.length]
       case command
-        when *CMDS
-          __send__ "_#{command}", *message
-        else
-          '500 Unknown command'
+      when *CMDS
+        __send__ "_#{command}", *message
+      else
+        '500 Unknown command'
       end
     end
 
+
     ## FTP commands
     #
     #  Methods are prefixed with an underscore to avoid conflicts with internal server
@@ -105,25 +134,51 @@ module FakeFtp
     end
 
     def _cwd(*args)
+      @path = args[0]
+      @path = "/#{path}" if path[0].chr != "/"
+      '250 OK!'
+    end
+
+    def _cdup(*args)
       '250 OK!'
     end
-    alias :_cdup :_cwd
 
     def _list(*args)
+      wildcards = []
+      args.each do |arg|
+        next unless arg.include? '*'
+        wildcards << arg.gsub('*', '.*')
+      end
+
       respond_with('425 Ain\'t no data port!') && return if active? && @active_connection.nil?
 
       respond_with('150 Listing status ok, about to open data connection')
       data_client = active? ? @active_connection : @data_server.accept
 
-      data_client.write(@files.map do |f|
+      files = @files
+      if not wildcards.empty?
+        files = files.select do |f|
+          wildcards.any? { |wildcard| f.name =~ /#{wildcard}/ }
+        end
+      end
+      files = files.map do |f|
         "-rw-r--r--\t1\towner\tgroup\t#{f.bytes}\t#{f.created.strftime('%b %d %H:%M')}\t#{f.name}"
-      end.join("\n"))
+      end
+      data_client.write(files.join("\n"))
       data_client.close
       @active_connection = nil
 
       '226 List information transferred'
     end
 
+    def _mdtm(filename = '', local = false)
+      respond_with('501 No filename given') && return if filename.empty?
+      server_file = file(filename)
+      respond_with('550 File not found') && return if server_file.nil?
+
+      respond_with("213 #{server_file.last_modified_time.strftime("%Y%m%d%H%M%S")}")
+    end
+
     def _nlst(*args)
       respond_with('425 Ain\'t no data port!') && return if active? && @active_connection.nil?
 
@@ -165,7 +220,7 @@ module FakeFtp
     end
 
     def _pwd(*args)
-      "257 \"/pub\" is current directory"
+      "257 \"#{path}\" is current directory"
     end
 
     def _quit(*args)
@@ -192,13 +247,35 @@ module FakeFtp
       '226 File transferred'
     end
 
+    def _rnfr(rename_from='')
+      return '501 Send path name.' if rename_from.nil? || rename_from.size < 1
+
+      @rename_from = rename_from
+      '350 Send RNTO to complete rename.'
+    end
+
+    def _rnto(rename_to='')
+      return '501 Send path name.' if rename_to.nil? || rename_to.size < 1
+
+      return '503 Send RNFR first.' unless @rename_from
+
+      if file = file(@rename_from)
+        file.name = rename_to
+        @rename_from = nil
+        '250 Path renamed.'
+      else
+        @rename_from = nil
+        '550 File not found.'
+      end
+    end
+
     def _stor(filename = '')
       respond_with('425 Ain\'t no data port!') && return if active? && @active_connection.nil?
 
       respond_with('125 Do it!')
       data_client = active? ? @active_connection : @data_server.accept
 
-      data = data_client.recv(1024)
+      data = data_client.read(nil).chomp
       file = FakeFtp::File.new(::File.basename(filename.to_s), data, @mode)
       @files << file
 
@@ -207,14 +284,23 @@ module FakeFtp
       '226 Did it!'
     end
 
+    def _dele(filename = '')
+      files_to_delete = @files.select{ |file| file.name == filename }
+      return '550 Delete operation failed.' if files_to_delete.count == 0
+
+      @files = @files - files_to_delete
+
+      '250 Delete operation successful.'
+    end
+
     def _type(type = 'A')
       case type.to_s
-        when 'A'
-          '200 Type set to A.'
-        when 'I'
-          '200 Type set to I.'
-        else
-          '504 We don\'t allow those'
+      when 'A'
+        '200 Type set to A.'
+      when 'I'
+        '200 Type set to I.'
+      else
+        '504 We don\'t allow those'
       end
     end
 
@@ -222,6 +308,10 @@ module FakeFtp
       (name.to_s == 'anonymous') ? '230 logged in' : '331 send your password'
     end
 
+    def _mkd(directory)
+      "257 OK!"
+    end
+
     def active?
       @mode == :active
     end