reform 2.3.2 → 2.6.1

data/test/{module_new_api.rb → module_test.rb}

@@ -25,8 +25,8 @@ class ModuleInclusionTest < MiniTest::Spec
       params { required(:band).filled }
     end
 
-    include Dry::Types.module # allows using Types::* in module.
-    property :cool, type: DRY_TYPES_CONSTANT::Bool # test coercion.
+    include Dry.Types(default: :nominal) # allows using Types::* in module.
+    property :cool, type: Types::Params::Bool # test coercion.
   end
 
   # TODO: test if works, move stuff into inherit_schema!
@@ -60,24 +60,24 @@ class ModuleInclusionTest < MiniTest::Spec
   let(:song) { OpenStruct.new(band: OpenStruct.new(title: "Time Again")) }
 
   # nested form from module is present and creates accessor.
-  it { SongForm.new(song).band.title.must_equal "Time Again" }
+  it { assert_equal SongForm.new(song).band.title, "Time Again" }
 
   # methods from module get included.
-  it { SongForm.new(song).id.must_equal 1 }
-  it { SongForm.new(song).band.id.must_equal 2 }
+  it { assert_equal SongForm.new(song).id, 1 }
+  it { assert_equal SongForm.new(song).band.id, 2 }
 
   # validators get inherited.
   it do
     form = SongForm.new(OpenStruct.new)
     form.validate({})
-    form.errors.messages.must_equal(band: ["must be filled"])
+    assert_equal form.errors.messages, band: ["must be filled"]
   end
 
   # coercion works
   it do
     form = SongForm.new(OpenStruct.new)
     form.validate(cool: "1")
-    form.cool.must_equal true
+    assert form.cool
   end
 
   # include a module into a module into a class :)
@@ -106,7 +106,7 @@ class ModuleInclusionTest < MiniTest::Spec
   it do
     form = AlbumForm.new(OpenStruct.new(band: OpenStruct.new))
     form.validate("band" => {})
-    form.errors.messages.must_equal("band.title": ["must be filled"], "band.label": ["must be filled"], name: ["must be filled"])
+    assert_equal form.errors.messages, "band.title": ["must be filled"], "band.label": ["must be filled"], name: ["must be filled"]
   end
 
   describe "module with custom accessors" do
@@ -130,8 +130,8 @@ class ModuleInclusionTest < MiniTest::Spec
     let(:song) { OpenStruct.new(id: 1, title: "Instant Mash") }
 
     it do
-      IncludingSongForm.new(song).id.must_equal 1
-      IncludingSongForm.new(song).title.must_equal "INSTANT MASH"
+      assert_equal IncludingSongForm.new(song).id, 1
+      assert_equal IncludingSongForm.new(song).title, "INSTANT MASH"
     end
   end
 end

data/test/parse_option_test.rb

@@ -13,13 +13,13 @@ class ParseOptionTest < MiniTest::Spec
   let(:form) { CommentForm.new(Comment.new, user: current_user) }
 
   it do
-    form.user.must_equal current_user
+    assert_equal form.user, current_user
 
     lorem = "Lorem ipsum dolor sit amet..."
     form.validate("content" => lorem, "user" => "not the current user")
 
-    form.content.must_equal lorem
-    form.user.must_equal current_user
+    assert_equal form.content, lorem
+    assert_equal form.user, current_user
   end
 
   describe "using ':parse' option doesn't override other ':deserialize' options" do
@@ -30,11 +30,11 @@ class ParseOptionTest < MiniTest::Spec
     end
 
     it do
-      ArticleCommentForm.definitions.get(:user)[:deserializer][:writeable].must_equal false
-      ArticleCommentForm.definitions.get(:user)[:deserializer][:instance].must_equal "Instance"
+      assert_equal ArticleCommentForm.definitions.get(:user)[:deserializer][:writeable], false
+      assert_equal ArticleCommentForm.definitions.get(:user)[:deserializer][:instance], "Instance"
 
-      ArticleCommentForm.definitions.get(:article)[:deserializer][:writeable].must_equal true
-      ArticleCommentForm.definitions.get(:article)[:deserializer][:instance].must_equal "Instance"
+      assert ArticleCommentForm.definitions.get(:article)[:deserializer][:writeable]
+      assert_equal ArticleCommentForm.definitions.get(:article)[:deserializer][:instance], "Instance"
     end
   end
 end

