acs-aem-commons icon indicating copy to clipboard operation
acs-aem-commons copied to clipboard

add Mockito inline dependency

Open MikhailNavitski opened this issue 10 months ago • 3 comments

https://github.com/Adobe-Consulting-Services/acs-aem-commons/issues/3327

MikhailNavitski avatar Apr 18 '24 15:04 MikhailNavitski

Why adding a new dependency without changing any code? What does this fix?

kwin avatar Apr 18 '24 16:04 kwin

Why adding a new dependency without changing any code? What does this fix?

Currently, there are several open issues related to the unit tests that I am planning to work on ( as an example https://github.com/Adobe-Consulting-Services/acs-aem-commons/issues/2279). There is no possibility to mock static methods right now, so, it fixes that issue and I can proceed with the https://github.com/Adobe-Consulting-Services/acs-aem-commons/issues/2279

MikhailNavitski avatar Apr 18 '24 16:04 MikhailNavitski

But It seems like need to rewrite a ton of tests for it :)

MikhailNavitski avatar Apr 18 '24 18:04 MikhailNavitski