superstore 2.4.4 → 2.5.0

data/test/unit/base_test.rb

@@ -17,4 +17,8 @@ class Superstore::BaseTest < Superstore::TestCase
     assert_equal 'sons', Son.table_name
     assert_equal 'sons', Grandson.table_name
   end
+
+  test 'translations' do
+    assert_equal :activerecord, Son.i18n_scope
+  end
 end

data/test/unit/caching_test.rb

@@ -16,6 +16,6 @@ class Superstore::CachingTest < Superstore::TestCase
     updated_at = Time.now
     issue = Issue.create!(id: 1, updated_at: updated_at)
 
-    assert_equal "issues/1-#{updated_at.utc.to_s(:nsec)}", issue.cache_key
+    assert_equal "issues/1-#{updated_at.utc.to_s(:usec)}", issue.cache_key
   end
 end

data/test/unit/callbacks_test.rb

@@ -3,7 +3,7 @@ require 'test_helper'
 class Superstore::CallbacksTest < Superstore::TestCase
   class TestIssue < Superstore::Base
     self.table_name = 'issues'
-    string :description
+    attribute :description, type: :string
 
     %w(
       before_validation
@@ -35,8 +35,8 @@ class Superstore::CallbacksTest < Superstore::TestCase
       before_validation
       after_validation
       before_save
-      after_save
       after_create
+      after_save
     )
     assert_equal expected, issue.callback_history
   end
@@ -47,7 +47,7 @@ class Superstore::CallbacksTest < Superstore::TestCase
 
     issue.update_attribute :description, 'foo'
 
-    assert_equal %w(before_save after_save after_update), issue.callback_history
+    assert_equal %w(before_save after_update after_save), issue.callback_history
   end
 
   test 'destroy' do
@@ -62,7 +62,7 @@ class Superstore::CallbacksTest < Superstore::TestCase
   test 'new_record during callbacks' do
     class NewRecordTestClass < Superstore::Base
       self.table_name = 'issues'
-      string :description
+      attribute :description, type: :string
 
       before_create :expect_new_record
       before_save   :expect_new_record

data/test/unit/core_test.rb

@@ -1,30 +1,13 @@
 require 'test_helper'
 
 class Superstore::CoreTest < Superstore::TestCase
-  test 'initialiaze' do
+  test 'initialize' do
     issue = Issue.new
 
     assert issue.new_record?
     assert !issue.destroyed?
   end
 
-  test 'initialize yields self' do
-    issue = Issue.new { |i| i.description = 'bar' }
-    assert_equal 'bar', issue.description
-  end
-
-  test 'dup' do
-    issue = Issue.create description: 'foo'
-
-    dup_issue = issue.dup
-
-    assert dup_issue.new_record?
-    assert_not_equal issue.id, dup_issue.id
-    assert_nil dup_issue.created_at
-    assert_nil dup_issue.updated_at
-    assert_equal 'foo', issue.description
-  end
-
   test 'equality of new records' do
     assert_not_equal Issue.new, Issue.new
   end
@@ -45,7 +28,14 @@ class Superstore::CoreTest < Superstore::TestCase
 
   test 'hash' do
     issue = Issue.create
-    assert_equal issue.id.hash, issue.hash
+    issue2 = Issue.create
+    refute_equal issue.hash, issue2.hash
+
+    issue3 = Issue.new(id: issue.id)
+    assert_equal issue.hash, issue3.hash
+
+    user = User.new(id: issue.id)
+    refute_equal issue.hash, user.hash
   end
 
   test 'inspect' do

data/test/unit/persistence_test.rb

@@ -4,47 +4,8 @@
 require 'test_helper'
 
 class Superstore::PersistenceTest < Superstore::TestCase
