tilia-dav 3.1.0.pre.alpha2

data/test/dav/http_delete_test.rb

@@ -0,0 +1,110 @@
+require 'test_helper'
+
+module Tilia
+  module Dav
+    # Tests related to the PUT request.
+    class HttpDeleteTest < DavServerTest
+      # Sets up the DAV tree.
+      #
+      # @return void
+      def set_up_tree
+        @tree = Mock::Collection.new(
+          'root',
+          'file1' => 'foo',
+          'dir' => {
+            'subfile' => 'bar',
+            'subfile2' => 'baz'
+          }
+        )
+      end
+
+      # A successful DELETE
+      def test_delete
+        request = Http::Request.new('DELETE', '/file1')
+
+        response = self.request(request)
+
+        assert_equal(
+          204,
+          response.status,
+          "Incorrect status code. Response body:  #{response.body_as_string}"
+        )
+
+        assert_equal(
+          {
+            'X-Sabre-Version' => [Version::VERSION],
+            'Content-Length' => ['0']
+          },
+          response.headers
+        )
+      end
+
+      # Deleting a Directory
+      def test_delete_directory
+        request = Http::Request.new('DELETE', '/dir')
+
+        response = self.request(request)
+
+        assert_equal(
+          204,
+          response.status,
+          "Incorrect status code. Response body:  #{response.body_as_string}"
+        )
+
+        assert_equal(
+          {
+            'X-Sabre-Version' => [Version::VERSION],
+            'Content-Length' => ['0']
+          },
+          response.headers
+        )
+      end
+
+      # DELETE on a node that does not exist
+      def test_delete_not_found
+        request = Http::Request.new('DELETE', '/file2')
+        response = self.request(request)
+
+        assert_equal(
+          404,
+          response.status,
+          "Incorrect status code. Response body:  #{response.body_as_string}"
+        )
+      end
+
+      # DELETE with preconditions
+      def test_delete_preconditions
+        request = Http::Request.new(
+          'DELETE',
+          '/file1',
+          'If-Match' => "\"#{Digest::MD5.hexdigest('foo')}\""
+        )
+
+        response = self.request(request)
+
+        assert_equal(
+          204,
+          response.status,
+          "Incorrect status code. Response body:  #{response.body_as_string}"
+        )
+      end
+
+      # DELETE with incorrect preconditions
+      def test_delete_preconditions_failed
+        request = Http::Request.new(
+          'DELETE',
+          '/file1',
+          'If-Match' => "\"#{Digest::MD5.hexdigest('bar')}\""
+        )
+
+        response = self.request(request)
+
+        assert_equal(
+          412,
+          response.status,
+          "Incorrect status code. Response body:  #{response.body_as_string}"
+        )
+      end
+    end
+  end
+end

data/test/dav/http_get_test.rb

