fixing import and test definitions

Author: timductive

x-pack/legacy/plugins/canvas/common/lib/dataurl.test.ts

@@ -22,13 +22,13 @@ const RAW_PIXEL =
 
 describe('dataurl', () => {
   describe('isValidDataUrl', () => {
-    it('invalid data url', () => {
+    it('returns false for an invalid data url', () => {
       expect(isValidDataUrl('somestring')).toBe(false);
     });
-    it('empty string returns false', () => {
+    it('returns false for an empty string', () => {
       expect(isValidDataUrl('')).toBe(false);
     });
-    it('valid data urls', () => {
+    it('returns true for valid data urls', () => {
       expect(isValidDataUrl(BASE64_TEXT)).toBe(true);
       expect(isValidDataUrl(BASE64_SVG)).toBe(true);
       expect(isValidDataUrl(BASE64_PIXEL)).toBe(true);
@@ -39,13 +39,13 @@ describe('dataurl', () => {
   });
 
   describe('dataurl.parseDataUrl', () => {
-    it('invalid data url', () => {
+    it('returns null for an invalid data url', () => {
       expect(parseDataUrl('somestring')).toBeNull();
     });
-    it('invalid base64 image returns null', () => {
+    it('returns null for an invalid base64 image', () => {
       expect(parseDataUrl(INVALID_BASE64_PIXEL)).toBeNull();
     });
-    it('text data urls', () => {
+    it('returns correct values for text data urls', () => {
       expect(parseDataUrl(BASE64_TEXT)).toEqual({
         charset: 'utf-8',
         data: null,
@@ -63,7 +63,7 @@ describe('dataurl', () => {
         mimetype: 'text/plain',
       });
     });
-    it('png data urls', () => {
+    it('returns correct values for png data urls', () => {
       expect(parseDataUrl(RAW_PIXEL)).toBeNull();
       expect(parseDataUrl(BASE64_PIXEL)).toEqual({
         charset: undefined,
@@ -74,7 +74,7 @@ describe('dataurl', () => {
         mimetype: 'image/png',
       });
     });
-    it('svg data urls', () => {
+    it('returns correct values for svg data urls', () => {
       expect(parseDataUrl(RAW_SVG)).toEqual({
         charset: undefined,
         data: null,

x-pack/legacy/plugins/canvas/common/lib/get_colors_from_palette.test.js

@@ -4,11 +4,11 @@
  * you may not use this file except in compliance with the Elastic License.
  */
 
-import { getColorsFromPalette } from './get_colors_from_palette';
 import {
   grayscalePalette,
   gradientPalette,
-} from './../../canvas_plugin_src/functions/common/__tests__/fixtures/test_styles';
+} from '../../canvas_plugin_src/functions/common/__tests__/fixtures/test_styles';
+import { getColorsFromPalette } from './get_colors_from_palette';
 
 describe('getColorsFromPalette', () => {
   it('returns the array of colors from a palette object when gradient is false', () => {

x-pack/legacy/plugins/canvas/common/lib/get_field_type.test.ts

@@ -4,11 +4,11 @@
  * you may not use this file except in compliance with the Elastic License.
  */
 
-import { getFieldType } from './get_field_type';
 import {
   emptyTable,
   testTable,
-} from './../../canvas_plugin_src/functions/common/__tests__/fixtures/test_tables';
+} from '../../canvas_plugin_src/functions/common/__tests__/fixtures/test_tables';
+import { getFieldType } from './get_field_type';
 
 describe('getFieldType', () => {
   it('returns type of a field in a datatable', () => {

x-pack/legacy/plugins/canvas/common/lib/handlebars.test.js

@@ -4,8 +4,8 @@
  * you may not use this file except in compliance with the Elastic License.
  */
 
+import { testTable } from '../../canvas_plugin_src/functions/common/__tests__/fixtures/test_tables';
 import { Handlebars } from './handlebars';
-import { testTable } from './../../canvas_plugin_src/functions/common/__tests__/fixtures/test_tables';
 
 describe('handlebars', () => {
   it('registers math function and returns argument error', () => {

x-pack/legacy/plugins/canvas/common/lib/hex_to_rgb.test.ts

@@ -7,18 +7,18 @@
 import { hexToRgb } from './hex_to_rgb';
 
 describe('hexToRgb', () => {
-  it('invalid hex', () => {
+  it('returns null for an invalid hex', () => {
     expect(hexToRgb('hexadecimal')).toBeNull();
     expect(hexToRgb('#00')).toBeNull();
     expect(hexToRgb('#00000')).toBeNull();
   });
-  it('shorthand', () => {
+  it('returns correct value for shorthand hex codes', () => {
     expect(hexToRgb('#000')).toEqual([0, 0, 0]);
     expect(hexToRgb('#FFF')).toEqual([255, 255, 255]);
     expect(hexToRgb('#fff')).toEqual([255, 255, 255]);
     expect(hexToRgb('#fFf')).toEqual([255, 255, 255]);
   });
-  it('longhand', () => {
+  it('returns correct value for longhand hex codes', () => {
     expect(hexToRgb('#000000')).toEqual([0, 0, 0]);
     expect(hexToRgb('#ffffff')).toEqual([255, 255, 255]);
     expect(hexToRgb('#fffFFF')).toEqual([255, 255, 255]);

x-pack/legacy/plugins/canvas/common/lib/resolve_dataurl.test.js

@@ -4,8 +4,8 @@
  * you may not use this file except in compliance with the Elastic License.
  */
 
+import { missingImage } from '../../common/lib/missing_asset';
 import { resolveFromArgs, resolveWithMissingImage } from './resolve_dataurl';
-import { missingImage } from './../../common/lib/missing_asset';
 
 describe('resolve_dataurl', () => {
   describe('resolveFromArgs', () => {

x-pack/legacy/plugins/canvas/common/lib/url.test.js

@@ -4,8 +4,8 @@
  * you may not use this file except in compliance with the Elastic License.
  */
 
+import { missingImage } from '../../common/lib/missing_asset';
 import { isValidUrl } from './url';
-import { missingImage } from './../../common/lib/missing_asset';
 
 describe('resolve_dataurl', () => {
   it('returns valid dataurl', () => {