-  test 'instantiate removes unknowns' do
-    assert_nil Issue.instantiate('theid', 'z' => 'nooo').attributes['z']
-  end
-
-  test 'encode_attributes' do
-    klass = temp_object do
-      string :description
-    end
-
-    assert_equal(
-      {},
-      klass.encode_attributes({})
-    )
-
-    assert_equal(
-      {'description' => nil},
-      klass.encode_attributes({'description' => nil})
-    )
-
-    assert_equal(
-      {'description' => 'lol'},
-      klass.encode_attributes({'description' => 'lol'})
-    )
-  end
-
-  test "batch" do
-    first_issue = second_issue = nil
-
-    Issue.batch do
-      assert Issue.batching?
-
-      first_issue = Issue.create
-      second_issue = Issue.create
-
-      assert_raise(Superstore::RecordNotFound) { Issue.find(first_issue.id) }
-      assert_raise(Superstore::RecordNotFound) { Issue.find(second_issue.id) }
-    end
-
-    assert !Issue.batching?
-    assert_nothing_raised { Issue.find(first_issue.id) }
-    assert_nothing_raised { Issue.find(second_issue.id) }
+  test 'instantiate with unknowns' do
+    assert_equal 'nooo', Issue.instantiate('id' => 'theid', 'document' => {'z' => 'nooo'}.to_json).attributes['z']
   end
 
   test 'persistence inquiries' do
@@ -81,14 +42,14 @@ class Superstore::PersistenceTest < Superstore::TestCase
 
   test 'save!' do
     klass = temp_object do
-      string :description
+      attribute :description, type: :string
       validates :description, presence: true
     end
 
     record = klass.new(description: 'bad')
     record.save!
 
-    assert_raise Superstore::RecordInvalid do
+    assert_raise ActiveRecord::RecordInvalid do
       record = klass.new
       record.save!
     end
@@ -126,7 +87,7 @@ class Superstore::PersistenceTest < Superstore::TestCase
       issue = Issue.new(description: 'bad')
       issue.save!
 
-      assert_raise Superstore::RecordInvalid do
+      assert_raise ActiveRecord::RecordInvalid do
         issue.update! description: ''
       end
     ensure
@@ -151,7 +112,7 @@ class Superstore::PersistenceTest < Superstore::TestCase
 
   test 'becomes includes changed_attributes' do
     klass = temp_object do
-      string :title
+      attribute :title, type: :string
     end
 
     issue = Issue.new(title: 'Something is wrong')
@@ -171,14 +132,9 @@ class Superstore::PersistenceTest < Superstore::TestCase
     assert_equal persisted_issue, reloaded_issue
   end
 
-  test 'quote_columns' do
-    klass = Class.new { include Superstore::Persistence }
-    assert_equal %w{'a' 'b'}, klass.__send__(:quote_columns, %w{a b})
-  end
-
   test 'delete' do
     klass = temp_object do
-      string :name
+      attribute :name, type: :string
     end
 
     record = klass.new(name: 'cool')
@@ -189,14 +145,14 @@ class Superstore::PersistenceTest < Superstore::TestCase
 
     klass.delete(id)
 
-    assert_raise Superstore::RecordNotFound do
+    assert_raise ActiveRecord::RecordNotFound do
       klass.find(id)
     end
   end
 
   test 'delete multiple' do
     klass = temp_object do
-      string :name
+      attribute :name, type: :string
     end
 
     ids = []
@@ -207,7 +163,14 @@ class Superstore::PersistenceTest < Superstore::TestCase
 
     klass.delete(ids)
 
-    assert_equal [], klass.find(ids)
+    assert_equal [], klass.where(id: ids)
   end
 
+  test 'find_by_id' do
+    Issue.create.tap do |issue|
+      assert_equal issue, Issue.find_by_id(issue.id)
+    end
+
+    assert_nil Issue.find_by_id('what')
+  end
 end

data/test/unit/{scope/batches_test.rb → relation/scrolling_test.rb}

@@ -1,25 +1,29 @@
 require 'test_helper'
 
-class Superstore::BatchesTest < Superstore::TestCase
-  test 'find_each' do
+class Superstore::ScrollingTest < Superstore::TestCase
+  ActiveRecord::Relation.class_eval do
+    include Superstore::Relation::Scrolling
+  end
+  
+  test 'scroll_each' do
     Issue.create
     Issue.create
 
     issues = []
-    Issue.find_each do |issue|
+    Issue.all.scroll_each do |issue|
       issues << issue
     end
 
     assert_equal Issue.all.to_set, issues.to_set
   end
 
-  test 'find_in_batches' do
+  test 'scroll_in_batches' do
     Issue.create
     Issue.create
     Issue.create
 
     issue_batches = []
