tilia-dav 3.1.0.pre.alpha2

data/test/dav/server_plugin_test.rb

@@ -0,0 +1,85 @@
+require 'test_helper'
+
+module Tilia
+  module Dav
+    class ServerPluginTest < AbstractServer
+      # @var Sabre\DAV\TestPlugin
+      attr_accessor :plugin
+
+      def setup
+        super
+
+        @plugin = TestPlugin.new
+        @server.add_plugin(@plugin)
+      end
+
+      def test_base_class
+        p = ServerPluginMock.new
+        assert_equal([], p.features)
+        assert_equal([], p.http_methods(''))
+        assert_equal(
+          {
+            'name' => 'Tilia::Dav::ServerPluginMock',
+            'description' => nil,
+            'link' => nil
+          },
+          p.plugin_info
+        )
+      end
+
+      def test_options
+        server_vars = {
+          'REQUEST_PATH'   => '/',
+          'REQUEST_METHOD' => 'OPTIONS'
+        }
+
+        request = Http::Sapi.create_from_server_array(server_vars)
+        @server.http_request = request
+        @server.exec
+
+        assert_equal(
+          {
+            'DAV'             => ['1, 3, extended-mkcol, drinking'],
+            'MS-Author-Via'   => ['DAV'],
+            'Allow'           => ['OPTIONS, GET, HEAD, DELETE, PROPFIND, PUT, PROPPATCH, COPY, MOVE, REPORT, BEER, WINE'],
+            'Accept-Ranges'   => ['bytes'],
+            'Content-Length'  => ['0'],
+            'X-Sabre-Version' => [Version::VERSION]
+          },
+          @response.headers
+        )
+
+        assert_equal(200, @response.status)
+        assert_equal('', @response.body_as_string)
+        assert_equal('OPTIONS', @plugin.saved_before_method)
+      end
+
+      def test_get_plugin
+        assert_equal(@plugin, @server.plugin(@plugin.class.to_s))
+      end
+
+      def test_unknown_plugin
+        assert_nil(@server.plugin('SomeRandomClassName'))
+      end
+
+      def test_get_supported_report_set
+        assert_equal([], @plugin.supported_report_set('/'))
+      end
+
+      def test_plugins
+        assert_equal(
+          {
+            @plugin.class.to_s => @plugin,
+            'core' => @server.plugin('core')
+          },
+          @server.plugins
+        )
+      end
+    end
+
+    class ServerPluginMock < ServerPlugin
+      def setup(s)
+      end
+    end
+  end
+end

data/test/dav/server_precondition_test.rb

