This also doesn't seem to be testing anything, but I'm not sure. If you remove the code inside `update_attr` does this still pass?