Removed usage of Mock<BaseProcessor>

[ngruson]

Changes

On ask of @vishweshbankwar, the usage of Mock<BaseProcessor> was removed in OpenTelemetry.Instrumentation.Http.Tests.
Instead, InMemoryExporter is used to verify exported activities.

Merge requirement checklist

• CONTRIBUTING guidelines followed (nullable enabled, static analysis, etc.)
• Unit tests added/updated
• Appropriate CHANGELOG.md files updated for non-trivial changes
• Changes in public API reviewed (if applicable)

[codecov]

Codecov Report

Merging #5095 (6a2db61) into main (4e390e3) will increase coverage by 0.12%.
Report is 1 commits behind head on main.
The diff coverage is n/a.

❗ Current head 6a2db61 differs from pull request most recent head c3215a9. Consider uploading reports for the commit c3215a9 to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #5095      +/-   ##
==========================================
+ Coverage   83.15%   83.28%   +0.12%     
==========================================
  Files         296      296              
  Lines       12351    12351              
==========================================
+ Hits        10271    10287      +16     
+ Misses       2080     2064      -16     

Flag	Coverage Δ
unittests	83.28% <ø> (+0.12%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 5 files with indirect coverage changes

[vishweshbankwar]

Thanks @ngruson!. For a separate PR, see if you can get rid of Mock propagator also.

[utpilla]

Thanks @ngruson! Could you also remove the package reference to Moq from the csproj:

opentelemetry-dotnet/test/OpenTelemetry.Instrumentation.Http.Tests/OpenTelemetry.Instrumentation.Http.Tests.csproj

Line 19 in 4e390e3

<PackageReference Include="Moq" />

[utpilla]

Thanks @ngruson! Could you also remove the package reference to Moq from the csproj:

opentelemetry-dotnet/test/OpenTelemetry.Instrumentation.Http.Tests/OpenTelemetry.Instrumentation.Http.Tests.csproj

Line 19 in 4e390e3

<PackageReference Include="Moq" />

Never mind! I got to know from @vishweshbankwar that this project still uses Moq for the propagator tests.

[ngruson]

I will do so in the next PR ;)

[vishweshbankwar]

I will do so in the next PR ;)

@ngruson If you are able to remove the mock propagator part, please also check other test projects where it is used.