@@ -0,0 +1,253 @@
+require 'test_helper'
+
+module Tilia
+  module Dav
+    class ServerPreconditionsTest < Minitest::Test
+      # @expectedException Sabre\DAV\Exception\PreconditionFailed
+      def test_if_match_no_node
+        root = SimpleCollection.new('root', [ServerPreconditionsNode.new])
+        server = ServerMock.new(root)
+        http_request = Http::Request.new('GET', '/bar', 'If-Match' => '*')
+        http_response = Http::Response.new
+        assert_raises(Exception::PreconditionFailed) do
+          server.check_preconditions(http_request, http_response)
+        end
+      end
+
+      def test_if_match_has_node
+        root = SimpleCollection.new('root', [ServerPreconditionsNode.new])
+        server = ServerMock.new(root)
+        http_request = Http::Request.new('GET', '/foo', 'If-Match' => '*')
+        http_response = Http::Response.new
+        assert(server.check_preconditions(http_request, http_response))
+      end
+
+      def test_if_match_wrong_etag
+        root = SimpleCollection.new('root', [ServerPreconditionsNode.new])
+        server = ServerMock.new(root)
+        http_request = Http::Request.new('GET', '/foo', 'If-Match' => '1234')
+        http_response = Http::Response.new
+        assert_raises(Exception::PreconditionFailed) do
+          server.check_preconditions(http_request, http_response)
+        end
+      end
+
+      def test_if_match_correct_etag
+        root = SimpleCollection.new('root', [ServerPreconditionsNode.new])
+        server = ServerMock.new(root)
+        http_request = Http::Request.new('GET', '/foo', 'If-Match' => '"abc123"')
+        http_response = Http::Response.new
+        assert(server.check_preconditions(http_request, http_response))
+      end
+
+      # Evolution sometimes uses \" instead of " for If-Match headers.
+      def test_if_match_evolution_etag
+        root = SimpleCollection.new('root', [ServerPreconditionsNode.new])
+        server = ServerMock.new(root)
+        http_request = Http::Request.new('GET', '/foo', 'If-Match' => '\\"abc123\\"')
+        http_response = Http::Response.new
+        assert(server.check_preconditions(http_request, http_response))
+      end
+
+      def test_if_match_multiple
+        root = SimpleCollection.new('root', [ServerPreconditionsNode.new])
+        server = ServerMock.new(root)
+        http_request = Http::Request.new('GET', '/foo', 'If-Match' => '"hellothere", "abc123"')
+        http_response = Http::Response.new
+        assert(server.check_preconditions(http_request, http_response))
+      end
+
+      def test_if_none_match_no_node
+        root = SimpleCollection.new('root', [ServerPreconditionsNode.new])
+        server = ServerMock.new(root)
+        http_request = Http::Request.new('GET', '/bar', 'If-None-Match' => '*')
+        http_response = Http::Response.new
+        assert(server.check_preconditions(http_request, http_response))
+      end
+
+      def test_if_none_match_has_node
+        root = SimpleCollection.new('root', [ServerPreconditionsNode.new])
+        server = ServerMock.new(root)
+        http_request = Http::Request.new('POST', '/foo', 'If-None-Match' => '*')
+        http_response = Http::Response.new
+        assert_raises(Exception::PreconditionFailed) do
+          server.check_preconditions(http_request, http_response)
+        end
+      end
+
+      def test_if_none_match_wrong_etag
+        root = SimpleCollection.new('root', [ServerPreconditionsNode.new])
+        server = ServerMock.new(root)
+        http_request = Http::Request.new('POST', '/foo', 'If-None-Match' => '"1234"')
+        http_response = Http::Response.new
+        assert(server.check_preconditions(http_request, http_response))
+      end
+
+      def test_if_none_match_wrong_etag_multiple
+        root = SimpleCollection.new('root', [ServerPreconditionsNode.new])
+        server = ServerMock.new(root)
+        http_request = Http::Request.new('POST', '/foo', 'If-None-Match' => '"1234", "5678"')
+        http_response = Http::Response.new
+        assert(server.check_preconditions(http_request, http_response))
+      end
+
+      def test_if_none_match_correct_etag
+        root = SimpleCollection.new('root', [ServerPreconditionsNode.new])
+        server = ServerMock.new(root)
+        http_request = Http::Request.new('POST', '/foo', 'If-None-Match' => '"abc123"')
+        http_response = Http::Response.new
+        assert_raises(Exception::PreconditionFailed) do
+          server.check_preconditions(http_request, http_response)
+        end
+      end
+
+      def test_if_none_match_correct_etag_multiple
+        root = SimpleCollection.new('root', [ServerPreconditionsNode.new])
+        server = ServerMock.new(root)
+        http_request = Http::Request.new('POST', '/foo', 'If-None-Match' => '"1234, "abc123"')
+        http_response = Http::Response.new
+        assert_raises(Exception::PreconditionFailed) do
+          server.check_preconditions(http_request, http_response)
+        end
+      end
+
+      def test_if_none_match_correct_etag_as_get
+        root = SimpleCollection.new('root', [ServerPreconditionsNode.new])
+        server = ServerMock.new(root)
+        http_request = Http::Request.new('GET', '/foo', 'If-None-Match' => '"abc123"')
+        server.http_response = Http::ResponseMock.new
+
+        refute(server.check_preconditions(http_request, server.http_response))
+        assert_equal(304, server.http_response.status)
+        assert_equal({ 'ETag' => ['"abc123"'] }, server.http_response.headers)
+      end
+
+      # This was a test written for issue #515.
+      def test_none_match_correct_etag_ensure_sapi_sent
+        root = SimpleCollection.new('root', [ServerPreconditionsNode.new])
+        server = ServerMock.new(root)
+        server.sapi = Http::SapiMock.new
+        Http::SapiMock.sent = 0
+        http_request = Http::Request.new('GET', '/foo', 'If-None-Match' => '"abc123"')
+        server.http_request = http_request
+        server.http_response = Http::ResponseMock.new
+
+        server.exec
+
+        refute(server.check_preconditions(http_request, server.http_response))
+        assert_equal(304, server.http_response.status)
+        assert_equal(
+          {
+            'ETag' => ['"abc123"'],
+            'X-Sabre-Version' => [Version::VERSION]
+          },
+          server.http_response.headers
+        )
+        assert_equal(1, Http::SapiMock.sent)
+      end
+
+      def test_if_modified_since_un_modified
+        root = SimpleCollection.new('root', [ServerPreconditionsNode.new])
+        server = ServerMock.new(root)
+        http_request = Http::Sapi.create_from_server_array(
+          'HTTP_IF_MODIFIED_SINCE' => 'Sun, 06 Nov 1994 08:49:37 GMT',
+          'REQUEST_PATH'  => '/foo'
+        )
+        server.http_response = Http::ResponseMock.new
+        refute(server.check_preconditions(http_request, server.http_response))
+
+        assert_equal(304, server.http_response.status)
+        assert_equal(
+          { 'Last-Modified' => ['Sat, 06 Apr 1985 23:30:00 GMT'] },
+          server.http_response.headers
+        )
+      end
+
+      def test_if_modified_since_modified
+        root = SimpleCollection.new('root', [ServerPreconditionsNode.new])
+        server = ServerMock.new(root)
+        http_request = Http::Sapi.create_from_server_array(
+          'HTTP_IF_MODIFIED_SINCE' => 'Tue, 06 Nov 1984 08:49:37 GMT',
+          'REQUEST_PATH'  => '/foo'
+        )
+
+        http_response = Http::ResponseMock.new
+        assert(server.check_preconditions(http_request, http_response))
+      end
+
+      def test_if_modified_since_invalid_date
+        root = SimpleCollection.new('root', [ServerPreconditionsNode.new])
+        server = ServerMock.new(root)
+        http_request = Http::Sapi.create_from_server_array(
+          'HTTP_IF_MODIFIED_SINCE' => 'Your mother',
+          'REQUEST_PATH'  => '/foo'
+        )
+        http_response = Http::ResponseMock.new
+
+        # Invalid dates must be ignored, so this should return true
+        assert(server.check_preconditions(http_request, http_response))
+      end
+
+      def test_if_modified_since_invalid_date2
+        root = SimpleCollection.new('root', [ServerPreconditionsNode.new])
+        server = ServerMock.new(root)
+        http_request = Http::Sapi.create_from_server_array(
+          'HTTP_IF_MODIFIED_SINCE' => 'Sun, 06 Nov 1994 08:49:37 EST',
+          'REQUEST_PATH'  => '/foo'
+        )
+        http_response = Http::ResponseMock.new
+        assert(server.check_preconditions(http_request, http_response))
+      end
+
+      def test_if_unmodified_since_un_modified
+        root = SimpleCollection.new('root', [ServerPreconditionsNode.new])
+        server = ServerMock.new(root)
+        http_request = Http::Sapi.create_from_server_array(
+          'HTTP_IF_UNMODIFIED_SINCE' => 'Sun, 06 Nov 1994 08:49:37 GMT',
+          'REQUEST_PATH'  => '/foo'
+        )
+        http_response = Http::Response.new
+        assert(server.check_preconditions(http_request, http_response))
+      end
+
+      def test_if_unmodified_since_modified
+        root = SimpleCollection.new('root', [ServerPreconditionsNode.new])
+        server = ServerMock.new(root)
+        http_request = Http::Sapi.create_from_server_array(
+          'HTTP_IF_UNMODIFIED_SINCE' => 'Tue, 06 Nov 1984 08:49:37 GMT',
+          'REQUEST_PATH'  => '/foo'
+        )
+        http_response = Http::ResponseMock.new
+        assert_raises(Exception::PreconditionFailed) do
+          server.check_preconditions(http_request, http_response)
+        end
+      end
+
+      def test_if_unmodified_since_invalid_date
+        root = SimpleCollection.new('root', [ServerPreconditionsNode.new])
+        server = ServerMock.new(root)
+        http_request = Http::Sapi.create_from_server_array(
+          'HTTP_IF_UNMODIFIED_SINCE' => 'Sun, 06 Nov 1984 08:49:37 CET',
+          'REQUEST_PATH'  => '/foo'
+        )
+        http_response = Http::ResponseMock.new
+        assert(server.check_preconditions(http_request, http_response))
+      end
+    end
+
+    class ServerPreconditionsNode < File
+      def etag
+        '"abc123"'
+      end
+
+      def last_modified
+        # my birthday & time, I believe (Evert ^^)
+        Time.zone.parse('1985-04-07 01:30 +02:00')
+      end
+
+      def name
+        'foo'
+      end
+    end
+  end
+end

