tilia-dav 3.1.0.pre.alpha2

data/test/dav/property_storage/backend/abstract_sequel_test.rb

@@ -0,0 +1,147 @@
+module Tilia
+  module Dav
+    module PropertyStorage
+      module Backend
+        module AbstractSequelTest
+          # Should return an instance of \PDO with the current tables initialized,
+          # and some test records.
+          def sequel
+            fail NotImplementedError
+          end
+
+          def backend
+            Sequel.new(sequel)
+          end
+
+          def test_prop_find
+            backend = self.backend
+
+            prop_find = PropFind.new('dir', ['{DAV:}displayname'])
+            backend.prop_find('dir', prop_find)
+
+            assert_equal('Directory', prop_find.get('{DAV:}displayname'))
+          end
+
+          def test_prop_find_nothing_to_do
+            backend = self.backend
+
+            prop_find = PropFind.new('dir', ['{DAV:}displayname'])
+            prop_find.set('{DAV:}displayname', 'foo')
+            backend.prop_find('dir', prop_find)
+
+            assert_equal('foo', prop_find.get('{DAV:}displayname'))
+          end
+
+          def test_prop_patch_update
+            backend = self.backend
+
+            prop_patch = PropPatch.new('{DAV:}displayname' => 'bar')
+            backend.prop_patch('dir', prop_patch)
+            prop_patch.commit
+
+            prop_find = PropFind.new('dir', ['{DAV:}displayname'])
+            backend.prop_find('dir', prop_find)
+
+            assert_equal('bar', prop_find.get('{DAV:}displayname'))
+          end
+
+          def test_prop_patch_complex
+            backend = self.backend
+
+            complex = Xml::Property::Complex.new('<foo xmlns="DAV:">somevalue</foo>')
+
+            prop_patch = PropPatch.new('{DAV:}complex' => complex)
+            backend.prop_patch('dir', prop_patch)
+            prop_patch.commit
+
+            prop_find = PropFind.new('dir', ['{DAV:}complex'])
+            backend.prop_find('dir', prop_find)
+
+            assert_equal(complex, prop_find.get('{DAV:}complex'))
+          end
+
+          def test_prop_patch_custom
+            backend = self.backend
+
+            custom = Xml::Property::Href.new('/foo/bar/')
+
+            prop_patch = PropPatch.new('{DAV:}custom' => custom)
+            backend.prop_patch('dir', prop_patch)
+            prop_patch.commit
+
+            prop_find = PropFind.new('dir', ['{DAV:}custom'])
+            backend.prop_find('dir', prop_find)
+
+            assert_instance_equal(custom, prop_find.get('{DAV:}custom'))
+          end
+
+          def test_prop_patch_remove
+            backend = self.backend
+
+            prop_patch = PropPatch.new('{DAV:}displayname' => nil)
+            backend.prop_patch('dir', prop_patch)
+            prop_patch.commit
+
+            prop_find = PropFind.new('dir', ['{DAV:}displayname'])
+            backend.prop_find('dir', prop_find)
+
+            assert_equal(nil, prop_find.get('{DAV:}displayname'))
+          end
+
+          def test_delete
+            backend = self.backend
+            backend.delete('dir')
+
+            prop_find = PropFind.new('dir', ['{DAV:}displayname'])
+            backend.prop_find('dir', prop_find)
+
+            assert_equal(nil, prop_find.get('{DAV:}displayname'))
+          end
+
+          def test_move
+            backend = self.backend
+            # Creating a new child property.
+            prop_patch = PropPatch.new('{DAV:}displayname' => 'child')
+            backend.prop_patch('dir/child', prop_patch)
+            prop_patch.commit
+
+            backend.move('dir', 'dir2')
+
+            # Old 'dir'
+            prop_find = PropFind.new('dir', ['{DAV:}displayname'])
+            backend.prop_find('dir', prop_find)
+            assert_equal(nil, prop_find.get('{DAV:}displayname'))
+
+            # Old 'dir/child'
+            prop_find = PropFind.new('dir/child', ['{DAV:}displayname'])
+            backend.prop_find('dir/child', prop_find)
+            assert_equal(nil, prop_find.get('{DAV:}displayname'))
+
+            # New 'dir2'
+            prop_find = PropFind.new('dir2', ['{DAV:}displayname'])
+            backend.prop_find('dir2', prop_find)
+            assert_equal('Directory', prop_find.get('{DAV:}displayname'))
+
+            # New 'dir2/child'
+            prop_find = PropFind.new('dir2/child', ['{DAV:}displayname'])
+            backend.prop_find('dir2/child', prop_find)
+            assert_equal('child', prop_find.get('{DAV:}displayname'))
+          end
+
+          def test_deep_delete
+            backend = self.backend
+            prop_patch = PropPatch.new('{DAV:}displayname' => 'child')
+            backend.prop_patch('dir/child', prop_patch)
+            prop_patch.commit
+            backend.delete('dir')
+
+            prop_find = PropFind.new('dir/child', ['{DAV:}displayname'])
+            backend.prop_find('dir/child', prop_find)
+
+            assert_equal(nil, prop_find.get('{DAV:}displayname'))
+          end
+        end
+      end
+    end
+  end
+end

