-
Notifications
You must be signed in to change notification settings - Fork 492
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[trivial] Remove IRepository
and replace it with IFileSystemBlockRepository
#12236
[trivial] Remove IRepository
and replace it with IFileSystemBlockRepository
#12236
Conversation
@@ -109,7 +109,7 @@ public async Task TestServicesAsync(string networkString) | |||
ExpirationScanFrequency = TimeSpan.FromSeconds(30) | |||
}); | |||
|
|||
IRepository<uint256, Block> blockRepository = bitcoinStore.BlockRepository; | |||
IFileSystemBlockRepository blockRepository = bitcoinStore.BlockRepository; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is what the PR is about. Just change IRepository<uint256, Block>
to IFileSystemBlockRepository
.
/// <summary> | ||
/// IRepository is a generic abstraction of a repository pattern | ||
/// </summary> | ||
public interface IRepository<TID, TElement> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Replaced with IFileSystemBlockRepository.cs
|
||
namespace WalletWasabi.Tests.UnitTests.Wallet; | ||
|
||
public class MockBlockRepository : IRepository<uint256, Block> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Replaced with MockFileSystemBlockRepository
IRepository
and replace it with IFileSystemBlockRepository
IRepository
and replace it with IFileSystemBlockRepository
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This is a helper PR for #12184 that allows easier mocking. It's also a general cleanup because
IRepository<TID, TElem>
is used only for block repository purposes and as such it's needlessly generic.