data/test/parse_pipeline_test.rb

@@ -10,6 +10,6 @@ class ParsePipelineTest < MiniTest::Spec
   it "allows passing :parse_pipeline directly" do
     form = AlbumForm.new(Album.new)
     form.validate("name" => "Greatest Hits")
-    form.name.must_equal "{\"name\"=>\"Greatest Hits\"}"
+    assert_equal form.name, "{\"name\"=>\"Greatest Hits\"}"
   end
 end

data/test/{populate_new_api.rb → populate_test.rb}

@@ -48,7 +48,7 @@ class PopulatorTest < MiniTest::Spec
       "name" => "override me!"
     )
 
-    form.name.must_equal "!em edirrevo"
+    assert_equal form.name, "!em edirrevo"
   end
 
   # changing existing property :artist.
@@ -60,7 +60,7 @@ class PopulatorTest < MiniTest::Spec
       "artist" => {"name" => "Marcus Miller"}
     )
 
-    form.artist.model.object_id.must_equal old_id
+    assert_equal form.artist.model.object_id, old_id
   end
 
   # use populator for default value on scalars?
@@ -68,36 +68,36 @@ class PopulatorTest < MiniTest::Spec
   # adding to collection via :populator.
   # valid.
   it "yyy" do
-    form.validate(
+    assert form.validate(
       "songs"  => [{"title" => "Fallout"}, {"title" => "Roxanne"},
         {"title" => "Rime Of The Ancient Mariner"}, # new song.
         {"title" => "Re-Education", "composer" => {"name" => "Rise Against"}}], # new song with new composer.
-    ).must_equal true
+    )
 
-    form.errors.messages.inspect.must_equal "{}"
+    assert_equal form.errors.messages.inspect, "{}"
 
     # form has updated.
-    form.name.must_equal "The Dissent Of Man"
-    form.songs[0].title.must_equal "Fallout"
-    form.songs[1].title.must_equal "Roxanne"
-    form.songs[1].composer.name.must_equal "Greg Graffin"
+    assert_equal form.name, "The Dissent Of Man"
+    assert_equal form.songs[0].title, "Fallout"
+    assert_equal form.songs[1].title, "Roxanne"
+    assert_equal form.songs[1].composer.name, "Greg Graffin"
 
-    form.songs[1].composer.model.must_be_instance_of Artist
+    form.songs[1].composer.model.is_a? Artist
 
-    form.songs[1].title.must_equal "Roxanne"
-    form.songs[2].title.must_equal "Rime Of The Ancient Mariner" # new song added.
-    form.songs[3].title.must_equal "Re-Education"
-    form.songs[3].composer.name.must_equal "Rise Against"
-    form.songs.size.must_equal 4
-    form.artist.name.must_equal "Bad Religion"
+    assert_equal form.songs[1].title, "Roxanne"
+    assert_equal form.songs[2].title, "Rime Of The Ancient Mariner" # new song added.
+    assert_equal form.songs[3].title, "Re-Education"
+    assert_equal form.songs[3].composer.name, "Rise Against"
+    assert_equal form.songs.size, 4
+    assert_equal form.artist.name, "Bad Religion"
 
     # model has not changed, yet.
-    album.name.must_equal "The Dissent Of Man"
-    album.songs[0].title.must_equal "Broken"
-    album.songs[1].title.must_equal "Resist Stance"
-    album.songs[1].composer.name.must_equal "Greg Graffin"
-    album.songs.size.must_equal 2
-    album.artist.name.must_equal "Bad Religion"
+    assert_equal album.name, "The Dissent Of Man"
+    assert_equal album.songs[0].title, "Broken"
+    assert_equal album.songs[1].title, "Resist Stance"
+    assert_equal album.songs[1].composer.name, "Greg Graffin"
+    assert_equal album.songs.size, 2
+    assert_equal album.artist.name, "Bad Religion"
   end
 end
 
@@ -117,7 +117,7 @@ class PopulateWithMethodTest < Minitest::Spec
   it "runs populator method" do
     form.validate("title" => "override me!")
 
-    form.title.must_equal "!em edirrevo"
+    assert_equal form.title, "!em edirrevo"
   end
 end
 
@@ -127,6 +127,14 @@ class PopulateWithCallableTest < Minitest::Spec
   class TitlePopulator
     include Uber::Callable
 