@@ -0,0 +1,130 @@
+require 'test_helper'
+
+module Tilia
+  module Dav
+    # Tests related to the GET request.
+    class HttpGetTest < DavServerTest
+      # Sets up the DAV tree.
+      #
+      # @return void
+      def set_up_tree
+        @tree = Mock::Collection.new(
+          'root',
+          'file1' => 'foo',
+          'a' => Mock::Collection.new('dir', []),
+          'b' => Mock::StreamingFile.new('streaming', 'stream')
+        )
+      end
+
+      def test_get
+        request = Http::Request.new('GET', '/file1')
+        response = self.request(request)
+
+        assert_equal(200, response.status)
+
+        # Removing Last-Modified because it keeps changing.
+        response.remove_header('Last-Modified')
+
+        assert_equal(
+          {
+            'X-Sabre-Version' => [Version::VERSION],
+            'Content-Type'    => ['application/octet-stream'],
+            'Content-Length'  => [3],
+            'ETag'            => ["\"#{Digest::MD5.hexdigest('foo')}\""]
+          },
+          response.headers
+        )
+
+        assert_equal('foo', response.body_as_string)
+      end
+
+      def test_get_http10
+        request = Http::Request.new('GET', '/file1')
+        request.http_version = '1.0'
+        response = self.request(request)
+
+        assert_equal(200, response.status)
+
+        # Removing Last-Modified because it keeps changing.
+        response.remove_header('Last-Modified')
+
+        assert_equal(
+          {
+            'X-Sabre-Version' => [Version::VERSION],
+            'Content-Type'    => ['application/octet-stream'],
+            'Content-Length'  => [3],
+            'ETag'            => ["\"#{Digest::MD5.hexdigest('foo')}\""]
+          },
+          response.headers
+        )
+
+        assert_equal('1.0', response.http_version)
+        assert_equal('foo', response.body_as_string)
+      end
+
+      def test_get404
+        request = Http::Request.new('GET', '/notfound')
+        response = self.request(request)
+
+        assert_equal(404, response.status)
+      end
+
+      def test_get404_aswell
+        request = Http::Request.new('GET', '/file1/subfile')
+        response = self.request(request)
+
+        assert_equal(404, response.status)
+      end
+
+      # We automatically normalize double slashes.
+      def test_get_double_slash
+        request = Http::Request.new('GET', '//file1')
+        response = self.request(request)
+
+        assert_equal(200, response.status)
+
+        # Removing Last-Modified because it keeps changing.
+        response.remove_header('Last-Modified')
+
+        assert_equal(
+          {
+            'X-Sabre-Version' => [Version::VERSION],
+            'Content-Type'    => ['application/octet-stream'],
+            'Content-Length'  => [3],
+            'ETag'            => ["\"#{Digest::MD5.hexdigest('foo')}\""]
+          },
+          response.headers
+        )
+
+        assert_equal('foo', response.body_as_string)
+      end
+
+      def test_get_collection
+        request = Http::Request.new('GET', '/dir')
+        response = self.request(request)
+
+        assert_equal(501, response.status)
+      end
+
+      def test_get_streaming
+        request = Http::Request.new('GET', '/streaming')
+        response = self.request(request)
+
+        assert_equal(200, response.status)
+
+        # Removing Last-Modified because it keeps changing.
+        response.remove_header('Last-Modified')
+
+        assert_equal(
+          {
+            'X-Sabre-Version' => [Version::VERSION],
+            'Content-Type'    => ['application/octet-stream']
+          },
+          response.headers
+        )
+
+        assert_equal('stream', response.body_as_string)
+      end
+    end
+  end
+end

data/test/dav/http_head_test.rb

@@ -0,0 +1,80 @@
+require 'test_helper'
+
+module Tilia
+  module Dav
+    # Tests related to the HEAD request.
+    class HttpHeadTest < DavServerTest
+      # Sets up the DAV tree.
+      #
+      # @return void
+      def set_up_tree
+        @tree = Mock::Collection.new(
+          'root',
+          'file1' => 'foo',
+          'a' => Mock::Collection.new('dir', []),
+          'b' => Mock::StreamingFile.new('streaming', 'stream')
+        )
+      end
+
+      def test_head
+        request = Http::Request.new('HEAD', '//file1')
+        response = self.request(request)
+
+        assert_equal(200, response.status)
+
+        # Removing Last-Modified because it keeps changing.
+        response.remove_header('Last-Modified')
+
+        assert_equal(
+          {
+            'X-Sabre-Version' => [Version::VERSION],
+            'Content-Type'    => ['application/octet-stream'],
+            'Content-Length'  => [3],
+            'ETag'            => ["\"#{Digest::MD5.hexdigest('foo')}\""]
+          },
+          response.headers
+        )
+
+        assert_equal('', response.body_as_string)
+      end
+
+      # According to the specs, HEAD should behave identical to GET. But, broken
+      # clients needs HEAD requests on collections to respond with a 200, so
+      # that's what we do.
+      def test_head_collection
+        request = Http::Request.new('HEAD', '/dir')
+        response = self.request(request)
+
+        assert_equal(200, response.status)
+      end
+
+      # HEAD automatically internally maps to GET via a sub-request.
+      # The Auth plugin must not be triggered twice for these, so we'll
+      # test for that.
+      def test_double_auth
+        count = 0
+
+        auth_backend = Auth::Backend::BasicCallBack.new(
+          lambda do |_user_name, _password|
+            count += 1
+            true
+          end
+        )
+        @server.add_plugin(
+          Auth::Plugin.new(
+            auth_backend
+          )
+        )
+        request = Http::Request.new(
+          'HEAD',
+          '/file1',
+          'Authorization' => "Basic #{Base64.strict_encode64('user:pass')}"
+        )
+        response = self.request(request)
+
+        assert_equal(200, response.status)
+        assert_equal(1, count, 'Auth was triggered twice :(')
+      end
+    end
+  end
+end

