modify dialog
add test