add handling of undefined/null options.

Author: wcastand

src/sync.js

@@ -11,15 +11,22 @@ class SyncSelect extends Component {
     options: [],
     searchValue: '',
   }
+
+  testOptions = options =>
+    typeof options === 'undefined' || options === null
+      ? []
+      : options.map(this.transform)
+
   componentWillMount() {
-    const opts = this.props.options.map(this.transform)
+    const opts = this.testOptions(this.props.options)
+
     this.setState({
       sourceOptions: opts,
       options: this.computeOptions('', opts),
     })
   }
   componentWillReceiveProps(nextProps) {
-    const newOpts = nextProps.options.map(this.transform)
+    const newOpts = this.testOptions(nextProps.options)
     if (this.props.sourceOptions !== newOpts) {
       this.setState({
         sourceOptions: newOpts,

stories/async.js

@@ -17,7 +17,7 @@ const createQuery = query =>
   query === ''
     ? 'https://swapi.co/api/people'
     : `https://swapi.co/api/people/?search=${query}`
-const fakeApiCb = (query, cb) => fakeApi(query).then(r => cb(null, r.results))
+const fakeApiCb = (query, cb) => fakeApi(query).then(r => cb(null, r))
 const fakeApi = query =>
   fetch(createQuery(query))
     .then(response => response.json())
@@ -34,7 +34,7 @@ storiesOf('Async', module)
       transform={data => ({label: data.name, value: data.name})}>
       <Label />
       <List renderItem={Item} />
-    </Container>,
+    </Container>
   )
   .add('With promise', () =>
     <Container
@@ -52,7 +52,7 @@ storiesOf('Async', module)
           <List renderItem={<Item render={renderingItem} />} />
         </div>
       </div>
-    </Container>,
+    </Container>
   )
   .add('With custom debounce', () =>
     <Container
@@ -69,5 +69,5 @@ storiesOf('Async', module)
         <TagList renderTag={Tag} />
         <List renderItem={<Item render={renderingItem} />} />
       </div>
-    </Container>,
+    </Container>
   )

stories/sync.js

@@ -88,6 +88,12 @@ storiesOf('Sync', module)
       <List renderItem={Item} />
     </Container>
   )
+  .add('Basic - Wraning on undefined options', () =>
+    <Container name="context" onChange={onChange}>
+      <Label />
+      <List renderItem={Item} />
+    </Container>
+  )
   .add('Basic withCustom Key', () =>
     <Container
       name="context"