Make hydrate more robust and add first JS test

Author: alandipert

inst/www/react-tools/react-tools.js

@@ -14,10 +14,10 @@ window.reactR = (function () {
         var elem = components.hasOwnProperty(tag.name) ? components[tag.name] : tag.name,
             args = [elem, tag.attribs];
         for (var i = 0; i < tag.children.length; i++) {
-            if (typeof tag.children[i] === 'string') {
-                args.push(tag.children[i]);
-            } else {
+            if (typeof tag.children[i] === 'object') {
                 args.push(hydrate(components, tag.children[i]));
+            } else {
+                args.push(tag.children[i]);
             }
         }
         return React.createElement.apply(null, args);

js-tests/js-tests.jsx

@@ -0,0 +1,24 @@
+import React from 'react';
+
+describe('window.reactR', () => {
+    describe('#hydrate()', () => {
+        it('should throw an exception with an unknown component', () => {
+            assert.throws(() => {
+                reactR.hydrate(
+                    {
+                        Foo: class Foo extends React.Component {
+                            render() {
+                                return <h1>Foo!</h1>;
+                            }
+                        }
+                    },
+                    {
+                        name: "Bar",
+                        attribs: {},
+                        children: {}
+                    }
+                )
+            }, Error, /Unknown component/);
+        });
+    });
+});

karma.conf.js

@@ -0,0 +1,36 @@
+// TODO: Get source maps for failures working.
+module.exports = function (config) {
+    config.set({
+        frameworks: ['mocha', 'chai'],
+        files: ['inst/www/react-tools/react-tools.js', 'js-tests/js-tests.jsx'],
+        preprocessors: {
+            'js-tests/*.js': ['webpack'],
+            'js-tests/*.jsx': ['webpack', 'sourcemap']
+        },
+        webpack: {
+            module: {
+                rules: [
+                    {
+                        test: [/\.js$/, /\.jsx$/],
+                        loader: 'babel-loader',
+                        options: {
+                            presets: ['@babel/preset-env', '@babel/preset-react']
+                        }
+                    }
+                ]
+            },
+            devtool: 'source-map'
+        },
+        webpackMiddleware: {
+            stats: 'errors-only'
+        },
+        reporters: ['progress'],
+        port: 9876,  // karma web server port
+        colors: true,
+        logLevel: config.LOG_INFO,
+        browsers: ['ChromeHeadless'],
+        autoWatch: false,
+        // singleRun: false, // Karma captures browsers, runs the tests and exits
+        concurrency: Infinity
+    })
+} 

package.json

@@ -0,0 +1,22 @@
+{
+    "devDependencies": {
+        "chai": "^4.2.0",
+        "karma": "^3.1.4",
+        "karma-chai": "^0.1.0",
+        "karma-chrome-launcher": "^2.2.0",
+        "karma-mocha": "^1.3.0",
+        "karma-webpack": "^3.0.5",
+        "karma-sourcemap-loader": "^0.3.7",
+        "mocha": "^5.2.0",
+        "react": "^16.7.0",
+        "react-dom": "^16.7.0",
+        "webpack": "^3.0.0",
+        "@babel/core": "^7.2.0",
+        "babel-loader": "^8.0.4",
+        "@babel/preset-env": "^7.2.0",
+        "@babel/preset-react": "^7.0.0"
+    },
+    "scripts": {
+        "test": "karma start --single-run --browsers ChromeHeadless karma.conf.js"
+    }
+}