#90 Consolidate mapper tests and add a new test

valentinajemuovic / banking-kata-java

Banking Kata (Java)

MIT License

346 stars 92 forks source link

#90 Consolidate mapper tests and add a new test #92

Closed JoaoCipriano closed 1 year ago

JoaoCipriano commented 1 year ago

Hi @valentinacupac! Following the instructions from issue #90, I deleted the test class BankAccountMapperTest and added a new test to verify the update method at class BankAccountStorage.

JoaoCipriano commented 1 year ago

With the new test should_only_update_filled_attributes() the BankAccountStorageTest will be complete and will test all three methods in the BankAccountStorage (find, add, and update).