+    def call(form:, **options)
+      form.title = options[:fragment].reverse
+    end
+  end
+
+  class TitlePopulatorWithOldSignature
+    include Uber::Callable
+
     def call(form, options)
       form.title = options[:fragment].reverse
     end
@@ -136,12 +144,28 @@ class PopulateWithCallableTest < Minitest::Spec
     property :title, populator: TitlePopulator.new
   end
 
+  class AlbumFormWithOldPopulator < TestForm
+    property :title, populator: TitlePopulatorWithOldSignature.new
+  end
+
   let(:form) { AlbumForm.new(Album.new) }
 
   it "runs populator method" do
     form.validate("title" => "override me!")
 
-    form.title.must_equal "!em edirrevo"
+    assert_equal form.title, "!em edirrevo"
+  end
+
+  it "gives warning when `form` is accepted as a positional argument" do
+    _, warnings = capture_io do
+      form = AlbumFormWithOldPopulator.new(Album.new)
+      form.validate("title" => "override me!")
+
+      assert_equal form.title, "!em edirrevo"
+    end
+
+    assert_equal warnings, %{[Reform] Accepting `form` as a positional argument in `:populator` will be deprecated. Please use `def call(form:, **options)` signature instead.
+}
   end
 end
 
@@ -161,7 +185,7 @@ class PopulateWithProcTest < Minitest::Spec
   it "runs populator method" do
     form.validate("title" => "override me!")
 
-    form.title.must_equal "!em edirrevo"
+    assert_equal form.title, "!em edirrevo"
   end
 end
 
@@ -216,35 +240,35 @@ class PopulateIfEmptyTest < MiniTest::Spec
   let(:form) { AlbumForm.new(album) }
 
   it do
-    form.songs.size.must_equal 2
+    assert_equal form.songs.size, 2
 
-    form.validate(
+    assert form.validate(
       "songs" => [{"title" => "Fallout"}, {"title" => "Roxanne"},
         {"title" => "Rime Of The Ancient Mariner"}, # new song.
         {"title" => "Re-Education", "composer" => {"name" => "Rise Against"}}], # new song with new composer.
-    ).must_equal true
+    )
 
-    form.errors.messages.inspect.must_equal "{}"
+    assert_equal form.errors.messages.inspect, "{}"
 
     # form has updated.
-    form.name.must_equal "The Dissent Of Man"
-    form.songs[0].title.must_equal "Fallout"
-    form.songs[1].title.must_equal "Roxanne"
-    form.songs[1].composer.name.must_equal "Greg Graffin"
-    form.songs[1].title.must_equal "Roxanne"
-    form.songs[2].title.must_equal "Rime Of The Ancient Mariner" # new song added.
-    form.songs[3].title.must_equal "Re-Education"
-    form.songs[3].composer.name.must_equal "Rise Against"
-    form.songs.size.must_equal 4
-    form.artist.name.must_equal "Bad Religion"
+    assert_equal form.name, "The Dissent Of Man"
+    assert_equal form.songs[0].title, "Fallout"
+    assert_equal form.songs[1].title, "Roxanne"
+    assert_equal form.songs[1].composer.name, "Greg Graffin"
+    assert_equal form.songs[1].title, "Roxanne"
+    assert_equal form.songs[2].title, "Rime Of The Ancient Mariner" # new song added.
+    assert_equal form.songs[3].title, "Re-Education"
+    assert_equal form.songs[3].composer.name, "Rise Against"
+    assert_equal form.songs.size, 4
+    assert_equal form.artist.name, "Bad Religion"
 
     # model has not changed, yet.
-    album.name.must_equal "The Dissent Of Man"
-    album.songs[0].title.must_equal "Broken"
-    album.songs[1].title.must_equal "Resist Stance"
-    album.songs[1].composer.name.must_equal "Greg Graffin"
-    album.songs.size.must_equal 2
-    album.artist.name.must_equal "Bad Religion"
+    assert_equal album.name, "The Dissent Of Man"
+    assert_equal album.songs[0].title, "Broken"
+    assert_equal album.songs[1].title, "Resist Stance"
+    assert_equal album.songs[1].composer.name, "Greg Graffin"
+    assert_equal album.songs.size, 2
+    assert_equal album.artist.name, "Bad Religion"
   end
 
   # trigger artist populator. lambda calling form instance method.
