refactor: simplify PropertySetup

[vbreuss]

This PR simplifies the PropertySetup class by removing redundant logic and improving test coverage for callback handling.

Key Changes:

• Removed unnecessary foundCallback tracking variable in InvokeGetter
• Simplified conditional logic for value initialization
• Added tests for MethodSetupResult.TriggerCallbacks with null parameters
• Improved test coverage for parameter callback invocation scenarios

[Copilot]

Pull request overview

This PR simplifies the PropertySetup class by removing redundant logic and improving test coverage for callback handling.

Changes:

• Removed unnecessary foundCallback tracking variable in InvokeGetter
• Simplified conditional logic for value initialization
• Added tests for MethodSetupResult.TriggerCallbacks with null parameters
• Improved test coverage for parameter callback invocation scenarios

Reviewed changes

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

File	Description
Source/Mockolate/Setup/PropertySetup.cs	Removed redundant foundCallback variable and simplified initialization logic in InvokeGetter and InitializeValue
Tests/Mockolate.Tests/MockMethods/SetupMethodTests.cs	Added tests for null parameter handling and parameter callback invocation with matching/non-matching counts

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[github-actions]

Test Results

    21 files  ± 0      21 suites  ±0   7m 46s ⏱️ + 1m 12s
 2 379 tests + 2   2 378 ✅ + 2  1 💤 ±0  0 ❌ ±0 
15 946 runs  +14  15 945 ✅ +14  1 💤 ±0  0 ❌ ±0 

Results for commit adc78ad. ± Comparison against base commit 198f8cb.

This pull request removes 1 and adds 3 tests. Note that renamed tests count towards both.

Mockolate.Tests.MockMethods.SetupMethodTests ‑ TriggerCallbacks_ArrayLengthDoesNotMatch_ShouldNotThrow

Mockolate.Tests.MockMethods.SetupMethodTests ‑ MethodSetupResult_TriggerCallbacks_Null_ShouldTriggerCallbacksWithNullArray
Mockolate.Tests.MockMethods.SetupMethodTests ‑ TriggerCallbacks_WhenParameterCountDoesNotMatch_ShouldNotInvokeParameterCallbacks
Mockolate.Tests.MockMethods.SetupMethodTests ‑ TriggerCallbacks_WhenParameterCountMatches_ShouldInvokeParameterCallbacks

[github-actions]

🚀 Benchmark Results

Details

BenchmarkDotNet v0.15.8, Linux Ubuntu 24.04.3 LTS (Noble Numbat)

AMD EPYC 7763 2.45GHz, 1 CPU, 4 logical and 2 physical cores

.NET SDK 10.0.102

[Host] : .NET 10.0.2 (10.0.2, 10.0.225.61305), X64 RyuJIT x86-64-v3
Job=InProcess  Toolchain=InProcessEmitToolchain  IterationCount=15

LaunchCount=1  WarmupCount=10

Method	Mean	Error	StdDev	Gen0	Gen1	Allocated
Simple_Mockolate	1.589 μs	0.0142 μs	0.0126 μs	0.2270	-	3.73 KB
Simple_Moq	180.212 μs	0.7719 μs	0.6446 μs	0.4883	-	14.77 KB
Simple_NSubstitute	5.745 μs	0.0206 μs	0.0172 μs	0.5569	0.0076	9.14 KB
Simple_FakeItEasy	6.490 μs	0.0190 μs	0.0159 μs	0.4959	-	8.11 KB

[github-actions]

This is addressed in release v1.0.3.