dolly 3.1.1 → 3.1.3

data/test/mango_test.rb

@@ -26,7 +26,9 @@ class MangoTest < Test::Unit::TestCase
 
     view_resp   = build_view_response [data]
     empty_resp  =  build_view_response []
-    not_found_resp = generic_response [{ key: "foo_bar/2", error: "not_found" }]
+
+    generic_response [{ key: "foo_bar/2", error: "not_found" }]
+
     @multi_resp = build_view_response all_docs
     @multi_type_resp = build_view_collation_response all_docs
 
@@ -68,22 +70,6 @@ class MangoTest < Test::Unit::TestCase
     assert_equal(FooBar.find_by(foo: 'bar').class, FooBar)
   end
 
-  test '#find_by for a property that does not have an index' do
-    #TODO: clean up all the fake request creation
-    resp = { docs: [{ foo: 'bar', id: "foo_bar/1"} ] }
-    key = 'date'
-
-    stub_request(:post, query_base_path).
-     to_return(body: resp.to_json)
-
-    stub_request(:get, "#{all_docs_path}?key=\"_design/index_#{key}\"").
-     to_return(body: { rows: [] }.to_json)
-
-    assert_raise Dolly::IndexNotFoundError do
-      FooBar.find_by(date: Date.today)
-    end
-  end
-
   test '#find_by with no returned data' do
     resp = { docs: [] }
 
@@ -125,22 +111,6 @@ class MangoTest < Test::Unit::TestCase
     assert_equal(FooBar.where(foo: { eq: 'bar' }).map(&:class).uniq, [FooBar])
   end
 
-  test '#where for a property that does not have an index' do
-    #TODO: clean up all the fake request creation
-    resp = { docs: [{ foo: 'bar', id: "foo_bar/1"} ] }
-
-    stub_request(:post, query_base_path).
-     to_return(body: resp.to_json)
-
-    key = 'date'
-    stub_request(:get, "#{all_docs_path}?key=\"_design/index_#{key}\"").
-     to_return(body: { rows: [] }.to_json)
-
-    assert_raise Dolly::IndexNotFoundError do
-      FooBar.where(date: Date.today)
-    end
-  end
-
   test '#where with no returned data' do
     resp = { docs: [] }
 

data/test/partitioned_document_test.rb

@@ -0,0 +1,38 @@
+require 'test_helper'
+
+class PartitionedDocumentTest < Test::Unit::TestCase
+  PARTITIONED_DB_PATH = 'http://localhost:5984/partitioned/'
+
+  test 'unpartitioned DB will raise exception' do
+    stub_request(:get, PARTITIONED_DB_PATH).
+      with(headers: { 'Content-Type'=>'application/json' }).
+      to_return(status: 200, body: { db_name: 'partitioned' }.to_json)
+
+    assert_raise(Dolly::PartitionedDataBaseExpectedError) do
+      class Partitioned < Dolly::Document
+        partitioned!
+        property :foo, class_name: String
+      end
+    end
+  end
+
+  test 'document id is partitioned' do
+    stub_request(:get, PARTITIONED_DB_PATH).
+      with(headers: { 'Content-Type'=>'application/json' }).
+      to_return(status: 200, body: { db_name: 'partitioned', props: { partitioned: true } }.to_json)
+
+    class Partitioned < Dolly::Document
+      set_namespace 'partitioned'
+      partitioned!
+
+      property :foo, class_name: String
+    end
+
+    assert_equal Partitioned.absolute_id("partitioned:abc"), "abc"
+
+    doc = Partitioned.new(foo: "something")
+    assert doc.id =~ /^partitioned:.+/
+  end
+end
+
+

data/test/property_manager_test.rb

@@ -4,15 +4,17 @@ class TestDoc < Dolly::Document
   property :name, class_name: String
   property :email, class_name: String
   property :last_name, class_name: String
+  property :active, class_name: TrueClass
 end
 
 class PropertyManagerTest < Test::Unit::TestCase
   test 'write_attribute with nil value' do
