Show TC Executions card when viewing a bug. Refs #699

- [db] add db_index=True to LinkReference.url field
- modify TC Executions card to show bugs only when specified.
  When viewing the Bug page showing the same info again doesn't
  give us any more additional information but occupies screen
  space.

tcms/bugs/migrations/0001_initial.py

@@ -2,6 +2,7 @@
 
 from django.conf import settings
 from django.db import migrations, models
+import tcms.core.models.base
 
 
 class Migration(migrations.Migration):
@@ -43,5 +44,6 @@ class Migration(migrations.Migration):
                 ('version', models.ForeignKey(on_delete=models.deletion.CASCADE,
                                               to='management.Version')),
             ],
+            bases=(models.Model, tcms.core.models.base.UrlMixin),
         ),
     ]

tcms/bugs/models.py

@@ -2,11 +2,14 @@
 
 # Licensed under the GPL 2.0: https://www.gnu.org/licenses/old-licenses/gpl-2.0.html
 
-from django.conf import settings
 from django.db import models
+from django.conf import settings
+from django.urls import reverse
 
+from tcms.core.models.base import UrlMixin
 
-class Bug(models.Model):
+
+class Bug(models.Model, UrlMixin):
     summary = models.CharField(max_length=255, db_index=True)
     created_at = models.DateTimeField(auto_now_add=True,
                                       db_index=True)
@@ -31,3 +34,6 @@ class Bug(models.Model):
 
     def __str__(self):
         return "BUG-%d: %s " % (self.pk, self.summary)
+
+    def _get_absolute_url(self):
+        return reverse('bugs-get', args=[self.pk, ])

tcms/bugs/static/bugs/js/get.js

@@ -4,4 +4,7 @@ $(document).ready(function() {
 
     // bind everything in tags table
     tagsCard('Bug', object_id, {bugs: object_id}, perm_remove_tag);
+
+    // executions tree view
+    treeViewBind();
 });

tcms/bugs/templates/bugs/get.html

@@ -117,7 +117,12 @@ <h2 class="card-pf-title">
                 </div>
             </div>
         </div>
+    </div>
 
+    <div class="row row-cards-pf">
+        <div class="col-xs-12 col-sm-12 col-md-12">
+            {% include 'include/tc_executions.html' %}
+        </div>
     </div>
 
     {% get_comment_list for object as comments %}

tcms/bugs/views.py

@@ -8,14 +8,15 @@
 from django.http import HttpResponseRedirect
 from django.shortcuts import render
 from django.utils.decorators import method_decorator
-from django.utils.translation import ugettext_lazy as _
 from django.views.generic import DetailView
 from django.views.generic.base import TemplateView
 from django.views.generic.base import View
 
 from tcms.bugs.models import Bug
+from tcms.testruns.models import TestExecution
 from tcms.bugs.forms import NewBugForm, BugCommentForm
 from tcms.core.helpers.comments import add_comment
+from tcms.core.contrib.linkreference.models import LinkReference
 
 
 class Get(DetailView):  # pylint: disable=missing-permission-required
@@ -27,6 +28,14 @@ def get_context_data(self, **kwargs):
         context = super().get_context_data(**kwargs)
         context['comment_form'] = BugCommentForm()
         context['comment_form'].populate(self.object.pk)
+
+        context['executions'] = TestExecution.objects.filter(
+            pk__in=LinkReference.objects.filter(
+                is_defect=True,
+                url=self.object.get_full_url(),
+            ).values('execution')
+        )
+
         return context
 
 

tcms/core/contrib/linkreference/migrations/0003_add_db_index_to_url_field.py

@@ -0,0 +1,18 @@
+# Generated by Django 2.2.5 on 2019-09-14 18:58
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('linkreference', '0002_update_fields'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='linkreference',
+            name='url',
+            field=models.URLField(db_index=True),
+        ),
+    ]

tcms/core/contrib/linkreference/models.py

@@ -7,7 +7,7 @@ class LinkReference(models.Model):
     execution = models.ForeignKey('testruns.TestExecution', on_delete=models.CASCADE)
 
     name = models.CharField(max_length=64, blank=True, default='')
-    url = models.URLField()
+    url = models.URLField(db_index=True)
     created_on = models.DateTimeField(auto_now_add=True, db_index=True)
     is_defect = models.BooleanField(default=False, db_index=True)
 

tcms/core/models/base.py

@@ -11,4 +11,4 @@ class UrlMixin:  # pylint: disable=too-few-public-methods
     def get_full_url(self):
         site = Site.objects.get(pk=settings.SITE_ID)
         host_link = request_host_link(None, site.domain)
-        return '{}/{}'.format(host_link, self._get_absolute_url().strip('/'))
+        return '{}/{}/'.format(host_link, self._get_absolute_url().strip('/'))

tcms/templates/include/tc_executions.html

@@ -40,7 +40,7 @@ <h2 class="card-pf-title">
                         <div class="list-group-item-header">
                             <div class="list-view-pf-main-info">
                                 <div class="list-view-pf-left">
-                                    <span class="fa {% if execution_comments or bugs %}fa-angle-right{% else %}fa-exclamation{% endif %}"></span>
+                                    <span class="fa {% if execution_comments or show_bugs and bugs %}fa-angle-right{% else %}fa-exclamation{% endif %}"></span>
                                 </div>
 
                                 <div class="list-view-pf-body">
@@ -58,7 +58,7 @@ <h2 class="card-pf-title">
                                             <span class="{{ execution.status.icon }}"></span>
                                             <strong>{{ execution.status.name }}</strong>
                                         </div>
-                                        {% if bugs %}
+                                        {% if show_bugs and bugs %}
                                         <div class="list-view-pf-additional-info-item">
                                             <span class="fa fa-bug"></span>
                                             <strong>{{ bugs|length }}</strong>
@@ -82,6 +82,7 @@ <h2 class="card-pf-title">
                                 </div>
                             </div>
                         </div> <!-- / caserun header -->
+                {% if show_bugs %}
                     {% for bug in bugs %}
                         <div class="list-group-item-container container-fluid">
                             <div class="list-group-item">
@@ -114,6 +115,7 @@ <h2 class="card-pf-title">
                             </div>
                         </div> <!-- /bug -->
                     {% endfor %}
+                {% endif %}
 
                     {% for comment in execution_comments %}
                         <!-- comments -->

tcms/testcases/templates/testcases/get.html

@@ -125,7 +125,7 @@ <h2 class="card-pf-title" style="text-align: left">
 
     <div class="row row-cards-pf">
         <div class="col-xs-12 col-sm-12 col-md-12">
-            {% include 'include/tc_executions.html' %}
+            {% include 'include/tc_executions.html' with show_bugs=True %}
         </div>
     </div>