labels added final

Author: olyapka84

task_manager/labels/forms.py

@@ -4,4 +4,10 @@
 class LabelForm(forms.ModelForm):
     class Meta:
         model = Label
-        fields = ("name",)
+        fields = ["name"]
+        widgets = {
+            "name": forms.TextInput(attrs={"class": "form-control"}),
+        }
+        labels = {
+            "name": "Имя",
+        }

task_manager/tasks/forms.py

@@ -5,10 +5,18 @@
 class TaskForm(forms.ModelForm):
     class Meta:
         model = Task
-        fields = ["name", "description", "status", "executor"]
+        fields = ["name", "description", "status", "executor", "labels"]
         widgets = {
             "name": forms.TextInput(attrs={"class": "form-control"}),
             "description": forms.Textarea(attrs={"class": "form-control", "rows": 4}),
             "status": forms.Select(attrs={"class": "form-select"}),
             "executor": forms.Select(attrs={"class": "form-select"}),
+            "labels": forms.SelectMultiple(attrs={"class": "form-select", "size": 5}),
+        }
+        labels = {
+            "name": "Имя",
+            "description": "Описание",
+            "status": "Статус",
+            "executor": "Исполнитель",
+            "labels": "Метки",
         }

task_manager/templates/tasks/detail.html

@@ -7,6 +7,14 @@ <h1>{{ task.name }}</h1>
 <hr>
 <p style="white-space: pre-wrap">{{ task.description|default:"(без описания)" }}</p>
 
+{% if task.labels.exists %}
+  <p><strong>Метки:</strong>
+    {% for label in task.labels.all %}
+      <span class="badge bg-secondary">{{ label.name }}</span>
+    {% endfor %}
+  </p>
+{% endif %}
+
 <p class="mt-3">
   <a class="btn btn-outline-secondary" href="{% url 'tasks_update' task.pk %}">Изменить</a>
   <a class="btn btn-outline-danger" href="{% url 'tasks_delete' task.pk %}">Удалить</a>

task_manager/templates/tasks/form.html

@@ -30,7 +30,14 @@ <h1>{{ object|default_if_none:"Создать задачу" }}</h1>
     {% if form.executor.errors %}<div class="text-danger">{{ form.executor.errors|striptags }}</div>{% endif %}
   </div>
 
+  <div class="mb-3">
+    <label for="id_labels" class="form-label">Метки</label>
+    {{ form.labels }}
+  <div class="form-text">Можно выбрать несколько (Ctrl/⌘ + клик).</div>
+  </div>
+
   <button type="submit" class="btn btn-primary">Сохранить</button>
   <a href="{% url 'tasks_index' %}" class="btn btn-link">Отмена</a>
 </form>
+
 {% endblock %}

tests/test_labels_crud.py

@@ -0,0 +1,83 @@
+import pytest
+from django.urls import reverse
+from django.contrib.auth import get_user_model
+
+from task_manager.labels.models import Label
+from task_manager.tasks.models import Task, Status
+
+User = get_user_model()
+
+
+@pytest.mark.django_db
+def test_labels_list_requires_login(client):
+    url = reverse("labels:index")
+    resp = client.get(url)
+    assert resp.status_code == 302
+    assert reverse("users:login") in resp["Location"]
+
+
+@pytest.mark.django_db
+def test_label_create_requires_login(client):
+    url = reverse("labels:create")
+    resp = client.post(url, {"name": "bug"})
+    assert resp.status_code == 302
+    assert reverse("users:login") in resp["Location"]
+    assert not Label.objects.filter(name="bug").exists()
+
+
+@pytest.mark.django_db
+def test_label_create_ok(client, django_user_model):
+    user = django_user_model.objects.create_user(username="u", password="p")
+    client.login(username="u", password="p")
+
+    url = reverse("labels:create")
+    resp = client.post(url, {"name": "bug"})
+    assert resp.status_code == 302
+    obj = Label.objects.get(name="bug")
+    assert obj.name == "bug"  # имя поля формы совпадает с демо: "name"
+
+
+@pytest.mark.django_db
+def test_label_update_ok(client, django_user_model):
+    user = django_user_model.objects.create_user(username="u", password="p")
+    client.login(username="u", password="p")
+
+    l = Label.objects.create(name="old")
+    url = reverse("labels:update", args=[l.pk])
+    resp = client.post(url, {"name": "new"})
+    assert resp.status_code == 302
+
+    l.refresh_from_db()
+    assert l.name == "new"
+
+
+@pytest.mark.django_db
+def test_label_delete_blocked_when_in_use(client, django_user_model):
+    """Нельзя удалить метку, если она связана с задачей."""
+    user = django_user_model.objects.create_user(username="u", password="p")
+    client.login(username="u", password="p")
+
+    l = Label.objects.create(name="bug")
+    s = Status.objects.create(name="open")
+    t = Task.objects.create(name="t1", author=user, status=s)
+    t.labels.add(l)
+
+    url = reverse("labels:delete", args=[l.pk])
+    resp_get = client.get(url)           # страница подтверждения не должна падать
+    assert resp_get.status_code in (200, 302)
+
+    resp_post = client.post(url)         # попытка удалить
+    assert resp_post.status_code == 302  # редирект назад к списку
+    assert Label.objects.filter(pk=l.pk).exists()
+
+
+@pytest.mark.django_db
+def test_label_delete_when_unused_ok(client, django_user_model):
+    user = django_user_model.objects.create_user(username="u", password="p")
+    client.login(username="u", password="p")
+
+    l = Label.objects.create(name="orphan")
+    url = reverse("labels:delete", args=[l.pk])
+    resp = client.post(url)
+    assert resp.status_code == 302
+    assert not Label.objects.filter(pk=l.pk).exists()