-    Issue.find_in_batches(batch_size: 2) do |issues|
+    Issue.all.scroll_in_batches(batch_size: 2) do |issues|
       issue_batches << issues
     end
 

data/test/unit/serialization_test.rb

@@ -3,8 +3,16 @@ require 'test_helper'
 class Superstore::SerializationTest < Superstore::TestCase
   test 'as_json' do
     issue = Issue.new
-    expected = {"id" => issue.id}
+    expected = {
+      "id"          => issue.id,
+      "created_at"  => nil,
+      "updated_at"  => nil,
+      "description" => nil,
+      "title"       => nil,
+      "parent_issue_id" => nil,
+      "comments"    => nil
+    }
 
     assert_equal expected, issue.as_json
   end
-end
+end

data/test/unit/{timestamps_test.rb → timestamp_test.rb}

@@ -1,8 +1,8 @@
 require 'test_helper'
 
-class Superstore::TimestampsTest < Superstore::TestCase
+class Superstore::TimestampTest < Superstore::TestCase
   test 'timestamps set on create' do
-    issue = Issue.create
+    issue = Issue.create#issue.created_at.to_i = #{issue.created_at.to_i}
 
     assert_in_delta Time.now.to_i, issue.created_at.to_i, 3
     assert_in_delta Time.now.to_i, issue.updated_at.to_i, 3
@@ -30,13 +30,13 @@ class Superstore::TimestampsTest < Superstore::TestCase
     time = 5.days.ago
     issue = Issue.create created_at: time
 
-    assert_equal time, issue.created_at
+    assert_in_delta time.to_i, issue.created_at.to_i, 3
   end
 
   test 'updated_at sets only if nil' do
-    time = 5.days.ago
+    time = 5.days.ago.utc
     issue = Issue.create updated_at: time
 
-    assert_equal time, issue.updated_at
+    assert_in_delta time.to_i, issue.updated_at.to_i, 3
   end
 end

data/test/unit/types/array_type_test.rb

@@ -1,23 +1,8 @@
 require 'test_helper'
 
 class Superstore::Types::ArrayTypeTest < Superstore::Types::TestCase
-  if Superstore::Base.adapter.class.name == 'Superstore::Adapters::CassandraAdapter'
-    test 'encode' do
-      assert_equal ['1', '2'].to_json, type.encode(['1', '2'])
-
-      assert_raise ArgumentError do
-        type.encode('wtf')
-      end
-    end
-
-    test 'decode' do
-      assert_equal ['1', '2'], type.decode(['1', '2'].to_json)
-      assert_equal nil, type.decode(nil)
-      assert_equal nil, type.decode('')
-    end
-  end
-
-  test 'typecast' do
-    assert_equal ['x', 'y'], type.typecast(['x', 'y'].to_set)
+  test 'cast_value' do
+    assert_equal ['x', 'y'], type.cast_value(['x', 'y'].to_set)
+    assert_equal ['x'], type.cast_value('x')
   end
 end

data/test/unit/types/boolean_type_test.rb

@@ -1,26 +1,12 @@
 require 'test_helper'
 
 class Superstore::Types::BooleanTypeTest < Superstore::Types::TestCase
-  if Superstore::Base.adapter.class.name == 'Superstore::Adapters::CassandraAdapter'
-    test 'encode' do
-      assert_equal '1', type.encode(true)
-      assert_equal '1', type.encode('true')
-      assert_equal '1', type.encode('1')
-
-      assert_equal '0', type.encode(false)
-      assert_equal '0', type.encode('false')
-      assert_equal '0', type.encode('0')
-      assert_equal '0', type.encode('')
-
-      assert_raise ArgumentError do
-        type.encode('wtf')
-      end
-    end
-  end
-
-  test 'decode' do
-    assert_equal true, type.decode('1')
-    assert_equal false, type.decode('0')
-    # assert_nil type.decode(nil)
+  test 'cast_value' do
+    assert_equal true, type.cast_value('1')
+    assert_equal true, type.cast_value('true')
+    assert_equal true, type.cast_value('true')
+    assert_equal false, type.cast_value('0')
+    assert_equal false, type.cast_value(false)
+    assert_nil type.cast_value('')
   end
 end

