Merge pull request #1487 from mzazrivec/add_ids_to_ansible_credential…

…_form_selects Add IDs to ansible credential form selects
ManageIQ · Jun 6, 2017 · 3ab2b70 · 3ab2b70
2 parents 96631ab + 5fc0bfa
commit 3ab2b70
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/app/assets/javascripts/components/ansible-credential-options.js b/app/assets/javascripts/components/ansible-credential-options.js
@@ -59,7 +59,7 @@ ManageIQ.angular.app.component('ansibleCredentialOptions', {
          '<a href="" ng-switch-when="password" ng-hide="!vm[name] || vm.newRecord" ng-click="vm.cancelPassword(name)">{{__("Cancel")}}</a>',
          // select
          '<div ng-switch-when="choice" class="col-md-8">',
-            '<select pf-select ng-options="opt as opt for opt in attr.choices" class="form-control" ng-model="vm.model[name]" />',
+            '<select id="{{name}}" pf-select ng-options="opt as opt for opt in attr.choices" class="form-control" ng-model="vm.model[name]" />',
           '</div>',
          // text
          '<div ng-switch-when="string" class="col-md-8">',

diff --git a/app/views/ansible_credential/_credential_form.html.haml b/app/views/ansible_credential/_credential_form.html.haml
@@ -30,6 +30,7 @@
         .col-md-8
           %select{'ng-model'   => 'vm.credentialModel.type',
                   'ng-options' => 'cred.value as cred.label for cred in vm.select_options',
+                  'id'         => 'type',
                   'required'   => 'true',
                   'pf-select'  => 'true'}
       - else