rack_dav 0.1.3 → 0.3.1

data/spec/file_resource_spec.rb

@@ -0,0 +1,11 @@
+require 'spec_helper'
+
+describe RackDAV::FileResource do
+
+  describe "#get" do
+    context "with a directory" do
+      # it "sets a compliant rack response"
+    end
+  end
+
+end

data/spec/fixtures/requests/lock.xml

@@ -0,0 +1,12 @@
+<?xml version="1.0" encoding="utf-8" ?>
+<D:lockinfo xmlns:D='DAV:'>
+  <D:lockscope>
+    <D:exclusive/>
+  </D:lockscope>
+  <D:locktype>
+    <D:write/>
+  </D:locktype>
+  <D:owner>
+    <D:href>http://example.com/</D:href>
+  </D:owner>
+</D:lockinfo>

data/spec/handler_spec.rb

@@ -1,270 +1,37 @@
-$:.unshift(File.expand_path(File.dirname(__FILE__) + '/../lib'))
-
-require 'rubygems'
-require 'rack_dav'
-require 'fileutils'
+require 'spec_helper'
 
 describe RackDAV::Handler do
-  DOC_ROOT = File.expand_path(File.dirname(__FILE__) + '/htdocs')
-  METHODS = %w(GET PUT POST DELETE PROPFIND PROPPATCH MKCOL COPY MOVE OPTIONS HEAD LOCK UNLOCK)  
-  
-  before do
-    FileUtils.mkdir(DOC_ROOT) unless File.exists?(DOC_ROOT)
-    @controller = RackDAV::Handler.new(:root => DOC_ROOT)
-  end
-
-  after do
-    FileUtils.rm_rf(DOC_ROOT) if File.exists?(DOC_ROOT)
-  end
-  
-  attr_reader :response
-  
-  def request(method, uri, options={})
-    options = {
-      'HTTP_HOST' => 'localhost',
-      'REMOTE_USER' => 'manni'
-    }.merge(options)
-    request = Rack::MockRequest.new(@controller)
-    @response = request.request(method, uri, options)
-  end
-
-  METHODS.each do |method|
-    define_method(method.downcase) do |*args|
-      request(method, *args)
-    end
-  end  
-  
-  def render
-    xml = Builder::XmlMarkup.new
-    xml.instruct! :xml, :version => "1.0", :encoding => "UTF-8"      
-    xml.namespace('d') do
-      yield xml
-    end
-    xml.target!
-  end
- 
-  def url_escape(string)
-    string.gsub(/([^ a-zA-Z0-9_.-]+)/n) do
-      '%' + $1.unpack('H2' * $1.size).join('%').upcase
-    end.tr(' ', '+')
-  end
-  
-  def response_xml
-    REXML::Document.new(@response.body)
-  end
-  
-  def multistatus_response(pattern)
-    @response.should be_multi_status
-    REXML::XPath::match(response_xml, "/multistatus/response", '' => 'DAV:').should_not be_empty
-    REXML::XPath::match(response_xml, "/multistatus/response" + pattern, '' => 'DAV:')
-  end
 
-  def propfind_xml(*props)
-    render do |xml|
-      xml.propfind('xmlns:d' => "DAV:") do
-        xml.prop do
-          props.each do |prop|
-          xml.tag! prop
-          end
-        end
-      end
-    end
-  end
-  
-  it 'should return all options' do
-    options('/').should be_ok
-    
-    METHODS.each do |method|
-      response.headers['allow'].should include(method)
+  describe "#initialize" do
+    it "accepts zero parameters" do
+      lambda do
+        klass.new
+      end.should_not raise_error
     end
