fake_ftp 0.0.1 → 0.0.2

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    fake_ftp (0.0.1)
+    fake_ftp (0.0.2)
 
 GEM
   remote: http://rubygems.org/

data/README.markdown

@@ -35,13 +35,15 @@ Usage
     ftp.put('some_file.txt')
     ftp.close
 
+    server.files.should include('some_file.txt')
+    server.file('some_file.txt').bytes.should == 25
+
     server.stop
 
 TODO
 ----
 
 * Active file upload
-* Matchers
 
 References
 ----------

data/lib/fake_ftp/file.rb

@@ -0,0 +1,10 @@
+module FakeFtp
+  class File
+    attr_accessor :bytes, :name
+
+    def initialize(name = nil, bytes = nil)
+      self.name = name
+      self.bytes = bytes
+    end
+  end
+end

data/lib/fake_ftp/server.rb

@@ -1,11 +1,10 @@
 require 'socket'
-require "thread"
+require 'thread'
 
 module FakeFtp
   class Server
 
-    attr_accessor :directory, :port, :passive_port
-    attr_reader :files
+    attr_accessor :port, :passive_port
 
     CMDS = %w[acct cwd cdup pass pasv port pwd quit stor type user]
     LNBK = "\r\n"
@@ -16,10 +15,16 @@ module FakeFtp
       raise(Errno::EADDRINUSE, "#{port}") if is_running?
       raise(Errno::EADDRINUSE, "#{passive_port}") if passive_port && is_running?(passive_port)
       @connection = nil
-      @data = nil
       @options = options
       @files = []
-      self.directory = "#{Rails.root}/tmp/ftp" rescue '/tmp'
+    end
+
+    def files
+      @files.map(&:name)
+    end
+
+    def file(name)
+      @files.detect { |file| file.name == name}
     end
 
     def start
@@ -117,7 +122,7 @@ module FakeFtp
     end
 
     def _pwd(*args)
-      "257 \"#{self.directory}\" is current directory"
+      "257 \"/pub\" is current directory"
     end
 
     def _quit(*args)
@@ -127,11 +132,13 @@ module FakeFtp
     end
 
     def _stor(filename)
-      @files << File.basename(filename.to_s)
       respond_with('125 Do it!')
 
       data_client = @data_server.accept
-      @data = data_client.recv(1024)
+      data = data_client.recv(1024)
+
+      file = FakeFtp::File.new(::File.basename(filename.to_s), data.length)
+      @files << file
 
       data_client.close
       '226 Did it!'

data/lib/fake_ftp/version.rb

@@ -1,3 +1,3 @@
 module FakeFtp
-  VERSION = "0.0.1"
+  VERSION = "0.0.2"
 end

data/lib/fake_ftp.rb

@@ -1,4 +1,5 @@
 require 'fake_ftp/server'
+require 'fake_ftp/file'
 
 module FakeFtp
 end

data/spec/models/fake_ftp/file_spec.rb

@@ -0,0 +1,27 @@
+require "spec_helper.rb"
+
+describe FakeFtp::File do
+  context 'attributes' do
+    before :each do
+      @file = FakeFtp::File.new
+    end
+
+    it "has a name attribute" do
+      @file.name = "some name"
+      @file.name.should == "some name"
+    end
+
+    it "has a bytes attribute" do
+      @file.bytes = 87
+      @file.bytes.should == 87
+    end
+  end
+
+  context 'setup' do
+    it "can be initialized with name and bytes" do
+      file = FakeFtp::File.new('filename', 104)
+      file.name.should == 'filename'
+      file.bytes.should == 104
+    end
+  end
+end

data/spec/models/fake_ftp/server_spec.rb

@@ -4,15 +4,10 @@ require 'net/ftp'
 describe FakeFtp::Server do
 
   before :each do
-    @directory = File.expand_path("../fixtures/destination", File.dirname(__FILE__))
     @text_filename = File.expand_path("../../fixtures/text_file.txt", File.dirname(__FILE__))
     @bin_filename = File.expand_path("../../fixtures/invisible_bike.jpg", File.dirname(__FILE__))
   end
 
-  after :each do
-    FileUtils.rm_rf(@directory+"/*")
-  end
-
   context 'setup' do
     it "starts a server on port n" do
       server = FakeFtp::Server.new(21212)