data/test/dav/property_storage/backend/mock.rb

@@ -0,0 +1,96 @@
+module Tilia
+  module Dav
+    module PropertyStorage
+      module Backend
+        class Mock
+          include BackendInterface
+
+          attr_accessor :data
+
+          def initialize
+            @data = {}
+          end
+
+          # Fetches properties for a path.
+          #
+          # This method received a PropFind object, which contains all the
+          # information about the properties that need to be fetched.
+          #
+          # Ususually you would just want to call 'get404Properties' on this object,
+          # as this will give you the _exact_ list of properties that need to be
+          # fetched, and haven't yet.
+          #
+          # @param string path
+          # @param PropFind prop_find
+          # @return void
+          def prop_find(path, prop_find)
+            return nil unless @data.key?(path)
+
+            @data[path].each do |name, value|
+              prop_find.set(name, value)
+            end
+          end
+
+          # Updates properties for a path
+          #
+          # This method received a PropPatch object, which contains all the
+          # information about the update.
+          #
+          # Usually you would want to call 'handleRemaining' on this object, to get
+          # a list of all properties that need to be stored.
+          #
+          # @param string path
+          # @param PropPatch prop_patch
+          # @return void
+          def prop_patch(path, prop_patch)
+            @data[path] = {} unless @data.key?(path)
+
+            prop_patch.handle_remaining(
+              lambda do |properties|
+                properties.each do |prop_name, prop_value|
+                  if prop_value.nil?
+                    @data[path].delete(prop_name)
+                  else
+                    @data[path][prop_name] = prop_value
+                  end
+                  return true
+                end
+              end
+            )
+          end
+
+          # This method is called after a node is deleted.
+          #
+          # This allows a backend to clean up all associated properties.
+          def delete(path)
+            @data.delete(path)
+          end
+
+          # This method is called after a successful MOVE
+          #
+          # This should be used to migrate all properties from one path to another.
+          # Note that entire collections may be moved, so ensure that all properties
+          # for children are also moved along.
+          #
+          # @param string source
+          # @param string destination
+          # @return void
+          def move(source, destination)
+            @data.each do |path, props|
+              if path == source
+                @data[destination] = props
+                @data.delete(path)
+                next
+              end
+
+              if path.index("#{source}/") == 0
+                @data["#{destination}#{path[source.length + 1..-1]}"] = props
+                @data.delete(path)
+              end
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/test/dav/property_storage/backend/sequel_mysql_test.rb

@@ -0,0 +1,32 @@
+require 'test_helper'
+
+module Tilia
+  module Dav
+    module PropertyStorage
+      module Backend
+        class SequelMysqlTest < Minitest::Test
+          include AbstractSequelTest
+
+          def sequel
+            db = TestUtil.mysql
+            db.run('DROP TABLE IF EXISTS propertystorage')
+            db.run(<<QUERY
+CREATE TABLE propertystorage (
+  id INT UNSIGNED NOT NULL PRIMARY KEY AUTO_INCREMENT,
+  path VARBINARY(1024) NOT NULL,
+  name VARBINARY(100) NOT NULL,
+  valuetype INT UNSIGNED,
+  value VARBINARY(255)
+) #{TestUtil.mysql_engine}
+QUERY
+                  )
+            db.run('CREATE UNIQUE INDEX path_property ON propertystorage (path(600), name(100))')
+            db.run('INSERT INTO propertystorage (path, name, value) VALUES ("dir", "{DAV:}displayname", "Directory")')
+
+            db
+          end
+        end
+      end
+    end
+  end
+end

