Fix test runability. Remove tests that were not adding much value. Need to rethink overall testing strategy.

Author: jasonkuhrt

.testem.yml

@@ -1,15 +1,14 @@
+
 framework: mocha
 
 src_files:
   - lib/**/*.js
   - test/**/*.coffee
 
 serve_files:
-  - build/**/*.js
+  - build/**/index.js
 
-before_tests: ./node_modules/.bin/webpack --config webpack.test.config.js
+before_tests: ./node_modules/.bin/webpack --config webpack.test.config.babel.js
 
 launch_in_dev:
   - chrome
-  - safari
-  - firefox

package.json

@@ -44,6 +44,7 @@
     "react-hot-loader": "^1.2.4",
     "react-tappable": "^0.6.0",
     "style-loader": "^0.12.3",
+    "testem": "^0.9.6",
     "webpack": "^1.12.1",
     "webpack-dev-server": "^1.10.1"
   },

test/_globals.coffee

@@ -6,30 +6,3 @@ g.testUtils = React.addons.TestUtils
 g.sim = testUtils.Simulate
 g.Popover = require '../lib'
 g.e = React.DOM
-Object.assign = require 'object.assign'
-
-
-
-g.$ = (query, el)->
-  if el
-    React.findDOMNode(el).querySelector query
-  else
-    document.querySelector query
-
-g.renderApp = (reactElementFactory, cb)->
-  React.render reactElementFactory, $('#app'), cb
-
-g.getPopoverPosition = ->
-  measure $ '.Popover'
-
-Object.assign window, require '../lib/utils'
-
-before ->
-  el = document.createElement 'div'
-  el.id = 'app'
-  document.body.appendChild el
-
-afterEach ->
-  #$ '#app'
-  #.innerHTML = ''
-

test/index.coffee

@@ -1,91 +1,9 @@
 require './_globals.coffee'
 
-Tracer = React.createFactory React.createClass
-  name: 'tracer'
-
-  getInitialState: ->
-    isOpen: false
-
-  getInitialProps: ->
-    style: this.props.style || {}
-
-  togglePopover: ->
-    this.setState
-      isOpen: !this.state.isOpen
-
-  render: ->
-    defaultStyle =
-      position: 'absolute'
-      background: 'red'
-      top: 0
-      left: 0
-      width: 50
-      height: 50
-
-    style = Object.assign defaultStyle, this.props.style
-
-    target = (
-      e.div
-        className: 'handle Target'
-        style: style
-        onClick: this.togglePopover
-    )
-
-    React.createElement(Popover, {
-      isOpen: this.props.isOpen
-    }, target)
-
-
-
-renderTracer = (props = {})->
-  new Promise (resolve)->
-    renderApp Tracer(props), ->
-      targetEl = $ '.Target'
-      sim.click targetEl, this
-      resolve [targetEl, $ '.Popover']
-  .tap -> Promise.delay 500
-
-
-
-describe 'react-popover', ->
-
-  it 'is a rendered element', ->
-    renderTracer (target, popover)->
-      a popover, 'Popover is rendered'
-
 
 
 describe 'react-popover rendering', ->
 
-  it 'should layout in the zone with the largest area', ->
-    renderTracer style: left: '90%', transform: 'translateX(-50%)'
-    .spread (l, p)->
-      pb = calcBounds p
-      lb = calcBounds l
-      if isLandscape()
-        console.log pb.y, lb.y
-        a pb.x2 < lb.x, 'Popover is left of target'
-        a pb.y is lb.y, 'Popover is equal y of target'
-      else
-        console.log pb.y, lb.y2
-        a pb.y > lb.y2, 'Popover is bottom of target'
-        a (pb.x < lb.x2 and pb.x2 > lb.x2), 'Popover is cross-axes-centered of target'
-
+  it 'should layout in the zone with the largest area'
   it 'body should autosize to dimensions of content'
   it 'body should not autosize beyond frame bounds'
-
-
-
-
-
-
-
-calcOrientation = ->
-  if window.innerWidth > window.innerHeight then 'landscape' else 'portrait'
-
-isLandscape = ->
-  calcOrientation() is 'landscape'
-
-
-
-

webpack.test.config.babel.js

@@ -12,7 +12,7 @@ export default {
   },
   output: {
     path: outputDir,
-    filename: `[name].js`,
+    filename: `index.js`,
   },
   module: {
     loaders: [
@@ -21,7 +21,4 @@ export default {
     ],
   },
   devtool: `source-map`,
-  devServer: {
-    contentBase: outputDir,
-  },
 }