Migrate some tests from Newtonsoft.Json to System.Text.Json

[JamieMagee]

This pull request contains the following changes:

• Migration of some tests from Newtonsoft.Json.JsonConvert to System.Text.Json.JsonSerializer
• Migration of PipInstallationReport from Newtonsoft.Json to System.Text.Json
  • No intermediate step was possible as it used JObject and JArray properties directly which cannot be handled by System.Text.Json
• Made parameterless constructors in the TypedComponent hierarchy public instead of private or internal
  • See Non-public, parameterless constructors not used for deserialization

Part of #231

[github-actions]

👋 Hi! It looks like you modified some files in the Detectors folder.
You may need to bump the detector versions if any of the following scenarios apply:

• The detector detects more or fewer components than before
• The detector generates different parent/child graph relationships than before
• The detector generates different devDependencies values than before

If none of the above scenarios apply, feel free to ignore this comment 🙂

[codecov]

Codecov Report

❌ Patch coverage is 88.37209% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.2%. Comparing base (6d46b96) to head (54ba4b8).
⚠️ Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
...tection.Contracts/TypedComponent/CondaComponent.cs	0.0%	1 Missing ⚠️
...ntracts/TypedComponent/DockerReferenceComponent.cs	0.0%	1 Missing ⚠️
...ection.Contracts/TypedComponent/DotNetComponent.cs	0.0%	1 Missing ⚠️
...etection.Contracts/TypedComponent/SpdxComponent.cs	0.0%	1 Missing ⚠️
...tection.Contracts/TypedComponent/VcpkgComponent.cs	0.0%	1 Missing ⚠️

Additional details and impacted files

@@          Coverage Diff          @@
##            main   #1523   +/-   ##
=====================================
  Coverage   90.2%   90.2%           
=====================================
  Files        426     426           
  Lines      35965   35965           
  Branches    2216    2216           
=====================================
+ Hits       32464   32465    +1     
  Misses      3045    3045           
+ Partials     456     455    -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[Copilot]

Pull request overview

This PR migrates test code from Newtonsoft.Json to System.Text.Json as part of issue #231. The migration maintains backward compatibility by adding dual serialization attributes to shared contract classes, allowing production code to continue using Newtonsoft.Json while tests use System.Text.Json.

Key Changes

• Replaced JsonConvert.Serialize/Deserialize with JsonSerializer.Serialize/Deserialize in test files
• Added System.Text.Json.Serialization attributes alongside existing Newtonsoft.Json attributes in pip contract classes
• Fixed a comment in PyPiClientTests.cs to accurately reflect that an API call is made twice instead of once

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
test/Microsoft.ComponentDetection.VerificationTests/ComponentDetectionIntegrationTests.cs	Updated serialization calls and imports for integration tests
test/Microsoft.ComponentDetection.Orchestrator.Tests/Services/DetectorProcessingServiceTests.cs	Migrated telemetry deserialization to System.Text.Json
test/Microsoft.ComponentDetection.Detectors.Tests/YarnLockDetectorTests.cs	Updated workspace JSON serialization in yarn lock tests
test/Microsoft.ComponentDetection.Detectors.Tests/SimplePypiClientTests.cs	Migrated PyPI project serialization in simple client tests
test/Microsoft.ComponentDetection.Detectors.Tests/SimplePipComponentDetectorTests.cs	Updated dependency graph serialization for error messages
test/Microsoft.ComponentDetection.Detectors.Tests/PyPiClientTests.cs	Migrated PyPI client tests and fixed comment accuracy
test/Microsoft.ComponentDetection.Detectors.Tests/PipReportComponentDetectorTests.cs	Updated pip report deserialization in detector tests
src/Microsoft.ComponentDetection.Detectors/pip/Contracts/PipInstallationReportItem.cs	Added System.Text.Json attributes while maintaining Newtonsoft.Json compatibility
src/Microsoft.ComponentDetection.Detectors/pip/Contracts/PipInstallationReport.cs	Added dual serialization attributes for transitional compatibility
src/Microsoft.ComponentDetection.Detectors/pip/Contracts/PipInstallationMetadata.cs	Added System.Text.Json property name attributes alongside existing ones

[Copilot]

Pull request overview

Copilot reviewed 17 out of 17 changed files in this pull request and generated 8 comments.

[Copilot]

Pull request overview

Copilot reviewed 20 out of 20 changed files in this pull request and generated 1 comment.

[Copilot]

Pull request overview

Copilot reviewed 20 out of 20 changed files in this pull request and generated 2 comments.

[Copilot]

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated no new comments.

[Copilot]

Pull request overview

Copilot reviewed 31 out of 31 changed files in this pull request and generated 2 comments.