fix: eslint issues (#198)

apache-superset · Jul 27, 2019 · 12aeb4a · 12aeb4a
1 parent ca256cd
commit 12aeb4a
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 2 deletions.
diff --git a/packages/superset-ui-chart-composition/src/tooltip/TooltipTable.tsx b/packages/superset-ui-chart-composition/src/tooltip/TooltipTable.tsx
@@ -29,7 +29,7 @@ export default class TooltipTable extends PureComponent<Props, {}> {
     return (
       <table className={className}>
         <tbody>
-          {data.map(({ key, keyColumn, keyStyle, valueColumn, valueStyle }, i) => (
+          {data.map(({ key, keyColumn, keyStyle, valueColumn, valueStyle }) => (
             <tr key={key}>
               <td style={keyStyle}>{keyColumn || key}</td>
               <td style={valueStyle ? { ...VALUE_CELL_STYLE, ...valueStyle } : VALUE_CELL_STYLE}>

diff --git a/packages/superset-ui-chart/src/types/Column.ts b/packages/superset-ui-chart/src/types/Column.ts
@@ -1,4 +1,5 @@
 /* eslint-disable no-unused-vars */
+/* eslint-disable import/prefer-default-export */
 
 export enum ColumnType {
   DOUBLE = 'DOUBLE',

diff --git a/packages/superset-ui-chart/src/types/Datasource.ts b/packages/superset-ui-chart/src/types/Datasource.ts
@@ -1,3 +1,5 @@
+/* eslint-disable import/prefer-default-export */
+
 import { Column } from './Column';
 import { QueryObjectMetric } from './Query';
 

diff --git a/packages/superset-ui-chart/src/types/Metric.ts b/packages/superset-ui-chart/src/types/Metric.ts
@@ -1,4 +1,5 @@
 /* eslint-disable no-unused-vars */
+/* eslint-disable import/prefer-default-export */
 
 import { Column } from './Column';
 

diff --git a/packages/superset-ui-chart/test/clients/ChartClient.test.ts b/packages/superset-ui-chart/test/clients/ChartClient.test.ts
@@ -170,7 +170,8 @@ describe('ChartClient', () => {
         field1: 'abc',
         field2: 'def',
       });
-      expect(chartClient.loadDatasource('1__table')).resolves.toEqual({
+
+      return expect(chartClient.loadDatasource('1__table')).resolves.toEqual({
         field1: 'abc',
         field2: 'def',
       });