fake_ftp 0.0.3 → 0.0.4

data/Gemfile.lock

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

data/README.markdown

@@ -18,6 +18,8 @@ their intended destination rather than testing how our code does so.
 Usage
 -----
 
+To test passive upload:
+
     require 'fake_ftp'
     require 'net/ftp'
 
@@ -35,13 +37,32 @@ Usage
 
     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
 
-TODO
-----
+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
 
-* files should track if uploaded via active or passive
 
 References
 ----------

data/lib/fake_ftp/file.rb

@@ -1,10 +1,20 @@
 module FakeFtp
   class File
     attr_accessor :bytes, :name
+    attr_writer :type
 
-    def initialize(name = nil, bytes = nil)
+    def initialize(name = nil, bytes = nil, type = nil)
       self.name = name
       self.bytes = bytes
+      self.type = type
+    end
+
+    def passive?
+      @type == :passive
+    end
+
+    def active?
+      @type == :active
     end
   end
 end

data/lib/fake_ftp/server.rb

@@ -145,7 +145,7 @@ module FakeFtp
       end
 
       data = data_client.recv(1024)
-      file = FakeFtp::File.new(::File.basename(filename.to_s), data.length)
+      file = FakeFtp::File.new(::File.basename(filename.to_s), data.length, @mode)
       @files << file
 
       data_client.close

data/lib/fake_ftp/version.rb

@@ -1,3 +1,3 @@
 module FakeFtp
-  VERSION = "0.0.3"
+  VERSION = "0.0.4"
 end

data/spec/models/fake_ftp/file_spec.rb

@@ -18,10 +18,64 @@ describe FakeFtp::File do
   end
 
   context 'setup' do
+    it "can be initialized without attributes" do
+      file = FakeFtp::File.new
+      file.name.should be_nil
+      file.bytes.should be_nil
+      file.instance_variable_get(:@type).should be_nil
+    end
+
+    it "can be initialized with name" do
+      file = FakeFtp::File.new('filename')
+      file.name.should == 'filename'
+      file.bytes.should be_nil
+      file.instance_variable_get(:@type).should be_nil
+    end
+
     it "can be initialized with name and bytes" do
       file = FakeFtp::File.new('filename', 104)
       file.name.should == 'filename'
       file.bytes.should == 104
+      file.instance_variable_get(:@type).should be_nil
+    end
+    
+    it "can be initialized with name and bytes and type" do
+      file = FakeFtp::File.new('filename', 104, :passive)
+      file.name.should == 'filename'
+      file.bytes.should == 104
+      file.instance_variable_get(:@type).should == :passive
+    end
+  end
+
+  describe '#passive?' do
+    before :each do
+      @file = FakeFtp::File.new
+    end
+
+    it "should be true if type is :passive" do
+      @file.type = :passive
+      @file.passive?.should be_true
+    end
+
+    it "should be false if type is :active" do
+      @file.type = :active
+      @file.passive?.should be_false
+    end
+  end
+
+  describe '#active?' do
+    before :each do
+      @file = FakeFtp::File.new
+    end
+
+    it "should be true if type is :active" do
+      @file.type = :active
+      @file.active?.should be_true
+    end
+
+    it "should be false if type is :passive" do
+      @file.type = :passive
+      @file.active?.should be_false
     end
   end
 end

data/spec/models/fake_ftp/server_spec.rb

@@ -389,6 +389,8 @@ describe FakeFtp::Server do
 
         @server.files.should include('text_file.txt')
         @server.file('text_file.txt').bytes.should == 20
+        @server.file('text_file.txt').should be_passive
+        @server.file('text_file.txt').should_not be_active
       end
 
       it "should put files using active" do
@@ -400,6 +402,8 @@ describe FakeFtp::Server do
 
         @server.files.should include('text_file.txt')
         @server.file('text_file.txt').bytes.should == 20
+        @server.file('text_file.txt').should_not be_passive
+        @server.file('text_file.txt').should be_active
       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: 25
+  hash: 23
   prerelease: false
   segments: 
   - 0
   - 0
-  - 3
-  version: 0.0.3
+  - 4
+  version: 0.0.4
 platform: ruby
 authors: 
 - Colin Shield