add form to delete a ComputerSoftwareRelation

computers/templates/computers/computer_details.html

@@ -51,8 +51,8 @@
 
                 <h2>Software</h2>
                 <ul>
-                {% for relation in software_relations %}
-                    <li>{{ relation.software }}</li>
+                {% for software in software_relations %}
+                    <li>{{ software.software }} <a href="{% url 'software_relation_delete' software.pk %}" class="delete" title="Delete" data-toggle="tooltip"><i class="material-icons">clear</i></a></li>
                 {% endfor %}
                 </ul>
                 <p><a href="{% url 'software_relation_create' pk %}" class="add" title="Add" data-toggle="tooltip"><i class="material-icons">add</i></a></p>

computers/tests/test_computer_form_views.py

@@ -178,3 +178,16 @@ def test_disk_relation_delete_view(create_admin_user):
     url = '/delete/disk-relation/{}/'.format(disk_relation.id)
     response = client.post(url)
     assert response.status_code == 302
+
+
+def test_software_relation_delete_view(create_admin_user):
+    create_admin_user()
+    client = Client()
+    client.login(username="pharma-admin", password="password")
+    computer = mixer.blend('computers.Computer', customer=mixer.SELECT)
+    software = mixer.blend('softwares.Software')
+    software_relation = mixer.blend('computers.ComputerSoftwareRelation',
+                                    computer=computer, software=software)
+    url = '/delete/software-relation/{}/'.format(software_relation.id)
+    response = client.post(url)
+    assert response.status_code == 302

computers/urls.py

@@ -47,4 +47,7 @@ urlpatterns = [
     path('computer/<int:pk>/create/software-relation/',
          views.ComputerSoftwareRelationCreateView.as_view(),
          name='software_relation_create'),
+    path('delete/software-relation/<int:pk>/',
+         views.ComputerSoftwareRelationDeleteView.as_view(),
+         name='software_relation_delete'),
  ]

computers/views.py

@@ -266,3 +266,11 @@ class ComputerSoftwareRelationCreateView(LoginRequiredMixin, CreateView):
             'computer': self.computer,
         }
 
+
+class ComputerSoftwareRelationDeleteView(LoginRequiredMixin, DeleteView):
+    model = ComputerSoftwareRelation
+    template_name = 'computers/relation_confirm_delete.html'
+
+    def get_success_url(self):
+        return reverse('computer', args=(self.object.computer.pk,))
+