Skip to content

Move Azure Service Fabric unit-test binding redirects to active test/ project#1371

Draft
Copilot wants to merge 2 commits into
mainfrom
copilot/fix-code-review-suggestion
Draft

Move Azure Service Fabric unit-test binding redirects to active test/ project#1371
Copilot wants to merge 2 commits into
mainfrom
copilot/fix-code-review-suggestion

Conversation

Copilot AI commented Jun 19, 2026

Copy link
Copy Markdown

The review flagged that the added binding redirects were placed under the legacy Test/ tree, which is not referenced by DurableTask.sln. This update applies the same redirect configuration to the active unit test project under test/.

  • Scope

    • Addresses only review comment #discussion_r3444200514.
    • No production code changes.
  • What changed

    • Added test/DurableTask.AzureServiceFabric.Tests/App.config.
    • Included the binding redirects needed for the Azure Service Fabric unit test project (net48) in the active test path.
  • Resulting alignment

    • Redirect config now sits with the test project actually included in solution builds/runs.
<!-- test/DurableTask.AzureServiceFabric.Tests/App.config -->
<dependentAssembly>
  <assemblyIdentity name="Newtonsoft.Json" publicKeyToken="30ad4fe6b2a6aeed" culture="neutral" />
  <bindingRedirect oldVersion="0.0.0.0-13.0.0.0" newVersion="13.0.0.0" />
</dependentAssembly>

Copilot AI requested review from Copilot and removed request for Copilot June 19, 2026 19:19
Copilot AI requested review from Copilot and removed request for Copilot June 19, 2026 19:34
Copilot AI changed the title [WIP] Fix code issue from review comment Move Azure Service Fabric unit-test binding redirects to active test/ project Jun 19, 2026
Copilot AI requested a review from shankarsama June 19, 2026 19:35
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