update todo and change sample to constructor injection