Podcast #128: We chat with Kent C Dodds about why he loves React and discuss what life was like in the dark days before Git. Listen now.

New answers tagged


First refactor the implementation to use an internal constructor internal AdPersister(AdRepository<TEntity> adRepository, IAdImagePersister adImagePersister){ _adRepository = adRepository; _adImagePersister = adImagePersister; } Effectively hiding it from use externally and giving you full controll of the class's activation The lower layer ...


Your implementation is good, but I would suggest following points to improve: It is really hard to test/mock source2ToCheck.getSource(). Source2 contains a lot logic/functionality (it is source type and factory) it has to be simplified. getSource() creates new course object it is confusing getSource(...) calls getAction(...) that create a source object. ...

Top 50 recent answers are included