On 15/01/2019 18:59, Lance Andersen wrote:
Looks good. I also skimmed the update test and it looks okay too except setUp and tearDown where it still catches exception so the methods succeed when they fail. Maybe setUp could be changed to not catch the IOException. No need for a new webrev if you do that.OK thank you. I made a ‘minor update to the comments and removed “4"
-Alan
