From 3e0d717b97c4a5a32bd3ee17a147accb37edcaca Mon Sep 17 00:00:00 2001
From: Elliot Winkler <elliot.winkler@gmail.com>
Date: Wed, 31 May 2023 09:30:17 -0600
Subject: [PATCH] NetworkController: Improve test for destroy (#19314)

This test for the `destroy` method in NetworkController uses `toBe` on
the core side because the use of `toBeTruthy` produces a lint warning
there. It doesn't here, but it's good to be consistent anyway so that
comparing the differences in NetworkController tests between this
extension and core is easier.
---
 app/scripts/controllers/network/network-controller.test.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/scripts/controllers/network/network-controller.test.ts b/app/scripts/controllers/network/network-controller.test.ts
index f70f8fb98..c18ec8a25 100644
--- a/app/scripts/controllers/network/network-controller.test.ts
+++ b/app/scripts/controllers/network/network-controller.test.ts
@@ -259,7 +259,7 @@ describe('NetworkController', () => {
         blockTracker.addListener('latest', () => {
           // do nothing
         });
-        expect(blockTracker.isRunning()).toBeTruthy();
+        expect(blockTracker.isRunning()).toBe(true);
 
         await controller.destroy();