[ReubenBond]

Problem

Orleans.Journaling.Tests referenced Orleans.Azure.Tests. The only thing it used from that reference was MetricCollector<T>, which it received transitively from Microsoft.Extensions.Diagnostics.Testing. As a result, Orleans.Journaling.Tests and Orleans.Journaling.Json.Tests could not compile until the whole Azure.Tests chain had built: Reminders.Tests, Runtime.Internal.Tests, Streaming.Tests and Runtime.Tests. In clean Orleans.slnx builds, those two projects were usually the last compilers to finish.

Change

• Remove the Orleans.Azure.Tests project reference.
• Reference Microsoft.Extensions.Diagnostics.Testing directly. The package is already pinned in Directory.Packages.props and resolves to the same versions as before: 8.10.0 for net8.0 and 10.3.0 for net10.0.

The Journaling.Json.Tests → Journaling.Tests reference stays, because Json.Tests uses its shared helpers.

Verification

All results below cover both net8.0 and net10.0.

• Test discovery: identical, including theory arguments (1,272 Journaling tests, 152 Json tests).
• Test results: identical per-test outcomes before and after:
  • Journaling: 1,125 passed, 147 skipped
  • Json: 152 passed
  • AzureStorage category against Azurite: 75 passed, 3 skipped
• Dependency closure: packages and assemblies are only removed, with one exception. System.Security.Cryptography.ProtectedData now resolves to 4.5.0, the same version as TestExtensions, which owns the shared Azure token credential. Runtime.Tests, Runtime.Internal.Tests, Streaming.Tests and Reminders.Tests already use 4.5.0. The previous 8.0.0/10.0.3 came only from Azure.Tests' System.Diagnostics.PerformanceCounter dependency. A Windows DPAPI round-trip with the 4.5.0 runtime library succeeds.
• Compiler work: the clean-build Csc/Fsc inventory is unchanged (294 Csc, 8 Fsc).

Build impact

These are clean dotnet build Orleans.slnx -m:32 runs on the rc.3 SDK, with npm ci in every sample, interleaved A-B-B-A then B-A-A-B. All times are in seconds.

	Samples (s)	Median (s)
Base (625b2ba)	115.1, 115.2, 107.1	115.1
This PR	119.7, 107.1, 106.6, 104.4	106.9

The base table leaves out one cold first build of a fresh worktree (188.0s).

The samples are noisy and the ranges overlap. A more conservative estimate holds every other project's schedule fixed: in the base runs, the last Journaling compile ends 0.9–3.2s after the last other compile. With this change, Journaling.Tests and Json.Tests finish 30–50s before the build ends and are never the last compiler. A focused Journaling build also no longer builds the Azure, Streaming, Reminders and Runtime test projects.

Microsoft Reviewers: Open in CodeFlow

[Copilot]

🟢 Approval recommended

No active review issues are reported for the test dependency change.

No open issues 🎉

What changed in this PR

This PR removes an unnecessary test-project dependency so Orleans journaling tests can build without waiting for the Azure test dependency chain.

Changes:

• Replace the Orleans.Azure.Tests project reference with a direct Microsoft.Extensions.Diagnostics.Testing package reference for MetricCollector<T>.
• Preserve shared test infrastructure references while reducing build dependencies for Journaling and its downstream JSON tests.

File	Description
test/​Orleans.Journaling.Tests/​Orleans.Journaling.Tests.csproj	Reference diagnostics testing directly and remove the Azure test-project dependency.

_{Review effort: Balanced}

[github-actions]

Code coverage

Metric	Pull request
Lines	84.15% (121,372 / 144,230)
Branches	74.47% (36,793 / 49,405)

Report-only conclusion: current-main baseline stale.

The newest successful coverage run tested 9a0c764, not current main 625b2ba.

Coverage combines every CI test matrix job, including providers, CodeGen, .NET 8/10, Linux, Windows, and macOS, using canonical physical source and branch identities.

The comparison remains report-only while normal line and branch variance is calibrated.

Coverage details

• Pull request commit: 3ffb82d2fa
• View the pull request test run and coverage artifacts
• View the trusted coverage reporting run