add test unit
remove magic number