From dc535a6121f3a4f0955c65ab63b5e425f09e4dd7 Mon Sep 17 00:00:00 2001
From: 0mkar <0mkar@protonmail.com>
Date: Fri, 25 May 2018 07:46:23 +0530
Subject: [PATCH] foolish number check fix, add test

---
 src/js/TreeNode.js | 2 +-
 test/TreeNode.js   | 9 +++++++++
 2 files changed, 10 insertions(+), 1 deletion(-)

diff --git a/src/js/TreeNode.js b/src/js/TreeNode.js
index 655b8432..ca60f933 100644
--- a/src/js/TreeNode.js
+++ b/src/js/TreeNode.js
@@ -195,7 +195,7 @@ class TreeNode extends React.Component {
         } = this.props;
 
         const clickable = onClick.toString() !== TreeNode.defaultProps.onClick.toString();
-        const inputId = `${treeId}-${Number.isNaN(value) ? value : value.split(' ').join('_')}`;
+        const inputId = `${treeId}-${Number.isInteger(value) ? value : value.split(' ').join('_')}`;
 
         const render = [(
             <label key={0} htmlFor={inputId}>
diff --git a/test/TreeNode.js b/test/TreeNode.js
index f4a0321b..50c9de43 100644
--- a/test/TreeNode.js
+++ b/test/TreeNode.js
@@ -36,6 +36,15 @@ describe('<TreeNode />', () => {
             assert.equal('planets-jupiter', wrapper.find('label').prop('htmlFor'));
             assert.equal('planets-jupiter', wrapper.find('label NativeCheckbox').prop('id'));
         });
+
+        it('should render a label associated with a checkbox given integer value', () => {
+            const wrapper = shallow(
+                <TreeNode {...baseProps} treeId="planets" value={0} />,
+            );
+
+            assert.equal('planets-0', wrapper.find('label').prop('htmlFor'));
+            assert.equal('planets-0', wrapper.find('label NativeCheckbox').prop('id'));
+        });
     });
 
     describe('checked', () => {