@@ -60,28 +55,22 @@ describe FakeFtp::Server do
       proc { FakeFtp::Server.new(21214, 21213) }.should raise_error(Errno::EADDRINUSE, "Address already in use - 21213")
       server.stop
     end
+  end
 
-    it "can be configured with a directory store" do
-      server = FakeFtp::Server.new
-      server.directory = @directory
-      server.directory.should == @directory
+  context 'files' do
+    before :each do
+      @file = FakeFtp::File.new('filename', 34)
+      @server = FakeFtp::Server.new(21212)
+      @server.instance_variable_set(:@files, [@file])
     end
 
-    it "should default directory to /tmp if Rails.root does not exist" do
-      server = FakeFtp::Server.new
-      server.directory.should == '/tmp'
+    it "returns filenames from :files" do
+      @server.files.should include('filename')
     end
 
-    it "should default directory to Rails.root/tmp/ftp if exists" do
-      module Rails; end
-      Rails.stub!(:root).and_return('/somewhere')
-      server = FakeFtp::Server.new
-      server.directory.should == '/somewhere/tmp/ftp'
+    it "can be accessed with :file" do
+      @server.file('filename').should == @file
     end
-
-    it "should clean up directory after itself"
-
-    it "should raise if attempting to delete a directory with contents other than its own"
   end
 
   context 'socket' do
@@ -234,25 +223,15 @@ describe FakeFtp::Server do
         end
 
         it "returns directory on PWD" do
-          @server.directory = @directory
           @client.puts "PWD"
-          @client.gets.should == "257 \"#{@directory}\" is current directory\r\n"
-        end
-
-        it "returns default directory on PWD" do
-          @server.directory = '/tmp'
-          @client.puts "PWD"
-          @client.gets.should == "257 \"/tmp\" is current directory\r\n"
+          @client.gets.should == "257 \"/pub\" is current directory\r\n"
         end
 
         it "says OK to any CWD, CDUP, without doing anything" do
-          directory = @server.directory
           @client.puts "CWD somewhere/else"
           @client.gets.should == "250 OK!\r\n"
-          @server.directory.should == directory
           @client.puts "CDUP"
           @client.gets.should == "250 OK!\r\n"
-          @server.directory.should == directory
         end
 
         it "does not respond to MKD" do
@@ -302,6 +281,7 @@ describe FakeFtp::Server do
           @data_client.close
           @client.gets.should == "226 Did it!\r\n"
           @server.files.should include('some_file')
+          @server.file('some_file').bytes.should == 10
         end
       end
     end
@@ -334,11 +314,15 @@ describe FakeFtp::Server do
         @ftp.close
       end
 
-      it "should put files to directory store using PASV" do
+      it "should put files using PASV" do
+        File.stat(@text_filename).size.should == 20
+
         @ftp.connect('127.0.0.1', 21212)
         @ftp.passive = true
         proc { @ftp.put(@text_filename) }.should_not raise_error
+
         @server.files.should include('text_file.txt')
+        @server.file('text_file.txt').bytes.should == 20
       end
 
       xit "should disconnect clients on close" do

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: fake_ftp
 version: !ruby/object:Gem::Version 
-  hash: 29
+  hash: 27
   prerelease: false
   segments: 
   - 0
   - 0
-  - 1
-  version: 0.0.1
+  - 2
+  version: 0.0.2
 platform: ruby
 authors: 
 - Colin Shield
@@ -16,7 +16,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-02-28 00:00:00 -08:00
+date: 2011-03-05 00:00:00 -08:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -83,10 +83,12 @@ files:
 - Rakefile
 - fake_ftp.gemspec
 - lib/fake_ftp.rb
+- lib/fake_ftp/file.rb
 - lib/fake_ftp/server.rb
 - lib/fake_ftp/version.rb
 - spec/fixtures/invisible_bike.jpg
 - spec/fixtures/text_file.txt
+- spec/models/fake_ftp/file_spec.rb
 - spec/models/fake_ftp/server_spec.rb
 - spec/spec_helper.rb
 has_rdoc: true
@@ -128,5 +130,6 @@ summary: Creates a fake FTP server for use in testing
 test_files: 
 - spec/fixtures/invisible_bike.jpg
 - spec/fixtures/text_file.txt
+- spec/models/fake_ftp/file_spec.rb
 - spec/models/fake_ftp/server_spec.rb
 - spec/spec_helper.rb