Add testing config and lint errors (#10)

* add tests: jest config and common testing
* remove unnecessary file
* lint: renamed files, apply ESLint
* group similar features in same file

Signed-off-by: inge4pres <francesco.gualazzi@elastic.co>

src/plugins/profiling/common/index.test.ts

@@ -0,0 +1,21 @@
+/*
+ * Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one
+ * or more contributor license agreements. Licensed under the Elastic License
+ * 2.0 and the Server Side Public License, v 1; you may not use this file except
+ * in compliance with, at your election, the Elastic License 2.0 or the Server
+ * Side Public License, v 1.
+ */
+
+import { timeRangeFromRequest } from './index';
+
+describe('Common profiling helpers', () => {
+  test('convert query parameters time ranges into tuple', () => {
+    const request = {
+      query: {
+        timeFrom: 123,
+        timeTo: 456,
+      },
+    };
+    expect(timeRangeFromRequest(request)).toEqual([123, 456]);
+  });
+});

src/plugins/profiling/common/index.ts

@@ -5,7 +5,6 @@
  * in compliance with, at your election, the Elastic License 2.0 or the Server
  * Side Public License, v 1.
  */
-import {KibanaRequest} from 'kibana/server';
 
 export const PLUGIN_ID = 'profiling';
 export const PLUGIN_NAME = 'profiling';
@@ -24,7 +23,7 @@ export function getLocalRoutePaths() {
     TopNThreads: `${BASE_LOCAL_ROUTE_PATH}/topn/threads`,
     TopNTraces: `${BASE_LOCAL_ROUTE_PATH}/topn/traces`,
     FlamechartElastic: `${BASE_LOCAL_ROUTE_PATH}/flamechart/elastic`,
-    FlamechartPixi: `${BASE_LOCAL_ROUTE_PATH}/flamechart/pixi`
+    FlamechartPixi: `${BASE_LOCAL_ROUTE_PATH}/flamechart/pixi`,
   };
 }
 
@@ -37,7 +36,7 @@ export function getRemoteRoutePaths() {
     TopNThreads: `${BASE_REMOTE_ROUTE_PATH}/topn/threads`,
     TopNTraces: `${BASE_REMOTE_ROUTE_PATH}/topn/traces`,
     FlamechartElastic: `${BASE_REMOTE_ROUTE_PATH}/flamechart/elastic`,
-    FlamechartPixi: `${BASE_REMOTE_ROUTE_PATH}/flamechart/pixi`
+    FlamechartPixi: `${BASE_REMOTE_ROUTE_PATH}/flamechart/pixi`,
   };
 }
 

src/plugins/profiling/jest.config.js

@@ -0,0 +1,16 @@
+/*
+ * Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one
+ * or more contributor license agreements. Licensed under the Elastic License
+ * 2.0 and the Server Side Public License, v 1; you may not use this file except
+ * in compliance with, at your election, the Elastic License 2.0 or the Server
+ * Side Public License, v 1.
+ */
+
+module.exports = {
+  preset: '@kbn/test',
+  rootDir: '../../..',
+  roots: ['<rootDir>/src/plugins/profiling'],
+  coverageDirectory: '<rootDir>/target/kibana-coverage/jest/src/plugins/profiling',
+  coverageReporters: ['text', 'html'],
+  collectCoverageFrom: ['<rootDir>/src/plugins/profiling/{common,public,server}/**/*.{ts,tsx}'],
+};

src/plugins/profiling/server/routes/index.ts

@@ -5,4 +5,39 @@
  * in compliance with, at your election, the Elastic License 2.0 or the Server
  * Side Public License, v 1.
  */
-export { registerRoutes } from './register_routes';
+import type { IRouter } from 'kibana/server';
+import { DataRequestHandlerContext } from '../../../data/server';
+import { registerFlameChartElasticRoute, registerFlameChartPixiRoute } from './load_flamechart';
+import {
+  registerTraceEventsTopNContainersRoute,
+  registerTraceEventsTopNDeploymentsRoute,
+  registerTraceEventsTopNHostsRoute,
+  registerTraceEventsTopNStackTracesRoute,
+  registerTraceEventsTopNThreadsRoute,
+} from './load_topn';
+
+import { registerFlameChartSearchRoute } from './search_flamechart';
+import {
+  registerTraceEventsTopNContainersSearchRoute,
+  registerTraceEventsTopNDeploymentsSearchRoute,
+  registerTraceEventsTopNHostsSearchRoute,
+  registerTraceEventsTopNStackTracesSearchRoute,
+  registerTraceEventsTopNThreadsSearchRoute,
+} from './search_topn';
+
+export function registerRoutes(router: IRouter<DataRequestHandlerContext>) {
+  registerFlameChartElasticRoute(router);
+  registerFlameChartPixiRoute(router);
+  registerTraceEventsTopNContainersRoute(router);
+  registerTraceEventsTopNDeploymentsRoute(router);
+  registerTraceEventsTopNHostsRoute(router);
+  registerTraceEventsTopNStackTracesRoute(router);
+  registerTraceEventsTopNThreadsRoute(router);
+
+  registerFlameChartSearchRoute(router);
+  registerTraceEventsTopNContainersSearchRoute(router);
+  registerTraceEventsTopNDeploymentsSearchRoute(router);
+  registerTraceEventsTopNHostsSearchRoute(router);
+  registerTraceEventsTopNStackTracesSearchRoute(router);
+  registerTraceEventsTopNThreadsSearchRoute(router);
+}

src/plugins/profiling/server/routes/load_flameChartElastic.ts → src/plugins/profiling/server/routes/load_flamechart.ts

@@ -6,8 +6,8 @@
  * Side Public License, v 1.
  */
 import { schema } from '@kbn/config-schema';
+import type { IRouter } from 'kibana/server';
 import type { DataRequestHandlerContext } from '../../../data/server';
-import type { IRouter } from '../../../../core/server';
 import { getLocalRoutePaths, timeRangeFromRequest } from '../../common';
 import { mapFlamechart } from './mappings';
 
@@ -33,3 +33,26 @@ export function registerFlameChartElasticRoute(router: IRouter<DataRequestHandle
     }
   );
 }
+
+export function registerFlameChartPixiRoute(router: IRouter<DataRequestHandlerContext>) {
+  const paths = getLocalRoutePaths();
+  router.get(
+    {
+      path: paths.FlamechartPixi,
+      validate: {
+        query: schema.object({
+          index: schema.maybe(schema.string()),
+          projectID: schema.maybe(schema.string()),
+          timeFrom: schema.maybe(schema.string()),
+          timeTo: schema.maybe(schema.string()),
+        }),
+      },
+    },
+    async (ctx, request, response) => {
+      const [timeFrom, timeTo] = timeRangeFromRequest(request);
+      const src = await import(`../fixtures/flamechart_${timeTo - timeFrom}`);
+      delete src.default;
+      return response.ok({ body: src });
+    }
+  );
+}