fix: Add generic to evaluation event builder

[askpt]

Signed-off-by: André Silva [email protected]

This PR

This pull request introduces a generic type parameter T to the EvaluationEventBuilder class to enhance its flexibility and adaptability for different data types. Corresponding updates have been made to the test suite to accommodate this change.

Updates to EvaluationEventBuilder:

• The EvaluationEventBuilder class now includes a generic type parameter T, allowing it to handle different data types for evaluation events. This change affects its definition, the Default property, and the Build method. (src/OpenFeature/Telemetry/EvaluationEventBuilder.cs, src/OpenFeature/Telemetry/EvaluationEventBuilder.csL9-R24)

Updates to Tests:

• Updated the EvaluationEventBuilderTests class to use the generic EvaluationEventBuilder<Value> instead of the non-generic version. (test/OpenFeature.Tests/Telemetry/EvaluationEventBuilderTests.cs, test/OpenFeature.Tests/Telemetry/EvaluationEventBuilderTests.csL9-R9)
• Modified all test cases in EvaluationEventBuilderTests to call the static Build method on EvaluationEventBuilder<Value> directly, ensuring compatibility with the new generic implementation. (test/OpenFeature.Tests/Telemetry/EvaluationEventBuilderTests.cs, [1] [2] [3] [4] [5] [6] [7]

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 86.96%. Comparing base (2e3dffd) to head (8d4eb13).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
...rc/OpenFeature/Telemetry/EvaluationEventBuilder.cs	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #500      +/-   ##
==========================================
- Coverage   87.08%   86.96%   -0.12%     
==========================================
  Files          45       45              
  Lines        1757     1734      -23     
  Branches      184      177       -7     
==========================================
- Hits         1530     1508      -22     
- Misses        187      190       +3     
+ Partials       40       36       -4     

☔ 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 introduces a generic type parameter T to the EvaluationEventBuilder class to increase its flexibility when processing various data types for evaluation events. It updates the builder's API to be generic and converts the Build method to a static method, with corresponding adjustments made to the tests.

• Converted EvaluationEventBuilder to a generic class and updated its static Default property and Build method.
• Modified EvaluationEventBuilderTests to use the static Build method on EvaluationEventBuilder rather than an instance method.

Reviewed Changes

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

File	Description
src/OpenFeature/Telemetry/EvaluationEventBuilder.cs	Updated class and methods to be generic with a static API.
test/OpenFeature.Tests/Telemetry/EvaluationEventBuilderTests.cs	Adjusted tests to adopt the new generic static Build method.