ResolveAll is broken about abstractions HOT 9 CLOSED

Actually, this looks like this is related to the interface of IUnityContainer changing, which seems to have broken Prism. Still investigating.

from abstractions.

Could you be more specific: what is failing and under what circumstances.
The code you referring to is working fine in all existing tests. Granted these are the original tests and only cover 87% of code...

Could you give me a test case where it fails?

from abstractions.

This test fails: https://github.com/PrismLibrary/Prism/blob/master/Source/Wpf/Prism.Unity.Wpf.Tests/UnityServiceLocatorAdapterFixture.cs#L34

This actually seems to be related to the fact that the IUnityContainer interface was changed. ResolveAll is no longer on the interface, which means the new extension method is being used instead of the interface method (which has been removed), hence breaking our tests.

Do you have a list of all the breaking changes you have made?

from abstractions.

Looking at the coverage date I can see it is being tested properly: https://codecov.io/gh/unitycontainer/unity/src/master/src/Unity/Container/UnityContainerExtensions.cs

from abstractions.

I listed all breaking changes in release notes every time there was a change.

from abstractions.

I don't see them:

https://github.com/unitycontainer/abstractions/releases

https://github.com/unitycontainer/container/releases

Should I be looking somewhere else?

from abstractions.

I see the problem. Mocks will not work.

from abstractions.

https://github.com/unitycontainer/unity/releases

Did not have time to update these everywhere

from abstractions.

I see... I will have to find a way to update my tests to account for all the breaking changes.

from abstractions.