Use type alias for controller state rather than interface

The mock controller state in the base controller tests now uses a type
alias for the controller state rather than an interface. This was
required to get around an incompatibility between
`Record<string, unknown>` and interfaces[1].

The `@typescript-eslint/consistent-type-definitions` ESLint rule has
been disabled, as this problem will be encountered fairly frequently.

[1]: microsoft/TypeScript#15300 (comment)

Author: Gudahtt

Diff for: .eslintrc.js

@@ -25,6 +25,9 @@ module.exports = {
     'no-shadow': 'off',
     '@typescript-eslint/no-shadow': 'error',
     '@typescript-eslint/indent': 'off',
+    // disabled due to incompatibility with Record<string, unknown>
+    // See https://github.com/Microsoft/TypeScript/issues/15300#issuecomment-702872440
+    '@typescript-eslint/consistent-type-definitions': 'off',
 
     // TODO re-enable most of these rules
     '@typescript-eslint/no-non-null-assertion': 'off',

Diff for: src/BaseControllerV2.test.ts

@@ -3,9 +3,9 @@ import * as sinon from 'sinon';
 
 import { BaseController } from './BaseControllerV2';
 
-interface MockControllerState {
+type MockControllerState = {
   count: number;
-}
+};
 
 class MockController extends BaseController<MockControllerState> {
   update(callback: (state: Draft<MockControllerState>) => void | MockControllerState) {