Skip to content

Commit a013f7f

Browse files
committed
Add test for admin case
1 parent a8dae12 commit a013f7f

File tree

2 files changed

+14
-2
lines changed

2 files changed

+14
-2
lines changed

simple_history/tests/admin.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
from django.contrib import admin
44

55
from simple_history.admin import SimpleHistoryAdmin
6-
from .models import Poll, Choice, Person, Book, Document, Paper
6+
from .models import Poll, Choice, Person, Book, Document, Paper, Employee
77

88

99
class PersonAdmin(SimpleHistoryAdmin):
@@ -17,3 +17,4 @@ def has_change_permission(self, request, obj=None):
1717
admin.site.register(Book, SimpleHistoryAdmin)
1818
admin.site.register(Document, SimpleHistoryAdmin)
1919
admin.site.register(Paper, SimpleHistoryAdmin)
20+
admin.site.register(Employee, SimpleHistoryAdmin)

simple_history/tests/tests/test_admin.py

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
from django.conf import settings
1313
from simple_history.models import HistoricalRecords
1414

15-
from ..models import Book, Person, Poll, State
15+
from ..models import Book, Person, Poll, State, Employee
1616

1717

1818
today = datetime(2021, 1, 1, 10, 0)
@@ -234,3 +234,14 @@ def test_deleteting_user(self):
234234

235235
historical_poll = poll.history.all()[0]
236236
self.assertEqual(historical_poll.history_user, None)
237+
238+
def test_missing_one_to_one(self):
239+
"""A relation to a missing one-to-one model should still show history"""
240+
self.login()
241+
manager = Employee.objects.create()
242+
employee = Employee.objects.create(manager=manager)
243+
employee.manager = None
244+
employee.save()
245+
manager.delete()
246+
response = self.app.get(get_history_url(employee, 0))
247+
self.assertEqual(response.status_code, 200)

0 commit comments

Comments
 (0)