Create index pattern / create_index_pattern/lib => typescript (elasti…

…c#56811) (elastic#56880) * Create index pattern / create_index_pattern/lib => typescript
Dosant · Feb 5, 2020 · 0855f12 · 0855f12
1 parent 736ed90
commit 0855f12
Show file tree

Hide file tree

Showing 21 changed files with 283 additions and 229 deletions.
diff --git a/..._patterns/create_index_pattern_wizard/components/step_index_pattern/step_index_pattern.js b/..._patterns/create_index_pattern_wizard/components/step_index_pattern/step_index_pattern.js
@@ -279,13 +279,12 @@ export class StepIndexPattern extends Component {
 
   render() {
     const { isIncludingSystemIndices, allIndices } = this.props;
-    const { query, partialMatchedIndices, exactMatchedIndices } = this.state;
+    const { partialMatchedIndices, exactMatchedIndices } = this.state;
 
     const matchedIndices = getMatchedIndices(
       allIndices,
       partialMatchedIndices,
       exactMatchedIndices,
-      query,
       isIncludingSystemIndices
     );
 

diff --git a/...e_index_pattern_wizard/constants/index.js → ...e_index_pattern_wizard/constants/index.ts b/...e_index_pattern_wizard/constants/index.js → ...e_index_pattern_wizard/constants/index.ts
diff --git a/...ctions/index_patterns/create_index_pattern_wizard/lib/__jest__/api/get_indices.error.json b/...ctions/index_patterns/create_index_pattern_wizard/lib/__jest__/api/get_indices.error.json
diff --git a/...ns/index_patterns/create_index_pattern_wizard/lib/__jest__/api/get_indices.exception.json b/...ns/index_patterns/create_index_pattern_wizard/lib/__jest__/api/get_indices.exception.json
diff --git a/...ions/index_patterns/create_index_pattern_wizard/lib/__jest__/api/get_indices.success.json b/...ions/index_patterns/create_index_pattern_wizard/lib/__jest__/api/get_indices.success.json
diff --git a/...ment/sections/index_patterns/create_index_pattern_wizard/lib/__jest__/get_indices.test.js b/...ment/sections/index_patterns/create_index_pattern_wizard/lib/__jest__/get_indices.test.js
diff --git a/.../lib/__jest__/can_append_wildcard.test.js → ...rn_wizard/lib/can_append_wildcard.test.ts b/.../lib/__jest__/can_append_wildcard.test.js → ...rn_wizard/lib/can_append_wildcard.test.ts
@@ -17,13 +17,9 @@
  * under the License.
  */
 
-import { canAppendWildcard } from '../can_append_wildcard';
+import { canAppendWildcard } from './can_append_wildcard';
 
 describe('canAppendWildcard', () => {
-  test('ignores no data', () => {
-    expect(canAppendWildcard({})).toBeFalsy();
-  });
-
   test('ignores symbols', () => {
     expect(canAppendWildcard('%')).toBeFalsy();
   });

diff --git a/...pattern_wizard/lib/can_append_wildcard.js → ...pattern_wizard/lib/can_append_wildcard.ts b/...pattern_wizard/lib/can_append_wildcard.js → ...pattern_wizard/lib/can_append_wildcard.ts
@@ -17,7 +17,7 @@
  * under the License.
  */
 
-export const canAppendWildcard = keyPressed => {
+export const canAppendWildcard = (keyPressed: string) => {
   // If it's not a letter, number or is something longer, reject it
   if (!keyPressed || !/[a-z0-9]/i.test(keyPressed) || keyPressed.length !== 1) {
     return false;

diff --git a/...wizard/lib/contains_illegal_characters.js → ...wizard/lib/contains_illegal_characters.ts b/...wizard/lib/contains_illegal_characters.js → ...wizard/lib/contains_illegal_characters.ts
@@ -17,6 +17,6 @@
  * under the License.
  */
 
-export function containsIllegalCharacters(pattern, illegalCharacters) {
+export function containsIllegalCharacters(pattern: string, illegalCharacters: string[]) {
   return illegalCharacters.some(char => pattern.includes(char));
 }
diff --git a/...est__/contains_invalid_characters.test.js → ...d/lib/contains_invalid_characters.test.ts b/...est__/contains_invalid_characters.test.js → ...d/lib/contains_invalid_characters.test.ts
@@ -17,7 +17,7 @@
  * under the License.
  */
 
-import { containsIllegalCharacters } from '../contains_illegal_characters';
+import { containsIllegalCharacters } from './contains_illegal_characters';
 
 describe('containsIllegalCharacters', () => {
   it('returns true with illegal characters', () => {

diff --git a/.../lib/__jest__/ensure_minimum_time.test.js → ...rn_wizard/lib/ensure_minimum_time.test.ts b/.../lib/__jest__/ensure_minimum_time.test.js → ...rn_wizard/lib/ensure_minimum_time.test.ts
@@ -17,7 +17,7 @@
  * under the License.
  */
 
-import { ensureMinimumTime } from '../ensure_minimum_time';
+import { ensureMinimumTime } from './ensure_minimum_time';
 
 describe('ensureMinimumTime', () => {
   it('resolves single promise', async done => {

diff --git a/...pattern_wizard/lib/ensure_minimum_time.js → ...pattern_wizard/lib/ensure_minimum_time.ts b/...pattern_wizard/lib/ensure_minimum_time.js → ...pattern_wizard/lib/ensure_minimum_time.ts
@@ -27,7 +27,7 @@
 export const DEFAULT_MINIMUM_TIME_MS = 300;
 
 export async function ensureMinimumTime(
-  promiseOrPromises,
+  promiseOrPromises: Promise<any> | Array<Promise<any>>,
   minimumTimeMs = DEFAULT_MINIMUM_TIME_MS
 ) {
   let returnValue;

diff --git a/.../lib/__jest__/extract_time_fields.test.js → ...rn_wizard/lib/extract_time_fields.test.ts b/.../lib/__jest__/extract_time_fields.test.js → ...rn_wizard/lib/extract_time_fields.test.ts
@@ -17,11 +17,14 @@
  * under the License.
  */
 
-import { extractTimeFields } from '../extract_time_fields';
+import { extractTimeFields } from './extract_time_fields';
 
 describe('extractTimeFields', () => {
   it('should handle no date fields', () => {
-    const fields = [{ type: 'text' }, { type: 'text' }];
+    const fields = [
+      { type: 'text', name: 'name' },
+      { type: 'text', name: 'name' },
+    ];
 
     expect(extractTimeFields(fields)).toEqual([
       { display: `The indices which match this index pattern don't contain any time fields.` },

diff --git a/...pattern_wizard/lib/extract_time_fields.js → ...pattern_wizard/lib/extract_time_fields.ts b/...pattern_wizard/lib/extract_time_fields.js → ...pattern_wizard/lib/extract_time_fields.ts
@@ -18,8 +18,9 @@
  */
 
 import { i18n } from '@kbn/i18n';
+import { IFieldType } from '../../../../../../../../../plugins/data/public';
 
-export function extractTimeFields(fields) {
+export function extractTimeFields(fields: IFieldType[]) {
   const dateFields = fields.filter(field => field.type === 'date');
   const label = i18n.translate('kbn.management.createIndexPattern.stepTime.noTimeFieldsLabel', {
     defaultMessage: "The indices which match this index pattern don't contain any time fields.",