One of these days I'll remember to test BEFORE pushing...

RIGS/models.py

@@ -473,7 +473,7 @@ class Event(models.Model, RevisionMixin):
         return reverse_lazy('event_detail', kwargs={'pk': self.pk})
 
     def __str__(self):
-        return self.display_id + ": " + self.name
+        return "{}: {}".format(self.display_id, self.name)
 
     def clean(self):
         errdict = {}

RIGS/views.py

@@ -20,7 +20,7 @@ from RIGS import models, forms
 from assets import models as asset_models
 from functools import reduce
 
-from PyRIGS.views import GenericListView,GenericDetailView
+from PyRIGS.views import GenericListView, GenericDetailView
 
 
 class PersonList(GenericListView):
@@ -43,8 +43,8 @@ class PersonDetail(GenericDetailView):
         context['history_link'] = 'person_history'
         context['detail_link'] = 'person_detail'
         context['update_link'] = 'person_update'
-        context['associated'] ='partials/associated_organisations.html'
-        context['associated2'] ='partials/associated_events.html'
+        context['associated'] = 'partials/associated_organisations.html'
+        context['associated2'] = 'partials/associated_events.html'
         return context
 
 
@@ -95,6 +95,7 @@ class OrganisationList(GenericListView):
         context['union_account'] = True
         return context
 
+
 class OrganisationDetail(GenericDetailView):
     model = models.Organisation
 
@@ -103,10 +104,11 @@ class OrganisationDetail(GenericDetailView):
         context['history_link'] = 'organisation_history'
         context['detail_link'] = 'organisation_detail'
         context['update_link'] = 'organisation_update'
-        context['associated'] ='partials/associated_people.html'
-        context['associated2'] ='partials/associated_events.html'
+        context['associated'] = 'partials/associated_people.html'
+        context['associated2'] = 'partials/associated_events.html'
         return context
 
+
 class OrganisationCreate(generic.CreateView):
     template_name = 'organisation_form.html'
     model = models.Organisation

assets/tests/pages.py

@@ -124,7 +124,7 @@ class SupplierList(BasePage):
     URL_TEMPLATE = reverse('supplier_list')
 
     _supplier_item_locator = (By.ID, 'row_item')
-    _search_text_locator = (By.XPATH, '//*[@id="main"]/div[2]/form/div/input')
+    _search_text_locator = (By.ID, 'id_search_text')
     _go_button_locator = (By.ID, 'id_search')
 
     class SupplierListRow(Region):

templates/generic_list.html

@@ -12,7 +12,7 @@
             {% csrf_token %}
             <div class="input-group">
                 <input type="search" name="q" placeholder="Search" value="{{ request.GET.q }}"
-                       class="form-control"/>
+                       class="form-control" id="id_search_text"/>
                 <span class="input-group-append">{% button 'search' id="id_search" %}</span>
             </div>
         </form>

users/templates/profile_detail.html

@@ -145,6 +145,6 @@
 </div>
 <h4>Events</h4>
 {% with object.latest_events as events %}
-    {% include 'event_table.html' %}
+    {% include 'partials/event_table.html' %}
 {% endwith %}
 {% endblock %}