devDeps: @metamask/eslint* packages 9->10 (#106)

* devDeps: bump electron; dedupe browserslist

* yarn dedupe

* devDeps: bump @metamask/eslint* packages 9->10

* lint:fix

.eslintrc.js

@@ -7,6 +7,9 @@ module.exports = {
     {
       files: ['*.ts'],
       extends: ['@metamask/eslint-config-typescript'],
+      rules: {
+        '@typescript-eslint/consistent-type-definitions': 'off',
+      },
     },
 
     {

package.json

@@ -39,25 +39,25 @@
     "@jest-runner/electron": "^3.0.1",
     "@lavamoat/allow-scripts": "^2.3.1",
     "@metamask/auto-changelog": "^3.0.0",
-    "@metamask/eslint-config": "^9.0.0",
-    "@metamask/eslint-config-jest": "^9.0.0",
-    "@metamask/eslint-config-nodejs": "^9.0.0",
-    "@metamask/eslint-config-typescript": "^9.0.1",
+    "@metamask/eslint-config": "^10.0.0",
+    "@metamask/eslint-config-jest": "^10.0.0",
+    "@metamask/eslint-config-nodejs": "^10.0.0",
+    "@metamask/eslint-config-typescript": "^10.0.0",
     "@types/chrome": "^0.0.204",
     "@types/jest": "^26.0.13",
     "@types/node": "^18.0.0",
     "@types/readable-stream": "^2.3.9",
-    "@typescript-eslint/eslint-plugin": "^4.33.0",
-    "@typescript-eslint/parser": "^4.33.0",
+    "@typescript-eslint/eslint-plugin": "^5.33.0",
+    "@typescript-eslint/parser": "^5.33.0",
     "browserify": "^17.0.0",
-    "electron": "^18.3.7",
-    "eslint": "^7.30.0",
+    "electron": "^18.3.15",
+    "eslint": "^8.21.0",
     "eslint-config-prettier": "^8.1.0",
-    "eslint-plugin-import": "^2.22.1",
-    "eslint-plugin-jest": "^24.3.4",
+    "eslint-plugin-import": "^2.26.0",
+    "eslint-plugin-jest": "^26.8.2",
     "eslint-plugin-jsdoc": "^39.3.3",
     "eslint-plugin-node": "^11.1.0",
-    "eslint-plugin-prettier": "^3.3.1",
+    "eslint-plugin-prettier": "^4.2.1",
     "jest": "^26.6.3",
     "jest-it-up": "^2.0.2",
     "prettier": "^2.7.1",

src/BasePostMessageStream.test.ts

@@ -37,7 +37,7 @@ describe('BasePostMessageStream', () => {
       };
 
       stream.once('error', (error) => {
-        expect(error.message).toStrictEqual('push error');
+        expect(error.message).toBe('push error');
         resolve();
       });
 

src/WebWorker/WebWorker.test.ts

@@ -37,7 +37,7 @@ describe('WebWorker Streams', () => {
     // Send message to worker, triggering a response
     parentStream.write(111);
 
-    expect(await responsePromise).toStrictEqual(555);
+    expect(await responsePromise).toBe(555);
 
     // Check that events with falsy data are ignored as expected
     parentStream.once('data', (data) => {
@@ -48,7 +48,7 @@ describe('WebWorker Streams', () => {
     // Terminate worker, destroy parent, and check that parent was destroyed
     worker.terminate();
     parentStream.destroy();
-    expect(parentStream.destroyed).toStrictEqual(true);
+    expect(parentStream.destroyed).toBe(true);
   });
 
   describe('WebWorkerPostMessageStream', () => {

src/browser.test.ts

@@ -16,7 +16,7 @@ describe('post-message-stream', () => {
       );
 
       for (const exportName of expectedExports) {
-        expect(exportName in PostMessageStream).toStrictEqual(true);
+        expect(exportName in PostMessageStream).toBe(true);
       }
     });
   });

src/index.test.ts

@@ -20,7 +20,7 @@ describe('post-message-stream', () => {
       );
 
       for (const exportName of expectedExports) {
-        expect(exportName in PostMessageStream).toStrictEqual(true);
+        expect(exportName in PostMessageStream).toBe(true);
       }
     });
   });

src/node-process/node-process.test.ts

@@ -51,7 +51,7 @@ describe('Node Child Process Streams', () => {
     // Send message to child, triggering a response
     parentStream.write(111);
 
-    expect(await responsePromise).toStrictEqual(555);
+    expect(await responsePromise).toBe(555);
 
     // Check that events with falsy data are ignored as expected
     parentStream.once('data', (data) => {
@@ -63,7 +63,7 @@ describe('Node Child Process Streams', () => {
     // was destroyed
     childProcess.kill();
     parentStream.destroy();
-    expect(parentStream.destroyed).toStrictEqual(true);
+    expect(parentStream.destroyed).toBe(true);
   });
 
   describe('ProcessParentMessageStream', () => {

src/node-thread/node-thread.test.ts

@@ -49,7 +49,7 @@ describe('Node Thread Streams', () => {
     // Send message to child, triggering a response
     parentStream.write(111);
 
-    expect(await responsePromise).toStrictEqual(555);
+    expect(await responsePromise).toBe(555);
 
     // Check that events with falsy data are ignored as expected
     parentStream.once('data', (data) => {
@@ -61,7 +61,7 @@ describe('Node Thread Streams', () => {
     // was destroyed
     thread.terminate();
     parentStream.destroy();
-    expect(parentStream.destroyed).toStrictEqual(true);
+    expect(parentStream.destroyed).toBe(true);
   });
 
   describe('ThreadParentMessageStream', () => {

src/runtime/BrowserRuntimePostMessageStream.test.ts

@@ -115,7 +115,7 @@ describe('BrowserRuntimePostMessageStream', () => {
     // Write to stream A, triggering a response from stream B
     streamA.write(111);
 
-    expect(await responsePromise).toStrictEqual(555);
+    expect(await responsePromise).toBe(555);
 
     const throwingListener = (data: any) => {
       throw new Error(`Unexpected data on stream: ${data}`);
@@ -129,7 +129,7 @@ describe('BrowserRuntimePostMessageStream', () => {
     // Destroy streams and confirm that they were destroyed
     streamA.destroy();
     streamB.destroy();
-    expect(streamA.destroyed).toStrictEqual(true);
-    expect(streamB.destroyed).toStrictEqual(true);
+    expect(streamA.destroyed).toBe(true);
+    expect(streamB.destroyed).toBe(true);
   });
 });

src/window/WindowPostMessageStream.test.ts

@@ -51,7 +51,7 @@ describe('WindowPostMessageStream', () => {
     // Write to stream A, triggering a response from stream B
     streamA.write(111);
 
-    expect(await responsePromise).toStrictEqual(555);
+    expect(await responsePromise).toBe(555);
 
     // Check that events without e.g. the correct event.source are ignored as
     // expected
@@ -65,8 +65,8 @@ describe('WindowPostMessageStream', () => {
     // Destroy streams and confirm that they were destroyed
     streamA.destroy();
     streamB.destroy();
-    expect(streamA.destroyed).toStrictEqual(true);
-    expect(streamB.destroyed).toStrictEqual(true);
+    expect(streamA.destroyed).toBe(true);
+    expect(streamB.destroyed).toBe(true);
   });
 
   it('can take targetOrigin as an option', () => {
@@ -75,7 +75,7 @@ describe('WindowPostMessageStream', () => {
       target: 'target',
       targetOrigin: '*',
     });
-    expect((stream as any)._targetOrigin).toStrictEqual('*');
+    expect((stream as any)._targetOrigin).toBe('*');
     stream.destroy();
   });
 });