Merge pull request #22 from afhammad/fix-single-clause-component

fix rendering of single clause when using components
stoeffel · Nov 8, 2015 · 38fdfdc · 38fdfdc
2 parents c63c9e7 + 1326df4
commit 38fdfdc
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 0 deletions.
diff --git a/__tests__/component-test.js b/__tests__/component-test.js
@@ -11,6 +11,18 @@ const {
 } = require('../module/');
 describe('React-Cond components', () => {
 
+  it('should render the child component for a single condition if it matches', () => {
+	  	const isTrue = (i) => i;
+
+		let component = TestUtils.renderIntoDocument(
+			<Cond value={true}>
+				<Clause test={isTrue}><h1>Works!</h1></Clause>
+			</Cond>
+		);
+		let val = TestUtils.findRenderedDOMComponentWithTag(component, 'h1');
+		equal(val.getDOMNode().textContent, 'Works!');
+	});
+
   it('should render the child component if the condition value is equal to `props.value`', () => {
 		let component = TestUtils.renderIntoDocument(
 			<Cond value={1}>

diff --git a/module/index.js b/module/index.js
@@ -34,7 +34,11 @@ export const Cond = React.createClass({
 		if (Array.isArray(children[0])) {
 			clauses = children;
 		} else if (children[0] && children[0].props && children[0].props[CASE_SYMBOL]) {
+      // multiple clauses
 			clauses = children.map(c => [c.props.test, c.props.children]);
+		} else if (children && children.props && children.props[CASE_SYMBOL]) {
+      // single clause
+			clauses = [[children.props.test, children.props.children]];
 		} else {
 			clauses = [children];
 		}