data/test/dav/http_move_test.rb

@@ -0,0 +1,105 @@
+require 'test_helper'
+
+module Tilia
+  module Dav
+    # Tests related to the MOVE request.
+    class HttpMoveTest < DavServerTest
+      # Sets up the DAV tree.
+      #
+      # @return void
+      def set_up_tree
+        @tree = Mock::Collection.new(
+          'root',
+          'file1' => 'content1',
+          'file2' => 'content2'
+        )
+      end
+
+      def test_move_to_self
+        request = Http::Request.new(
+          'MOVE',
+          '/file1',
+          'Destination' => '/file1'
+        )
+        response = self.request(request)
+        assert_equal(403, response.status)
+        assert_equal('content1', @tree.child('file1').get)
+      end
+
+      def test_move
+        request = Http::Request.new(
+          'MOVE',
+          '/file1',
+          'Destination' => '/file3'
+        )
+        response = self.request(request)
+        assert_equal(201, response.status, response.inspect)
+        assert_equal('content1', @tree.child('file3').get)
+        refute(@tree.child_exists('file1'))
+      end
+
+      def test_move_to_existing
+        request = Http::Request.new(
+          'MOVE',
+          '/file1',
+          'Destination' => '/file2'
+        )
+        response = self.request(request)
+        assert_equal(204, response.status, response.inspect)
+        assert_equal('content1', @tree.child('file2').get)
+        refute(@tree.child_exists('file1'))
+      end
+
+      def test_move_to_existing_overwrite_t
+        request = Http::Request.new(
+          'MOVE',
+          '/file1',
+          'Destination' => '/file2',
+          'Overwrite' => 'T'
+        )
+        response = self.request(request)
+        assert_equal(204, response.status, response.inspect)
+        assert_equal('content1', @tree.child('file2').get)
+        refute(@tree.child_exists('file1'))
+      end
+
+      def test_move_to_existing_overwrite_f
+        request = Http::Request.new(
+          'MOVE',
+          '/file1',
+          'Destination' => '/file2',
+          'Overwrite' => 'F'
+        )
+        response = self.request(request)
+        assert_equal(412, response.status, response.inspect)
+        assert_equal('content1', @tree.child('file1').get)
+        assert_equal('content2', @tree.child('file2').get)
+        assert(@tree.child_exists('file1'))
+        assert(@tree.child_exists('file2'))
+      end
+
+      # If we MOVE to an existing file, but a plugin prevents the original from
+      # being deleted, we need to make sure that the server does not delete
+      # the destination.
+      def test_move_to_existing_blocked_delete_source
+        @server.on(
+          'beforeUnbind',
+          lambda do |path|
+            fail Exception::Forbidden, 'uh oh' if path == 'file1'
+          end
+        )
+        request = Http::Request.new(
+          'MOVE',
+          '/file1',
+          'Destination' => '/file2'
+        )
+        response = self.request(request)
+        assert_equal(403, response.status, response.inspect)
+        assert_equal('content1', @tree.child('file1').get)
+        assert_equal('content2', @tree.child('file2').get)
+        assert(@tree.child_exists('file1'))
+        assert(@tree.child_exists('file2'))
+      end
+    end
+  end
+end

data/test/dav/http_prefer_parsing_test.rb

