From 4f6cc0bed627963a64aac8bbf8d0f805c0c3fdff Mon Sep 17 00:00:00 2001
From: Sebastijan K <58827427+sebastijankuzner@users.noreply.github.com>
Date: Thu, 13 May 2021 17:02:28 +0200
Subject: [PATCH] chore(core-state): remove valuesByIndex (#4383)

---
 .../unit/core-state/wallets/wallet-repository.test.ts  | 10 +++++-----
 packages/core-kernel/src/contracts/state/wallets.ts    |  2 --
 packages/core-state/src/wallets/wallet-repository.ts   |  4 ----
 3 files changed, 5 insertions(+), 11 deletions(-)

diff --git a/__tests__/unit/core-state/wallets/wallet-repository.test.ts b/__tests__/unit/core-state/wallets/wallet-repository.test.ts
index ca1871fc15..7c8ddd306e 100644
--- a/__tests__/unit/core-state/wallets/wallet-repository.test.ts
+++ b/__tests__/unit/core-state/wallets/wallet-repository.test.ts
@@ -281,11 +281,11 @@ describe("Wallet Repository", () => {
         );
     });
 
-    describe("valuesByIndex", () => {
+    describe("allByIndex", () => {
         it("should return values on index", () => {
             const wallet = walletRepo.findByAddress("address");
 
-            expect(walletRepo.valuesByIndex("addresses")).toEqual([wallet]);
+            expect(walletRepo.allByIndex("addresses")).toEqual([wallet]);
         });
     });
 
@@ -294,18 +294,18 @@ describe("Wallet Repository", () => {
             const wallet = walletRepo.findByAddress("address");
             walletRepo.setOnIndex("addresses", "address2", wallet);
 
-            expect(walletRepo.valuesByIndex("addresses")).toEqual([wallet, wallet]);
+            expect(walletRepo.allByIndex("addresses")).toEqual([wallet, wallet]);
         });
     });
 
     describe("forgetOnIndex", () => {
         it("should forget wallet on index", () => {
             const wallet = walletRepo.findByAddress("address");
-            expect(walletRepo.valuesByIndex("addresses")).toEqual([wallet]);
+            expect(walletRepo.allByIndex("addresses")).toEqual([wallet]);
 
             walletRepo.forgetOnIndex("addresses", "address");
 
-            expect(walletRepo.valuesByIndex("addresses")).toEqual([]);
+            expect(walletRepo.allByIndex("addresses")).toEqual([]);
         });
     });
 });
diff --git a/packages/core-kernel/src/contracts/state/wallets.ts b/packages/core-kernel/src/contracts/state/wallets.ts
index 6b07615b92..f978365906 100644
--- a/packages/core-kernel/src/contracts/state/wallets.ts
+++ b/packages/core-kernel/src/contracts/state/wallets.ts
@@ -191,8 +191,6 @@ export interface WalletRepository {
 
     index(wallet: Wallet): void;
 
-    valuesByIndex(index: string): ReadonlyArray<Wallet>;
-
     setOnIndex(index: string, key: string, wallet: Wallet): void;
 
     forgetOnIndex(index: string, key: string): void;
diff --git a/packages/core-state/src/wallets/wallet-repository.ts b/packages/core-state/src/wallets/wallet-repository.ts
index 9d78106c18..43ca9feda2 100644
--- a/packages/core-state/src/wallets/wallet-repository.ts
+++ b/packages/core-state/src/wallets/wallet-repository.ts
@@ -136,10 +136,6 @@ export class WalletRepository implements Contracts.State.WalletRepository {
         }
     }
 
-    public valuesByIndex(index: string): ReadonlyArray<Contracts.State.Wallet> {
-        return this.getIndex(index).values();
-    }
-
     public setOnIndex(index: string, key: string, wallet: Contracts.State.Wallet): void {
         this.getIndex(index).set(key, wallet);
     }