Skip to content

Conversation

@StuartFerguson
Copy link
Member

Replaced instances of IEstateClient with IIntermediateEstateClient across multiple test files and service implementations. This change affects test classes such as FloatDomainServiceTests, SettlementDomainServiceTests, TransactionDomainServiceTests, and TransactionValidationServiceTests, as well as service classes like FloatDomainService, SettlementDomainService, and VoucherDomainService. The new IIntermediateEstateClient interface wraps the existing IEstateClient, enhancing the architecture by introducing an intermediate layer for estate client operations, improving separation of concerns and facilitating easier testing.

Replaced instances of IEstateClient with IIntermediateEstateClient across multiple test files and service implementations. This change affects test classes such as FloatDomainServiceTests, SettlementDomainServiceTests, TransactionDomainServiceTests, and TransactionValidationServiceTests, as well as service classes like FloatDomainService, SettlementDomainService, and VoucherDomainService. The new IIntermediateEstateClient interface wraps the existing IEstateClient, enhancing the architecture by introducing an intermediate layer for estate client operations, improving separation of concerns and facilitating easier testing.
@StuartFerguson StuartFerguson linked an issue Jan 20, 2025 that may be closed by this pull request
@StuartFerguson StuartFerguson merged commit 81def56 into master Jan 20, 2025
5 checks passed
@github-actions github-actions bot deleted the task/#693_intermediate_estate_client branch March 22, 2025 01:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Introduce intermediate estate client

2 participants