Skip to content

Conversation

Copy link

Copilot AI commented Oct 30, 2025

Integration tests referenced incorrect namespaces for source-generated types, causing build failures. The generator creates namespaces based on schema version (e.g., Versioning.Test.V2 for version="2" schemas), then appends message versions as sub-namespaces (e.g., V2.V1 for version 1 messages).

Changes

  • VersioningIntegrationTests.cs: Updated namespace aliases from Versioning.Test.V{0,1,2} to Versioning.Test.V2{,.V1,.V2} and corrected file paths to include schema hash folder
  • DeprecatedFieldsIntegrationTests.cs: Changed namespace from Deprecated.Test.V0 to Deprecated.Test.V2 and updated file search patterns from "OrderWithDeprecated.cs" to "OrderWithDeprecated*.cs" to match all versioned files

Example of corrected namespace imports:

// Before
using V0 = Versioning.Test.V0;
using V1 = Versioning.Test.V1;
using V2 = Versioning.Test.V2;

// After
using V0 = Versioning.Test.V2;      // Base version (schema v2)
using V1 = Versioning.Test.V2.V1;   // Message version 1
using V2 = Versioning.Test.V2.V2;   // Message version 2
Original prompt

Temos alguns erros ocorrendo no build, favor corrigir


💡 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.

Co-authored-by: pedrosakuma <39205549+pedrosakuma@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix errors occurring in the build Fix integration test namespace references to match generated code structure Oct 30, 2025
Copilot AI requested a review from pedrosakuma October 30, 2025 14:32
@pedrosakuma pedrosakuma marked this pull request as ready for review October 30, 2025 15:32
@pedrosakuma pedrosakuma merged commit 7d46eae into master Oct 30, 2025
2 checks passed
@pedrosakuma pedrosakuma deleted the copilot/fix-build-errors branch October 30, 2025 15:32
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