all these changes made the tests harder to understand. maybe having just a single logger that would log `newValue` and `oldValue` would be better. OR not change this logs at all and just write an explicit test that will verify that the old value  is being correctly passed as an argument.