Fixing ids for UI automation (mattermost#5772)

stevepartridge · Mar 16, 2017 · 81a48af · 81a48af
1 parent 631bafc
commit 81a48af
Show file tree

Hide file tree

Showing 2 changed files with 26 additions and 3 deletions.
diff --git a/components/setting_item_max.jsx b/components/setting_item_max.jsx
@@ -31,12 +31,30 @@ export default class SettingItemMax extends React.Component {
  render() {
  var clientError = null;
  if (this.props.client_error) {
- clientError = (<div className='form-group'><label className='col-sm-12 has-error'>{this.props.client_error}</label></div>);
+ clientError = (
+ <div className='form-group'>
+ <label
+ id='clientError'
+ className='col-sm-12 has-error'
+ >
+ {this.props.client_error}
+ </label>
+ </div>
+ );
  }
 
  var serverError = null;
  if (this.props.server_error) {
- serverError = (<div className='form-group'><label className='col-sm-12 has-error'>{this.props.server_error}</label></div>);
+ serverError = (
+ <div className='form-group'>
+ <label
+ id='serverError'
+ className='col-sm-12 has-error'
+ >
+ {this.props.server_error}
+ </label>
+ </div>
+ );
  }
 
  var extraInfo = null;

diff --git a/components/setting_item_min.jsx b/components/setting_item_min.jsx
@@ -34,7 +34,12 @@ export default class SettingItemMin extends React.Component {
  >
  <li className='col-xs-12 col-sm-9 section-title'>{this.props.title}</li>
  {editButton}
- <li className='col-xs-12 section-describe'>{this.props.describe}</li>
+ <li
+ id={this.props.title + 'Desc'}
+ className='col-xs-12 section-describe'
+ >
+ {this.props.describe}
+ </li>
  </ul>
  );
  }