design feedback to avoid handling the entire config getter

Author: tsullivan

x-pack/legacy/plugins/reporting/export_types/common/execute_job/decrypt_job_headers.test.ts

@@ -5,37 +5,25 @@
  */
 
 import { cryptoFactory } from '../../../server/lib/crypto';
-import { ReportingConfig } from '../../../server/types';
 import { Logger } from '../../../types';
 import { decryptJobHeaders } from './decrypt_job_headers';
 
-const getMockConfig = (mockGet: jest.Mock<any, any>) => ({
-  get: mockGet,
-  kbnConfig: { get: mockGet },
-});
-const mockConfigGet = jest.fn().mockImplementation((key: string) => {
-  if (key === 'encryptionKey') {
-    return 'testencryptionkey';
-  }
-});
-const mockConfig = getMockConfig(mockConfigGet);
-
-const encryptHeaders = async (config: ReportingConfig, headers: Record<string, string>) => {
-  const crypto = cryptoFactory(config);
+const encryptHeaders = async (encryptionKey: string, headers: Record<string, string>) => {
+  const crypto = cryptoFactory(encryptionKey);
   return await crypto.encrypt(headers);
 };
 
 describe('headers', () => {
   test(`fails if it can't decrypt headers`, async () => {
     const getDecryptedHeaders = () =>
       decryptJobHeaders({
+        encryptionKey: 'abcsecretsauce',
         job: {
           headers: 'Q53+9A+zf+Xe+ceR/uB/aR/Sw/8e+M+qR+WiG+8z+EY+mo+HiU/zQL+Xn',
         },
         logger: ({
           error: jest.fn(),
         } as unknown) as Logger,
-        config: mockConfig,
       });
     await expect(getDecryptedHeaders()).rejects.toMatchInlineSnapshot(
       `[Error: Failed to decrypt report job data. Please ensure that xpack.reporting.encryptionKey is set and re-generate this report. Error: Invalid IV length]`
@@ -48,15 +36,15 @@ describe('headers', () => {
       baz: 'quix',
     };
 
-    const encryptedHeaders = await encryptHeaders(mockConfig, headers);
+    const encryptedHeaders = await encryptHeaders('abcsecretsauce', headers);
     const decryptedHeaders = await decryptJobHeaders({
+      encryptionKey: 'abcsecretsauce',
       job: {
         title: 'cool-job-bro',
         type: 'csv',
         headers: encryptedHeaders,
       },
       logger: {} as Logger,
-      config: mockConfig,
     });
     expect(decryptedHeaders).toEqual(headers);
   });

x-pack/legacy/plugins/reporting/export_types/common/execute_job/decrypt_job_headers.ts

@@ -6,7 +6,6 @@
 
 import { i18n } from '@kbn/i18n';
 import { cryptoFactory } from '../../../server/lib/crypto';
-import { ReportingConfig } from '../../../server/types';
 import { CryptoFactory, Logger } from '../../../types';
 
 interface HasEncryptedHeaders {
@@ -18,15 +17,15 @@ export const decryptJobHeaders = async <
   JobParamsType,
   JobDocPayloadType extends HasEncryptedHeaders
 >({
-  config,
+  encryptionKey,
   job,
   logger,
 }: {
-  config: ReportingConfig;
+  encryptionKey?: string;
   job: JobDocPayloadType;
   logger: Logger;
 }): Promise<Record<string, string>> => {
-  const crypto: CryptoFactory = cryptoFactory(config);
+  const crypto: CryptoFactory = cryptoFactory(encryptionKey);
   try {
     const decryptedHeaders: Record<string, string> = await crypto.decrypt(job.headers);
     return decryptedHeaders;

x-pack/legacy/plugins/reporting/export_types/common/layouts/create_layout.ts

@@ -4,17 +4,17 @@
  * you may not use this file except in compliance with the Elastic License.
  */
 
-import { ReportingConfig } from '../../../server/types';
+import { CaptureConfig } from '../../../server/types';
 import { LayoutTypes } from '../constants';
 import { Layout, LayoutParams } from './layout';
 import { PreserveLayout } from './preserve_layout';
 import { PrintLayout } from './print_layout';
 
-export function createLayout(config: ReportingConfig, layoutParams?: LayoutParams): Layout {
+export function createLayout(captureConfig: CaptureConfig, layoutParams?: LayoutParams): Layout {
   if (layoutParams && layoutParams.id === LayoutTypes.PRESERVE_LAYOUT) {
     return new PreserveLayout(layoutParams.dimensions);
   }
 
   // this is the default because some jobs won't have anything specified
-  return new PrintLayout(config);
+  return new PrintLayout(captureConfig);
 }

x-pack/legacy/plugins/reporting/export_types/common/layouts/print_layout.ts

@@ -8,7 +8,7 @@ import path from 'path';
 import { EvaluateFn, SerializableOrJSHandle } from 'puppeteer';
 import { HeadlessChromiumDriver } from '../../../server/browsers';
 import { LevelLogger } from '../../../server/lib';
-import { ReportingConfig } from '../../../server/types';
+import { ReportingConfigType } from '../../../server/core';
 import { LayoutTypes } from '../constants';
 import { getDefaultLayoutSelectors, Layout, LayoutSelectorDictionary, Size } from './layout';
 import { CaptureConfig } from './types';
@@ -21,9 +21,9 @@ export class PrintLayout extends Layout {
   public readonly groupCount = 2;
   private captureConfig: CaptureConfig;
 
-  constructor(config: ReportingConfig) {
+  constructor(captureConfig: ReportingConfigType['capture']) {
     super(LayoutTypes.PRINT);
-    this.captureConfig = config.get('capture');
+    this.captureConfig = captureConfig;
   }
 
   public getCssOverridesPath() {

x-pack/legacy/plugins/reporting/export_types/common/lib/screenshots/get_number_of_items.ts

@@ -7,9 +7,9 @@
 import { i18n } from '@kbn/i18n';
 import { HeadlessChromiumDriver as HeadlessBrowser } from '../../../../server/browsers';
 import { LevelLogger } from '../../../../server/lib';
+import { CaptureConfig } from '../../../../server/types';
 import { LayoutInstance } from '../../layouts/layout';
 import { CONTEXT_GETNUMBEROFITEMS, CONTEXT_READMETADATA } from './constants';
-import { CaptureConfig } from './types';
 
 export const getNumberOfItems = async (
   captureConfig: CaptureConfig,

x-pack/legacy/plugins/reporting/export_types/common/lib/screenshots/observable.test.ts

@@ -16,13 +16,12 @@ jest.mock('../../../../server/browsers/chromium/puppeteer', () => ({
 }));
 
 import * as Rx from 'rxjs';
-import { get } from 'lodash';
 // eslint-disable-next-line @kbn/eslint/no-restricted-paths
 import { loggingServiceMock } from '../../../../../../../../src/core/server/mocks';
 import { LevelLogger } from '../../../../server/lib';
 import { createMockBrowserDriverFactory, createMockLayoutInstance } from '../../../../test_helpers';
 import { ConditionalHeaders, HeadlessChromiumDriver } from '../../../../types';
-import { ReportingConfig } from '../../../../server/types';
+import { CaptureConfig } from '../../../../server/types';
 import { screenshotsObservableFactory } from './observable';
 import { ElementsPositionAndAttribute } from './types';
 
@@ -32,10 +31,7 @@ import { ElementsPositionAndAttribute } from './types';
 const mockLogger = jest.fn(loggingServiceMock.create);
 const logger = new LevelLogger(mockLogger());
 
-const mockConfig = {
-  get: (key: string) => get({ capture: { timeouts: { openUrl: 13 } } }, key, null),
-  kbnConfig: { get: jest.fn() },
-} as ReportingConfig;
+const mockConfig = { timeouts: { openUrl: 13 } } as CaptureConfig;
 const mockLayout = createMockLayoutInstance(mockConfig);
 
 /*

x-pack/legacy/plugins/reporting/export_types/common/lib/screenshots/observable.ts

@@ -6,7 +6,7 @@
 
 import * as Rx from 'rxjs';
 import { catchError, concatMap, first, mergeMap, take, takeUntil, toArray } from 'rxjs/operators';
-import { ReportingConfig } from '../../../../server';
+import { CaptureConfig } from '../../../../server/types';
 import { HeadlessChromiumDriverFactory } from '../../../../types';
 import { getElementPositionAndAttributes } from './get_element_position_data';
 import { getNumberOfItems } from './get_number_of_items';
@@ -19,11 +19,9 @@ import { waitForRenderComplete } from './wait_for_render';
 import { waitForVisualizations } from './wait_for_visualizations';
 
 export function screenshotsObservableFactory(
-  config: ReportingConfig,
+  captureConfig: CaptureConfig,
   browserDriverFactory: HeadlessChromiumDriverFactory
 ) {
-  const captureConfig = config.get('capture');
-
   return function screenshotsObservable({
     logger,
     urls,

x-pack/legacy/plugins/reporting/export_types/common/lib/screenshots/open_url.ts

@@ -7,9 +7,9 @@
 import { i18n } from '@kbn/i18n';
 import { HeadlessChromiumDriver as HeadlessBrowser } from '../../../../server/browsers';
 import { LevelLogger } from '../../../../server/lib';
+import { CaptureConfig } from '../../../../server/types';
 import { ConditionalHeaders } from '../../../../types';
 import { PAGELOAD_SELECTOR } from '../../constants';
-import { CaptureConfig } from './types';
 
 export const openUrl = async (
   captureConfig: CaptureConfig,

x-pack/legacy/plugins/reporting/export_types/common/lib/screenshots/types.ts

@@ -5,12 +5,9 @@
  */
 
 import { LevelLogger } from '../../../../server/lib';
-import { ReportingConfigType } from '../../../../server/types';
 import { ConditionalHeaders, ElementPosition } from '../../../../types';
 import { LayoutInstance } from '../../layouts/layout';
 
-export type CaptureConfig = ReportingConfigType['capture'];
-
 export interface ScreenshotObservableOpts {
   logger: LevelLogger;
   urls: string[];

x-pack/legacy/plugins/reporting/export_types/common/lib/screenshots/wait_for_render.ts

@@ -7,9 +7,9 @@
 import { i18n } from '@kbn/i18n';
 import { HeadlessChromiumDriver as HeadlessBrowser } from '../../../../server/browsers';
 import { LevelLogger } from '../../../../server/lib';
+import { CaptureConfig } from '../../../../server/types';
 import { LayoutInstance } from '../../layouts/layout';
 import { CONTEXT_WAITFORRENDER } from './constants';
-import { CaptureConfig } from './types';
 
 export const waitForRenderComplete = async (
   captureConfig: CaptureConfig,