Skip to content

Refactor event handling and test data usage - #768

Merged
StuartFerguson merged 1 commit into
mainfrom
nightly/#767_fix_failed_build
Jul 14, 2026
Merged

StuartFerguson merged 1 commit into
mainfrom
nightly/#767_fix_failed_build

Conversation

@StuartFerguson

Copy link
Copy Markdown
Member

Removed AppliedEventCount tracking from FileProfileAggregate and related extension methods. Refactored update logic to use IsDomainEventRequired for more precise event application, trimming whitespace except line breaks. Replaced hardcoded test data with TestData constants in tests and added SafaricomListeningDirectory. Updated test project to use full debug type.

closes #767

Removed AppliedEventCount tracking from FileProfileAggregate and related extension methods. Refactored update logic to use IsDomainEventRequired for more precise event application, trimming whitespace except line breaks. Replaced hardcoded test data with TestData constants in tests and added SafaricomListeningDirectory. Updated test project to use full debug type.
@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 16 complexity · 0 duplication

Metric Results
Complexity 16
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@StuartFerguson
StuartFerguson merged commit d9a548d into main Jul 14, 2026
6 checks passed
@github-actions
github-actions Bot deleted the nightly/#767_fix_failed_build branch September 13, 2026 02:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Investigate Nightly Build Failure [Codacy] Define a constant instead of using this literal 'Voucher Profile' 4 times.

1 participant