-    doc = TestDoc.new(name: 'name', last_name: nil, email: 'does not change')
+    doc = TestDoc.new(name: 'name', last_name: nil, email: 'does not change', active: 'true')
     assert_equal(doc.name, 'name')
     doc.update_properties(name: nil)
     assert_equal(doc.name, nil)
     assert_equal(doc.last_name, nil)
     assert_equal(doc.email, 'does not change')
+    assert_equal(doc.active, true)
   end
 end

data/test/slugable_document_test.rb

@@ -0,0 +1,51 @@
+require 'test_helper'
+
+class SlugedDocument < Dolly::Document
+  include Dolly::Slugable
+  property :name, class_name: String
+  set_slug :name
+end
+
+class MultiSlugedDocument < Dolly::Document
+  include Dolly::Slugable
+  property :foo, class_name: String
+  property :bar, class_name: Integer
+
+  set_slug :foo, :bar
+end
+
+class CustomSeparatorSlug < Dolly::Document
+  include Dolly::Slugable
+  property :a, class_name: Integer
+  property :b, class_name: String
+  property :c, class_name: String
+
+  set_slug :a, :b, :c, separator: '*'
+end
+
+class SlugableDocumentTest < Test::Unit::TestCase
+  test 'default id of slugable document is name' do
+    assert_equal SlugedDocument.new(name: "a").id, "sluged_document/a"
+  end
+
+  test 'bundled slug id' do
+    assert_equal MultiSlugedDocument.new(foo: "a", bar: 1).id, "multi_sluged_document/a_1"
+  end
+
+  test 'bundled slug id cant be empty' do
+    assert_equal MultiSlugedDocument.new(foo: "a").id, "multi_sluged_document/a_"
+  end
+
+  test 'custom seperator slug' do
+    assert_equal CustomSeparatorSlug.new(a: 1, b: 'x', c: 'c').id, "custom_separator_slug/1*x*c"
+  end
+
+  test 'raise missing slug exception' do
+    assert_raise(Dolly::MissingSlugableProperties) do
+      class MissingSlugDocument < Dolly::Document
+        include Dolly::Slugable
+        set_slug :foo
+      end
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: dolly
 version: !ruby/object:Gem::Version
-  version: 3.1.1
+  version: 3.1.3
 platform: ruby
 authors:
 - javierg
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2021-10-05 00:00:00.000000000 Z
+date: 2022-10-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: oj
@@ -66,6 +66,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '13.0'
+- !ruby/object:Gem::Dependency
+  name: rexml
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: test-unit-full
   requirement: !ruby/object:Gem::Requirement
@@ -141,6 +155,7 @@ files:
 - lib/dolly/query_arguments.rb
 - lib/dolly/request.rb
 - lib/dolly/request_header.rb
+- lib/dolly/slugable.rb
 - lib/dolly/timestamp.rb
 - lib/dolly/version.rb
 - lib/dolly/view_query.rb
@@ -151,12 +166,12 @@ files:
 - test/bulk_document_test.rb
 - test/document_test.rb
 - test/document_type_test.rb
-- test/dummy/config/initializers/filter_parameter_logging.rb
-- test/dummy/log/test.log
 - test/inheritance_test.rb
 - test/mango_index_test.rb
 - test/mango_test.rb
+- test/partitioned_document_test.rb
 - test/property_manager_test.rb
+- test/slugable_document_test.rb
 - test/support/test.txt
 - test/test_helper.rb
 - test/view_query_test.rb
@@ -183,8 +198,8 @@ signing_key:
 specification_version: 4
 summary: will write something
 test_files:
-- test/dummy/config/initializers/filter_parameter_logging.rb
-- test/dummy/log/test.log
+- test/partitioned_document_test.rb
+- test/slugable_document_test.rb
 - test/inheritance_test.rb
 - test/document_type_test.rb
 - test/mango_index_test.rb

data/test/dummy/config/initializers/filter_parameter_logging.rb

@@ -1,4 +0,0 @@
-# Be sure to restart your server when you modify this file.
-
-# Configure sensitive parameters which will be filtered from the log file.
-Rails.application.config.filter_parameters += [:password]