From 8e5b523315c93d076451978da5f36d48b584c5b2 Mon Sep 17 00:00:00 2001
From: Levi Thomason <me@levithomason.com>
Date: Mon, 23 May 2016 17:38:34 -0700
Subject: [PATCH 01/13] chore(coverage): setup coverage

---
 .babelrc                  | 30 +++++++++++++++++++++++++++++-
 .gitignore                |  1 +
 build/karma.conf.babel.js | 14 +++++++++++---
 build/webpack.config.js   | 18 ------------------
 package.json              |  2 ++
 5 files changed, 43 insertions(+), 22 deletions(-)

diff --git a/.babelrc b/.babelrc
index 80e4f50e51..24522a9d57 100644
--- a/.babelrc
+++ b/.babelrc
@@ -6,5 +6,33 @@
   ],
   "plugins": [
     "lodash"
-  ]
+  ],
+  "env": {
+    "development": {
+      "plugins": [
+        ["react-transform", {
+          "transforms": [{
+              "transform": "react-transform-hmr",
+              "imports": [
+                "react"
+              ],
+              "locals": [
+                "module"
+              ]
+            }, {
+              "transform": "react-transform-catch-errors",
+              "imports": [
+                "react",
+                "redbox-react"
+              ]
+            }]
+        }]
+      ]
+    },
+    "test": {
+      "plugins": [
+        ["__coverage__", { "ignore": "test/" }]
+      ]
+    }
+  }
 }
diff --git a/.gitignore b/.gitignore
index a7331188dd..8a16ce4047 100644
--- a/.gitignore
+++ b/.gitignore
@@ -6,6 +6,7 @@
 npm-debug.log*
 node_modules/
 bower_components/
+coverage/
 dist/
 docs/build
 docs/app/docgenInfo.json
diff --git a/build/karma.conf.babel.js b/build/karma.conf.babel.js
index 71faed10f6..21e0540182 100644
--- a/build/karma.conf.babel.js
+++ b/build/karma.conf.babel.js
@@ -1,5 +1,4 @@
 import { argv } from 'yargs'
-import webpack from 'webpack'
 
 import config from '../config'
 import webpackConfig from './webpack.config'
