grafana/toolkit: Remove hack to expose plugin/e2e exports & types (#19467)

packages/grafana-toolkit/package.json

@@ -100,6 +100,5 @@
   },
   "_moduleAliases": {
     "puppeteer": "node_modules/puppeteer-core"
-  },
-  "types": "src/index.ts"
+  }
 }

packages/grafana-toolkit/src/cli/tasks/toolkit.build.ts

@@ -3,7 +3,6 @@ import * as fs from 'fs';
 import chalk from 'chalk';
 import { useSpinner } from '../utils/useSpinner';
 import { Task, TaskRunner } from './task';
-import escapeRegExp from 'lodash/escapeRegExp';
 
 const path = require('path');
 
@@ -120,21 +119,6 @@ const toolkitBuildTaskRunner: TaskRunner<ToolkitBuildOptions> = async () => {
   fs.mkdirSync('./dist/sass');
   await copyFiles();
   await copySassFiles();
-
-  // RYAN HACK HACK HACK
-  // when Dominik is back from vacation, we can find a better way
-  // This moves the index to the root so plugin e2e tests can import them
-  console.warn('hacking an index.js file for toolkit.  Help!');
-  const index = `${distDir}/src/index.js`;
-  fs.readFile(index, 'utf8', (err, data) => {
-    const pattern = 'require("./';
-    const js = data.replace(new RegExp(escapeRegExp(pattern), 'g'), 'require("./src/');
-    fs.writeFile(`${distDir}/index.js`, js, err => {
-      if (err) {
-        throw new Error('Error writing index: ' + err);
-      }
-    });
-  });
 };
 
 export const toolkitBuildTask = new Task<ToolkitBuildOptions>('@grafana/toolkit build', toolkitBuildTaskRunner);

packages/grafana-toolkit/src/index.ts

-export * from './e2e';
-
-// Namespace for Plugins
-import * as plugins from './plugins';
-
-export { plugins };

packages/grafana-toolkit/src/plugins/e2e/commonPluginTests.ts

 import { Browser, Page } from 'puppeteer-core';
 
-import { e2eScenario, takeScreenShot, plugins, pages } from '@grafana/toolkit';
+import { e2eScenario, takeScreenShot, pages } from '@grafana/toolkit/src/e2e';
+import { getEndToEndSettings } from '@grafana/toolkit/src/plugins';
 
 // ****************************************************************
 // NOTE, This file is copied to plugins at runtime, it is not run locally
@@ -11,7 +12,7 @@ const sleep = (milliseconds: number) => {
 };
 
 e2eScenario('Common Plugin Test', 'should pass', async (browser: Browser, page: Page) => {
-  const settings = plugins.getEndToEndSettings();
+  const settings = getEndToEndSettings();
   const pluginPage = pages.getPluginPage(settings.plugin.id);
   await pluginPage.init(page);
   await pluginPage.navigateTo();

packages/grafana-toolkit/tsconfig.json

@@ -7,7 +7,7 @@
     "rootDirs": ["."],
     "outDir": "dist/src",
     "declaration": true,
-    "declarationDir": "dist",
+    "declarationDir": "dist/src",
     "typeRoots": ["./node_modules/@types"],
     "esModuleInterop": true,
     "lib": ["es2015", "es2017.string", "dom"]

public/e2e-test/pages/dashboards/createDashboardPage.ts

-import { TestPage, ClickablePageObjectType, ClickablePageObject, Selector } from '@grafana/toolkit';
+import { TestPage, ClickablePageObjectType, ClickablePageObject, Selector } from '@grafana/toolkit/src/e2e';
 
 export interface CreateDashboardPage {
   addQuery: ClickablePageObjectType;

public/e2e-test/pages/dashboards/dashboardsPage.ts

-import { TestPage, ClickablePageObjectType, ClickablePageObject, Selector } from '@grafana/toolkit';
+import { TestPage, ClickablePageObjectType, ClickablePageObject, Selector } from '@grafana/toolkit/src/e2e';
 
 export interface DashboardsPage {
   dashboard: ClickablePageObjectType;

public/e2e-test/pages/dashboards/saveDashboardModal.ts

@@ -6,7 +6,7 @@ import {
   InputPageObjectType,
   InputPageObject,
   PageObject,
-} from '@grafana/toolkit';
+} from '@grafana/toolkit/src/e2e';
 
 export interface SaveDashboardModal {
   name: InputPageObjectType;

public/e2e-test/pages/datasources/addDataSourcePage.ts

-import { TestPage, ClickablePageObject, Selector, ClickablePageObjectType } from '@grafana/toolkit';
+import { TestPage, ClickablePageObject, Selector, ClickablePageObjectType } from '@grafana/toolkit/src/e2e';
 
 export interface AddDataSourcePage {
   testDataDB: ClickablePageObjectType;

public/e2e-test/pages/datasources/dataSources.ts

-import { TestPage } from '@grafana/toolkit';
+import { TestPage } from '@grafana/toolkit/src/e2e';
 
 export interface DataSourcesPage {}
 

public/e2e-test/pages/datasources/editDataSourcePage.ts

@@ -5,7 +5,7 @@ import {
   ClickablePageObject,
   PageObject,
   Selector,
-} from '@grafana/toolkit';
+} from '@grafana/toolkit/src/e2e';
 
 export interface EditDataSourcePage {
   saveAndTest: ClickablePageObjectType;

public/e2e-test/pages/panels/editPanel.ts

@@ -5,7 +5,7 @@ import {
   Selector,
   ClickablePageObjectType,
   ClickablePageObject,
-} from '@grafana/toolkit';
+} from '@grafana/toolkit/src/e2e';
 
 export interface EditPanelPage {
   queriesTab: ClickablePageObjectType;

public/e2e-test/pages/panels/panel.ts

-import { TestPage, ClickablePageObjectType, ClickablePageObject, Selector } from '@grafana/toolkit';
+import { TestPage, ClickablePageObjectType, ClickablePageObject, Selector } from '@grafana/toolkit/src/e2e';
 
 export interface Panel {
   panelTitle: ClickablePageObjectType;

public/e2e-test/pages/panels/sharePanelModal.ts

-import { TestPage, ClickablePageObjectType, ClickablePageObject, Selector } from '@grafana/toolkit';
+import { TestPage, ClickablePageObjectType, ClickablePageObject, Selector } from '@grafana/toolkit/src/e2e';
 
 export interface SharePanelModal {
   directLinkRenderedImage: ClickablePageObjectType;

public/e2e-test/scenarios/smoke.test.ts

 import { Browser, Page, Target } from 'puppeteer-core';
 
-import { e2eScenario, constants, takeScreenShot, compareScreenShots } from '@grafana/toolkit';
+import { e2eScenario, constants, takeScreenShot, compareScreenShots } from '@grafana/toolkit/src/e2e';
 import { addDataSourcePage } from 'e2e-test/pages/datasources/addDataSourcePage';
 import { editDataSourcePage } from 'e2e-test/pages/datasources/editDataSourcePage';
 import { dataSourcesPage } from 'e2e-test/pages/datasources/dataSources';