disposable 0.4.6 → 0.4.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0d17fbfc73eb6b25b90136d515733b694ac2631ad3861f47d62dde5517ec394b
-  data.tar.gz: be69f4aaed8fda60749dcb3dc3188bb57def014cd91715198baab0eebd801386
+  metadata.gz: 60b151f3e5bb984a08ae4631bb836caa4b08b62e64b57d260a93e156129c4729
+  data.tar.gz: f793d20d7b3fb398d07baddbaf9c835e1944b4f6bfe60f1fb36eb325bf6a5919
 SHA512:
-  metadata.gz: eb4b2eca60ed3001be80d50eff19fe008abf68030a99101eb0282b5fe65d091c21d62537a3dcbfafb32484e985d4e670377d701413de25832b8195f0ff08cdeb
-  data.tar.gz: b95131034f38c0907672943108c0633164817981ba0107c508e618467b69f0c7f2e914c91d1152219b3488e6432cd1a9839e394704a1465e1792ef540140613e
+  metadata.gz: e8ab3254e28140e886dc65341cb8d3f8ae53328f7bb304cd8e0760869e894440c615cafaae1b2d5b2aab1bf721d86d5d659fc40ac1f7301bb6a39d1303cbea02
+  data.tar.gz: ac09aaa979ddc3de44d409740d3d8a1121d07b498c59ac2d77496091c799b692fcc22d7f495174886a75491d71f230e1544fb87ebc89713e0d97d5bf22db9d82

data/.gitignore

@@ -15,3 +15,4 @@ test/tmp
 test/version_tmp
 tmp
 Gemfile*.lock
+.rubocop*

data/.travis.yml

@@ -1,42 +1,44 @@
 language: ruby
-sudo: false
 rvm:
   - ruby-head
+  - 2.6
   - 2.5
   - 2.4
-  - 2.3
-  - 2.2
 env:
-  - "DRY_TYPES=1.1"
-  - "DRY_TYPES=1.0"
-  - "DRY_TYPES=0.15"
-  - "DRY_TYPES=0.13"
-  - "DRY_TYPES=0.12"
-  - "ACTIVERECORD=5.0"
-  - "ACTIVERECORD=5.1"
-  - "ACTIVERECORD=5.2"
+  - "ACTIVERECORD=5.0 DRY_TYPES=0.13"
+  - "ACTIVERECORD=5.0 DRY_TYPES=0.14"
+  - "ACTIVERECORD=5.0 DRY_TYPES=0.15"
+  - "ACTIVERECORD=5.0 DRY_TYPES=1.0"
+  - "ACTIVERECORD=5.0 DRY_TYPES=1.1"
+  - "ACTIVERECORD=5.0 DRY_TYPES=1.2"
+  - "ACTIVERECORD=5.1 DRY_TYPES=0.13"
+  - "ACTIVERECORD=5.1 DRY_TYPES=0.14"
+  - "ACTIVERECORD=5.1 DRY_TYPES=0.15"
+  - "ACTIVERECORD=5.1 DRY_TYPES=1.0"
+  - "ACTIVERECORD=5.1 DRY_TYPES=1.1"
+  - "ACTIVERECORD=5.1 DRY_TYPES=1.2"
+  - "ACTIVERECORD=5.2 DRY_TYPES=0.13"
+  - "ACTIVERECORD=5.2 DRY_TYPES=0.14"
+  - "ACTIVERECORD=5.2 DRY_TYPES=0.15"
+  - "ACTIVERECORD=5.2 DRY_TYPES=1.0"
+  - "ACTIVERECORD=5.2 DRY_TYPES=1.1"
+  - "ACTIVERECORD=5.2 DRY_TYPES=1.2"
+  - "ACTIVERECORD=6.0 DRY_TYPES=0.13"
+  - "ACTIVERECORD=6.0 DRY_TYPES=0.14"
+  - "ACTIVERECORD=6.0 DRY_TYPES=0.15"
+  - "ACTIVERECORD=6.0 DRY_TYPES=1.0"
+  - "ACTIVERECORD=6.0 DRY_TYPES=1.1"
+  - "ACTIVERECORD=6.0 DRY_TYPES=1.2"
 gemfile:
   - Gemfile
 matrix:
   fast_finish: true
