Less navigation; same destination...

Less navigation; same destination... (Remove NavigationAccessor and its cache)

We were previously looking at the CLR type of the navigation to determine whether it was a collection or not. Now we're using the IsUnique metadata from the foreign key. This means that the CLR type info caching for navigations is not required anymore.
Add a test for constructor argument checks