test_logentry.py 17 KB

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