Skip to content

Commit

Permalink
add tests, notifications content
Browse files Browse the repository at this point in the history
  • Loading branch information
NameeLesS committed Jun 6, 2021
1 parent 63826d3 commit b825181
Show file tree
Hide file tree
Showing 35 changed files with 462 additions and 72 deletions.
2 changes: 1 addition & 1 deletion backend-project/small_eod/cases/factories.py
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ class CaseFactory(AbstractTimestampUserFactory, factory.django.DjangoModelFactor
m2m_field_name="responsible_users", factory_cls=UserFactory
)
notified_users = ManyToManyPostGeneration(
m2m_field_name="notified_users", factory_cls=UserFactory
m2m_field_name="notified_users", factory_cls=UserFactory, size=3
)
tags = ManyToManyPostGeneration(
m2m_field_name="tags", factory_cls=TagFactory, size=3
Expand Down
Original file line number Diff line number Diff line change
@@ -1,2 +1,14 @@
<p>Case create</p>>
<p>Case create</p>
{% extends "notifications/email/mail_base.html" %}
{% block subject %}
<p>Utworzono nową sprawę - {{instance.name}}</p>
{% endblock %}

{% block content %}
<p>W systemie small_eod pojawiła się nowa sprawa.</p>

<p>
Sprawa: {{instance.name}}<br/>
Utworzył: {{instance.created_by}}<br/>
Data utworzenia: {{instance.created_on}}<br/>
</p>
{% endblock %}
Original file line number Diff line number Diff line change
@@ -1,2 +1,12 @@
Case create
Case create
{% extends "notifications/email/mail_base.txt" %}
{% block subject %}
Utworzono nową sprawę - {{instance.name}}
{% endblock %}

{% block content %}
W systemie small_eod pojawiła się nowa sprawa.

Sprawa: {{instance.name}}
Utworzył: {{instance.created_by}}
Data utworzenia: {{instance.created_on}}
{% endblock %}

This file was deleted.

Original file line number Diff line number Diff line change
@@ -1,2 +1,8 @@
<p>Case removed</p>
<p>Case removed</p>
{% extends "notifications/email/mail_base.html" %}
{% block subject %}
<p>Zamknięto sprawę - {{instance.name}}</p>
{% endblock %}

{% block content %}
<p>{{instance.modified_by}} zamknął obserwowaną przez ciebie sprawę - {{instance.name}}</p>
{% endblock %}
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
{% extends "notifications/email/mail_base.txt" %}
{% block subject %}
Zamknięto sprawę - {{instance.name}}
{% endblock %}

{% block content %}
{{instance.modified_by}} zamknął obserwowaną przez ciebie sprawę - {{instance.name}}
{% endblock %}
Original file line number Diff line number Diff line change
@@ -1,2 +1,14 @@
<p>Case updated</p>
<p>Case updated</p>
{% extends "notifications/email/mail_base.html" %}
{% block subject %}
<p>Zaktualizowano sprawę - {{instance.name}}</p>
{% endblock %}

{% block content %}
<p>{{modified_by}} dokonał zmian w obserwowanej przez ciebie sprawie - {{instance.name}}.</p>

<p>
Sprawa: {{instance.name}}<br/>
Zmodyfikował: {{instance.modified_by}}<br/>
Data modyfikacji: {{instance.modified_on}}<br/>
</p>
{% endblock %}
Original file line number Diff line number Diff line change
@@ -1,2 +1,11 @@
Case update
Case update
{% extends "notifications/email/mail_base.txt" %}
{% block subject %}
Zaktualizowano sprawę - {{instance.name}}
{% endblock %}
{% block content %}
{{instance.modified_by}} dokonał zmian w obserwowanej przez ciebie sprawie - {{instance.name}}.

Sprawa: {{instance.name}}
Zmodyfikował: {{instance.modified_by}}
Data modyfikacji: {{instance.modified_on}}
{% endblock %}
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
{% extends "notifications/email/mail_base.html" %}
{% block subject %}
<p>Nowe wydarzenie w sprawie - {{instance.case.name}}</p>
{% endblock %}

{% block content %}
<p>{{instance.created_by}} utworzył nowe wydarzenie - {{instance.name}}.</p>

<p>
Wydarzenie: {{instance.name}}<br/>
Dotyczy: {{instance.case.name}}<br/>
Zaplanowano na: {{instance.date}}<br/>
Utworzył: {{instance.created_by}}<br/>
Data utworzenia: {{instance.created_on}}<br/>
</p>
{% endblock %}
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
{% extends "notifications/email/mail_base.txt" %}
{% block subject %}
Utworzono nowe wydarzenie w sprawie - {{instance.case.name}}
{% endblock %}

{% block content %}
{{instance.created_by}} utworzył nowe wydarzenie - {{instance.name}}.


Wydarzenie: {{instance.name}}
Dotyczy: {{instance.case.name}}
Zaplanowano na: {{instance.date}}
Utworzył: {{instance.created_by}}
Data utworzenia: {{instance.created_on}}
{% endblock %}
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
{% extends "notifications/email/mail_base.html" %}
{% block subject %}
<p>Usunięto wydarzenie - {{instance.name}}</p>
{% endblock %}

{% block content %}
<p>Usunięto zaplanowane wydarzenie - {{instance.name}}</p>

<p>
Wydarzenie: {{instance.name}}<br/>
Dotyczy: {{instance.case.name}}<br/>
</p>
{% endblock %}
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
{% extends "notifications/email/mail_base.txt" %}
{% block subject %}
Usunięto wydarzenie - {{instance.name}}
{% endblock %}

{% block content %}
Usunięto zaplanowane wydarzenie - {{instance.name}}

Wydarzenie: {{instance.name}}
Dotyczy: {{instance.case.name}}
{% endblock %}
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
{% extends "notifications/email/mail_base.html" %}
{% block subject %}
<p>Zaktualizowano wydarzenie - {{instance.name}}</p>
{% endblock %}

{% block content %}
<p>{{instance.modified_by}} dokonał zmian w zaplanowanym wydarzeniu - {{instance.name}}</p>

<p>
Wydarzenie: {{instance.name}}<br/>
Dotyczy: {{instance.case.name}}<br/>
Zmodyfikował: {{instance.modified_by}}<br/>
Data modyfikacji: {{instance.modified_on}}<br/>
</p>

{% endblock %}
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
{% extends "notifications/email/mail_base.txt" %}
{% block subject %}
Zaktualizowano wydarzenie - {{instance.name}}
{% endblock %}

{% block content %}
{{instance.modified_by}} dokonał zmian w zaplanowanym wydarzeniu - {{instance.name}}


Wydarzenie: {{instance.name}}
Dotyczy: {{instance.case.name}}
Zmodyfikował: {{instance.modified_by}}
Data modyfikacji: {{instance.modified_on}}
{% endblock %}
20 changes: 20 additions & 0 deletions backend-project/small_eod/events/tests/test_views.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
from test_plus.test import TestCase
from django.core import mail

from ...authkey.factories import KeyFactory
from ...generic.tests.test_views import GenericViewSetMixin, OrderingViewSetMixin
Expand Down Expand Up @@ -53,3 +54,22 @@ def test_ical_validate_response_format(self):
self.assertIn(self.obj.name, body)
self.assertIn(self.obj.comment, body)
self.assertIn(self.obj.case.name, body)

def test_send_post_notifications(self):
super().test_create_plain()
self.assertGreater(len(mail.outbox), 0)

def test_send_delete_notifications(self):
response = self.client.delete(
self.get_url(name="detail", pk=self.obj.pk, **self.get_extra_kwargs()),
)
self.assertTrue(response.status_code, 200)
self.assertGreater(len(mail.outbox), 0)

This comment has been minimized.

Copy link
@ad-m

ad-m Jun 7, 2021

Member

Czy możesz testować, że te wiadomości, które masz są faktycznie tymi wiadomości, których oczekuejsz np. przez dodanie nagłówka z rodzajem wiadomości, a potem policzeniem tylko wiadomości, które są określonego rodzaju?


def test_send_patch_notifications(self):
super().test_update_partial_plain()
self.assertGreater(len(mail.outbox), 0)

def test_notify_user_only_once(self):
super().test_update_partial_plain()
self.assertEqual(len(mail.outbox), len(self.obj.case.notified_users.all()))
Original file line number Diff line number Diff line change
@@ -0,0 +1,18 @@
{% extends "notifications/email/mail_base.html" %}
{% block subject %}
<p>Nowa wiadomość w sprawie - {{instance.case.name}}</p>
{% endblock %}

{% block content %}
{% if instance.direction == "received" %}
<p>Otrzymano nową wiadomość - {{instance.reference_number}} - dotyczącą obserwowanej przez ciebie sprawy - {{instance.case.name}}</p>

This comment has been minimized.

Copy link
@ad-m

ad-m Jun 7, 2021

Member

Proponuje to dalej uprościć wskazująć:

Zarejestrowaną nową wiadomość - {{instance.reference_number}} - dotyczącą obserwowanej przez ciebie sprawy - {{instance.case.name}}

{% else %}
<p>Wysłano wiadomość - {{instance.reference_number}} - w sprawie - {{instance.case.name}}</p>
{% endif %}

<p>
Dotyczy: {{instance.case.name}}<br/>
Utworzył: {{instance.created_by}}<br/>
Data utworzenia: {{instance.created_on}}<br/>
</p>
{% endblock %}
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
{% extends "notifications/email/mail_base.txt" %}
{% block subject %}
Nowa wiadomość w sprawie - {{instance.case.name}}
{% endblock %}

{% block content %}
{% if instance.direction == "received" %}
Otrzymano nową wiadomość - {{instance.reference_number}} - dotyczącą obserwowanej przez ciebie sprawy - {{instance.case.name}}
{% else %}
Wysłano wiadomość - {{instance.reference_number}} - w sprawie - {{instance.case.name}}
{% endif %}


Dotyczy: {{instance.case.name}}
Utworzył: {{instance.created_by}}
Data utworzenia: {{instance.created_on}}
{% endblock %}
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
{% extends "notifications/email/mail_base.html" %}
{% block subject %}
<p>Usunięto wiadomość - {{instance.reference_number}}</p>
{% endblock %}

{% block content %}
<p>Usunięto wiadomość - {{instance.reference_number}} - dotyczącą obserwowanej przez ciebie sprawy - {{instance.case.name}}.</p>
{% endblock %}
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
{% extends "notifications/email/mail_base.txt" %}
{% block subject %}
Usunięto wiadomość - {{instance.reference_number}}
{% endblock %}

{% block content %}
Usunięto wiadomość - {{instance.reference_number}} - dotyczącą obserwowanej przez ciebie sprawy - {{instance.case.name}}.
{% endblock %}
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
{% extends "notifications/email/mail_base.html" %}
{% block subject %}
<p>Zmodyfikowano wiadomość - {{instance.reference_number}}</p>
{% endblock %}

{% block content %}
<p>{{instance.modified_by}} dokonał zmian w wiadomości - {{instance.reference_number}}.</p>

<p>
Dotyczy: {{instance.case.name}}<br/>
Zmodyfikował: {{instance.modified_by}}<br/>
Data modyfikacji: {{instance.modified_on}}<br/>
</p>
{% endblock %}
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
{% extends "notifications/email/mail_base.txt" %}
{% block subject %}
Zmodyfikowano wiadomość - {{instance.reference_number}}
{% endblock %}

{% block content %}
{{instance.modified_by}} dokonał zmian w wiadomości - {{instance.reference_number}}.

Dotyczy: {{instance.case.name}}
Zmodyfikował: {{instance.modified_by}}
Data modyfikacji: {{instance.modified_on}}
{% endblock %}
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
{% extends "notifications/email/mail_base.html" %}
{% block subject %}
<p>Dodano notatkę do sprawy - {{case}}</p>
{% endblock %}

{% block content %}
<p>{{instance.created_by}} dodał nową notatkę do sprawy - {{instance.case.name}}</p>

<p>
Dotyczy: {{instance.case.name}}<br/>
Utworzył: {{instance.created_by}}<br/>
Data utworzenia: {{instance.created_on}}<br/>
Treść: {{instance.comment}}<br/>
</p>
{% endblock %}
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
{% extends "notifications/email/mail_base.txt" %}
{% block subject %}
Dodano notatkę do sprawy - {{case}}
{% endblock %}

{% block content %}
{{instance.created_by}} dodał nową notatkę do sprawy - {{instance.case.name}}

Dotyczy: {{instance.case.name}}
Utworzył: {{instance.created_by}}
Data utworzenia: {{instance.created_on}}
Treść: {{instance.comment}}
{% endblock %}
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
{% extends "notifications/email/mail_base.html" %}
{% block subject %}
<p>Usunięto notatkę dotyczącą sprawy - {{instance.case.name}}</p>
{% endblock %}

{% block content %}
<p>Usunięto notatkę dotyczącą obserwowanej przez ciebie sprawy - {{instance.case.name}}</p>
{% endblock %}
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
{% extends "notifications/email/mail_base.txt" %}
{% block subject %}
Usunięto notatkę dotyczącą sprawy - {{instance.case.name}}
{% endblock %}

{% block content %}
Usunięto notatkę dotyczącą obserwowanej przez ciebie sprawy - {{instance.case.name}}
{% endblock %}
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
{% extends "notifications/email/mail_base.html" %}
{% block subject %}
<p>Zaktualizowano notatkę dotyczącą sprawy - {{instance.case.name}}</p>
{% endblock %}

{% block content %}

<p>{{instance.modified_by}} dokonał zmian w notatce dotyczącej obserwowanej przez ciebie sprawy - {{instance.case.name}}.</p>

<p>
Sprawa: {{instance.case.name}}<br/>
Zmieniona przez: {{instance.modified_by}}<br/>
Data modyfikacji: {{instance.midofied_on}}<br/>
Treść: {{instance.comment}}<br/>
</p>
{% endblock %}
Loading

0 comments on commit b825181

Please sign in to comment.