Add some tests

src/Select.jsx

@@ -182,6 +182,7 @@ const Select = React.createClass({
       });
       this.fireChange(nextValue);
       this.setOpenState(false, true);
+      this.setInputValue('', false);
       return;
     }
     this.setInputValue(val);

src/util.js

@@ -105,11 +105,8 @@ export function findFirstMenuItem(children) {
 }
 
 export function includesSeparators(string, separators) {
-  if (~separators.indexOf(string)) {
-    return false;
-  }
   for (let i = 0; i < separators.length; ++i) {
-    if (~string.indexOf(separators[i])) {
+    if (string.lastIndexOf(separators[i]) > 0) {
       return true;
     }
   }
@@ -118,5 +115,12 @@ export function includesSeparators(string, separators) {
 
 export function splitBySeparators(string, separators) {
   const reg = new RegExp(`[${separators.join()}]`);
-  return string.split(reg);
+  const array = string.split(reg);
+  if (array[0] === '') {
+    array.shift();
+  }
+  if (array[array.length - 1] === '') {
+    array.pop();
+  }
+  return array;
 }

tests/util.spec.js

@@ -0,0 +1,40 @@
+import expect from 'expect.js';
+import { includesSeparators, splitBySeparators } from '../src/util';
+
+describe('includesSeparators', () => {
+  const separators = [' ', ','];
+  it('return true when given includes separators', () => {
+    expect(includesSeparators(',foo,bar', separators)).to.be(true);
+  });
+
+  it('return false when given do not include separators', () => {
+    expect(includesSeparators('foobar', separators)).to.be(false);
+  });
+
+  it('return false when string only has a leading separator', () => {
+    expect(includesSeparators(',foobar', separators)).to.be(false);
+  });
+});
+
+describe('splitBySeparators', () => {
+  const separators = [' ', ','];
+  it('split given string by separators', () => {
+    const string = 'foo bar,baz';
+    expect(splitBySeparators(string, separators)).to.eql(['foo', 'bar', 'baz']);
+  });
+
+  it('split string with leading separator ', () => {
+    const string = ',foo';
+    expect(splitBySeparators(string, separators)).to.eql(['foo']);
+  });
+
+  it('split string with trailling separator', () => {
+    const string = 'foo,';
+    expect(splitBySeparators(string, separators)).to.eql(['foo']);
+  });
+
+  it('split a separator', () => {
+    const string = ',';
+    expect(splitBySeparators(string, separators)).to.eql([]);
+  });
+});