-  include:
-    - { rvm: 2.3, env: ACTIVERECORD=4.2  }
-    - { rvm: 2.2, env: ACTIVERECORD=4.2  }
-    - { rvm: 2.3, env: ACTIVERECORD=4.1  }
-    - { rvm: 2.2, env: ACTIVERECORD=4.1  }
-    - rvm: 2.1
-      env:
-        - ACTIVERECORD=4.1
-        - DRY_TYPES=0.10
   allow_failures:
     - rvm: ruby-head
-    - env: ACTIVERECORD=4.1
-    - rvm: 2.1
-    - rvm: 2.2
   exclude:
-    - { rvm: 2.3, env: DRY_TYPES=1.1 }
-    - { rvm: 2.2, env: DRY_TYPES=1.1 }
-    - { rvm: 2.3, env: DRY_TYPES=1.0 }
-    - { rvm: 2.2, env: DRY_TYPES=1.0 }
-    - { rvm: 2.3, env: DRY_TYPES=0.15 }
-    - { rvm: 2.2, env: DRY_TYPES=0.15 }
+    - {rvm: 2.4, env: "ACTIVERECORD=6.0 DRY_TYPES=0.13"}
+    - {rvm: 2.4, env: "ACTIVERECORD=6.0 DRY_TYPES=0.14"}
+    - {rvm: 2.4, env: "ACTIVERECORD=6.0 DRY_TYPES=0.15"}
+    - {rvm: 2.4, env: "ACTIVERECORD=6.0 DRY_TYPES=1.0"}
+    - {rvm: 2.4, env: "ACTIVERECORD=6.0 DRY_TYPES=1.1"}
+    - {rvm: 2.4, env: "ACTIVERECORD=6.0 DRY_TYPES=1.2"}

data/CHANGES.md

@@ -1,3 +1,8 @@
+# 0.4.7
+
+* Deprecation warning for nilify options for dry-v >= 1.x
+* Fix initializer struct when using `false` as value
+
 # 0.4.6
 
 * Remove deprecation warnings for newer versions of dry-types

data/Gemfile

@@ -7,4 +7,4 @@ gem "minitest-line"
   gem gem_name, dependency
 end
 
-
+gem "sqlite3", ENV.fetch('ACTIVERECORD', '5.2').to_f >= 6 ? '~> 1.4' : '~> 1.3.0'

data/disposable.gemspec

@@ -26,7 +26,6 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "rake"
   spec.add_development_dependency "minitest"
   spec.add_development_dependency "activerecord"#, "4.2.5"
-  spec.add_development_dependency "sqlite3", '~> 1.3.0'
   spec.add_development_dependency "dry-types"# "~> 0.6"
   # spec.add_development_dependency "database_cleaner"
 end

data/lib/disposable/twin/coercion.rb

@@ -18,7 +18,10 @@ module Disposable::Twin::Coercion
     end
 
     def coercing_setter!(name, type, nilify=false)
-     type = type ? (DRY_TYPES_CONSTANT::Nil | type) : DRY_TYPES_CONSTANT::Nil if nilify
+      # TODO: remove nilily with next release (0.5) for new dry-type versions
+      type = type ? (DRY_TYPES_CONSTANT::Nil | type) : DRY_TYPES_CONSTANT::Nil if nilify
+
+      warn "DEPRECATION WARNING [Disposable]: nilify is deprecated and it will be removed with the next release" if nilify && DRY_TYPES_VERSION >= Gem::Version.new("1.0.0")
 
       mod = Module.new do
         define_method("#{name}=") do |value|

data/lib/disposable/twin/property/struct.rb

@@ -6,7 +6,9 @@ class Disposable::Twin
     module Struct
       def read_value_for(dfn, options)
         name = dfn[:name]
-        @model[name.to_s] || @model[name.to_sym] # TODO: test sym vs. str.
+        # TODO: test sym vs. str.
+        return unless key_value = model.to_h.find { |k, _| k.to_sym == name.to_sym }
+        key_value.last
       end
 
       def sync_hash_representer # TODO: make this without representable, please.

