Merge pull request #1131 from brettkail-wk/validate-css-url-token

Validate CSS url tokens

arelle/plugin/validate/ESEF/ESEF_Current/ValidateXbrlFinally.py

@@ -1079,3 +1079,7 @@ def validateCssUrlContent(cssRules: List[Any], normalizedUri:str, modelXbrl: Mod
                     css_rule_url = css_rule.arguments[0].value  # url or base64
                     evaluatedMsg = _('On line {line}').format(line=1) #css_element.source_line)
                     validateImage(normalizedUri, css_rule_url, modelXbrl, val, elt, evaluatedMsg, contentOtherThanXHTMLGuidance)
+        elif isinstance(css_rule, tinycss2.ast.URLToken):
+            value = css_rule.value
+            evaluatedMsg = _('On line {line}').format(line=1) #css_element.source_line)
+            validateImage(normalizedUri, value, modelXbrl, val, elt, evaluatedMsg, contentOtherThanXHTMLGuidance)

tests/unit_tests/arelle/plugin/validate/ESEF/ESEF_Current/test_validate_css_url.py

@@ -0,0 +1,19 @@
+from unittest import TestCase
+from unittest.mock import MagicMock
+
+from arelle.plugin.validate.ESEF.ESEF_Current.ValidateXbrlFinally import validateCssUrl
+
+class TestValidateCssUrl(TestCase):
+    def test_url_function(self) -> None:
+        modelXbrl = MagicMock()
+        validateCssUrl(
+            '* { background: url("http://example.com") }',
+            MagicMock(), modelXbrl, MagicMock(), MagicMock(), MagicMock())
+        self.assertEqual(modelXbrl.error.call_args.args[0], 'ESEF.4.1.6.xHTMLDocumentContainsExternalReferences')
+
+    def test_url_token(self) -> None:
+        modelXbrl = MagicMock()
+        validateCssUrl(
+            '* { background: url(http://example.com) }',
+            MagicMock(), modelXbrl, MagicMock(), MagicMock(), MagicMock())
+        self.assertEqual(modelXbrl.error.call_args.args[0], 'ESEF.4.1.6.xHTMLDocumentContainsExternalReferences')