we didn't test the update function