[Plugin Generator] Generate tsconfig and useDefaultBehaviors (#72040)

* improve test stability * add a tsconfig file and useDefaultBehaviors Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
elastic · Jul 20, 2020 · 8cc5196 · 8cc5196
1 parent 2a1ce2b
commit 8cc5196
Show file tree

Hide file tree

Showing 4 changed files with 26 additions and 1 deletion.
diff --git a/packages/kbn-plugin-generator/sao_template/sao.js b/packages/kbn-plugin-generator/sao_template/sao.js
@@ -120,6 +120,11 @@ module.exports = function ({ name, targetPath }) {
           return !customPath;
         },
       },
+      generateTsconfig: {
+        type: 'confirm',
+        message: 'Would you like to use a custom tsconfig file?',
+        default: true,
+      },
     },
     filters: {
       'public/**/index.scss': 'generateScss',
@@ -128,6 +133,7 @@ module.exports = function ({ name, targetPath }) {
       'translations/**/*': 'generateTranslations',
       'i18nrc.json': 'generateTranslations',
       'eslintrc.js': 'generateEslint',
+      'tsconfig.json': 'generateTsconfig',
     },
     move: {
       'eslintrc.js': '.eslintrc.js',

diff --git a/packages/kbn-plugin-generator/sao_template/sao.test.js b/packages/kbn-plugin-generator/sao_template/sao.test.js
@@ -80,12 +80,14 @@ describe('plugin generator sao integration', () => {
       generateApi: true,
       generateScss: false,
       generateEslint: false,
+      generateTsconfig: false,
     });
 
     // check output files
     expect(res.fileList).toContain('public/plugin.ts');
     expect(res.fileList).not.toContain('public/index.scss');
     expect(res.fileList).not.toContain('.eslintrc.js');
+    expect(res.fileList).not.toContain('tsconfig.json');
   });
 
   it('plugin package has correct title', async () => {
@@ -136,6 +138,7 @@ describe('plugin generator sao integration', () => {
 
   it('includes dotfiles', async () => {
     const res = await sao.mockPrompt(template);
+    expect(res.files['tsconfig.json']).toBeTruthy();
     expect(res.files['.eslintrc.js']).toBeTruthy();
     expect(res.files['.i18nrc.json']).toBeTruthy();
   });

diff --git a/packages/kbn-plugin-generator/sao_template/template/public/components/app.tsx b/packages/kbn-plugin-generator/sao_template/template/public/components/app.tsx
@@ -73,7 +73,7 @@ export const <%= upperCamelCaseName %>App = ({ basename, notifications, http, na
     <Router basename={basename}>
       <I18nProvider>
         <>
-          <navigation.ui.TopNavMenu appName={ PLUGIN_ID } showSearchBar={true} />
+          <navigation.ui.TopNavMenu appName={ PLUGIN_ID } showSearchBar={true} useDefaultBehaviors={true}/>
           <EuiPage restrictWidth="1000px">
             <EuiPageBody>
               <EuiPageHeader>

diff --git a/packages/kbn-plugin-generator/sao_template/template/tsconfig.json b/packages/kbn-plugin-generator/sao_template/template/tsconfig.json
@@ -0,0 +1,16 @@
+{
+  "extends": "../../tsconfig.json",
+  "compilerOptions": {
+    "outDir": "./target",
+    "skipLibCheck": true
+  },
+  "include": [
+    "index.ts",
+    "common/**/*.ts",
+    "public/**/*.ts",
+    "public/**/*.tsx",
+    "server/**/*.ts",
+    "../../typings/**/*",
+  ],
+  "exclude": []
+}