switching from setter DI to constructor based DI