@@ -252,11 +276,11 @@ class PopulateIfEmptyTest < MiniTest::Spec
     form = AlbumForm.new(album = Album.new)
     form.validate("artist" => {"name" => "From Autumn To Ashes"})
 
-    form.artist.name.must_equal "From Autumn To Ashes"
+    assert_equal form.artist.name, "From Autumn To Ashes"
     # test lambda was executed in form context.
-    form.artist.model.must_be_instance_of AlbumForm::Sting
+    assert form.artist.model.is_a? AlbumForm::Sting
     # test lambda block arguments.
-    form.artist.model.args.to_s.must_equal "[{\"name\"=>\"From Autumn To Ashes\"}, nil]"
+    assert_equal form.artist.model.args.to_s, "[{\"name\"=>\"From Autumn To Ashes\"}, nil]"
 
     assert_nil album.artist
   end
@@ -292,13 +316,72 @@ class PopulateIfEmptyWithDeletionTest < MiniTest::Spec
   let(:form) { AlbumForm.new(album) }
 
   it do
-    form.validate(
+    assert form.validate(
       "songs" => [{"title" => "Broken, delete me!"}, {"title" => "Roxanne"}]
-    ).must_equal true
+    )
+
+    assert_equal form.errors.messages.inspect, "{}"
+
+    assert_equal form.songs.size, 1
+    assert_equal form.songs[0].title, "Roxanne"
+  end
+end
 
-    form.errors.messages.inspect.must_equal "{}"
+class PopulateWithFormKeyTest < MiniTest::Spec
+  Song  = Struct.new(:title, :album, :composer)
+  Album = Struct.new(:name, :songs, :artist)
+
+  let(:song)  { Song.new('Broken') }
+  let(:song2) { Song.new('Resist Stance') }
+  let(:album) { Album.new('The Dissent Of Man', [song, song2]) }
+
+  class SongForm < TestForm
+    property :title
+
+    validation do
+      params { required(:title).filled }
+    end
+  end
+
+  class AlbumForm < TestForm
+    property :name
+
+    collection :songs, form: SongForm, populator: :populator!, model_identifier: :title
 
-    form.songs.size.must_equal 1
-    form.songs[0].title.must_equal "Roxanne"
+    def populator!(fragment:, **)
+      item = songs.find { |song| song.title == fragment['title'] }
+      if item && fragment['delete'] == '1'
+        songs.delete(item)
+        return skip!
+      end
+      item || songs.append(Song.new)
+    end
+  end
+
+  let(:form) { AlbumForm.new(album) }
+
+  it do
+    assert_equal 2, form.songs.size
+
+    assert form.validate(
+      'songs' => [
+        { 'title' => 'Broken' },
+        { 'title' => 'Resist Stance' },
+        { 'title' => 'Rime Of The Ancient Mariner' }
+      ]
+    )
+
+    assert_equal 3, form.songs.size
+
+    assert form.validate(
+      'songs' => [
+        { 'title' => 'Broken', 'delete' => '1' },
+        { 'title' => 'Resist Stance' },
+        { 'title' => 'Rime Of The Ancient Mariner' }
+      ]
+    )
+    assert_equal 2, form.songs.size
+    assert_equal 'Resist Stance', form.songs.first.title
+    assert_equal 'Rime Of The Ancient Mariner', form.songs.last.title
   end
 end

data/test/populator_skip_test.rb

@@ -21,8 +21,8 @@ class PopulatorSkipTest < MiniTest::Spec
 
     form.validate(hash)
 
-    form.songs.size.must_equal 2
+    assert_equal form.songs.size, 2
     assert_nil form.songs[0].title
-    form.songs[1].title.must_equal "Bad"
+    assert_equal form.songs[1].title, "Bad"
   end
 end

data/test/prepopulator_test.rb

@@ -37,25 +37,25 @@ class PrepopulatorTest < MiniTest::Spec
   it do
     form = AlbumForm.new(OpenStruct.new(length: 1)).prepopulate!(title: "Potemkin City Limits")
 
