Fix test's warnings (#2697)

* tests: adding required props to FilterableTable mockedProps

* tests: adding required prop `height` to QuerySearch mockedProps

* tests: adding required prop `height` to ResultSet mockedProps

* tests: adding required prop `height` to SqlEditorLeftBar mockedProps

* tests: fix warning in Timer component

superset/assets/spec/javascripts/components/FilterableTable/FilterableTable_spec.jsx

@@ -4,7 +4,12 @@ import { expect } from 'chai';
 import FilterableTable from '../../../../javascripts/components/FilterableTable/FilterableTable';
 
 describe('FilterableTable', () => {
+  const mockedProps = {
+    orderedColumnKeys: [],
+    data: [],
+    height: 0,
+  };
   it('is valid element', () => {
-    expect(React.isValidElement(<FilterableTable />)).to.equal(true);
+    expect(React.isValidElement(<FilterableTable {...mockedProps} />)).to.equal(true);
   });
 });

superset/assets/spec/javascripts/sqllab/QuerySearch_spec.jsx

@@ -11,6 +11,7 @@ import QuerySearch from '../../../javascripts/SqlLab/components/QuerySearch';
 describe('QuerySearch', () => {
   const mockedProps = {
     actions: {},
+    height: 0,
   };
   it('is valid', () => {
     expect(

superset/assets/spec/javascripts/sqllab/ResultSet_spec.jsx

@@ -9,14 +9,10 @@ import { queries } from './fixtures';
 describe('ResultSet', () => {
   const mockedProps = {
     query: queries[0],
+    height: 0,
   };
-  it('renders', () => {
-    expect(React.isValidElement(<ResultSet />)).to.equal(true);
-  });
-  it('renders with props', () => {
-    expect(
-      React.isValidElement(<ResultSet />),
-    ).to.equal(true);
+  it('is valid', () => {
+    expect(React.isValidElement(<ResultSet {...mockedProps} />)).to.equal(true);
   });
   it('renders a Table', () => {
     const wrapper = shallow(<ResultSet {...mockedProps} />);

superset/assets/spec/javascripts/sqllab/SqlEditorLeftBar_spec.jsx

@@ -12,6 +12,7 @@ describe('SqlEditorLeftBar', () => {
   const mockedProps = {
     tables: [table],
     queryEditor: defaultQueryEditor,
+    height: 0,
   };
   it('is valid', () => {
     expect(

superset/assets/spec/javascripts/sqllab/Timer_spec.jsx

@@ -14,10 +14,7 @@ describe('Timer', () => {
     isRunning: true,
     state: 'warning',
   };
-  it('renders', () => {
-    expect(React.isValidElement(<Timer />)).to.equal(true);
-  });
-  it('renders with props', () => {
+  it('is valid', () => {
     expect(React.isValidElement(<Timer {...mockedProps} />))
     .to.equal(true);
   });