@@ -11,7 +10,7 @@ module.exports = (karmaConfig) => {
     basePath: process.cwd(),
     browsers: ['PhantomJS'],
     singleRun: !argv.watch,
-    reporters: ['mocha'],
+    reporters: ['mocha', 'coverage'],
     files: [
       './test/tests.bundle.js',
     ],
@@ -23,7 +22,16 @@ module.exports = (karmaConfig) => {
       // exit on ResourceError, useful if karma exits without killing phantom
       exitOnResourceError: true,
     },
+    coverageReporter: {
+      reporters: [
+        { type: 'lcov', dir: 'coverage', subdir: '.' },
+        { type: 'text-summary' },
+      ],
+      includeAllSources: true,
+    },
     preprocessors: {
+      // do not include 'coverage' preprocessor for karma-coverage
+      // code is already instrumented by babel-plugin-__coverage__
       '**/*.bundle.js': ['webpack'],
     },
     client: {
@@ -45,7 +53,7 @@ module.exports = (karmaConfig) => {
         ],
       },
       plugins: [
-        new webpack.DefinePlugin(config.compiler_globals),
+        ...webpackConfig.plugins,
       ],
       resolve: {
         ...webpackConfig.resolve,
diff --git a/build/webpack.config.js b/build/webpack.config.js
index 22b10cd48e..6afc3d4afe 100644
--- a/build/webpack.config.js
+++ b/build/webpack.config.js
@@ -133,24 +133,6 @@ webpackConfig.module.loaders = [{
   loader: 'babel',
   query: {
     cacheDirectory: true,
-    plugins: [],
-    presets: ['es2015', 'react', 'stage-1'],
-    env: {
-      development: {
-        plugins: [
-          ['react-transform', {
-            transforms: [{
-              transform: 'react-transform-hmr',
-              imports: ['react'],
-              locals: ['module'],
-            }, {
-              transform: 'react-transform-catch-errors',
-              imports: ['react', 'redbox-react'],
-            }],
-          }],
-        ],
-      },
-    },
   },
 }, {
   //
diff --git a/package.json b/package.json
index 541a7dfe39..1a2ff22eb5 100644
--- a/package.json
+++ b/package.json
@@ -51,6 +51,7 @@
     "babel-core": "^6.5.2",
     "babel-eslint": "^6.0.4",
     "babel-loader": "^6.2.0",
+    "babel-plugin-__coverage__": "^0.111111.11",
     "babel-plugin-lodash": "^3.1.4",
     "babel-plugin-react-transform": "^2.0.2",
     "babel-preset-es2015": "^6.5.0",
@@ -85,6 +86,7 @@
     "json-loader": "^0.5.3",
     "karma": "^0.13.22",
     "karma-cli": "^1.0.0",
+    "karma-coverage": "^1.0.0",
     "karma-mocha": "^1.0.1",
     "karma-mocha-reporter": "^2.0.0",
     "karma-phantomjs-launcher": "^1.0.0",

From d4ec4bdf25ecdeb06e9b38f3b0fcff9b08716b9d Mon Sep 17 00:00:00 2001
From: Levi Thomason <me@levithomason.com>
Date: Mon, 23 May 2016 17:39:30 -0700
Subject: [PATCH 02/13] chore(ci): report coverage

---
 circle.yml | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/circle.yml b/circle.yml
index db4cb8969a..21a42c9dee 100644
--- a/circle.yml
+++ b/circle.yml
@@ -15,6 +15,8 @@ dependencies:
 test:
   pre:
     - npm run lint
+  post:
+    - bash <(curl -s https://codecov.io/bash)
 
 deployment:
   development:

From f2dd1ef6fcd7b47a14f2a7dbdf504e922f44707b Mon Sep 17 00:00:00 2001
From: Levi Thomason <me@levithomason.com>
Date: Mon, 23 May 2016 17:39:16 -0700
Subject: [PATCH 03/13] docs(readme): add coverage badge

---
 README.md | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/README.md b/README.md
index 4dea620375..7f8f929910 100644
--- a/README.md
+++ b/README.md
@@ -1,4 +1,8 @@
-# Stardust  [![Code Climate](https://img.shields.io/codeclimate/github/TechnologyAdvice/stardust.svg?style=flat-square)](https://codeclimate.com/github/TechnologyAdvice/stardust) [![Gemnasium](https://img.shields.io/gemnasium/TechnologyAdvice/stardust.svg?style=flat-square)](https://gemnasium.com/TechnologyAdvice/stardust) [![Circle CI](https://img.shields.io/circleci/project/TechnologyAdvice/stardust/master.svg?style=flat-square)](https://circleci.com/gh/TechnologyAdvice/stardust/tree/master)
+# Stardust
+[![Circle CI](https://img.shields.io/circleci/project/TechnologyAdvice/stardust/master.svg?style=flat-square)](https://circleci.com/gh/TechnologyAdvice/stardust/tree/master)
+[![Codecov](https://img.shields.io/codecov/c/github/TechnologyAdvice/stardust/master.svg?style=flat-square)](https://codecov.io/gh/TechnologyAdvice/stardust)
+[![Code Climate](https://img.shields.io/codeclimate/github/TechnologyAdvice/stardust.svg?style=flat-square)](https://codeclimate.com/github/TechnologyAdvice/stardust)
+[![Gemnasium](https://img.shields.io/gemnasium/TechnologyAdvice/stardust.svg?style=flat-square)](https://gemnasium.com/TechnologyAdvice/stardust)
 
 The (soon to be) [official][8] integration between [Semantic UI][5] and [React][3].
 

From ba8fbf407d0001e25c491eea4f7f0b711d721829 Mon Sep 17 00:00:00 2001
From: Levi Thomason <me@levithomason.com>
Date: Mon, 23 May 2016 17:43:51 -0700
Subject: [PATCH 04/13] test(common): enforce named function components

---
 test/specs/commonTests.js | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/test/specs/commonTests.js b/test/specs/commonTests.js
index e1cb2c24a4..17817df9c2 100644
--- a/test/specs/commonTests.js
+++ b/test/specs/commonTests.js
@@ -47,6 +47,16 @@ const componentInfo = componentCtx.keys().map(key => {
  * @param {Object} [requiredProps={}] Props required to render Component without errors or warnings.
  */
 export const isConformant = (Component, requiredProps = {}) => {
+  // tests depend on Component constructor names, enforce them
+  if (!Component.prototype.constructor.name) {
+    const ReactDOMServer = require('react-dom/server')
+    throw new Error([
+      'Component is not a named function. This should help identify it:',
+      `static _meta = ${JSON.stringify(Component._meta, null, 2)}`,
+      `Rendered:\n${ReactDOMServer.renderToStaticMarkup(<Component />)}`,
+    ].join('\n'))
+  }
+
   const info = _.find(componentInfo, i => i.constructorName === Component.prototype.constructor.name)
   const { _meta, constructorName, componentClassName, filenameWithoutExt, sdClass } = info
 

From ae97dd85f771bb38f763b490d060a34959ca0293 Mon Sep 17 00:00:00 2001
From: Levi Thomason <me@levithomason.com>
Date: Mon, 23 May 2016 17:41:40 -0700
Subject: [PATCH 05/13] refactor(Select): use named function

---
 src/addons/Select/Select.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/addons/Select/Select.js b/src/addons/Select/Select.js
index 1cb8654313..0fbdb2e702 100644
--- a/src/addons/Select/Select.js
+++ b/src/addons/Select/Select.js
@@ -9,7 +9,7 @@ import Dropdown from '../../modules/Dropdown/Dropdown'
  * A <Select /> is sugar for <Dropdown selection />.
  * @see Dropdown
  */
-const Select = ({ className, ...rest }) => {
+function Select({ className, ...rest }) {
   const classes = cx('sd-select', className)
 
   return <Dropdown {...rest} className={classes} selection />

From 0f26719282e5f3ebefb4f381df1dbd0ff392bcea Mon Sep 17 00:00:00 2001
From: Levi Thomason <me@levithomason.com>
Date: Mon, 23 May 2016 17:41:56 -0700
Subject: [PATCH 06/13] refactor(Label): use named function

---
 src/elements/Label/Label.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/elements/Label/Label.js b/src/elements/Label/Label.js
index d029d49bc6..deb5b6e912 100644
--- a/src/elements/Label/Label.js
+++ b/src/elements/Label/Label.js
@@ -19,7 +19,7 @@ import Image from '../Image/Image'
 /**
  * A label displays content classification
  */
-const Label = (props) => {
+function Label(props) {
   const {
     attached, children, color, corner, className, circular, detail, detailLink, floating, horizontal, icon,
     image, link, onClick, onClickDetail, onClickRemove, pointing, removable, ribbon, size, tag, text,

From fa21e37bf3588be8e504d1eb007e854124c0f732 Mon Sep 17 00:00:00 2001
From: Levi Thomason <me@levithomason.com>
Date: Mon, 23 May 2016 17:42:11 -0700
Subject: [PATCH 07/13] refactor(MenuItem): use named function

---
 src/collections/Menu/MenuItem.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/collections/Menu/MenuItem.js b/src/collections/Menu/MenuItem.js
index 50aab87362..64d4cdfcf1 100644
--- a/src/collections/Menu/MenuItem.js
+++ b/src/collections/Menu/MenuItem.js
@@ -2,7 +2,7 @@ import React, { PropTypes } from 'react'
 import classNames from 'classnames'
 import META from '../../utils/Meta'
 
-const MenuItem = ({ __onClick, active, children, className, label, name, onClick, ...rest }) => {
+function MenuItem({ __onClick, active, children, className, label, name, onClick, ...rest }) {
   const handleClick = (e) => {
     if (__onClick) __onClick(name)
     if (onClick) onClick(name)

From f1ca2c7434477dd771b47ae8760825beee7929a5 Mon Sep 17 00:00:00 2001
From: Levi Thomason <me@levithomason.com>
Date: Mon, 23 May 2016 17:42:25 -0700
Subject: [PATCH 08/13] refactor(DropdownItem): use named function

---
 src/modules/Dropdown/DropdownItem.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/modules/Dropdown/DropdownItem.js b/src/modules/Dropdown/DropdownItem.js
index a9bdd5b81e..c67ef9daf2 100644
--- a/src/modules/Dropdown/DropdownItem.js
+++ b/src/modules/Dropdown/DropdownItem.js
@@ -6,7 +6,7 @@ import { someChildType } from '../../utils/childrenUtils'
 import { useKeyOnly } from '../../utils/propUtils'
 import META from '../../utils/Meta'
 
-const DropdownItem = (props) => {
+function DropdownItem(props) {
   const {
     active,
     children,

From dfd0bfa812795d0183f9c5ae71ef7265c42b9a06 Mon Sep 17 00:00:00 2001
From: Levi Thomason <me@levithomason.com>
Date: Mon, 23 May 2016 17:42:48 -0700
Subject: [PATCH 09/13] refactor(ItemItems): use named function

---
 src/views/Item/ItemItems.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/views/Item/ItemItems.js b/src/views/Item/ItemItems.js
index d37c664772..f82a24c6c9 100644
--- a/src/views/Item/ItemItems.js
+++ b/src/views/Item/ItemItems.js
@@ -2,7 +2,7 @@ import React, { PropTypes } from 'react'
 import cx from 'classnames'
 import META from '../../utils/Meta'
 
-const ItemItems = (props) => {
+function ItemItems(props) {
   const { className, children, ...rest } = props
   const classes = cx('sd-item-items ui', className, 'items')
 

From b79a4ec6034444d1a5c0dccf2a2d2490ead8fd5c Mon Sep 17 00:00:00 2001
From: Levi Thomason <me@levithomason.com>
Date: Mon, 23 May 2016 17:44:45 -0700
Subject: [PATCH 10/13] feat(debug): don't debug in tests

---
 src/utils/debug.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/utils/debug.js b/src/utils/debug.js
index b24817e5ca..bbc68d1b54 100644
--- a/src/utils/debug.js
+++ b/src/utils/debug.js
@@ -1,7 +1,7 @@
 let debug
 const noop = () => undefined
 
-if (process.env.NODE_ENV !== 'production') {
+if (process.env.NODE_ENV !== 'production' && process.env.NODE_ENV !== 'test') {
   debug = require('debug')
   localStorage.debug = 'stardust:*'
 } else {

From c80781f1db23e9908c0139bc14cb06d8c920db78 Mon Sep 17 00:00:00 2001
From: Levi Thomason <me@levithomason.com>
Date: Mon, 23 May 2016 17:45:13 -0700
Subject: [PATCH 11/13] test(FormFields): fix missing key

---
 test/specs/collections/Form/Fields-test.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/test/specs/collections/Form/Fields-test.js b/test/specs/collections/Form/Fields-test.js
index aeebf2a4ab..2761f80bc6 100644
--- a/test/specs/collections/Form/Fields-test.js
+++ b/test/specs/collections/Form/Fields-test.js
@@ -12,7 +12,7 @@ describe('Fields', () => {
 
   describe('evenlyDivided', () => {
     it('adds the word class for the number of child fields', () => {
-      const children = _.times(_.random(1, 16), () => <FormField />)
+      const children = _.times(_.random(1, 16), (i) => <FormField key={i} />)
       const className = `${numberToWord(children.length)} fields`
 
       shallow(<FormFields evenlyDivided>{children}</FormFields>)

From a8ce290e9e3244a2c1df52eadf074dbbbe2deccb Mon Sep 17 00:00:00 2001
From: Levi Thomason <me@levithomason.com>
Date: Mon, 23 May 2016 17:49:10 -0700
Subject: [PATCH 12/13] test(Dropdown): fix search selection test

---
 test/specs/modules/Dropdown/Dropdown-test.js | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/test/specs/modules/Dropdown/Dropdown-test.js b/test/specs/modules/Dropdown/Dropdown-test.js
index 73cebef891..06241e4c94 100644
--- a/test/specs/modules/Dropdown/Dropdown-test.js
+++ b/test/specs/modules/Dropdown/Dropdown-test.js
@@ -21,11 +21,11 @@ const wrapperMount = (node, opts) => {
   attachTo = document.createElement('div')
   document.body.appendChild(attachTo)
 
-  wrapper = global.mount(node, { ...opts, attachTo })
+  wrapper = mount(node, { ...opts, attachTo })
   return wrapper
 }
-const wrapperShallow = (...args) => (wrapper = global.shallow(...args))
-const wrapperRender = (...args) => (wrapper = global.render(...args))
+const wrapperShallow = (...args) => (wrapper = shallow(...args))
+const wrapperRender = (...args) => (wrapper = render(...args))
 
 // ----------------------------------------
 // Options
@@ -52,9 +52,6 @@ const dropdownMenuIsOpen = () => {
   menu.should.have.className('visible')
 }
 
-options = getOptions()
-defaultProps = { options }
-
 describe('Dropdown Component', () => {
   beforeEach(() => {
     attachTo = undefined
@@ -720,12 +717,15 @@ describe('Dropdown Component', () => {
     })
 
     it('still allows moving selection after blur/focus', () => {
+      // open, first item is selected
       const search = wrapperMount(<Dropdown {...defaultProps} search />)
         .find('input.search')
+        .simulate('focus')
+
+      domEvent.keyDown(document, { key: 'ArrowDown' })
+      dropdownMenuIsOpen()
 
-      // open, first item is selected
       const items = wrapper
-        .simulate('click')
         .find('DropdownItem')
 
       items

From f7dc0b6e2abdc1392f198e98a4caca15b0f63732 Mon Sep 17 00:00:00 2001
From: Levi Thomason <me@levithomason.com>
Date: Mon, 23 May 2016 17:49:46 -0700
Subject: [PATCH 13/13] test(AutoControlledComponent): silence logs

---
 test/specs/utils/AutoControlledComponent-test.js | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/test/specs/utils/AutoControlledComponent-test.js b/test/specs/utils/AutoControlledComponent-test.js
index 3d5e3541fe..b29360f746 100644
--- a/test/specs/utils/AutoControlledComponent-test.js
+++ b/test/specs/utils/AutoControlledComponent-test.js
@@ -39,6 +39,8 @@ describe('extending AutoControlledComponent', () => {
     })
 
     it('sets state for autoControlledProps', () => {
+      consoleUtil.disableOnce()
+
       const autoControlledProps = _.keys(makeProps())
       const randomProp = _.sample(autoControlledProps)
       const randomValue = faker.hacker.verb()
@@ -70,6 +72,8 @@ describe('extending AutoControlledComponent', () => {
     })
 
     it('does not set state for props defined by the parent', () => {
+      consoleUtil.disableOnce()
+
       const props = makeProps()
       const autoControlledProps = _.keys(props)
 
@@ -95,6 +99,8 @@ describe('extending AutoControlledComponent', () => {
 
   describe('initial state', () => {
     it('is derived from autoControlledProps in props', () => {
+      consoleUtil.disableOnce()
+
       const props = makeProps()
       const autoControlledProps = _.keys(props)
 
@@ -114,6 +120,8 @@ describe('extending AutoControlledComponent', () => {
 
   describe('default props', () => {
     it('are applied to state for props in autoControlledProps', () => {
+      consoleUtil.disableOnce()
+
       const props = makeProps()
       const autoControlledProps = _.keys(props)
       const defaultProps = makeDefaultProps(props)
@@ -134,6 +142,8 @@ describe('extending AutoControlledComponent', () => {
     })
 
     it('allows trySetState to work on non-default autoControlledProps', () => {
+      consoleUtil.disableOnce()
+
       const props = makeProps()
       const autoControlledProps = _.keys(props)
       const defaultProps = makeDefaultProps(props)
@@ -155,6 +165,8 @@ describe('extending AutoControlledComponent', () => {
 
   describe('changing props', () => {
     it('sets state for props in autoControlledProps', () => {
+      consoleUtil.disableOnce()
+
       const props = makeProps()
       const autoControlledProps = _.keys(props)
 
@@ -189,6 +201,8 @@ describe('extending AutoControlledComponent', () => {
     })
 
     it('does not set state for default props when changed', () => {
+      consoleUtil.disableOnce()
+
       const props = makeProps()
       const autoControlledProps = _.keys(props)
       const defaultProps = makeDefaultProps(props)