updated
add negative test