makes test leads not required, conditionally displays tests

Author: devGregA

dojo/engagement/views.py

@@ -309,7 +309,11 @@ def add_tests(request, eid):
         if form.is_valid():
             new_test = form.save(commit=False)
             new_test.engagement = eng
-            new_test.lead = User.objects.get(id=form['lead'].value())
+	    try:
+            	new_test.lead = User.objects.get(id=form['lead'].value())
+	    except:
+		new_test.lead = None
+		pass
             new_test.save()
             tags = request.POST.getlist('tags')
             t = ", ".join(tags)

dojo/forms.py

@@ -555,7 +555,7 @@ class TestForm(forms.ModelForm):
                                      "Choose from the list or add new tags.  Press TAB key to add.")
     lead = forms.ModelChoiceField(
 	queryset=User.objects.exclude(is_staff=False),
-	required=True, label="Testing Lead")
+	required=False, label="Testing Lead")
 
 
     def __init__(self, *args, **kwargs):

dojo/templates/dojo/engagement.html

@@ -63,10 +63,14 @@ <h3 class="has-filters">
 						<div class="lineContainer">
 							<a  style="display: inline" class="eng_link" href="{% url 'view_engagement' e.id %}">
                                                 {% if e.name %}{{ e.name }} {% endif %}{{ e.target_start }}</a>
-							| Lead : {{ e.lead.first_name }}
+							| Lead: {{ e.lead.first_name }}
                                                         |
                                                         {%  for test in e.test_set.all %}
-                                                                <a href="{% url 'view_test' test.id %}"> {{ test.test_type }} : {{ test.lead.first_name }} </a> |
+								{% if test.lead %}
+                                                                <a href="{% url 'view_test' test.id %}"> {{ test.test_type }}: {{ test.lead.first_name }} </a> |
+								{% else %}
+                                                                <a href="{% url 'view_test' test.id %}"> {{ test.test_type }} </a> |
+								{% endif %}
                                                         {% endfor %}
 						</div>
                                                 <sup>