update with test