-    form.length.must_equal 1
-    form.title.must_equal "Another Day At Work"
-    form.hit.model.must_equal Song.new("Potemkin City Limits")
-    form.songs.size.must_equal 2
-    form.songs[0].model.must_equal Song.new
-    form.songs[1].model.must_equal Song.new
-    form.songs[1].model.must_equal Song.new
+    assert_equal form.length, 1
+    assert_equal form.title, "Another Day At Work"
+    assert_equal form.hit.model, Song.new("Potemkin City Limits")
+    assert_equal form.songs.size, 2
+    assert_equal form.songs[0].model, Song.new
+    assert_equal form.songs[1].model, Song.new
+    assert_equal form.songs[1].model, Song.new
     # prepopulate works more than 1 level, recursive.
     # it also passes options properly down there.
-    form.hit.band.model.must_equal Band.new("Potemkin City Limits")
+    assert_equal form.hit.band.model, Band.new("Potemkin City Limits")
   end
 
   # add to existing collection.
   it do
-    form = AlbumForm.new(OpenStruct.new(songs: [Song.new])).prepopulate!
+    form = AlbumForm.new(OpenStruct.new(songs: [Song.new])).prepopulate!(title: "Potemkin City Limits")
 
-    form.songs.size.must_equal 2
-    form.songs[0].model.must_equal Song.new
-    form.songs[1].model.must_equal Song.new
+    assert_equal form.songs.size, 2
+    assert_equal form.songs[0].model, Song.new
+    assert_equal form.songs[1].model, Song.new
   end
 end
 
@@ -75,7 +75,7 @@ class PrepopulateWithoutConfiguration < MiniTest::Spec
 
   subject { AlbumForm.new(OpenStruct.new(songs: [], hit: nil)).prepopulate! }
 
-  it { subject.songs.size.must_equal 0 }
+  it { assert_equal subject.songs.size, 0 }
 end
 
 class ManualPrepopulatorOverridingTest < MiniTest::Spec
@@ -104,8 +104,8 @@ class ManualPrepopulatorOverridingTest < MiniTest::Spec
   it do
     form = AlbumForm.new(OpenStruct.new(length: 1)).prepopulate!(title: "Potemkin City Limits")
 
-    form.length.must_equal 1
-    form.hit.model.must_equal Song.new("Potemkin City Limits")
-    form.hit.title.must_equal "Potemkin City Limits"
+    assert_equal form.length, 1
+    assert_equal form.hit.model, Song.new("Potemkin City Limits")
+    assert_equal form.hit.title, "Potemkin City Limits"
   end
 end

data/test/read_only_test.rb

@@ -9,6 +9,6 @@ class ReadonlyTest < MiniTest::Spec
 
   let(:form) { SongForm.new(OpenStruct.new) }
 
-  it { form.readonly?(:artist).must_equal false }
-  it { form.readonly?(:title).must_equal true }
+  it { refute form.readonly?(:artist) }
+  it { assert form.readonly?(:title) }
 end

data/test/readable_test.rb

@@ -15,16 +15,16 @@ class ReadableTest < MiniTest::Spec
 
     form.validate("password" => "123")
 
-    form.password.must_equal "123"
+    assert_equal form.password, "123"
 
     form.sync
-    cred.password.must_equal "123" # password written.
+    assert_equal cred.password, "123" # password written.
 
     hash = {}
     form.save do |nested|
       hash = nested
     end
 
-    hash.must_equal("password" => "123")
+    assert_equal hash, "password" => "123"
   }
 end

data/test/{reform_new_api.rb → reform_test.rb}

@@ -19,7 +19,7 @@ class ReformTest < Minitest::Spec
 
     it "returns empty fields" do
       assert_nil form.title
-      form.name.must_be_nil
+      assert_nil form.name
     end
 
     describe "and submitted values" do
@@ -27,15 +27,15 @@ class ReformTest < Minitest::Spec
         form.validate("name" => "Duran Duran")
 
         assert_nil form.title
-        form.name.must_equal "Duran Duran"
+        assert_equal form.name, "Duran Duran"
       end
     end
   end
 
   describe "(edit) form with existing models" do
     it "returns filled-out fields" do
-      form.name.must_equal  "Duran Duran"
-      form.title.must_equal "Rio"
+      assert_equal form.name,  "Duran Duran"
+      assert_equal form.title, "Rio"
     end
   end
 
@@ -50,13 +50,13 @@ class ReformTest < Minitest::Spec
     end
 
     it "returns true when valid" do
