.

Author: SimonCropp

docs/verify-xml.md

@@ -66,7 +66,7 @@ public Task XmlIgnoreMember() =>
     VerifyXml(xml)
         .IgnoreMember("node");
 ```
-<sup><a href='/src/Verify.Tests/XmlTests.cs#L30-L37' title='Snippet source file'>snippet source</a> | <a href='#snippet-XmlIgnoreMember' title='Start of snippet'>anchor</a></sup>
+<sup><a href='/src/Verify.Tests/XmlTests.cs#L46-L53' title='Snippet source file'>snippet source</a> | <a href='#snippet-XmlIgnoreMember' title='Start of snippet'>anchor</a></sup>
 <!-- endSnippet -->
 
 Will produce
@@ -92,7 +92,7 @@ public Task XmlScrubMember() =>
     VerifyXml(xml)
         .ScrubMember("node");
 ```
-<sup><a href='/src/Verify.Tests/XmlTests.cs#L39-L46' title='Snippet source file'>snippet source</a> | <a href='#snippet-XmlScrubMember' title='Start of snippet'>anchor</a></sup>
+<sup><a href='/src/Verify.Tests/XmlTests.cs#L55-L62' title='Snippet source file'>snippet source</a> | <a href='#snippet-XmlScrubMember' title='Start of snippet'>anchor</a></sup>
 <!-- endSnippet -->
 
 Will produce

src/Verify.Tests/XmlTests.CData.verified.xml

@@ -0,0 +1 @@
+

src/Verify.Tests/XmlTests.cs

@@ -27,6 +27,22 @@ public Task NoDeclaration() =>
             </body>
             """);
 
+    [Fact]
+    public Task CData() =>
+        VerifyXml(
+            """
+            <?xml version="1.0" encoding="utf-8" standalone="yes"?>
+              <person><![CDATA[name is John Doe]]></person>
+            """);
+
+    [Fact]
+    public Task CDataMix() =>
+        VerifyXml(
+            """
+            <?xml version="1.0" encoding="utf-8" standalone="yes"?>
+              <person><![CDATA[name is John Doe]]>value</person>
+            """);
+
     #region XmlIgnoreMember
 
     [Fact]

src/Verify/Verifier/InnerVerifier_Xml.cs

@@ -63,52 +63,31 @@ Task<VerifyResult> VerifyXml(XContainer? target)
         }
 
         var serialization = settings.serialization;
-        var nodes = target
+        var elements = target
             .Descendants()
             .ToList();
-        foreach (var node in nodes)
+        foreach (var element in elements)
         {
-            if (serialization.TryGetScrubOrIgnoreByName(node.Name.LocalName, out var scrubOrIgnore))
+            if (serialization.TryGetScrubOrIgnoreByName(element.Name.LocalName, out var scrubOrIgnore))
             {
                 if (scrubOrIgnore == ScrubOrIgnore.Ignore)
                 {
-                    node.Remove();
+                    element.Remove();
                 }
                 else
                 {
-                    node.Value = "Scrubbed";
+                    element.Value = "Scrubbed";
                 }
 
                 continue;
             }
 
-            foreach (var attribute in node
-                         .Attributes()
-                         .ToList())
-            {
-                if (serialization.TryGetScrubOrIgnoreByName(attribute.Name.LocalName, out scrubOrIgnore))
-                {
-                    if (scrubOrIgnore == ScrubOrIgnore.Ignore)
-                    {
-                        attribute.Remove();
-                    }
-                    else
-                    {
-                        attribute.Value = "Scrubbed";
-                    }
-
-                    continue;
-                }
-
-                attribute.Value = ConvertValue(serialization, attribute.Value);
-            }
-
-            if (node.IsEmpty || node.HasElements)
-            {
-                continue;
-            }
+            ScrubAttributes(element, serialization);
+        }
 
-            node.Value = ConvertValue(serialization, node.Value);
+        foreach (var node in target.DescendantNodes())
+        {
+            Debug.WriteLine(node);
         }
 
         return VerifyString(target.ToString(), "xml");
@@ -124,4 +103,36 @@ string ConvertValue(SerializationSettings serialization, string value)
 
         return ApplyScrubbers.ApplyForPropertyValue(span, settings, counter).ToString();
     }
+
+    void ScrubAttributes(XElement node, SerializationSettings serialization)
+    {
+        foreach (var attribute in node
+                     .Attributes()
+                     .ToList())
+        {
+            if (serialization.TryGetScrubOrIgnoreByName(attribute.Name.LocalName, out var scrubOrIgnore))
+            {
+                if (scrubOrIgnore == ScrubOrIgnore.Ignore)
+                {
+                    attribute.Remove();
+                }
+                else
+                {
+                    attribute.Value = "Scrubbed";
+                }
+
+                continue;
+            }
+
+            var span = attribute.Value.AsSpan();
+            if (serialization.TryConvertString(counter, span, out var result))
+            {
+                attribute.Value = result;
+            }
+            else
+            {
+                attribute.Value = ApplyScrubbers.ApplyForPropertyValue(span, settings, counter).ToString();
+            }
+        }
+    }
 }