-  end
-  
-  it 'should return headers' do
-    put('/test.html', :input => '<html/>').should be_ok
-    head('/test.html').should be_ok
-    
-    response.headers['etag'].should_not be_nil
-    response.headers['content-type'].should match(/html/)
-    response.headers['last-modified'].should_not be_nil
-  end
-  
-  it 'should not find a nonexistent resource' do
-    get('/not_found').should be_not_found
-  end
-  
-  it 'should not allow directory traversal' do
-    get('/../htdocs').should be_forbidden
-  end
-  
-  it 'should create a resource and allow its retrieval' do
-    put('/test', :input => 'body').should be_ok    
-    get('/test').should be_ok
-    response.body.should == 'body'
-  end
-  it 'should create and find a url with escaped characters' do
-    put(url_escape('/a b'), :input => 'body').should be_ok    
-    get(url_escape('/a b')).should be_ok
-    response.body.should == 'body'
-  end
-  
-  it 'should delete a single resource' do
-    put('/test', :input => 'body').should be_ok
-    delete('/test').should be_no_content
-  end
-  
-  it 'should delete recursively' do
-    mkcol('/folder').should be_created
-    put('/folder/a', :input => 'body').should be_ok
-    put('/folder/b', :input => 'body').should be_ok
-    
-    delete('/folder').should be_no_content
-    get('/folder').should be_not_found
-    get('/folder/a').should be_not_found
-    get('/folder/b').should be_not_found
-  end
 
-  it 'should not allow copy to another domain' do
-    put('/test', :input => 'body').should be_ok
-    copy('http://localhost/', 'HTTP_DESTINATION' => 'http://another/').should be_bad_gateway
-  end
-
-  it 'should not allow copy to the same resource' do
-    put('/test', :input => 'body').should be_ok
-    copy('/test', 'HTTP_DESTINATION' => '/test').should be_forbidden
-  end
-
-  it 'should not allow an invalid destination uri' do
-    put('/test', :input => 'body').should be_ok
-    copy('/test', 'HTTP_DESTINATION' => '%').should be_bad_request
-  end
-
-  it 'should copy a single resource' do
-    put('/test', :input => 'body').should be_ok
-    copy('/test', 'HTTP_DESTINATION' => '/copy').should be_created
-    get('/copy').body.should == 'body'
-  end
-
-  it 'should copy a resource with escaped characters' do
-    put(url_escape('/a b'), :input => 'body').should be_ok
-    copy(url_escape('/a b'), 'HTTP_DESTINATION' => url_escape('/a c')).should be_created
-    get(url_escape('/a c')).should be_ok
-    response.body.should == 'body'
-  end
-  
-  it 'should deny a copy without overwrite' do
-    put('/test', :input => 'body').should be_ok
-    put('/copy', :input => 'copy').should be_ok
-    copy('/test', 'HTTP_DESTINATION' => '/copy', 'HTTP_OVERWRITE' => 'F')
-    
-    multistatus_response('/href').first.text.should == 'http://localhost/test'
-    multistatus_response('/status').first.text.should match(/412 Precondition Failed/)
-    
-    get('/copy').body.should == 'copy'
-  end
-  
-  it 'should allow a copy with overwrite' do
-    put('/test', :input => 'body').should be_ok
-    put('/copy', :input => 'copy').should be_ok
-    copy('/test', 'HTTP_DESTINATION' => '/copy', 'HTTP_OVERWRITE' => 'T').should be_no_content
-    get('/copy').body.should == 'body'
-  end
-  
-  it 'should copy a collection' do
-    mkcol('/folder').should be_created
-    copy('/folder', 'HTTP_DESTINATION' => '/copy').should be_created
-    propfind('/copy', :input => propfind_xml(:resourcetype))
-    multistatus_response('/propstat/prop/resourcetype/collection').should_not be_empty
-  end
-
-  it 'should copy a collection resursively' do
-    mkcol('/folder').should be_created
-    put('/folder/a', :input => 'A').should be_ok
-    put('/folder/b', :input => 'B').should be_ok
-    
-    copy('/folder', 'HTTP_DESTINATION' => '/copy').should be_created
-    propfind('/copy', :input => propfind_xml(:resourcetype))
-    multistatus_response('/propstat/prop/resourcetype/collection').should_not be_empty    
-    
-    get('/copy/a').body.should == 'A'
-    get('/copy/b').body.should == 'B'
-  end
-  
-  it 'should move a collection recursively' do
-    mkcol('/folder').should be_created
-    put('/folder/a', :input => 'A').should be_ok
-    put('/folder/b', :input => 'B').should be_ok
-    
-    move('/folder', 'HTTP_DESTINATION' => '/move').should be_created
-    propfind('/move', :input => propfind_xml(:resourcetype))
-    multistatus_response('/propstat/prop/resourcetype/collection').should_not be_empty    
-    
-    get('/move/a').body.should == 'A'
-    get('/move/b').body.should == 'B'
-    get('/folder/a').should be_not_found
-    get('/folder/b').should be_not_found
-  end
-  
-  it 'should create a collection' do
-    mkcol('/folder').should be_created
-    propfind('/folder', :input => propfind_xml(:resourcetype))
-    multistatus_response('/propstat/prop/resourcetype/collection').should_not be_empty
-  end
-  
-  it 'should not find properties for nonexistent resources' do
-    propfind('/non').should be_not_found
-  end
-  
-  it 'should find all properties' do
-    xml = render do |xml|
-      xml.propfind('xmlns:d' => "DAV:") do
-        xml.allprop
-      end
+    it "accepts a hash of options" do
+      lambda do
+        klass.new({})
+        klass.new :foo => "bar"
+      end.should_not raise_error
     end
