test_creating_and_deleting_css_files.py 12 KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401
  1. import pytest
  2. from django.urls import reverse
  3. from ....cache.test import assert_invalidates_cache
  4. from ....test import assert_contains, assert_has_error_message
  5. from ... import THEME_CACHE
  6. @pytest.fixture
  7. def create_link(theme):
  8. return reverse(
  9. "misago:admin:appearance:themes:new-css-file", kwargs={"pk": theme.pk}
  10. )
  11. @pytest.fixture
  12. def edit_link(theme, css):
  13. return reverse(
  14. "misago:admin:appearance:themes:edit-css-file",
  15. kwargs={"pk": theme.pk, "css_pk": css.pk},
  16. )
  17. @pytest.fixture
  18. def data():
  19. return {"name": "test.css", "source": ".page-header { padding: 0}"}
  20. def test_css_creation_form_is_displayed(admin_client, create_link):
  21. response = admin_client.get(create_link)
  22. assert response.status_code == 200
  23. assert_contains(response, "New CSS")
  24. def test_css_can_be_created(theme, admin_client, create_link, data):
  25. admin_client.post(create_link, data)
  26. assert theme.css.exists()
  27. def test_css_is_created_with_entered_name(theme, admin_client, create_link, data):
  28. admin_client.post(create_link, data)
  29. assert theme.css.last().name == data["name"]
  30. def test_created_source_file_contains_css_entered_by_user(
  31. theme, admin_client, create_link, data
  32. ):
  33. admin_client.post(create_link, data)
  34. css = theme.css.last()
  35. assert css.source_file.read().decode("utf-8") == data["source"]
  36. def test_source_file_is_created_in_theme_directory(
  37. theme, admin_client, create_link, data
  38. ):
  39. admin_client.post(create_link, data)
  40. css = theme.css.last()
  41. assert theme.dirname in str(css.source_file)
  42. def test_created_source_file_name_starts_with_asset_name(
  43. theme, admin_client, create_link, data
  44. ):
  45. admin_client.post(create_link, data)
  46. css = theme.css.last()
  47. source_filename = str(css.source_file).split("/")[-1]
  48. assert source_filename.startswith("test.")
  49. def test_created_source_file_has_css_extension(theme, admin_client, create_link, data):
  50. admin_client.post(create_link, data)
  51. css = theme.css.last()
  52. source_filename = str(css.source_file).split("/")[-1]
  53. assert source_filename.endswith(".css")
  54. def test_created_source_file_is_hashed(theme, admin_client, create_link, data):
  55. admin_client.post(create_link, data)
  56. css = theme.css.last()
  57. source_filename = str(css.source_file).split("/")[-1]
  58. assert ".%s." % css.source_hash in source_filename
  59. def test_css_creation_fails_if_name_is_not_given(
  60. theme, admin_client, create_link, data
  61. ):
  62. data["name"] = ""
  63. admin_client.post(create_link, data)
  64. assert not theme.css.exists()
  65. def test_css_creation_fails_if_name_is_not_valid(
  66. theme, admin_client, create_link, data
  67. ):
  68. data["name"] = "missing-extension"
  69. admin_client.post(create_link, data)
  70. assert not theme.css.exists()
  71. def test_css_creation_fails_if_name_is_already_taken_by_other_css_in_theme(
  72. theme, admin_client, create_link, data, css
  73. ):
  74. data["name"] = css.name
  75. admin_client.post(create_link, data)
  76. assert theme.css.count() == 1
  77. def test_css_name_usage_check_passess_if_name_is_used_by_other_theme_css(
  78. other_theme, admin_client, data, css
  79. ):
  80. create_link = reverse(
  81. "misago:admin:appearance:themes:new-css-file", kwargs={"pk": other_theme.pk}
  82. )
  83. data["name"] = css.name
  84. admin_client.post(create_link, data)
  85. assert other_theme.css.exists()
  86. def test_css_creation_fails_if_source_is_not_given(
  87. theme, admin_client, create_link, data
  88. ):
  89. data["source"] = ""
  90. admin_client.post(create_link, data)
  91. assert not theme.css.exists()
  92. def test_css_file_without_url_is_created_without_rebuilding_flag(
  93. theme, admin_client, create_link, data
  94. ):
  95. admin_client.post(create_link, data)
  96. css = theme.css.last()
  97. assert not css.source_needs_building
  98. def test_css_file_with_image_url_is_created_with_rebuilding_flag(
  99. theme, admin_client, create_link, data, image
  100. ):
  101. data["source"] = "body { background-image: url(/static/%s); }" % image.name
  102. admin_client.post(create_link, data)
  103. css = theme.css.last()
  104. assert css.source_needs_building
  105. def test_creating_css_file_without_image_url_doesnt_trigger_single_css_file_rebuild(
  106. theme, admin_client, create_link, data, image, mock_build_single_theme_css
  107. ):
  108. admin_client.post(create_link, data)
  109. mock_build_single_theme_css.assert_not_called()
  110. def test_creating_css_file_with_image_url_triggers_single_css_file_rebuild(
  111. theme, admin_client, create_link, data, image, mock_build_single_theme_css
  112. ):
  113. data["source"] = "body { background-image: url(/static/%s); }" % image.name
  114. admin_client.post(create_link, data)
  115. css = theme.css.last()
  116. mock_build_single_theme_css.assert_called_once_with(css.pk)
  117. def test_css_file_is_created_with_correct_order(
  118. theme, admin_client, create_link, css_link, data
  119. ):
  120. admin_client.post(create_link, data)
  121. css = theme.css.get(name=data["name"])
  122. assert css.order == 1
  123. def test_creating_css_file_invalidates_theme_cache(admin_client, create_link, data):
  124. with assert_invalidates_cache(THEME_CACHE):
  125. admin_client.post(create_link, data)
  126. def test_error_message_is_set_if_user_attempts_to_create_css_in_default_theme(
  127. default_theme, admin_client
  128. ):
  129. create_link = reverse(
  130. "misago:admin:appearance:themes:new-css-file", kwargs={"pk": default_theme.pk}
  131. )
  132. response = admin_client.get(create_link)
  133. assert_has_error_message(response)
  134. def test_error_message_is_set_if_user_attempts_to_create_css_in_nonexisting_theme(
  135. nonexisting_theme, admin_client
  136. ):
  137. create_link = reverse(
  138. "misago:admin:appearance:themes:new-css-file",
  139. kwargs={"pk": nonexisting_theme.pk},
  140. )
  141. response = admin_client.get(create_link)
  142. assert_has_error_message(response)
  143. def test_css_creation_form_redirects_user_to_edition_after_creation(
  144. theme, admin_client, create_link, data
  145. ):
  146. data["stay"] = "1"
  147. response = admin_client.post(create_link, data)
  148. assert response["location"] == reverse(
  149. "misago:admin:appearance:themes:edit-css-file",
  150. kwargs={"pk": theme.pk, "css_pk": theme.css.last().pk},
  151. )
  152. def test_css_edition_form_is_displayed(admin_client, edit_link, css):
  153. response = admin_client.get(edit_link)
  154. assert response.status_code == 200
  155. assert_contains(response, css.name)
  156. def test_css_edition_form_contains_source_file_contents(admin_client, edit_link, css):
  157. response = admin_client.get(edit_link)
  158. assert_contains(response, css.source_file.read().decode("utf-8"))
  159. def test_css_name_can_be_changed(admin_client, edit_link, css, data):
  160. data["name"] = "new-name.css"
  161. admin_client.post(edit_link, data)
  162. css.refresh_from_db()
  163. assert css.name == data["name"]
  164. def test_css_name_change_also_changes_source_file_name(
  165. admin_client, edit_link, css, data
  166. ):
  167. data["name"] = "new-name.css"
  168. admin_client.post(edit_link, data)
  169. css.refresh_from_db()
  170. assert "new-name" in str(css.source_file)
  171. def test_css_source_can_be_changed(admin_client, edit_link, css, data):
  172. data["source"] = ".misago-footer { display: none; }"
  173. admin_client.post(edit_link, data)
  174. css.refresh_from_db()
  175. assert css.source_file.read().decode("utf-8") == data["source"]
  176. def test_changing_css_source_also_changes_source_hash(
  177. admin_client, edit_link, css, data
  178. ):
  179. original_hash = css.source_hash
  180. data["source"] = ".misago-footer { display: none; }"
  181. admin_client.post(edit_link, data)
  182. css.refresh_from_db()
  183. assert css.source_hash != original_hash
  184. def test_changing_css_source_also_changes_hash_in_filename(
  185. admin_client, edit_link, css, data
  186. ):
  187. original_hash = css.source_hash
  188. data["source"] = ".misago-footer { display: none; }"
  189. admin_client.post(edit_link, data)
  190. css.refresh_from_db()
  191. assert original_hash not in str(css.source_file)
  192. assert css.source_hash in str(css.source_file)
  193. def test_hash_stays_same_if_source_is_not_changed(admin_client, edit_link, css, data):
  194. original_hash = css.source_hash
  195. data["name"] = "changed.css"
  196. data["source"] = css.source_file.read().decode("utf-8")
  197. admin_client.post(edit_link, data)
  198. css.refresh_from_db()
  199. assert original_hash == css.source_hash
  200. def test_file_is_not_updated_if_form_data_has_no_changes(
  201. admin_client, edit_link, css, data
  202. ):
  203. original_source_file = str(css.source_file)
  204. data["name"] = css.name
  205. data["source"] = css.source_file.read().decode("utf-8")
  206. admin_client.post(edit_link, data)
  207. css.refresh_from_db()
  208. assert original_source_file == str(css.source_file)
  209. def test_adding_image_url_to_edited_file_sets_rebuilding_flag(
  210. theme, admin_client, edit_link, css, data, image
  211. ):
  212. data["source"] = "body { background-image: url(/static/%s); }" % image.name
  213. admin_client.post(edit_link, data)
  214. css.refresh_from_db()
  215. assert css.source_needs_building
  216. def test_removing_url_from_edited_file_removes_rebuilding_flag(
  217. theme, admin_client, edit_link, css, data
  218. ):
  219. css.source_needs_building = True
  220. css.save()
  221. data["source"] = "body { background-image: none; }"
  222. admin_client.post(edit_link, data)
  223. css.refresh_from_db()
  224. assert not css.source_needs_building
  225. def test_adding_image_url_to_edited_file_triggers_single_css_file_rebuild(
  226. theme, admin_client, edit_link, css, data, image, mock_build_single_theme_css
  227. ):
  228. data["source"] = "body { background-image: url(/static/%s); }" % image.name
  229. admin_client.post(edit_link, data)
  230. mock_build_single_theme_css.assert_called_once_with(css.pk)
  231. def test_removing_url_from_edited_file_deosnt_trigger_single_css_file_rebuild(
  232. theme, admin_client, edit_link, css, data, mock_build_single_theme_css
  233. ):
  234. data["source"] = "body { background-image: none; }"
  235. admin_client.post(edit_link, data)
  236. mock_build_single_theme_css.assert_not_called()
  237. def test_css_order_stays_the_same_after_edit(admin_client, edit_link, css, data):
  238. original_order = css.order
  239. data["name"] = "changed.css"
  240. admin_client.post(edit_link, data)
  241. css.refresh_from_db()
  242. assert css.order == original_order
  243. def test_editing_css_file_invalidates_theme_cache(admin_client, edit_link, css, data):
  244. with assert_invalidates_cache(THEME_CACHE):
  245. admin_client.post(edit_link, data)
  246. def test_css_edit_form_redirects_user_to_edition_after_saving(
  247. theme, admin_client, edit_link, css, data
  248. ):
  249. data["stay"] = "1"
  250. response = admin_client.post(edit_link, data)
  251. assert response["location"] == edit_link
  252. def test_error_message_is_set_if_user_attempts_to_edit_css_file_in_default_theme(
  253. default_theme, admin_client
  254. ):
  255. edit_link = reverse(
  256. "misago:admin:appearance:themes:edit-css-file",
  257. kwargs={"pk": default_theme.pk, "css_pk": 1},
  258. )
  259. response = admin_client.get(edit_link)
  260. assert_has_error_message(response)
  261. def test_error_message_is_set_if_user_attempts_to_edit_css_file_in_nonexisting_theme(
  262. nonexisting_theme, admin_client
  263. ):
  264. edit_link = reverse(
  265. "misago:admin:appearance:themes:edit-css-file",
  266. kwargs={"pk": nonexisting_theme.pk, "css_pk": 1},
  267. )
  268. response = admin_client.get(edit_link)
  269. assert_has_error_message(response)
  270. def test_error_message_is_set_if_user_attempts_to_edit_css_belonging_to_other_theme(
  271. other_theme, admin_client, css
  272. ):
  273. edit_link = reverse(
  274. "misago:admin:appearance:themes:edit-css-file",
  275. kwargs={"pk": other_theme.pk, "css_pk": css.pk},
  276. )
  277. response = admin_client.get(edit_link)
  278. assert_has_error_message(response)
  279. def test_error_message_is_set_if_user_attempts_to_edit_nonexisting_css(
  280. theme, admin_client
  281. ):
  282. edit_link = reverse(
  283. "misago:admin:appearance:themes:edit-css-file",
  284. kwargs={"pk": theme.pk, "css_pk": 1},
  285. )
  286. response = admin_client.get(edit_link)
  287. assert_has_error_message(response)
  288. def test_error_message_is_set_if_user_attempts_to_edit_css_link_with_file_form(
  289. theme, admin_client, css_link
  290. ):
  291. edit_link = reverse(
  292. "misago:admin:appearance:themes:edit-css-file",
  293. kwargs={"pk": theme.pk, "css_pk": css_link.pk},
  294. )
  295. response = admin_client.get(edit_link)
  296. assert_has_error_message(response)