Fix up unnecessarily split strings, which were a side-effect of refor…

…matting
mozilla · Jan 24, 2024 · 0a83866 · 0a83866
1 parent f38dd2f
commit 0a83866
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/csp/tests/test_jinja_extension.py b/csp/tests/test_jinja_extension.py
@@ -42,7 +42,7 @@ def test_async_attribute_with_falsey(self):
                 var hello='world';
             {% endscript %}"""
 
-        expected = '<script nonce="{}" id="jeff" async=false>' "var hello='world';" "</script>"
+        expected = '<script nonce="{}" id="jeff" async=false>var hello=\'world\';</script>'
 
         self.assert_template_eq(*self.process_templates(tpl, expected))
 
@@ -52,7 +52,7 @@ def test_async_attribute_with_truthy(self):
                 var hello='world';
             {% endscript %}"""
 
-        expected = '<script nonce="{}" id="jeff" async>' "var hello='world';" "</script>"
+        expected = '<script nonce="{}" id="jeff" async>var hello=\'world\';</script>'
 
         self.assert_template_eq(*self.process_templates(tpl, expected))
 
@@ -66,7 +66,7 @@ def test_nested_script_tags_are_removed(self):
                 </script>
             {% endscript %}"""
 
-        expected = "<script" ' nonce="{}" id="jeff" type="application/javascript" defer>' "var hello='world';</script>"
+        expected = '<script nonce="{}" id="jeff" type="application/javascript" defer>var hello=\'world\';</script>'
 
         self.assert_template_eq(*self.process_templates(tpl, expected))
 
@@ -82,6 +82,6 @@ def test_regex_captures_script_content_including_brackets(self):
             {% endscript %}
             """
 
-        expected = '<script nonce="{}">' 'let capture_text = "<script></script>"' "</script>"
+        expected = '<script nonce="{}">let capture_text = "<script></script>"</script>'
 
         self.assert_template_eq(*self.process_templates(tpl, expected))
diff --git a/csp/tests/test_templatetags.py b/csp/tests/test_templatetags.py
@@ -29,7 +29,7 @@ def test_script_tag_sets_attrs_correctly(self):
                 var hello='world';
             {% endscript %}"""
 
-        expected = "<script" ' nonce="{}" id="jeff" type="application/javascript" defer>' "var hello='world';</script>"
+        expected = '<script nonce="{}" id="jeff" type="application/javascript" defer>var hello=\'world\';</script>'
 
         self.assert_template_eq(*self.process_templates(tpl, expected))
 
@@ -65,7 +65,7 @@ def test_nested_script_tags_are_removed(self):
                 </script>
             {% endscript %}"""
 
-        expected = "<script" ' nonce="{}" id="jeff" type="application/javascript" defer>' "var hello='world';</script>"
+        expected = '<script nonce="{}" id="jeff" type="application/javascript" defer>var hello=\'world\';</script>'
 
         self.assert_template_eq(*self.process_templates(tpl, expected))
 
@@ -82,6 +82,6 @@ def test_regex_captures_script_content_including_brackets(self):
             {% endscript %}
             """
 
-        expected = '<script nonce="{}">' 'let capture_text = "<script></script>"' "</script>"
+        expected = '<script nonce="{}">let capture_text = "<script></script>"</script>'
 
         self.assert_template_eq(*self.process_templates(tpl, expected))