data/test/dav/server_props_infinite_depth_test.rb

@@ -0,0 +1,144 @@
+require 'test_helper'
+
+module Tilia
+  module Dav
+    class ServerPropsInfiniteDepthTest < AbstractServer
+      def root_node
+        FsExt::Directory.new(@temp_dir)
+      end
+
+      def setup
+        super
+
+        ::File.open("#{@temp_dir}/test2.txt", 'w') { |f| f.write 'Test contents2' }
+        Dir.mkdir("#{@temp_dir}/col")
+        Dir.mkdir("#{@temp_dir}/col/col")
+        ::File.open("#{@temp_dir}/col/col/test.txt", 'w') { |f| f.write 'Test contents' }
+        @server.add_plugin(Locks::Plugin.new(Locks::Backend::File.new("#{@temp_dir}/.locksdb")))
+        @server.enable_propfind_depth_infinity = true
+      end
+
+      def send_request(body)
+        request = Http::Request.new('PROPFIND', '/', 'Depth' => 'infinity')
+        request.body = body
+
+        @server.http_request = request
+        @server.exec
+      end
+
+      def test_prop_find_empty_body
+        send_request('')
+
+        assert_equal(207, @response.status, "Incorrect status received. Full response body: #{@response.body_as_string}")
+
+        assert_equal(
+          {
+            'X-Sabre-Version' => [Version::VERSION],
+            'Content-Type' => ['application/xml; charset=utf-8'],
+            'DAV' => ['1, 3, extended-mkcol, 2'],
+            'Vary' => ['Brief,Prefer']
+          },
+          @response.headers
+        )
+
+        body = @response.body.gsub(/xmlns(:[A-Za-z0-9_])?=("|')DAV:("|')/) { "xmlns#{Regexp.last_match(1)}=\"urn:DAV\"" }
+        xml = LibXML::XML::Document.string(body)
+
+        data = xml.find_first('/d:multistatus/d:response/d:href')
+        assert_equal('/', data.content, 'href element should have been /')
+
+        data = xml.find('/d:multistatus/d:response/d:propstat/d:prop/d:resourcetype')
+        # 8 resources are to be returned: /, col, col/col, col/col/test.txt, dir, dir/child.txt, test.txt and test2.txt
+        assert_equal(8, data.size)
+      end
+
+      def test_supported_locks
+        xml = <<XML
+<?xml version="1.0"?>
+<d:propfind xmlns:d="DAV:">
+  <d:prop>
+    <d:supportedlock />
+  </d:prop>
+</d:propfind>
+XML
+
+        send_request(xml)
+
+        body = @response.body_as_string
+        assert_equal(207, @response.status, body)
+
+        body = @response.body.gsub(/xmlns(:[A-Za-z0-9_])?=("|')DAV:("|')/) { "xmlns#{Regexp.last_match(1)}=\"urn:DAV\"" }
+        xml = LibXML::XML::Document.string(body)
+
+        data = xml.find('/d:multistatus/d:response/d:propstat/d:prop/d:supportedlock/d:lockentry')
+        assert_equal(16, data.size, 'We expected sixteen \'d:lockentry\' tags')
+
+        data = xml.find('/d:multistatus/d:response/d:propstat/d:prop/d:supportedlock/d:lockentry/d:lockscope')
+        assert_equal(16, data.size, 'We expected sixteen \'d:lockscope\' tags')
+
+        data = xml.find('/d:multistatus/d:response/d:propstat/d:prop/d:supportedlock/d:lockentry/d:locktype')
+        assert_equal(16, data.size, 'We expected sixteen \'d:locktype\' tags')
+
+        data = xml.find('/d:multistatus/d:response/d:propstat/d:prop/d:supportedlock/d:lockentry/d:lockscope/d:shared')
+        assert_equal(8, data.size, 'We expected eight \'d:shared\' tags')
+
+        data = xml.find('/d:multistatus/d:response/d:propstat/d:prop/d:supportedlock/d:lockentry/d:lockscope/d:exclusive')
+        assert_equal(8, data.size, 'We expected eight \'d:exclusive\' tags')
+
+        data = xml.find('/d:multistatus/d:response/d:propstat/d:prop/d:supportedlock/d:lockentry/d:locktype/d:write')
+        assert_equal(16, data.size, 'We expected sixteen \'d:write\' tags')
+      end
+
+      def test_lock_discovery
+        xml = <<XML
+<?xml version="1.0"?>
+<d:propfind xmlns:d="DAV:">
+  <d:prop>
+    <d:lockdiscovery />
+  </d:prop>
+</d:propfind>
+XML
+
+        send_request(xml)
+
+        body = @response.body.gsub(/xmlns(:[A-Za-z0-9_])?=("|')DAV:("|')/) { "xmlns#{Regexp.last_match(1)}=\"urn:DAV\"" }
+        xml = LibXML::XML::Document.string(body)
+
+        data = xml.find('/d:multistatus/d:response/d:propstat/d:prop/d:lockdiscovery')
+        assert_equal(8, data.size, 'We expected eight \'d:lockdiscovery\' tags')
+      end
+
+      def test_unknown_property
+        xml = <<XML
+<?xml version="1.0"?>
+<d:propfind xmlns:d="DAV:">
+  <d:prop>
+    <d:macaroni />
+  </d:prop>
+</d:propfind>
+XML
+
+        send_request(xml)
+
+        body = @response.body.gsub(/xmlns(:[A-Za-z0-9_])?=("|')DAV:("|')/) { "xmlns#{Regexp.last_match(1)}=\"urn:DAV\"" }
+        xml = LibXML::XML::Document.string(body)
+
+        path_tests = [
+          '/d:multistatus',
+          '/d:multistatus/d:response',
+          '/d:multistatus/d:response/d:propstat',
+          '/d:multistatus/d:response/d:propstat/d:status',
+          '/d:multistatus/d:response/d:propstat/d:prop',
+          '/d:multistatus/d:response/d:propstat/d:prop/d:macaroni'
+        ]
+        path_tests.each do |test|
+          assert(xml.find(test).size > 0, "We expected the #{test} element to appear in the response, we got: #{body}")
+        end
+
+        val = xml.find('/d:multistatus/d:response/d:propstat/d:status')
+        assert_equal(8, val.size, body)
+        assert_equal('HTTP/1.1 404 Not Found', val[0].content)
+      end
+    end
+  end
+end