Merge pull request #19966 from code-dot-org/school-autocomplete-fix-r…

…emount-blank Fix bug where unmounting and remounting the school autocomplete would…
code-dot-org · Jan 12, 2018 · 774db2b · 774db2b
2 parents 815e572 + dc84f14
commit 774db2b
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/apps/src/templates/SchoolAutocompleteDropdown.jsx b/apps/src/templates/SchoolAutocompleteDropdown.jsx
@@ -67,6 +67,7 @@ export default class SchoolAutocompleteDropdown extends Component {
         name={this.props.fieldName}
         async={true}
         loadOptions={this.getOptions}
+        cache={false}
         filterOption={() => true}
         value={this.props.value}
         onChange={this.props.onChange}

diff --git a/apps/test/unit/templates/SchoolAutocompleteDropdownTest.js b/apps/test/unit/templates/SchoolAutocompleteDropdownTest.js
@@ -40,6 +40,10 @@ describe('SchoolAutocompleteDropdown', () => {
     expect(select).to.have.prop('value', '12345');
   });
 
+  it('Disables cache on the select control', () => {
+    expect(select).to.have.prop('cache', false);
+  });
+
   it("Calls props.onChange when the selection changes", () => {
     select.simulate("change", {value: '1', label: 'selected school'});
     expect(handleChange).to.be.calledOnce;