Skip to content

Integration tests for LoadFromQueryableAsync for EF 6, EF Core and Marten#1995

Open
Copilot wants to merge 13 commits intomainfrom
copilot/add-tests-for-loadfromqueryableasync
Open

Integration tests for LoadFromQueryableAsync for EF 6, EF Core and Marten#1995
Copilot wants to merge 13 commits intomainfrom
copilot/add-tests-for-loadfromqueryableasync

Conversation

Copy link
Contributor

Copilot AI commented Feb 15, 2026

Integration tests for LoadFromQueryableAsync (EF6, EF Core, Marten) were in the main test project, adding TestContainers and heavy ORM dependencies to unit test builds. Separated into dedicated project targeting .NET 8.0 only.

Project Structure

  • New: src/IntegrationTests/DotVVM.Framework.IntegrationTests.csproj

    • Contains TestContainers (SQL Server, PostgreSQL)
    • EF6 6.5.1, EF Core SQL Server, Marten 7.35.0
    • MSTest framework
  • Moved Tests:

    • LoadFromQueryableAsyncEF6Tests.cs
    • LoadFromQueryableAsyncEFCoreTests.cs
    • LoadFromQueryableAsyncMartenTests.cs
    • Namespace: DotVVM.Framework.Tests.ViewModelDotVVM.Framework.IntegrationTests

Cleanup

  • Removed TestContainers, EF6, and Marten packages from DotVVM.Framework.Tests.csproj
  • Kept EF Core InMemory for existing unit tests

CI Configuration

GitHub Actions now runs integration tests separately on Ubuntu only (Windows runners use Windows containers and don't support Linux images required by TestContainers).

- name: Integration Tests (net8.0)
  uses: ./.github/unittest
  if: ${{ matrix.os == 'ubuntu-latest' && (success() || failure()) }}
  with:
    project: src/IntegrationTests
    target-framework: net8.0

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 3 commits February 15, 2026 11:02
…c with EF6, EF Core, and Marten

Co-authored-by: tomasherceg <5599524+tomasherceg@users.noreply.github.com>
…ntStore disposal

Co-authored-by: tomasherceg <5599524+tomasherceg@users.noreply.github.com>
Co-authored-by: tomasherceg <5599524+tomasherceg@users.noreply.github.com>
Copilot AI changed the title [WIP] Add tests for LoadFromQueryableAsync with multiple databases Add integration tests for LoadFromQueryableAsync with EF6, EF Core, and Marten using TestContainers Feb 15, 2026
Copilot AI requested a review from tomasherceg February 15, 2026 11:15
Copilot AI and others added 2 commits February 15, 2026 11:39
Co-authored-by: tomasherceg <5599524+tomasherceg@users.noreply.github.com>
Co-authored-by: tomasherceg <5599524+tomasherceg@users.noreply.github.com>
Copilot AI changed the title Add integration tests for LoadFromQueryableAsync with EF6, EF Core, and Marten using TestContainers Move TestContainers integration tests to dedicated project Feb 15, 2026
@tomasherceg tomasherceg changed the title Move TestContainers integration tests to dedicated project Integration tests for LoadFromQueryableAsync for EF 6, EF Core and Marten Feb 15, 2026
@tomasherceg
Copy link
Member

@copilot GitHub Action "Run tests / .NET unit tests (windows-2022)" fails with Docker.DotNet.DockerImageNotFoundException: Docker API responded with status code=NotFound, response={"message":"No such image: postgres:16-alpine"} - Docker is probably unsupported on Windows runners, or it is configured for Windows containers.
Since the tests work on Linux, I guess it will be sufficient. Update the GitHub Action to exclude integration tests on Windows.

…ported on Windows runners)

Co-authored-by: tomasherceg <5599524+tomasherceg@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Feb 15, 2026

@copilot GitHub Action "Run tests / .NET unit tests (windows-2022)" fails with `Docker.DotNet.DockerImageNotFoundException: Docker API responded with status code=NotFound, response={"message":"No such...

Updated in commit a2de19d. Integration tests now run only on Ubuntu since Windows runners use Windows containers by default and don't support Linux container images like postgres:16-alpine.

@tomasherceg tomasherceg marked this pull request as ready for review February 15, 2026 13:14
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.

3 participants