test_logentry.py 14 KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347
  1. import json
  2. from datetime import datetime
  3. from django.contrib.admin.models import ADDITION, CHANGE, DELETION, LogEntry
  4. from django.contrib.admin.utils import quote
  5. from django.contrib.auth.models import User
  6. from django.contrib.contenttypes.models import ContentType
  7. from django.test import TestCase, override_settings
  8. from django.urls import reverse
  9. from django.utils import translation
  10. from django.utils.html import escape
  11. from .models import Article, ArticleProxy, Car, Site
  12. @override_settings(ROOT_URLCONF="admin_utils.urls")
  13. class LogEntryTests(TestCase):
  14. @classmethod
  15. def setUpTestData(cls):
  16. cls.user = User.objects.create_superuser(
  17. username="super", password="secret", email="super@example.com"
  18. )
  19. cls.site = Site.objects.create(domain="example.org")
  20. cls.a1 = Article.objects.create(
  21. site=cls.site,
  22. title="Title",
  23. created=datetime(2008, 3, 12, 11, 54),
  24. )
  25. content_type_pk = ContentType.objects.get_for_model(Article).pk
  26. LogEntry.objects.log_action(
  27. cls.user.pk,
  28. content_type_pk,
  29. cls.a1.pk,
  30. repr(cls.a1),
  31. CHANGE,
  32. change_message="Changed something",
  33. )
  34. def setUp(self):
  35. self.client.force_login(self.user)
  36. def test_logentry_save(self):
  37. """
  38. LogEntry.action_time is a timestamp of the date when the entry was
  39. created. It shouldn't be updated on a subsequent save().
  40. """
  41. logentry = LogEntry.objects.get(content_type__model__iexact="article")
  42. action_time = logentry.action_time
  43. logentry.save()
  44. self.assertEqual(logentry.action_time, action_time)
  45. def test_logentry_change_message(self):
  46. """
  47. LogEntry.change_message is stored as a dumped JSON structure to be able
  48. to get the message dynamically translated at display time.
  49. """
  50. post_data = {
  51. "site": self.site.pk,
  52. "title": "Changed",
  53. "hist": "Some content",
  54. "created_0": "2008-03-12",
  55. "created_1": "11:54",
  56. }
  57. change_url = reverse(
  58. "admin:admin_utils_article_change", args=[quote(self.a1.pk)]
  59. )
  60. response = self.client.post(change_url, post_data)
  61. self.assertRedirects(response, reverse("admin:admin_utils_article_changelist"))
  62. logentry = LogEntry.objects.filter(
  63. content_type__model__iexact="article"
  64. ).latest("id")
  65. self.assertEqual(logentry.get_change_message(), "Changed Title and History.")
  66. with translation.override("fr"):
  67. self.assertEqual(
  68. logentry.get_change_message(), "Modification de Title et Historique."
  69. )
  70. add_url = reverse("admin:admin_utils_article_add")
  71. post_data["title"] = "New"
  72. response = self.client.post(add_url, post_data)
  73. self.assertRedirects(response, reverse("admin:admin_utils_article_changelist"))
  74. logentry = LogEntry.objects.filter(
  75. content_type__model__iexact="article"
  76. ).latest("id")
  77. self.assertEqual(logentry.get_change_message(), "Added.")
  78. with translation.override("fr"):
  79. self.assertEqual(logentry.get_change_message(), "Ajout.")
  80. def test_logentry_change_message_not_json(self):
  81. """LogEntry.change_message was a string before Django 1.10."""
  82. logentry = LogEntry(change_message="non-JSON string")
  83. self.assertEqual(logentry.get_change_message(), logentry.change_message)
  84. def test_logentry_change_message_localized_datetime_input(self):
  85. """
  86. Localized date/time inputs shouldn't affect changed form data detection.
  87. """
  88. post_data = {
  89. "site": self.site.pk,
  90. "title": "Changed",
  91. "hist": "Some content",
  92. "created_0": "12/03/2008",
  93. "created_1": "11:54",
  94. }
  95. with translation.override("fr"):
  96. change_url = reverse(
  97. "admin:admin_utils_article_change", args=[quote(self.a1.pk)]
  98. )
  99. response = self.client.post(change_url, post_data)
  100. self.assertRedirects(
  101. response, reverse("admin:admin_utils_article_changelist")
  102. )
  103. logentry = LogEntry.objects.filter(
  104. content_type__model__iexact="article"
  105. ).latest("id")
  106. self.assertEqual(logentry.get_change_message(), "Changed Title and History.")
  107. def test_logentry_change_message_formsets(self):
  108. """
  109. All messages for changed formsets are logged in a change message.
  110. """
  111. a2 = Article.objects.create(
  112. site=self.site,
  113. title="Title second article",
  114. created=datetime(2012, 3, 18, 11, 54),
  115. )
  116. post_data = {
  117. "domain": "example.com", # domain changed
  118. "admin_articles-TOTAL_FORMS": "5",
  119. "admin_articles-INITIAL_FORMS": "2",
  120. "admin_articles-MIN_NUM_FORMS": "0",
  121. "admin_articles-MAX_NUM_FORMS": "1000",
  122. # Changed title for 1st article
  123. "admin_articles-0-id": str(self.a1.pk),
  124. "admin_articles-0-site": str(self.site.pk),
  125. "admin_articles-0-title": "Changed Title",
  126. # Second article is deleted
  127. "admin_articles-1-id": str(a2.pk),
  128. "admin_articles-1-site": str(self.site.pk),
  129. "admin_articles-1-title": "Title second article",
  130. "admin_articles-1-DELETE": "on",
  131. # A new article is added
  132. "admin_articles-2-site": str(self.site.pk),
  133. "admin_articles-2-title": "Added article",
  134. }
  135. change_url = reverse(
  136. "admin:admin_utils_site_change", args=[quote(self.site.pk)]
  137. )
  138. response = self.client.post(change_url, post_data)
  139. self.assertRedirects(response, reverse("admin:admin_utils_site_changelist"))
  140. self.assertSequenceEqual(Article.objects.filter(pk=a2.pk), [])
  141. logentry = LogEntry.objects.filter(content_type__model__iexact="site").latest(
  142. "action_time"
  143. )
  144. self.assertEqual(
  145. json.loads(logentry.change_message),
  146. [
  147. {"changed": {"fields": ["Domain"]}},
  148. {"added": {"object": "Added article", "name": "article"}},
  149. {
  150. "changed": {
  151. "fields": ["Title", "not_a_form_field"],
  152. "object": "Changed Title",
  153. "name": "article",
  154. }
  155. },
  156. {"deleted": {"object": "Title second article", "name": "article"}},
  157. ],
  158. )
  159. self.assertEqual(
  160. logentry.get_change_message(),
  161. "Changed Domain. Added article “Added article”. "
  162. "Changed Title and not_a_form_field for article “Changed Title”. "
  163. "Deleted article “Title second article”.",
  164. )
  165. with translation.override("fr"):
  166. self.assertEqual(
  167. logentry.get_change_message(),
  168. "Modification de Domain. Ajout de article « Added article ». "
  169. "Modification de Title et not_a_form_field pour l'objet "
  170. "article « Changed Title ». "
  171. "Suppression de article « Title second article ».",
  172. )
  173. def test_logentry_get_edited_object(self):
  174. """
  175. LogEntry.get_edited_object() returns the edited object of a LogEntry
  176. object.
  177. """
  178. logentry = LogEntry.objects.get(content_type__model__iexact="article")
  179. edited_obj = logentry.get_edited_object()
  180. self.assertEqual(logentry.object_id, str(edited_obj.pk))
  181. def test_logentry_get_admin_url(self):
  182. """
  183. LogEntry.get_admin_url returns a URL to edit the entry's object or
  184. None for nonexistent (possibly deleted) models.
  185. """
  186. logentry = LogEntry.objects.get(content_type__model__iexact="article")
  187. expected_url = reverse(
  188. "admin:admin_utils_article_change", args=(quote(self.a1.pk),)
  189. )
  190. self.assertEqual(logentry.get_admin_url(), expected_url)
  191. self.assertIn("article/%d/change/" % self.a1.pk, logentry.get_admin_url())
  192. logentry.content_type.model = "nonexistent"
  193. self.assertIsNone(logentry.get_admin_url())
  194. def test_logentry_unicode(self):
  195. log_entry = LogEntry()
  196. log_entry.action_flag = ADDITION
  197. self.assertTrue(str(log_entry).startswith("Added "))
  198. log_entry.action_flag = CHANGE
  199. self.assertTrue(str(log_entry).startswith("Changed "))
  200. log_entry.action_flag = DELETION
  201. self.assertTrue(str(log_entry).startswith("Deleted "))
  202. # Make sure custom action_flags works
  203. log_entry.action_flag = 4
  204. self.assertEqual(str(log_entry), "LogEntry Object")
  205. def test_logentry_repr(self):
  206. logentry = LogEntry.objects.first()
  207. self.assertEqual(repr(logentry), str(logentry.action_time))
  208. def test_log_action(self):
  209. content_type_pk = ContentType.objects.get_for_model(Article).pk
  210. log_entry = LogEntry.objects.log_action(
  211. self.user.pk,
  212. content_type_pk,
  213. self.a1.pk,
  214. repr(self.a1),
  215. CHANGE,
  216. change_message="Changed something else",
  217. )
  218. self.assertEqual(log_entry, LogEntry.objects.latest("id"))
  219. def test_recentactions_without_content_type(self):
  220. """
  221. If a LogEntry is missing content_type it will not display it in span
  222. tag under the hyperlink.
  223. """
  224. response = self.client.get(reverse("admin:index"))
  225. link = reverse("admin:admin_utils_article_change", args=(quote(self.a1.pk),))
  226. should_contain = """<a href="%s">%s</a>""" % (
  227. escape(link),
  228. escape(repr(self.a1)),
  229. )
  230. self.assertContains(response, should_contain)
  231. should_contain = "Article"
  232. self.assertContains(response, should_contain)
  233. logentry = LogEntry.objects.get(content_type__model__iexact="article")
  234. # If the log entry doesn't have a content type it should still be
  235. # possible to view the Recent Actions part (#10275).
  236. logentry.content_type = None
  237. logentry.save()
  238. should_contain = should_contain.encode()
  239. counted_presence_before = response.content.count(should_contain)
  240. response = self.client.get(reverse("admin:index"))
  241. counted_presence_after = response.content.count(should_contain)
  242. self.assertEqual(counted_presence_before - 1, counted_presence_after)
  243. def test_proxy_model_content_type_is_used_for_log_entries(self):
  244. """
  245. Log entries for proxy models should have the proxy model's contenttype
  246. (#21084).
  247. """
  248. proxy_content_type = ContentType.objects.get_for_model(
  249. ArticleProxy, for_concrete_model=False
  250. )
  251. post_data = {
  252. "site": self.site.pk,
  253. "title": "Foo",
  254. "hist": "Bar",
  255. "created_0": "2015-12-25",
  256. "created_1": "00:00",
  257. }
  258. changelist_url = reverse("admin:admin_utils_articleproxy_changelist")
  259. # add
  260. proxy_add_url = reverse("admin:admin_utils_articleproxy_add")
  261. response = self.client.post(proxy_add_url, post_data)
  262. self.assertRedirects(response, changelist_url)
  263. proxy_addition_log = LogEntry.objects.latest("id")
  264. self.assertEqual(proxy_addition_log.action_flag, ADDITION)
  265. self.assertEqual(proxy_addition_log.content_type, proxy_content_type)
  266. # change
  267. article_id = proxy_addition_log.object_id
  268. proxy_change_url = reverse(
  269. "admin:admin_utils_articleproxy_change", args=(article_id,)
  270. )
  271. post_data["title"] = "New"
  272. response = self.client.post(proxy_change_url, post_data)
  273. self.assertRedirects(response, changelist_url)
  274. proxy_change_log = LogEntry.objects.latest("id")
  275. self.assertEqual(proxy_change_log.action_flag, CHANGE)
  276. self.assertEqual(proxy_change_log.content_type, proxy_content_type)
  277. # delete
  278. proxy_delete_url = reverse(
  279. "admin:admin_utils_articleproxy_delete", args=(article_id,)
  280. )
  281. response = self.client.post(proxy_delete_url, {"post": "yes"})
  282. self.assertRedirects(response, changelist_url)
  283. proxy_delete_log = LogEntry.objects.latest("id")
  284. self.assertEqual(proxy_delete_log.action_flag, DELETION)
  285. self.assertEqual(proxy_delete_log.content_type, proxy_content_type)
  286. def test_action_flag_choices(self):
  287. tests = ((1, "Addition"), (2, "Change"), (3, "Deletion"))
  288. for action_flag, display_name in tests:
  289. with self.subTest(action_flag=action_flag):
  290. log = LogEntry(action_flag=action_flag)
  291. self.assertEqual(log.get_action_flag_display(), display_name)
  292. def test_hook_get_log_entries(self):
  293. LogEntry.objects.log_action(
  294. self.user.pk,
  295. ContentType.objects.get_for_model(Article).pk,
  296. self.a1.pk,
  297. "Article changed",
  298. CHANGE,
  299. change_message="Article changed message",
  300. )
  301. c1 = Car.objects.create()
  302. LogEntry.objects.log_action(
  303. self.user.pk,
  304. ContentType.objects.get_for_model(Car).pk,
  305. c1.pk,
  306. "Car created",
  307. ADDITION,
  308. change_message="Car created message",
  309. )
  310. response = self.client.get(reverse("admin:index"))
  311. self.assertContains(response, "Article changed")
  312. self.assertContains(response, "Car created")
  313. # site "custom_admin" only renders log entries of registered models
  314. response = self.client.get(reverse("custom_admin:index"))
  315. self.assertContains(response, "Article changed")
  316. self.assertNotContains(response, "Car created")