@@ -0,0 +1,186 @@
+require 'test_helper'
+
+module Tilia
+  module Dav
+    class HttpPreferParsingTest < DavServerTest
+      def test_parse_simple
+        http_request = Http::Sapi.create_from_server_array(
+          'HTTP_PREFER' => 'return-asynch'
+        )
+
+        server = ServerMock.new
+        server.http_request = http_request
+
+        assert_equal(
+          {
+            'respond-async' => true,
+            'return'        => nil,
+            'handling'      => false,
+            'wait'          => nil
+          },
+          server.http_prefer
+        )
+      end
+
+      def test_parse_value
+        http_request = Http::Sapi.create_from_server_array(
+          'HTTP_PREFER' => 'wait=10'
+        )
+
+        server = ServerMock.new
+        server.http_request = http_request
+
+        assert_equal(
+          {
+            'respond-async' => false,
+            'return'        => nil,
+            'handling'      => false,
+            'wait'          => '10'
+          },
+          server.http_prefer
+        )
+      end
+
+      def test_parse_multiple
+        http_request = Http::Sapi.create_from_server_array(
+          'HTTP_PREFER' => 'return-minimal, strict,lenient'
+        )
+
+        server = ServerMock.new
+        server.http_request = http_request
+
+        assert_equal(
+          {
+            'respond-async' => false,
+            'return'        => 'minimal',
+            'handling'      => 'lenient',
+            'wait'          => nil
+          },
+          server.http_prefer
+        )
+      end
+
+      def test_parse_weird_value
+        http_request = Http::Sapi.create_from_server_array(
+          'HTTP_PREFER' => 'BOOOH'
+        )
+
+        server = ServerMock.new
+        server.http_request = http_request
+
+        assert_equal(
+          {
+            'respond-async' => false,
+            'return'        => nil,
+            'handling'      => false,
+            'wait'          => nil,
+            'boooh'         => true
+          },
+          server.http_prefer
+        )
+      end
+
+      def test_brief
+        http_request = Http::Sapi.create_from_server_array(
+          'HTTP_BRIEF' => 't'
+        )
+
+        server = ServerMock.new
+        server.http_request = http_request
+
+        assert_equal(
+          {
+            'respond-async' => false,
+            'return'        => 'minimal',
+            'handling'      => false,
+            'wait'          => nil
+          },
+          server.http_prefer
+        )
+      end
+
+      # propfindMinimal
+      #
+      # @return void
+      def test_propfind_minimal
+        request = Http::Sapi.create_from_server_array(
+          'REQUEST_METHOD' => 'PROPFIND',
+          'REQUEST_PATH'   => '/',
+          'HTTP_PREFER'    => 'return-minimal'
+        )
+        request.body = <<BLA
+<?xml version="1.0"?>
+<d:propfind xmlns:d="DAV:">
+    <d:prop>
+        <d:something />
+        <d:resourcetype />
+    </d:prop>
+</d:propfind>
+BLA
+
+        response = self.request(request)
+
+        body = response.body_as_string
+
+        assert_equal(207, response.status, body)
+
+        assert(body.index('resourcetype'), body)
+        refute(body.index('something'), body)
+      end
+
+      def testproppatch_minimal
+        request = Http::Request.new('PROPPATCH', '/', ['Prefer' => 'return-minimal'])
+        request.body = <<BLA
+<?xml version="1.0"?>
+<d:propertyupdate xmlns:d="DAV:">
+    <d:set>
+        <d:prop>
+            <d:something>nope!</d:something>
+        </d:prop>
+    </d:set>
+</d:propertyupdate>
+BLA
+
+        @server.on(
+          'propPatch',
+          lambda do |_path, prop_patch|
+            prop_patch.handle(
+              '{DAV:}something',
+              lambda do |_props|
+                return true
+              end
+
+            )
+          end
+        )
+
+        response = self.request(request)
+
+        assert_equal(0, response.body.size, "Expected empty body: #{response.body}")
+        assert_equal(204, response.status)
+      end
+
+      def testproppatch_minimal_error
+        request = Http::Request.new('PROPPATCH', '/', ['Prefer' => 'return-minimal'])
+        request.body = <<BLA
+<?xml version="1.0"?>
+<d:propertyupdate xmlns:d="DAV:">
+    <d:set>
+        <d:prop>
+            <d:something>nope!</d:something>
+        </d:prop>
+    </d:set>
+</d:propertyupdate>
+BLA
+
+        response = self.request(request)
+
+        body = response.body_as_string
+
+        assert_equal(207, response.status)
+        assert(body.index('something'))
+        assert(body.index('403 Forbidden'), body)
+      end
+    end
+  end
+end