chore(core-state): remove unused clear method on stateStore (#4336)

ArkEcosystem · Feb 26, 2021 · c20652e · c20652e
1 parent 17ec422
commit c20652e
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 31 deletions.
diff --git a/__tests__/unit/core-state/stores/state.test.ts b/__tests__/unit/core-state/stores/state.test.ts
@@ -15,17 +15,13 @@ let factory: FactoryBuilder;
 let logger: jest.SpyInstance;
 let dispatchSpy: jest.SpyInstance;
 
-beforeAll(async () => {
+beforeEach(async () => {
     const initialEnv = await setUp();
     factory = initialEnv.factory;
     logger = initialEnv.spies.logger.info;
     dispatchSpy = initialEnv.spies.dispatchSpy;
     stateStorage = initialEnv.sandbox.app.get(Container.Identifiers.StateStore);
-});
-
-beforeEach(() => {
     blocks = makeChainedBlocks(101, factory.get("Block"));
-    stateStorage.clear();
 });
 
 afterAll(() => jest.clearAllMocks());
@@ -426,18 +422,6 @@ describe("State Storage", () => {
         });
     });
 
-    describe("clear", () => {
-        it("should clear the last blocks", () => {
-            for (let i = 0; i < 100; i++) {
-                stateStorage.setLastBlock(blocks[i]);
-            }
-
-            expect(stateStorage.getLastBlocks()).toHaveLength(100);
-            stateStorage.clear();
-            expect(stateStorage.getLastBlocks()).toHaveLength(0);
-        });
-    });
-
     describe("reset", () => {
         it("should reset initial blockchain state", () => {
             const mockBlockChainMachine = {
@@ -539,7 +523,7 @@ describe("State Storage", () => {
 
             stateStorage.pushPingBlock(blocks[5].data, true);
             expect(logger).toHaveBeenCalledWith(
-                `Previous block ${blocks[3].data.height.toLocaleString()} pinged blockchain 1 times`,
+                `Previous block ${blocks[3].data.height.toLocaleString()} pinged blockchain 0 times`,
             );
 
             const blockPing = stateStorage.getBlockPing()!;

diff --git a/packages/core-kernel/src/contracts/state/state-store.ts b/packages/core-kernel/src/contracts/state/state-store.ts
@@ -57,11 +57,6 @@ export interface StateStore {
 
     reset(blockchainMachine): void;
 
-    /**
-     * Clear last blocks.
-     */
-    clear(): void;
-
     /**
      * Is wakeup timeout set.
      */

diff --git a/packages/core-state/src/stores/state.ts b/packages/core-state/src/stores/state.ts
@@ -132,14 +132,6 @@ export class StateStore implements Contracts.State.StateStore {
         this.blockchain = blockchainMachine.initialState;
     }
 
-    /**
-     * Clear last blocks.
-     */
-    public clear(): void {
-        this.lastBlocks = this.lastBlocks.clear();
-        this.cachedTransactionIds = this.cachedTransactionIds.clear();
-    }
-
     public isWakeUpTimeoutSet(): boolean {
         return !!this.wakeUpTimeout;
     }