-    
-    propfind('http://localhost/', :input => xml)
-    
-    multistatus_response('/href').first.text.strip.should == 'http://localhost/'
 
-    props = %w(creationdate displayname getlastmodified getetag resourcetype getcontenttype getcontentlength)
-    props.each do |prop|
-      multistatus_response('/propstat/prop/' + prop).should_not be_empty
+    it "sets options from argument" do
+      instance = klass.new :foo => "bar"
+      instance.options[:foo].should == "bar"
     end
-  end
-  
-  it 'should find named properties' do
-    put('/test.html', :input => '<html/>').should be_ok    
-    propfind('/test.html', :input => propfind_xml(:getcontenttype, :getcontentlength))
-   
-    multistatus_response('/propstat/prop/getcontenttype').first.text.should == 'text/html'
-    multistatus_response('/propstat/prop/getcontentlength').first.text.should == '7'
-  end
 
-  it 'should lock a resource' do
-    put('/test', :input => 'body').should be_ok
-    
-    xml = render do |xml|
-      xml.lockinfo('xmlns:d' => "DAV:") do
-        xml.lockscope { xml.exclusive }
-        xml.locktype { xml.write }
-        xml.owner { xml.href "http://test.de/" }
-      end      
+    it "defaults option :resource_class to FileResource" do
+      instance = klass.new
+      instance.options[:resource_class].should be(RackDAV::FileResource)
     end
 
-    lock('/test', :input => xml)
-    
-    response.should be_ok
-
-    match = lambda do |pattern|
-      REXML::XPath::match(response_xml, "/prop/lockdiscovery/activelock" + pattern, '' => 'DAV:')
+    it "defaults option :root to current directory" do
+      path = File.expand_path("../../bin", __FILE__)
+      Dir.chdir(path)
+      instance = klass.new
+      instance.options[:root].should == path
     end
-    
-    match[''].should_not be_empty
-
-    match['/locktype'].should_not be_empty
-    match['/lockscope'].should_not be_empty
-    match['/depth'].should_not be_empty
-    match['/owner'].should_not be_empty
-    match['/timeout'].should_not be_empty
-    match['/locktoken'].should_not be_empty
   end
+
 end

data/spec/spec_helper.rb

@@ -0,0 +1,27 @@
+require 'rspec'
+require 'rack_dav'
+
+unless defined?(SPEC_ROOT)
+  SPEC_ROOT = File.expand_path("../", __FILE__)
+end
+
+module Helpers
+
+  private
+
+    # Gets the currently described class.
+    # Conversely to +subject+, it returns the class
+    # instead of an instance.
+    def klass
+      described_class
+    end
+
+    def fixture(*names)
+      File.join(SPEC_ROOT, "fixtures", *names)
+    end
+
+end
+
+RSpec.configure do |config|
+  config.include Helpers
+end

data/spec/support/lockable_file_resource.rb

