Skip to content

Add missing test overrides for Contains_on_Enumerable/MemoryExtensions#3821

Open
roji wants to merge 1 commit intohotfix/9.0.5from
roji/fix-ci-hotfix-9-0-5
Open

Add missing test overrides for Contains_on_Enumerable/MemoryExtensions#3821
roji wants to merge 1 commit intohotfix/9.0.5from
roji/fix-ci-hotfix-9-0-5

Conversation

@roji
Copy link
Copy Markdown
Member

@roji roji commented Apr 27, 2026

EF Core 9.0.14 added Contains_on_Enumerable and Contains_on_MemoryExtensions to PrimitiveCollectionsQueryTestBase (for C# 14 first-class span support). This adds the required overrides with SQL assertions to PrimitiveCollectionsQueryNpgsqlTest, fixing the Check_all_tests_overridden failure on all CI jobs.

Fixes the CI failure at https://github.com/npgsql/efcore.pg/actions/runs/24959840704.

EF Core 9.0.14 added Contains_on_Enumerable and Contains_on_MemoryExtensions
to PrimitiveCollectionsQueryTestBase. Add the required overrides with SQL
assertions to PrimitiveCollectionsQueryNpgsqlTest.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 27, 2026 06:49
@roji roji changed the base branch from main to hotfix/9.0.5 April 27, 2026 06:50
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds provider-specific overrides for newly introduced EF Core tests in the primitive collections query suite, ensuring Npgsql’s functional tests include the required SQL baselines and CI no longer fails on the “all tests overridden” check.

Changes:

  • Add Contains_on_Enumerable override with Npgsql SQL assertion baseline.
  • Add Contains_on_MemoryExtensions override with Npgsql SQL assertion baseline.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants