committing each function testing