data/test/dav/property_storage/backend/sequel_sqlite_test.rb

@@ -0,0 +1,31 @@
+require 'test_helper'
+
+module Tilia
+  module Dav
+    module PropertyStorage
+      module Backend
+        class SequelSqliteTest < Minitest::Test
+          include AbstractSequelTest
+
+          def sequel
+            db = TestUtil.sqlite
+            db.run(<<SQL
+CREATE TABLE propertystorage (
+    id integer primary key asc,
+    path text,
+    name text,
+    valuetype integer,
+    value string
+)
+SQL
+                  )
+            db.run('CREATE UNIQUE INDEX path_property ON propertystorage (path, name)')
+            db.run('INSERT INTO propertystorage (path, name, value) VALUES ("dir", "{DAV:}displayname", "Directory")')
+
+            db
+          end
+        end
+      end
+    end
+  end
+end

data/test/dav/property_storage/plugin_test.rb

@@ -0,0 +1,90 @@
+require 'yaml'
+
+module Tilia
+  module Dav
+    module PropertyStorage
+      class PluginTest < DavServerTest
+        def setup
+          @setup_files = true
+
+          super
+
+          @backend = Backend::Mock.new
+          @plugin = Plugin.new(@backend)
+
+          @server.add_plugin(@plugin)
+        end
+
+        def test_get_info
+          assert_has_key('name', @plugin.plugin_info)
+        end
+
+        def test_set_property
+          @server.update_properties('', '{DAV:}displayname' => 'hi')
+          assert_equal(
+            {
+              '' => { '{DAV:}displayname' => 'hi' }
+            },
+            @backend.data
+          )
+        end
+
+        def test_get_property
+          test_set_property
+          result = @server.properties('', ['{DAV:}displayname'])
+
+          assert_equal(
+            {
+              '{DAV:}displayname' => 'hi'
+            },
+            result
+          )
+        end
+
+        def test_delete_property
+          test_set_property
+          @server.emit('afterUnbind', [''])
+          assert_equal({}, @backend.data)
+        end
+
+        def def(_test_move)
+          @server.tree.node_for_path('files').create_file('source')
+          @server.update_properties('files/source', '{DAV:}displayname' => 'hi')
+
+          request = new Http::Request.new('MOVE', '/files/source', 'Destination' => '/files/dest')
+          assert_equal(201, request.status)
+
+          result = @server.properties('/files/dest', ['{DAV:}displayname'])
+
+          assert_equal(
+            {
+              '{DAV:}displayname' => 'hi'
+            },
+            result
+          )
+
+          @server.tree.node_for_path('files').create_file('source')
+          result = @server.properties('/files/source', ['{DAV:}displayname'])
+
+          assert_equal([], result)
+        end
+
+        def test_set_property_in_filtered_path
+          @plugin.path_filter = lambda do |_path|
+            return false
+          end
+
+          @server.update_properties('', '{DAV:}displayname' => 'hi')
+          assert_equal({}, @backend.data)
+        end
+
+        def test_get_property_in_filtered_path
+          test_set_property_in_filtered_path
+          result = @server.properties('', ['{DAV:}displayname'])
+
+          assert_equal({}, result)
+        end
+      end
+    end
+  end
+end

data/test/dav/server_copy_move_test.rb

