tidy up basic_spec
add some comments to helper