Skip to content

feat: add FakeXrmEasy mocked tests for BulkOperationExecutor (Phase 3)#97

Merged
joshsmithxrm merged 4 commits into
mainfrom
feature/integration-tests-phase3
Jan 2, 2026
Merged

feat: add FakeXrmEasy mocked tests for BulkOperationExecutor (Phase 3)#97
joshsmithxrm merged 4 commits into
mainfrom
feature/integration-tests-phase3

docs: improve review-bot-comments to handle duplicates

3ee9bd0
Select commit
Loading
Failed to load commit list.
GitHub Advanced Security / CodeQL succeeded Jan 2, 2026 in 2s

4 new alerts

New alerts in code changed by this pull request

  • 4 notes

See annotations below for details.

View all branch alerts.

Annotations

Check notice on line 81 in tests/PPDS.Dataverse.IntegrationTests/BulkOperations/ElasticTableTests.cs

See this annotation in the file changed.

Code scanning / CodeQL

Missed opportunity to use Select Note test

This foreach loop immediately
maps its iteration variable to another variable
- consider mapping the sequence explicitly using '.Select(...)'.

Check notice on line 165 in tests/PPDS.Dataverse.IntegrationTests/BulkOperations/ElasticTableTests.cs

See this annotation in the file changed.

Code scanning / CodeQL

Missed opportunity to use Select Note test

This foreach loop immediately
maps its iteration variable to another variable
- consider mapping the sequence explicitly using '.Select(...)'.

Check notice on line 29 in tests/PPDS.Dataverse.IntegrationTests/FakeMessageExecutors/CreateMultipleRequestExecutor.cs

See this annotation in the file changed.

Code scanning / CodeQL

Missed opportunity to use Select Note test

This foreach loop immediately
maps its iteration variable to another variable
- consider mapping the sequence explicitly using '.Select(...)'.

Check notice on line 30 in tests/PPDS.Dataverse.IntegrationTests/FakeMessageExecutors/UpsertMultipleRequestExecutor.cs

See this annotation in the file changed.

Code scanning / CodeQL

Missed opportunity to use Select Note test

This foreach loop immediately
maps its iteration variable to another variable
- consider mapping the sequence explicitly using '.Select(...)'.