@@ -0,0 +1,30 @@
+module RackDAV
+
+  # Quick & Dirty
+  class LockableFileResource < FileResource
+    @@locks = {}
+
+    def lock(token, timeout, scope = nil, type = nil, owner = nil)
+      if scope && type && owner
+        # Create lock
+        @@locks[token] = {
+          :timeout => timeout,
+          :scope   => scope,
+          :type    => type,
+          :owner   => owner
+        }
+        return true
+      else
+        # Refresh lock
+        lock = @@locks[token]
+        return false unless lock
+        return [ lock[:timeout], lock[:scope], lock[:type], lock[:owner] ]
+      end
+    end
+
+    def unlock(token)
+      !!@@locks.delete(token)
+    end
+  end
+
+end

metadata

@@ -1,79 +1,136 @@
---- !ruby/object:Gem::Specification 
+--- !ruby/object:Gem::Specification
 name: rack_dav
-version: !ruby/object:Gem::Version 
-  hash: 29
-  prerelease: false
-  segments: 
-  - 0
-  - 1
-  - 3
-  version: 0.1.3
+version: !ruby/object:Gem::Version
+  version: 0.3.1
+  prerelease: 
 platform: ruby
-authors: 
+authors:
 - Matthias Georgi
 autorequire: 
 bindir: bin
 cert_chain: []
-
-date: 2010-10-23 00:00:00 -07:00
-default_executable: 
-dependencies: []
-
-description: WebDAV handler for Rack
+date: 2013-01-24 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: rack
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 1.4.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 1.4.0
+- !ruby/object:Gem::Dependency
+  name: nokogiri
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 2.11.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 2.11.0
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 0.9.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 0.9.0
+description: WebDAV handler for Rack.
 email: matti.georgi@gmail.com
-executables: 
+executables:
 - rack_dav
 extensions: []
-
-extra_rdoc_files: 
+extra_rdoc_files:
 - README.md
-files: 
+files:
 - .gitignore
+- CHANGELOG.md
+- Gemfile
+- Gemfile.lock
 - LICENSE
-- rack_dav.gemspec
+- README.md
+- Rakefile
+- bin/rack_dav
 - lib/rack_dav.rb
+- lib/rack_dav/controller.rb
 - lib/rack_dav/file_resource.rb
 - lib/rack_dav/handler.rb
-- lib/rack_dav/controller.rb
-- lib/rack_dav/builder_namespace.rb
 - lib/rack_dav/http_status.rb
 - lib/rack_dav/resource.rb
-- bin/rack_dav
+- lib/rack_dav/string.rb
+- lib/rack_dav/version.rb
+- rack_dav.gemspec
+- spec/controller_spec.rb
+- spec/file_resource_spec.rb
+- spec/fixtures/folder/01.txt
+- spec/fixtures/folder/02.txt
+- spec/fixtures/requests/lock.xml
 - spec/handler_spec.rb
-- README.md
-has_rdoc: true
-homepage: http://www.matthias-georgi.de/rack_dav
+- spec/spec_helper.rb
+- spec/support/lockable_file_resource.rb
+homepage: http://georgi.github.com/rack_dav
 licenses: []
-
 post_install_message: 
 rdoc_options: []
-
-require_paths: 
+require_paths:
 - lib
-required_ruby_version: !ruby/object:Gem::Requirement 
+required_ruby_version: !ruby/object:Gem::Requirement
   none: false
-  requirements: 
-  - - ">="
-    - !ruby/object:Gem::Version 
-      hash: 3
-      segments: 
-      - 0
-      version: "0"
-required_rubygems_version: !ruby/object:Gem::Requirement 
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
+required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
-  requirements: 
-  - - ">="
-    - !ruby/object:Gem::Version 
-      hash: 3
-      segments: 
-      - 0
-      version: "0"
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
 requirements: []
-
 rubyforge_project: 
-rubygems_version: 1.3.7
+rubygems_version: 1.8.24
 signing_key: 
 specification_version: 3
-summary: WebDAV handler for Rack
+summary: WebDAV handler for Rack.
 test_files: []
-
+has_rdoc: