[ts] enable "resolveJsonModule" and disable existing failures

Author: spalger

tsconfig.base.json

@@ -39,6 +39,8 @@
     // Resolve modules in the same way as Node.js. Aka make `require` works the
     // same in TypeScript as it does in Node.js.
     "moduleResolution": "node",
+    // "resolveJsonModule" allows for importing, extracting types from and generating .json files.
+    "resolveJsonModule": true,
     // Disallow inconsistently-cased references to the same file.
     "forceConsistentCasingInFileNames": true,
     // Forbid unused local variables as the rule was deprecated by ts-lint

typings/index.d.ts

@@ -35,15 +35,6 @@ declare module '*.svg' {
   export default content;
 }
 
-// allow JSON files to be imported directly without lint errors
-// see: https://github.com/palantir/tslint/issues/1264#issuecomment-228433367
-// and: https://github.com/Microsoft/TypeScript/wiki/Breaking-Changes#arbitrary-expressions-are-forbidden-in-export-assignments-in-ambient-contexts
-declare module '*.json' {
-  const json: any;
-  // eslint-disable-next-line import/no-default-export
-  export default json;
-}
-
 type MethodKeysOf<T> = {
   [K in keyof T]: T[K] extends (...args: any[]) => any ? K : never;
 }[keyof T];

x-pack/plugins/apm/public/components/app/ErrorGroupOverview/List/__test__/List.test.tsx

@@ -43,6 +43,7 @@ describe('ErrorGroupOverview -> List', () => {
       <MemoryRouter>
         <MockApmPluginContextWrapper>
           <MockUrlParamsContextProvider>
+            {/* @ts-expect-error invalid json props */}
             <ErrorGroupList items={props.items} serviceName="opbeans-python" />
           </MockUrlParamsContextProvider>
         </MockApmPluginContextWrapper>

x-pack/plugins/apm/public/components/shared/Links/DiscoverLinks/__test__/DiscoverTransactionButton.test.tsx

@@ -15,6 +15,7 @@ import mockTransaction from './mockTransaction.json';
 
 describe('DiscoverTransactionLink component', () => {
   it('should render with data', () => {
+    // @ts-expect-error invalid json mock
     const transaction: Transaction = mockTransaction;
 
     expect(
@@ -25,6 +26,7 @@ describe('DiscoverTransactionLink component', () => {
 
 describe('getDiscoverQuery', () => {
   it('should return the correct query params object', () => {
+    // @ts-expect-error invalid json mock
     const transaction: Transaction = mockTransaction;
     const result = getDiscoverQuery(transaction);
     expect(result).toMatchSnapshot();

x-pack/plugins/apm/server/lib/service_map/group_resource_nodes.test.ts

@@ -10,14 +10,15 @@ import expectedGroupedData from './mock_responses/group_resource_nodes_grouped.j
 
 describe('groupResourceNodes', () => {
   it('should group external nodes', () => {
+    // @ts-expect-error invalid json mock
     const responseWithGroups = groupResourceNodes(preGroupedData);
     expect(responseWithGroups.elements).toHaveLength(
       expectedGroupedData.elements.length
     );
     for (const element of responseWithGroups.elements) {
       const expectedElement = expectedGroupedData.elements.find(
         ({ data: { id } }: { data: { id: string } }) => id === element.data.id
-      );
+      )!;
       expect(element).toMatchObject(expectedElement);
     }
   });

x-pack/plugins/apm/server/lib/services/annotations/index.test.ts

@@ -31,6 +31,7 @@ describe('getServiceAnnotations', () => {
             searchAggregatedTransactions: false,
           }),
         {
+          // @ts-expect-error invalid json mock
           mockResponse: () => noVersions,
         }
       );
@@ -50,6 +51,7 @@ describe('getServiceAnnotations', () => {
             searchAggregatedTransactions: false,
           }),
         {
+          // @ts-expect-error invalid json mock
           mockResponse: () => oneVersion,
         }
       );
@@ -74,6 +76,7 @@ describe('getServiceAnnotations', () => {
             searchAggregatedTransactions: false,
           }),
         {
+          // @ts-expect-error invalid json mock
           mockResponse: () => responses.shift(),
         }
       );

x-pack/plugins/canvas/canvas_plugin_src/functions/server/demodata/index.ts

@@ -61,6 +61,7 @@ export function demodata(): ExpressionFunctionDefinition<
             { id: 'project', name: 'project', meta: { type: 'string' } },
             { id: 'percent_uptime', name: 'percent_uptime', meta: { type: 'number' } },
           ],
+          // @ts-expect-error invalid json mock
           rows: sortBy(demoRows, 'time'),
         };
       } else if (args.type === DemoRows.SHIRTS) {

x-pack/plugins/canvas/shareable_runtime/components/__tests__/app.test.tsx

@@ -4,10 +4,10 @@
  * you may not use this file except in compliance with the Elastic License.
  */
 
-/*	
-  One test relies on react-dom at a version of 16.9... it can be enabled	
-  once renovate completes the upgrade.  Relevant code has been commented out	
-  in the meantime.	
+/*
+  One test relies on react-dom at a version of 16.9... it can be enabled
+  once renovate completes the upgrade.  Relevant code has been commented out
+  in the meantime.
 */
 
 import { mount, ReactWrapper } from 'enzyme';

x-pack/plugins/canvas/shareable_runtime/test/index.ts

@@ -11,4 +11,11 @@ import test from './workpads/test.json';
 export * from './utils';
 
 export type WorkpadNames = keyof typeof sharedWorkpads;
-export const sharedWorkpads = { hello, austin, test };
+export const sharedWorkpads = {
+  // TODO: the automatic types for these JSON files are insufficient, and "austin" is so massive
+  // that Typescript refuses to type it. These should be converted to TypeScript and typed to fit
+  // the requirements. "austin" should also be reduced to the necessary data
+  hello: hello as any,
+  austin: austin as any,
+  test: test as any,
+};

x-pack/plugins/infra/public/utils/apollo_client.ts

@@ -16,6 +16,7 @@ export const createApolloClient = (fetch: HttpHandler) => {
   const cache = new InMemoryCache({
     addTypename: false,
     fragmentMatcher: new IntrospectionFragmentMatcher({
+      // @ts-expect-error apollo-cache-inmemory types don't match actual introspection data
       introspectionQueryResultData,
     }),
   });