Renders query list in case-insensitive alpha order (#1126)

This commit is contained in:
Mike Stone 2017-01-30 15:11:13 -05:00 committed by Jason Meller
parent 54653127ee
commit cdc3d1bf3a
2 changed files with 6 additions and 5 deletions

View file

@ -76,10 +76,11 @@ class QueriesList extends Component {
}
render () {
const alphaSort = q => q.name.toLowerCase();
const { checkedQueryIDs, onCheckQuery, onSelectQuery, queries, selectedQuery } = this.props;
const { allQueriesChecked } = this.state;
const { renderHelpText, handleCheckAll } = this;
const sortedQueries = sortBy(queries, [alphaSort]);
const wrapperClassName = classnames(`${baseClass}__table`, {
[`${baseClass}__table--query-selected`]: size(checkedQueryIDs),
});
@ -101,7 +102,7 @@ class QueriesList extends Component {
</thead>
<tbody>
{renderHelpText()}
{!!queries.length && sortBy(queries, ['name']).map((query) => {
{!!sortedQueries.length && sortedQueries.map((query) => {
return (
<QueriesListRow
checked={this.isChecked(query)}

View file

@ -16,7 +16,7 @@ const store = {
101: {
...queryStub,
id: 101,
name: 'My unique query name',
name: 'alpha query',
},
},
},
@ -65,7 +65,7 @@ describe('ManageQueriesPage - component', () => {
expect(page.node.getQueries().length).toEqual(2);
fillInFormInput(queryFilterInput, 'My unique query name');
fillInFormInput(queryFilterInput, 'alpha query');
expect(page.node.getQueries().length).toEqual(1);
});
@ -115,7 +115,7 @@ describe('ManageQueriesPage - component', () => {
});
describe('bulk delete action', () => {
const queries = [queryStub, { ...queryStub, id: 101, name: 'My unique query name' }];
const queries = [queryStub, { ...queryStub, id: 101, name: 'alpha query' }];
it('displays the delete action button when a query is checked', () => {
const page = mount(<ManageQueriesPage queries={queries} />);