data/test/unit/types/date_range_type_test.rb

@@ -0,0 +1,28 @@
+require 'test_helper'
+
+class Superstore::Types::DateRangeTypeTest < Superstore::Types::TestCase
+  test 'serialize' do
+    assert_equal ["2004-04-25", "2004-05-15"], type.serialize(Date.new(2004, 4, 25) .. Date.new(2004, 5, 15))
+  end
+
+  test 'deserialize' do
+    assert_equal Date.new(2004, 4, 25)..Date.new(2004, 5, 15), type.deserialize(["2004-04-25", "2004-05-15"])
+    assert_nil type.deserialize(["2004-05-15", "2004-04-25"])
+
+    # decode returns argument if already a Range
+    range = Date.new(2019, 1, 1)..Date.new(2019, 2, 1)
+    assert_equal range, type.deserialize(range)
+  end
+
+  test 'cast_value' do
+    assert_equal Date.new(2004, 4, 25)..Date.new(2004, 5, 15), type.cast_value(Date.new(2004, 4, 25)..Date.new(2004, 5, 15))
+    assert_nil type.cast_value(Date.new(2004, 5, 15)..Date.new(2004, 4, 25))
+    assert_equal Date.new(2004, 4, 25)..Date.new(2004, 5, 15), type.cast_value([Date.new(2004, 4, 25), Date.new(2004, 5, 15)])
+    assert_nil type.cast_value([Date.new(2004, 5, 15), Date.new(2004, 4, 25)])
+    assert_equal Date.new(2004, 4, 25)..Date.new(2004, 5, 15), type.cast_value(["2004-04-25", "2004-05-15"])
+
+    assert_nil type.cast_value(["2004-04-25", nil])
+    assert_nil type.cast_value([nil, "2004-05-15"])
+    assert_nil type.cast_value(["xx", "2004-05-15"])
+  end
+end

data/test/unit/types/date_type_test.rb

@@ -1,15 +1,24 @@
 require 'test_helper'
 
 class Superstore::Types::DateTypeTest < Superstore::Types::TestCase
-  test 'encode' do
-    assert_equal '2004-04-25', type.encode(Date.new(2004, 4, 25))
+  test 'serialize' do
+    assert_equal '2004-04-25', type.serialize(Date.new(2004, 4, 25))
   end
 
-  test 'decode' do
-    assert_equal Date.new(2004, 4, 25), type.decode('2004-04-25')
+  test 'deserialize' do
+    assert_equal Date.new(2004, 4, 25), type.deserialize('2004-04-25')
   end
 
-  test 'decoding a blank dates' do
-    assert_nil type.decode('')
+  test 'cast_value' do
+    assert_nil type.cast_value(1000)
+    assert_nil type.cast_value(1000.0)
+    assert_nil type.cast_value('')
+    assert_nil type.cast_value('nil')
+    assert_nil type.cast_value('bad format')
+    assert_equal Date.new(2004, 4, 25), type.cast_value('2004-04-25')
+    assert_equal Date.new(2017, 5, 1), type.cast_value('2017-05-01T21:39:06.796897Z')
+
+    my_time = Time.current
+    assert_equal my_time.to_date, type.cast_value(my_time)
   end
 end

data/test/unit/types/float_type_test.rb

@@ -1,24 +1,9 @@
 require 'test_helper'
 
 class Superstore::Types::FloatTypeTest < Superstore::Types::TestCase
-  if Superstore::Base.adapter.class.name == 'Superstore::Adapters::CassandraAdapter'
-    test 'encode' do
-      assert_equal '5.01', type.encode(5.01)
-
-      assert_raise ArgumentError do
-        type.encode('x')
-      end
-    end
-  end
-
-  test 'decode' do
-    assert_equal 0.0, type.decode('xyz')
-    assert_equal 3.14, type.decode('3.14')
-    assert_equal 5, type.decode('5')
-  end
-
-  test 'typecast' do
-    assert_equal 1.1, type.typecast('1.1')
-    assert_equal 42.0, type.typecast(42)
+  test 'cast_value' do
+    assert_nil type.cast_value('xyz')
+    assert_equal 1.1, type.cast_value('1.1')
+    assert_equal 42.0, type.cast_value(42)
   end
 end