Remove tests using IdentityServer4.EntityFramework

[AndriySvyryd]

Fixes #34485

[roji]

@AndriySvyryd didn't we say it's maybe OK to keep these tests as we'll be moving test running to another pipeilne altogether?

[AndriySvyryd]

@AndriySvyryd didn't we say it's maybe OK to keep these tests as we'll be moving test running to another pipeilne altogether?

It might be OK from the perspective that we won't get alerts. But what's the point of testing against an obsolete and unlisted package?

[roji]

I'm guessing quite a few people may be still using it (as it's the last free version), and it does give us some coverage in terms of detecting breaking changes which also potentially affect other packages too (like the newer Duende package). But the comment was also from the perspective of what we actually gain by removing it (like why not leave it).

But in any case, am OK with removing if you prefer.

[AndriySvyryd]

But the comment was also from the perspective of what we actually gain by removing it (like why not leave it).

Less dependencies -> less maintenance cost