From 69e0f29a304d21586e69887f2f96f629be564311 Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Thu, 2 Nov 2023 21:42:14 +0000 Subject: [PATCH 1/2] Bump django from 4.1.10 to 4.1.13 Bumps [django](https://github.com/django/django) from 4.1.10 to 4.1.13. - [Commits](https://github.com/django/django/compare/4.1.10...4.1.13) --- updated-dependencies: - dependency-name: django dependency-type: indirect ... Signed-off-by: dependabot[bot] --- requirements.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/requirements.txt b/requirements.txt index a09c4b8d..14792843 100644 --- a/requirements.txt +++ b/requirements.txt @@ -14,7 +14,7 @@ click==8.0.3 # via pip-tools distlib==0.3.4 # via virtualenv -django==4.1.10 +django==4.1.13 # via # django-classy-tags # django-cms From 57bca3188963db68fb91fa1f65df886df30fdb93 Mon Sep 17 00:00:00 2001 From: "pre-commit-ci[bot]" <66853113+pre-commit-ci[bot]@users.noreply.github.com> Date: Thu, 2 Nov 2023 21:42:28 +0000 Subject: [PATCH 2/2] ci: auto fixes from pre-commit hooks for more information, see https://pre-commit.ci --- .github/PULL_REQUEST_TEMPLATE.md | 2 +- aldryn_config.py | 14 ++++++------- tests/settings.py | 25 ++++++++++++---------- tests/test_migrations.py | 17 +++++++-------- tests/test_models.py | 1 - tests/test_plugins.py | 34 ++++++++++++++++++++++-------- tests/test_templatetags.py | 36 +++++++++++++------------------- 7 files changed, 70 insertions(+), 59 deletions(-) diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md index dc334a20..8f0dc786 100644 --- a/.github/PULL_REQUEST_TEMPLATE.md +++ b/.github/PULL_REQUEST_TEMPLATE.md @@ -26,5 +26,5 @@ Use 'x' to check each item: [x] I have ... * [ ] I have opened this pull request against ``master`` * [ ] I have added or modified the tests when changing logic * [ ] I have followed [the conventional commits guidelines](https://www.conventionalcommits.org/) to add meaningful information into the changelog -* [ ] I have read the [contribution guidelines ](https://github.com/django-cms/django-cms/blob/develop/CONTRIBUTING.rst) and I have joined #workgroup-pr-review on +* [ ] I have read the [contribution guidelines ](https://github.com/django-cms/django-cms/blob/develop/CONTRIBUTING.rst) and I have joined #workgroup-pr-review on [Slack](https://www.django-cms.org/slack) to find a “pr review buddy” who is going to review my pull request. diff --git a/aldryn_config.py b/aldryn_config.py index 3e532b2c..981c5e67 100644 --- a/aldryn_config.py +++ b/aldryn_config.py @@ -11,16 +11,16 @@ class Form(forms.BaseForm): required=False, ) enable_search = forms.CheckboxField( - 'Enable snippet content to be searchable.', + "Enable snippet content to be searchable.", required=False, initial=False, ) def to_settings(self, data, settings): - if data['editor_theme']: - settings['DJANGOCMS_SNIPPET_THEME'] = data['editor_theme'] - if data['editor_mode']: - settings['DJANGOCMS_SNIPPET_MODE'] = data['editor_mode'] - if data['enable_search']: - settings['DJANGOCMS_SNIPPET_SEARCH'] = data['enable_search'] + if data["editor_theme"]: + settings["DJANGOCMS_SNIPPET_THEME"] = data["editor_theme"] + if data["editor_mode"]: + settings["DJANGOCMS_SNIPPET_MODE"] = data["editor_mode"] + if data["enable_search"]: + settings["DJANGOCMS_SNIPPET_SEARCH"] = data["enable_search"] return settings diff --git a/tests/settings.py b/tests/settings.py index 5656cebe..763998f2 100644 --- a/tests/settings.py +++ b/tests/settings.py @@ -1,23 +1,26 @@ #!/usr/bin/env python HELPER_SETTINGS = { - 'INSTALLED_APPS': [ - 'tests.utils', + "INSTALLED_APPS": [ + "tests.utils", ], - 'CMS_LANGUAGES': { - 1: [{ - 'code': 'en', - 'name': 'English', - }] + "CMS_LANGUAGES": { + 1: [ + { + "code": "en", + "name": "English", + } + ] }, - 'LANGUAGE_CODE': 'en', - 'ALLOWED_HOSTS': ['localhost'], + "LANGUAGE_CODE": "en", + "ALLOWED_HOSTS": ["localhost"], } def run(): from app_helper import runner - runner.cms('djangocms_snippet') + runner.cms("djangocms_snippet") -if __name__ == '__main__': + +if __name__ == "__main__": run() diff --git a/tests/test_migrations.py b/tests/test_migrations.py index 29c4a94b..fe27c860 100644 --- a/tests/test_migrations.py +++ b/tests/test_migrations.py @@ -7,24 +7,23 @@ class MigrationTestCase(TestCase): - @override_settings(MIGRATION_MODULES={}) def test_for_missing_migrations(self): output = StringIO() options = { - 'interactive': False, - 'dry_run': True, - 'stdout': output, - 'check_changes': True, + "interactive": False, + "dry_run": True, + "stdout": output, + "check_changes": True, } try: - call_command('makemigrations', **options) + call_command("makemigrations", **options) except SystemExit as e: status_code = str(e) else: # the "no changes" exit code is 0 - status_code = '0' + status_code = "0" - if status_code == '1': - self.fail('There are missing migrations:\n {}'.format(output.getvalue())) + if status_code == "1": + self.fail(f"There are missing migrations:\n {output.getvalue()}") diff --git a/tests/test_models.py b/tests/test_models.py index 077e0010..127a0a0d 100644 --- a/tests/test_models.py +++ b/tests/test_models.py @@ -4,7 +4,6 @@ class SnippetModelTestCase(TestCase): - def setUp(self): pass diff --git a/tests/test_plugins.py b/tests/test_plugins.py index 6e4e6767..58d92af9 100644 --- a/tests/test_plugins.py +++ b/tests/test_plugins.py @@ -5,7 +5,6 @@ class SnippetPluginsTestCase(CMSTestCase): - def setUp(self): self.language = "en" self.home = create_page( @@ -29,7 +28,9 @@ def tearDown(self): self.superuser.delete() def test_html_rendering(self): - request_url = self.page.get_absolute_url(self.language) + "?toolbar_off=true" + request_url = ( + self.page.get_absolute_url(self.language) + "?toolbar_off=true" + ) snippet = Snippet.objects.create( name="plugin_snippet", html="

Hello World

", @@ -52,7 +53,9 @@ def test_html_rendering(self): self.assertIn(b"

Hello World

", response.content) def test_failing_html_rendering(self): - request_url = self.page.get_absolute_url(self.language) + "?toolbar_off=true" + request_url = ( + self.page.get_absolute_url(self.language) + "?toolbar_off=true" + ) snippet = Snippet.objects.create( name="plugin_snippet", html="{% import weirdness %}", @@ -70,10 +73,14 @@ def test_failing_html_rendering(self): response = self.client.get(request_url) self.assertContains(response, "Invalid block tag on line 1") - self.assertContains(response, "Did you forget to register or load this tag?") + self.assertContains( + response, "Did you forget to register or load this tag?" + ) def test_template_rendering(self): - request_url = self.page.get_absolute_url(self.language) + "?toolbar_off=true" + request_url = ( + self.page.get_absolute_url(self.language) + "?toolbar_off=true" + ) template = "snippet.html" snippet = Snippet.objects.create( name="plugin_snippet", @@ -94,13 +101,22 @@ def test_template_rendering(self): with self.login_user_context(self.superuser): response = self.client.get(request_url) - self.assertNotIn("Template {} does not exist".format(template).encode(), response.content) - self.assertNotIn(b"context must be a dict rather than Context", response.content) - self.assertNotIn(b"context must be a dict rather than PluginContext", response.content) + self.assertNotIn( + f"Template {template} does not exist".encode(), response.content + ) + self.assertNotIn( + b"context must be a dict rather than Context", response.content + ) + self.assertNotIn( + b"context must be a dict rather than PluginContext", + response.content, + ) self.assertContains(response, "

Hello World Template

") def test_failing_template_rendering(self): - request_url = self.page.get_absolute_url(self.language) + "?toolbar_off=true" + request_url = ( + self.page.get_absolute_url(self.language) + "?toolbar_off=true" + ) template = "some_template" snippet = Snippet.objects.create( name="plugin_snippet", diff --git a/tests/test_templatetags.py b/tests/test_templatetags.py index b1bb45b5..344d74f9 100644 --- a/tests/test_templatetags.py +++ b/tests/test_templatetags.py @@ -7,7 +7,6 @@ class SnippetTemplateTagTestCase(TestCase): - def test_html_rendered(self): snippet = Snippet.objects.create( name="test snippet", @@ -20,17 +19,15 @@ def test_html_rendered(self): context = Context({"title": "world"}) template_to_render = Template( - '{% load snippet_tags %}' - '{% snippet_fragment "test_snippet" %}' + "{% load snippet_tags %}" '{% snippet_fragment "test_snippet" %}' ) rendered_template = template_to_render.render(context) - self.assertInHTML('

hello world

', rendered_template) + self.assertInHTML("

hello world

", rendered_template) # test html errors context = Context({"title": "world"}) template_to_render = Template( - '{% load snippet_tags %}' - '{% snippet_fragment "test_snippet_2" %}' + "{% load snippet_tags %}" '{% snippet_fragment "test_snippet_2" %}' ) with self.assertRaises(ObjectDoesNotExist): # Snippet matching query does not exist. @@ -50,29 +47,27 @@ def test_template_rendered(self): # use a string to identify context = Context({}) template_to_render = Template( - '{% load snippet_tags %}' - '{% snippet_fragment "test_snippet" %}' + "{% load snippet_tags %}" '{% snippet_fragment "test_snippet" %}' ) rendered_template = template_to_render.render(context) - self.assertInHTML('

Hello World Template

', rendered_template) + self.assertInHTML("

Hello World Template

", rendered_template) # use an id to identify context = Context({}) template_to_render = Template( - '{% load snippet_tags %}' - '{% snippet_fragment 1 %}' + "{% load snippet_tags %}" "{% snippet_fragment 1 %}" ) rendered_template = template_to_render.render(context) - self.assertInHTML('

Hello World Template

', rendered_template) + self.assertInHTML("

Hello World Template

", rendered_template) # tests "or" functionality context = Context({}) template_to_render = Template( - '{% load snippet_tags %}' + "{% load snippet_tags %}" '{% snippet_fragment "test_snippet_1" or %}

hello world

{% endsnippet_fragment %}' ) rendered_template = template_to_render.render(context) - self.assertInHTML('

hello world

', rendered_template) + self.assertInHTML("

hello world

", rendered_template) def test_template_errors(self): template = "does_not_exist.html" @@ -87,16 +82,16 @@ def test_template_errors(self): context = Context({}) template_to_render = Template( - '{% load snippet_tags %}' - '{% snippet_fragment "test_snippet" %}' + "{% load snippet_tags %}" '{% snippet_fragment "test_snippet" %}' ) rendered_template = template_to_render.render(context) - self.assertIn('Template does_not_exist.html does not exist.', rendered_template) + self.assertIn( + "Template does_not_exist.html does not exist.", rendered_template + ) context = Context({}) template_to_render = Template( - '{% load snippet_tags %}' - '{% snippet_fragment "test_snippet_1" %}' + "{% load snippet_tags %}" '{% snippet_fragment "test_snippet_1" %}' ) with self.assertRaises(ObjectDoesNotExist): # Snippet object does not exist @@ -106,6 +101,5 @@ def test_template_errors(self): with self.assertRaises(TemplateSyntaxError): # You need to specify at least a "snippet" ID, slug or instance template_to_render = Template( - '{% load snippet_tags %}' - '{% snippet_fragment %}' + "{% load snippet_tags %}" "{% snippet_fragment %}" )