More Dashboard UX unit tests (#1603)

.gitignore

@@ -1,4 +1,6 @@
 *.pyc
+yarn-error.log
+_modules
 superset/assets/coverage/*
 changelog.sh
 .DS_Store

superset/assets/javascripts/dashboard/Dashboard.jsx

@@ -74,7 +74,7 @@ function initDashboardView(dashboard) {
   $('[data-toggle="tooltip"]').tooltip({ container: 'body' });
 }
 
-function dashboardContainer(dashboard) {
+export function dashboardContainer(dashboard) {
   return Object.assign({}, dashboard, {
     type: 'dashboard',
     filters: {},

superset/assets/javascripts/dashboard/components/Controls.jsx

@@ -10,14 +10,9 @@ import CodeModal from './CodeModal';
 import SliceAdder from './SliceAdder';
 
 const propTypes = {
-  table: React.PropTypes.object,
   dashboard: React.PropTypes.object.isRequired,
 };
 
-const defaultProps = {
-  actions: {},
-};
-
 class Controls extends React.PureComponent {
   constructor(props) {
     super(props);
@@ -138,6 +133,5 @@ class Controls extends React.PureComponent {
   }
 }
 Controls.propTypes = propTypes;
-Controls.defaultProps = defaultProps;
 
 export default Controls;

superset/assets/spec/javascripts/dashboard/CodeModal_spec.jsx

@@ -0,0 +1,21 @@
+import React from 'react';
+import CodeModal from '../../../javascripts/dashboard/components/CodeModal';
+import { mount } from 'enzyme';
+import { describe, it } from 'mocha';
+import { expect } from 'chai';
+
+
+describe('CodeModal', () => {
+  const mockedProps = {
+    triggerNode: <i className="fa fa-edit" />,
+  };
+  it('is valid', () => {
+    expect(
+      React.isValidElement(<CodeModal {...mockedProps} />)
+    ).to.equal(true);
+  });
+  it('renders the trigger node', () => {
+    const wrapper = mount(<CodeModal {...mockedProps} />);
+    expect(wrapper.find('.fa-edit')).to.have.length(1);
+  });
+});

superset/assets/spec/javascripts/dashboard/CssEditor_spec.jsx

@@ -0,0 +1,21 @@
+import React from 'react';
+import CssEditor from '../../../javascripts/dashboard/components/CssEditor';
+import { mount } from 'enzyme';
+import { describe, it } from 'mocha';
+import { expect } from 'chai';
+
+
+describe('CssEditor', () => {
+  const mockedProps = {
+    triggerNode: <i className="fa fa-edit" />,
+  };
+  it('is valid', () => {
+    expect(
+      React.isValidElement(<CssEditor {...mockedProps} />)
+    ).to.equal(true);
+  });
+  it('renders the trigger node', () => {
+    const wrapper = mount(<CssEditor {...mockedProps} />);
+    expect(wrapper.find('.fa-edit')).to.have.length(1);
+  });
+});

superset/assets/spec/javascripts/dashboard/RefreshIntervalModal_spec.jsx

@@ -0,0 +1,21 @@
+import React from 'react';
+import RefreshIntervalModal from '../../../javascripts/dashboard/components/RefreshIntervalModal';
+import { mount } from 'enzyme';
+import { describe, it } from 'mocha';
+import { expect } from 'chai';
+
+
+describe('RefreshIntervalModal', () => {
+  const mockedProps = {
+    triggerNode: <i className="fa fa-edit" />,
+  };
+  it('is valid', () => {
+    expect(
+      React.isValidElement(<RefreshIntervalModal {...mockedProps} />)
+    ).to.equal(true);
+  });
+  it('renders the trigger node', () => {
+    const wrapper = mount(<RefreshIntervalModal {...mockedProps} />);
+    expect(wrapper.find('.fa-edit')).to.have.length(1);
+  });
+});