data/lib/disposable/version.rb

@@ -1,3 +1,3 @@
 module Disposable
-  VERSION = "0.4.6"
+  VERSION = "0.4.7"
 end

data/test/callback_group_test.rb

@@ -55,7 +55,7 @@ class CallbackGroupTest < MiniTest::Spec
     album = Album.new(songs: [Song.new(title: "Dead To Me"), Song.new(title: "Diesel Boy")])
     twin  = AlbumTwin.new(album)
 
-    Group.new(twin).().invocations.must_equal [
+    expect(Group.new(twin).().invocations).must_equal [
       [:on_change, :change!, []],
       [:on_add, :notify_album!, []],
       [:on_add, :reset_song!, []],
@@ -76,7 +76,7 @@ class CallbackGroupTest < MiniTest::Spec
 
     group = Group.new(twin).(content: content)
 
-    group.invocations.must_equal [
+    expect(group.invocations).must_equal [
       [:on_change, :change!, [twin]],
       [:on_add, :notify_album!, [twin.songs[0], twin.songs[1]]],
       [:on_add, :reset_song!,   [twin.songs[0], twin.songs[1]]],
@@ -85,8 +85,8 @@ class CallbackGroupTest < MiniTest::Spec
       [:on_update, :expire_cache!, []],
     ]
 
-    content.must_equal "notify_album!notify_album!reset_song!reset_song!"
-    group.output.must_equal "Album has changed!"
+    expect(content).must_equal "notify_album!notify_album!reset_song!reset_song!"
+    expect(group.output).must_equal "Album has changed!"
   end
 
 
@@ -129,7 +129,7 @@ class CallbackGroupTest < MiniTest::Spec
 
     # pp group.invocations
 
-    group.invocations.must_equal [
+    expect(group.invocations).must_equal [
       [:on_change, :change!, [twin]],
       [:on_add, :notify_album!, [twin.songs[0]]],
       [:on_add, :reset_song!,   [twin.songs[0]]],
@@ -138,7 +138,7 @@ class CallbackGroupTest < MiniTest::Spec
       [:on_update, :expire_cache!, []],
     ]
 
-    content.must_equal "Op: changed! [CallbackGroupTest::Operation]Op: notify_album! [CallbackGroupTest::Operation]Op: reset_song! [CallbackGroupTest::Operation]"
+    expect(content).must_equal "Op: changed! [CallbackGroupTest::Operation]Op: notify_album! [CallbackGroupTest::Operation]Op: reset_song! [CallbackGroupTest::Operation]"
   end
 end
 
@@ -157,12 +157,12 @@ class CallbackGroupInheritanceTest < MiniTest::Spec
   end
 
   it do
-    Group.hooks.size.must_equal 4
-    Group.hooks[0].to_s.must_equal "[:on_change, :change!, {}]"
+    expect(Group.hooks.size).must_equal 4
+    expect(Group.hooks[0].to_s).must_equal "[:on_change, :change!, {}]"
     # Group.hooks[1][1][:nested].hooks.to_s.must_equal "[[:on_add, [:notify_album!]],[:on_add, [:reset_song!]]]"
-    Group.hooks[2].to_s.must_equal "[:on_change, :rehash_name!, {:property=>:title}]"
+    expect(Group.hooks[2].to_s).must_equal "[:on_change, :rehash_name!, {:property=>:title}]"
 
-    Group.definitions.get(Group.hooks[3][1])[:nested].hooks.to_s.must_equal "[[:on_change, :sing!, {}]]"
+    expect(Group.definitions.get(Group.hooks[3][1])[:nested].hooks.to_s).must_equal "[[:on_change, :sing!, {}]]"
   end
 
   class EmptyGroup < Group
@@ -171,7 +171,7 @@ class CallbackGroupInheritanceTest < MiniTest::Spec
 
 
   it do
-    EmptyGroup.hooks.size.must_equal 4
+    expect(EmptyGroup.hooks.size).must_equal 4
     # TODO:
   end
 
@@ -183,10 +183,10 @@ class CallbackGroupInheritanceTest < MiniTest::Spec
   end
 
   it do
-    Group.hooks.size.must_equal 4
+    expect(Group.hooks.size).must_equal 4
     # pp EnhancedGroup.hooks
-    EnhancedGroup.hooks.size.must_equal 6
-    EnhancedGroup.definitions.get(EnhancedGroup.hooks[5][1])[:nested].hooks.to_s.must_equal "[[:on_add, :rewind!, {}]]"
+    expect(EnhancedGroup.hooks.size).must_equal 6
+    expect(EnhancedGroup.definitions.get(EnhancedGroup.hooks[5][1])[:nested].hooks.to_s).must_equal "[[:on_add, :rewind!, {}]]"
   end
 
   class EnhancedWithInheritGroup < EnhancedGroup
@@ -199,13 +199,13 @@ class CallbackGroupInheritanceTest < MiniTest::Spec
   end
 
   it do
-    Group.hooks.size.must_equal 4
-    EnhancedGroup.hooks.size.must_equal 6
+    expect(Group.hooks.size).must_equal 4
+    expect(EnhancedGroup.hooks.size).must_equal 6
 
-    EnhancedGroup.definitions.get(EnhancedGroup.hooks[5][1])[:nested].hooks.to_s.must_equal "[[:on_add, :rewind!, {}]]"
-    EnhancedWithInheritGroup.hooks.size.must_equal 6
-    EnhancedWithInheritGroup.definitions.get(EnhancedWithInheritGroup.hooks[1][1])[:nested].hooks.to_s.must_equal "[[:on_add, :rewind!, {}], [:on_add, :eat!, {}]]"
-    EnhancedWithInheritGroup.definitions.get(EnhancedWithInheritGroup.hooks[3][1])[:nested].hooks.to_s.must_equal "[[:on_change, :sing!, {}], [:on_delete, :yell!, {}]]"
+    expect(EnhancedGroup.definitions.get(EnhancedGroup.hooks[5][1])[:nested].hooks.to_s).must_equal "[[:on_add, :rewind!, {}]]"
+    expect(EnhancedWithInheritGroup.hooks.size).must_equal 6
+    expect(EnhancedWithInheritGroup.definitions.get(EnhancedWithInheritGroup.hooks[1][1])[:nested].hooks.to_s).must_equal "[[:on_add, :rewind!, {}], [:on_add, :eat!, {}]]"
+    expect(EnhancedWithInheritGroup.definitions.get(EnhancedWithInheritGroup.hooks[3][1])[:nested].hooks.to_s).must_equal "[[:on_change, :sing!, {}], [:on_delete, :yell!, {}]]"
   end
 
   class RemovingInheritGroup < Group
@@ -226,10 +226,10 @@ class CallbackGroupInheritanceTest < MiniTest::Spec
   # TODO: object_id tests for all nested representers.
 
   it do
-    Group.hooks.size.must_equal 4
-    RemovingInheritGroup.hooks.size.must_equal 3
-    RemovingInheritGroup.definitions.get(RemovingInheritGroup.hooks[0][1])[:nested].hooks.to_s.must_equal "[[:on_add, :reset_song!, {}]]"
-    RemovingInheritGroup.definitions.get(RemovingInheritGroup.hooks[2][1])[:nested].hooks.to_s.must_equal "[[:on_change, :sing!, {}]]"
+    expect(Group.hooks.size).must_equal 4
+    expect(RemovingInheritGroup.hooks.size).must_equal 3
+    expect(RemovingInheritGroup.definitions.get(RemovingInheritGroup.hooks[0][1])[:nested].hooks.to_s).must_equal "[[:on_add, :reset_song!, {}]]"
+    expect(RemovingInheritGroup.definitions.get(RemovingInheritGroup.hooks[2][1])[:nested].hooks.to_s).must_equal "[[:on_change, :sing!, {}]]"
   end
 
   # Group::clone
@@ -239,7 +239,7 @@ class CallbackGroupInheritanceTest < MiniTest::Spec
   end
 
   it do
-    Group.hooks.size.must_equal 4
-    ClonedGroup.hooks.size.must_equal 3
+    expect(Group.hooks.size).must_equal 4
+    expect(ClonedGroup.hooks.size).must_equal 3
   end
-end
+end

data/test/callbacks_test.rb

@@ -46,7 +46,7 @@ class CallbacksTest < MiniTest::Spec
     it do
       invokes = []
       Callback.new(twin).on_create { |t| invokes << t }
-      invokes.must_equal []
+      expect(invokes).must_equal []
     end
 
     # save, without any attributes changed.
@@ -55,7 +55,7 @@ class CallbacksTest < MiniTest::Spec
 
       invokes = []
       Callback.new(twin).on_create { |t| invokes << t }
-      invokes.must_equal [twin]
+      expect(invokes).must_equal [twin]
     end
 
     # before and after save, with attributes changed
@@ -64,12 +64,12 @@ class CallbacksTest < MiniTest::Spec
       twin.name = "Run For Cover"
       invokes = []
       Callback.new(twin).on_create { |t| invokes << t }
-      invokes.must_equal []
+      expect(invokes).must_equal []
 
       twin.save
 
       Callback.new(twin).on_create { |t| invokes << t }
-      invokes.must_equal [twin]
+      expect(invokes).must_equal [twin]
     end
 
     # for collections.
@@ -80,12 +80,12 @@ class CallbacksTest < MiniTest::Spec
       invokes = []
 
       Callback.new(twin.songs).on_create { |t| invokes << t }
-      invokes.must_equal []
+      expect(invokes).must_equal []
 
       twin.save
 
       Callback.new(twin.songs).on_create { |t| invokes << t }
-      invokes.must_equal [twin.songs[0], twin.songs[2]]
+      expect(invokes).must_equal [twin.songs[0], twin.songs[2]]
     end
   end
 
@@ -96,7 +96,7 @@ class CallbacksTest < MiniTest::Spec
     it do
       invokes = []
       Callback.new(twin).on_update { |t| invokes << t }
-      invokes.must_equal []
+      expect(invokes).must_equal []
     end
 
     # single twin.
@@ -106,40 +106,40 @@ class CallbacksTest < MiniTest::Spec
 
       invokes = []
       Callback.new(twin).on_update { |t| invokes << t }
-      invokes.must_equal []
+      expect(invokes).must_equal []
 
       invokes = []
       twin.save
 
       Callback.new(twin).on_update { |t| invokes << t }
-      invokes.must_equal []
+      expect(invokes).must_equal []
 
 
       # now with the persisted album.
       twin = AlbumTwin.new(album) # Album is persisted now.
 
       Callback.new(twin).on_update { |t| invokes << t }
-      invokes.must_equal []
+      expect(invokes).must_equal []
 
       invokes = []
       twin.save
 
       # nothing has changed, yet.
       Callback.new(twin).on_update { |t| invokes << t }
-      invokes.must_equal []
+      expect(invokes).must_equal []
 
       twin.name= "Corridors Of Power"
 
       # this will even trigger on_update before saving.
       Callback.new(twin).on_update { |t| invokes << t }
-      invokes.must_equal [twin]
+      expect(invokes).must_equal [twin]
 
       invokes = []
       twin.save
 
       # name changed.
       Callback.new(twin).on_update { |t| invokes << t }
-      invokes.must_equal [twin]
+      expect(invokes).must_equal [twin]
     end
 
     # for collections.
@@ -150,41 +150,41 @@ class CallbacksTest < MiniTest::Spec
 
       invokes = []
       Callback.new(twin.songs).on_update { |t| invokes << t }
-      invokes.must_equal []
+      expect(invokes).must_equal []
 
       invokes = []
       twin.save
 
       # initial save is no update.
       Callback.new(twin.songs).on_update { |t| invokes << t }
-      invokes.must_equal []
+      expect(invokes).must_equal []
 
 
       # now with the persisted album.
       twin = AlbumTwin.new(album) # Album is persisted now.
 
       Callback.new(twin.songs).on_update { |t| invokes << t }
-      invokes.must_equal []
+      expect(invokes).must_equal []
 
       invokes = []
       twin.save
 
       # nothing has changed, yet.
       Callback.new(twin.songs).on_update { |t| invokes << t }
-      invokes.must_equal []
+      expect(invokes).must_equal []
 
       twin.songs[1].title= "After The War"
       twin.songs[2].title= "Run For Cover"
 
       # # this will even trigger on_update before saving.
       Callback.new(twin.songs).on_update { |t| invokes << t }
-      invokes.must_equal [twin.songs[1], twin.songs[2]]
+      expect(invokes).must_equal [twin.songs[1], twin.songs[2]]
 
       invokes = []
       twin.save
 
       Callback.new(twin.songs).on_update { |t| invokes << t }
-      invokes.must_equal [twin.songs[1], twin.songs[2]]
+      expect(invokes).must_equal [twin.songs[1], twin.songs[2]]
     end
     # it do
     #   album.songs << song1 = Song.new
@@ -210,7 +210,7 @@ class CallbacksTest < MiniTest::Spec
     it do
       invokes = []
       Callback.new(twin.songs).on_add { |t| invokes << t }
-      invokes.must_equal []
+      expect(invokes).must_equal []
     end
 
     # collection present on initialize are not added.
@@ -220,7 +220,7 @@ class CallbacksTest < MiniTest::Spec
       album.songs = [ex_song, song]
 
       Callback.new(twin.songs).on_add { |t| invokes << t }
-      invokes.must_equal []
+      expect(invokes).must_equal []
     end
 
     # items added after initialization are added.
@@ -232,14 +232,14 @@ class CallbacksTest < MiniTest::Spec
       twin.songs << song
 
       Callback.new(twin.songs).on_add { |t| invokes << t }
-      invokes.must_equal [twin.songs[1]]
+      expect(invokes).must_equal [twin.songs[1]]
 
       twin.save
 
       # still shows the added after save.
       invokes = []
       Callback.new(twin.songs).on_add { |t| invokes << t }
-      invokes.must_equal [twin.songs[1]]
+      expect(invokes).must_equal [twin.songs[1]]
     end
   end
 
@@ -250,7 +250,7 @@ class CallbacksTest < MiniTest::Spec
     it do
       invokes = []
       Callback.new(twin.songs).on_add(:created) { |t| invokes << t }
-      invokes.must_equal []
+      expect(invokes).must_equal []
     end
 
     # collection present on initialize are not added.
@@ -260,7 +260,7 @@ class CallbacksTest < MiniTest::Spec
       album.songs = [ex_song, song]
 
       Callback.new(twin.songs).on_add(:created) { |t| invokes << t }
-      invokes.must_equal []
+      expect(invokes).must_equal []
     end
 
     # items added after initialization are added.
@@ -273,14 +273,14 @@ class CallbacksTest < MiniTest::Spec
       twin.songs << ex_song # already created.
 
       Callback.new(twin.songs).on_add(:created) { |t| invokes << t }
-      invokes.must_equal []
+      expect(invokes).must_equal []
 
       twin.save
 
       # still shows the added after save.
       invokes = []
       Callback.new(twin.songs).on_add(:created) { |t| invokes << t }
-      invokes.must_equal [twin.songs[1]] # only the created is invoked.
+      expect(invokes).must_equal [twin.songs[1]] # only the created is invoked.
     end
   end
 
@@ -291,7 +291,7 @@ class CallbacksTest < MiniTest::Spec
     it do
       invokes = []
       Callback.new(twin.songs).on_delete { |t| invokes << t }
-      invokes.must_equal []
+      expect(invokes).must_equal []
     end
 
     # collection present but nothing deleted.
@@ -301,7 +301,7 @@ class CallbacksTest < MiniTest::Spec
       album.songs = [ex_song, song]
 
       Callback.new(twin.songs).on_delete { |t| invokes << t }
-      invokes.must_equal []
+      expect(invokes).must_equal []
     end
 
     # items deleted.
@@ -313,14 +313,14 @@ class CallbacksTest < MiniTest::Spec
       twin.songs.delete(deleted = twin.songs[0])
 
       Callback.new(twin.songs).on_delete { |t| invokes << t }
-      invokes.must_equal [deleted]
+      expect(invokes).must_equal [deleted]
 
       twin.save
 
       # still shows the deleted after save.
       invokes = []
       Callback.new(twin.songs).on_delete { |t| invokes << t }
-      invokes.must_equal [deleted]
+      expect(invokes).must_equal [deleted]
     end
   end
 
@@ -331,7 +331,7 @@ class CallbacksTest < MiniTest::Spec
     it do
       invokes = []
       Callback.new(twin.songs).on_destroy { |t| invokes << t }
-      invokes.must_equal []
+      expect(invokes).must_equal []
     end
 
     # collection present but nothing deleted.
@@ -341,7 +341,7 @@ class CallbacksTest < MiniTest::Spec
       album.songs = [ex_song, song]
 
       Callback.new(twin.songs).on_destroy { |t| invokes << t }
-      invokes.must_equal []
+      expect(invokes).must_equal []
     end
 
     # items deleted, doesn't trigger on_destroy.
@@ -353,7 +353,7 @@ class CallbacksTest < MiniTest::Spec
       twin.songs.delete(twin.songs[0])
 
       Callback.new(twin.songs).on_destroy { |t| invokes << t }
-      invokes.must_equal []
+      expect(invokes).must_equal []
     end
 
     # items destroyed.
@@ -365,7 +365,7 @@ class CallbacksTest < MiniTest::Spec
       twin.songs.destroy(deleted = twin.songs[0])
 
       Callback.new(twin.songs).on_destroy { |t| invokes << t }
-      invokes.must_equal []
+      expect(invokes).must_equal []
 
       twin.extend(Disposable::Twin::Collection::Semantics) # now #save will destroy.
       twin.save
@@ -373,7 +373,7 @@ class CallbacksTest < MiniTest::Spec
       # still shows the deleted after save.
       invokes = []
       Callback.new(twin.songs).on_destroy { |t| invokes << t }
-      invokes.must_equal [deleted]
+      expect(invokes).must_equal [deleted]
     end
   end
 
@@ -384,7 +384,7 @@ class CallbacksTest < MiniTest::Spec
     # after initialization
     it do
       Callback.new(twin).on_change { |t| invokes << t }
-      invokes.must_equal []
+      expect(invokes).must_equal []
     end
 
     # save, without any attributes changed. unpersisted before.
@@ -394,7 +394,7 @@ class CallbacksTest < MiniTest::Spec
       twin.save
 
       Callback.new(twin).on_change { |t| invokes << t }
-      invokes.must_equal [] # nothing has changed, not even persisted?.
+      expect(invokes).must_equal [] # nothing has changed, not even persisted?.
     end
 
     # save, without any attributes changed. persisted before.
@@ -402,7 +402,7 @@ class CallbacksTest < MiniTest::Spec
       twin.save
 
       Callback.new(twin).on_change { |t| invokes << t }
-      invokes.must_equal [twin]
+      expect(invokes).must_equal [twin]
     end
 
     # before and after save, with attributes changed
@@ -411,24 +411,24 @@ class CallbacksTest < MiniTest::Spec
       twin.name = "Run For Cover"
       invokes = []
       Callback.new(twin).on_change { |t| invokes << t }
-      invokes.must_equal [twin]
+      expect(invokes).must_equal [twin]
 
       twin.save
 
       invokes = []
       Callback.new(twin).on_change { |t| invokes << t }
-      invokes.must_equal [twin]
+      expect(invokes).must_equal [twin]
     end
 
     # for scalars: on_change(:email).
     it do
       Callback.new(twin).on_change(property: :name) { |t| invokes << t }
-      invokes.must_equal []
+      expect(invokes).must_equal []
 
       twin.name = "Unforgiven"
 
       Callback.new(twin).on_change(property: :name) { |t| invokes << t }
-      invokes.must_equal [twin]
+      expect(invokes).must_equal [twin]
     end
 
     # for collections.
@@ -447,4 +447,4 @@ class CallbacksTest < MiniTest::Spec
     #   invokes.must_equal [twin.songs[0], twin.songs[2]]
     # end
   end
-end
+end