-      form.validate("name" => "Duran Duran").must_equal true
+      assert_equal form.validate("name" => "Duran Duran"), true
     end
 
     it "exposes input via property accessors" do
       form.validate("name" => "Duran Duran")
 
-      form.name.must_equal "Duran Duran"
+      assert_equal form.name, "Duran Duran"
     end
 
     it "doesn't change model properties" do
@@ -81,12 +81,12 @@ class ReformTest < Minitest::Spec
       let(:form) { ValidatingForm.new(comp) }
 
       it "returns false when invalid" do
-        form.validate({}).must_equal false
+        assert_equal form.validate({}), false
       end
 
       it "populates errors" do
         form.validate({})
-        form.errors.messages.must_equal(name: ["must be filled"], title: ["must be filled"])
+        assert_equal form.errors.messages, name: ["must be filled"], title: ["must be filled"]
       end
     end
   end
@@ -100,7 +100,7 @@ class ReformTest < Minitest::Spec
     it "xxpushes data to models" do
       form.save
 
-      comp.name.must_equal "Diesel Boy"
+      assert_equal comp.name, "Diesel Boy"
       assert_nil comp.title
     end
 
@@ -112,13 +112,13 @@ class ReformTest < Minitest::Spec
           hash = map
         end
 
-        hash.must_equal("name" => "Diesel Boy", "title" => nil)
+        assert_equal hash, "name" => "Diesel Boy", "title" => nil
       end
     end
   end
 
   describe "#model" do
-    it { form.model.must_equal comp }
+    it { assert_equal form.model, comp }
   end
 
   describe "inheritance" do
@@ -132,9 +132,9 @@ class ReformTest < Minitest::Spec
     let(:form) { HitForm.new(OpenStruct.new()) }
     it do
       form.validate("title" => "The Body")
-      form.title.must_equal "The Body"
+      assert_equal form.title, "The Body"
       assert_nil form.position
-      form.errors.messages.must_equal(name: ["must be filled"], position: ["must be filled"])
+      assert_equal form.errors.messages, name: ["must be filled"], position: ["must be filled"]
     end
   end
 end
@@ -156,18 +156,18 @@ class OverridingAccessorsTest < BaseTest
   subject { SongForm.new(song) }
 
   # override reader for presentation.
-  it { subject.title.must_equal "pray" }
+  it { assert_equal subject.title, "pray" }
 
   describe "#save" do
     before { subject.validate("title" => "Hey Little World") }
 
     # reader always used
-    it { subject.title.must_equal "hey little worldhey little world" }
+    it { assert_equal subject.title, "hey little worldhey little world" }
 
     # the reader is not used when saving/syncing.
     it do
       subject.save do |hash|
-        hash["title"].must_equal "Hey Little WorldHey Little World"
+        assert_equal hash["title"], "Hey Little WorldHey Little World"
       end
     end
 
@@ -175,7 +175,7 @@ class OverridingAccessorsTest < BaseTest
     it do
       song.extend(Saveable)
       subject.save
-      song.title.must_equal "Hey Little WorldHey Little World"
+      assert_equal song.title, "Hey Little WorldHey Little World"
     end
   end
 end
@@ -199,6 +199,6 @@ class MethodInFormTest < MiniTest::Spec
 
   # methods can be used instead of created accessors.
   subject { AlbumForm.new(OpenStruct.new(hit: OpenStruct.new)) }
-  it { subject.title.must_equal "The Suffer And The Witness" }
-  it { subject.hit.title.must_equal "Drones" }
+  it { assert_equal subject.title, "The Suffer And The Witness" }
+  it { assert_equal subject.hit.title, "Drones" }
 end

data/test/{save_new_api.rb → save_test.rb}

@@ -53,9 +53,9 @@ class SaveTest < BaseTest
 
     form.save
 
-    album.saved?.must_equal true
-    album.songs[0].title.must_equal "Fixed"
-    album.songs[0].saved?.must_equal true
+    assert album.saved?
+    assert_equal album.songs[0].title, "Fixed"
+    assert album.songs[0].saved?
     assert_nil album.artist.saved?
   end
 
@@ -70,7 +70,7 @@ class SaveTest < BaseTest
         nested_hash = hash
       end
 
-      nested_hash.must_equal({"name" => nil, "artist" => nil})
+      assert_equal nested_hash, "name" => nil, "artist" => nil
     end
   end
 end