@@ -0,0 +1,164 @@
+require 'test_helper'
+
+module Tilia
+  module Dav
+    class ServerCopyMoveTest < AbstractServer
+      def setup
+        super
+
+        @server.debug_exceptions = true
+
+        ::File.open("#{@temp_dir}/test2.txt", 'w') { |f| f.write 'Test contents2' }
+        Dir.mkdir("#{@temp_dir}/col")
+        ::File.open("#{@temp_dir}/col/test.txt", 'w') { |f| f.write 'Test contents' }
+      end
+
+      def test_copy_over_write
+        server_vars = {
+          'REQUEST_PATH'   => '/test.txt',
+          'REQUEST_METHOD' => 'COPY',
+          'HTTP_DESTINATION' => '/test2.txt'
+        }
+
+        request = Http::Sapi.create_from_server_array(server_vars)
+        @server.http_request = request
+        @server.exec
+
+        assert_equal(204, @response.status, "Received an incorrect HTTP status. Full body inspection: #{@response.body_as_string}")
+        assert_equal(
+          {
+            'X-Sabre-Version' => [Version::VERSION],
+            'Content-Length' => ['0']
+          },
+          @response.headers
+        )
+
+        assert_equal('Test contents', ::File.read("#{@temp_dir}/test2.txt"))
+      end
+
+      def test_copy_to_self
+        server_vars = {
+          'REQUEST_PATH'   => '/test.txt',
+          'REQUEST_METHOD' => 'COPY',
+          'HTTP_DESTINATION' => '/test.txt'
+        }
+
+        request = Http::Sapi.create_from_server_array(server_vars)
+        @server.http_request = request
+        @server.exec
+
+        assert_equal(403, @response.status, "Received an incorrect HTTP status. Full body inspection: #{@response.body_as_string}")
+        assert_equal('Test contents', ::File.read("#{@temp_dir}/test.txt"))
+      end
+
+      def test_non_existant_parent
+        server_vars = {
+          'REQUEST_PATH'     => '/test.txt',
+          'REQUEST_METHOD'   => 'COPY',
+          'HTTP_DESTINATION' => '/testcol2/test2.txt',
+          'HTTP_OVERWRITE'   => 'F'
+        }
+
+        request = Http::Sapi.create_from_server_array(server_vars)
+        @server.http_request = request
+        @server.exec
+
+        assert_equal(
+          {
+            'X-Sabre-Version' => [Version::VERSION],
+            'Content-Type' => ['application/xml; charset=utf-8']
+          },
+          @response.headers
+        )
+
+        assert_equal(409, @response.status)
+      end
+
+      def test_random_overwrite_header
+        server_vars = {
+          'REQUEST_PATH'     => '/test.txt',
+          'REQUEST_METHOD'   => 'COPY',
+          'HTTP_DESTINATION' => '/testcol2/test2.txt',
+          'HTTP_OVERWRITE'   => 'SURE!'
+        }
+
+        request = Http::Sapi.create_from_server_array(server_vars)
+        @server.http_request = request
+        @server.exec
+
+        assert_equal(400, @response.status)
+      end
+
+      def test_copy_directory
+        server_vars = {
+          'REQUEST_PATH'   => '/col',
+          'REQUEST_METHOD' => 'COPY',
+          'HTTP_DESTINATION' => '/col2'
+        }
+
+        request = Http::Sapi.create_from_server_array(server_vars)
+        @server.http_request = request
+        @server.exec
+
+        assert_equal(201, @response.status, "Full response: #{@response.body_as_string}")
+
+        assert_equal(
+          {
+            'X-Sabre-Version' => [Version::VERSION],
+            'Content-Length' => ['0']
+          },
+          @response.headers
+        )
+
+        assert_equal('Test contents', ::File.read("#{@temp_dir}/col/test.txt"))
+      end
+
+      def test_simple_copy_file
+        server_vars = {
+          'REQUEST_PATH'   => '/test.txt',
+          'REQUEST_METHOD' => 'COPY',
+          'HTTP_DESTINATION' => '/test3.txt'
+        }
+
+        request = Http::Sapi.create_from_server_array(server_vars)
+        @server.http_request = request
+        @server.exec
+
+        assert_equal(
+          {
+            'X-Sabre-Version' => [Version::VERSION],
+            'Content-Length' => ['0']
+          },
+          @response.headers
+        )
+
+        assert_equal(201, @response.status)
+        assert_equal('Test contents', ::File.read("#{@temp_dir}/test3.txt"))
+      end
+
+      def test_simple_copy_collection
+        server_vars = {
+          'REQUEST_PATH'   => '/col',
+          'REQUEST_METHOD' => 'COPY',
+          'HTTP_DESTINATION' => '/col2'
+        }
+
+        request = Http::Sapi.create_from_server_array(server_vars)
+        @server.http_request = request
+        @server.exec
+
+        assert_equal(201, @response.status, "Incorrect status received. Full response body: #{@response.body_as_string}")
+
+        assert_equal(
+          {
+            'X-Sabre-Version' => [Version::VERSION],
+            'Content-Length' => ['0']
+          },
+          @response.headers
+        )
+
+        assert_equal('Test contents', ::File.read("#{@temp_dir}/col2/test.txt"))
+      end
+    end
+  end
+end