add findAll and test