Merge pull request #60 from fahzee1/fix-permafrost-test-for-inputs

update permission id and update failed test

Author: devo-wm

permafrost/templates/permafrost/permafrostrole_form.html

@@ -132,7 +132,7 @@ <h6 class="m-0 text-primary font-weight-bold">
                     <div class="col-6">
                         {% for permission in category.optional %}
                         <div class="d-flex align-items-center">
-                            <label for="{{ permission.id }}">{{ permission.name }}</label>
+                            <label for="permission-{{ permission.id }}">{{ permission.name }}</label>
 
                             {% if permission.short_description %}
                             <i class="ml-2 fas fa-info-circle text-gray-600" aria-label="{% trans 'Info' %}"></i>
@@ -142,7 +142,7 @@ <h6 class="m-0 text-primary font-weight-bold">
                                 type="checkbox"
                                 name="{{ form.permissions.name }}"
                                 value="{{ permission.id }}"
-                                id="{{ permission.id }}"
+                                id="permission-{{ permission.id }}"
                                 {% if permission.selected %} checked{% endif %}
                             >
                         </div>

permafrost/tests.py

@@ -474,13 +474,15 @@ def test_update_form_has_selected_optional_permission(self):
                                 type="checkbox" 
                                 name="permissions" 
                                 value="37"
+                                id="permission-37"
                                  checked
                             >""")
             self.assertContains(response, """<input 
                                 class="ml-auto" 
                                 type="checkbox" 
                                 name="permissions" 
                                 value="38"
+                                id="permission-38"
                                  checked
                             >""")
         except: