Virtualize RepositoryBase methods

[ardalis]

Virtualize RepositoryBase methods

[fiseni]

Lets' first wait to merge the existing PR, and fix the EF6 project properly. Once we sort that out, then you can create an additional PR for these changes. Sounds OK?

For now, please use your own fully implemented Repository (you may just copy the implementations and update them as you wish). Anyhow, the RepositoryBase is given just as an optional base class and just as a reference. We try to avoid leading or enforcing users to use it.

[fiseni]

The methods of base repositories now are marked as virtual in v5.

[fiseni]

Hi @ngruson,

It's not about the method implementations only, but also the namespaces used within the "RepositoryBase".

The EntityFramework6 project should fully implement IRepositoryBase interface, and not inherit from the existing one in EntityFrameworkCore project. These two "addon/plugin" projects shouldn't reference each other. As for making the methods virtual, sure, why not.

Also, we should have a new project for tests, something like EntityFramework6.IntegrationTests. That way we can clearly test upon EF6, without referencing the EFCore at all.

This is my stand initially, @ardalis may have a different opinion.

PS. We have to refactor the test infrastructure here. It's in our backlog for some time. We should have the following:

• UnitTests project for the base package.
• IntegrationsTests project for each plugin package.

[EDIT]
Now that I read it again, I think I misunderstood you, sorry :). In the next release, we may refactor the methods as virtual.

[ngruson]

Hi @fiseni

Just to clarify further:
The EF6 base repository is not referencing the EF Core base repository and is implementing IRepositoryBase.
In my solution I have a repository which is inheriting from RepositoryBase (it's EF6 but it might as well be EF Core).
I have unit tests in my solution to test this repository. So this is not about the tests in Ardalis.Specification.

Shall I make a PR to virtualize these methods in both the EF Core and EF6 RepositoryBase classes?

[fiseni]

Implemented in version 5.0.3