BeXmlSerializable does not respect XmlIgnoreAttribute about fluentassertions HOT 5 OPEN

After further investigation, this is certainly not an issue with XML serialization, but rather how the assertion process handles equivalency:

As my screenshot shows, ignored members are not set during the deserialization process, and as a result the equivalency will fail. I am currently researching how involved of a change it would be to add support for this. I think the most surgical route would be to update EquivalencyOptions<TExpectation> to exclude ignored members and the BeXmlSerializable method to specify this as part of its options:

deserializedObject.Should().BeEquivalentTo(assertions.Subject, 
    static options => options
        .RespectingRuntimeTypes()
        .IncludingFields()
        .IncludingProperties()
        .ExcludeIgnoredMembers());

My biggest remaining question is if this new method should be specific to XML or not as ExcludeXmlIgnoredMembers for clarity that other serialization methods are not supported.

from fluentassertions.

I think a simpler approach could be to add a custom IMemberSelectionRule that is based on AllPropertiesSelectionRule and AllFieldsSelectionRule and filters out properties and fields marked with the XmlIgnore attribute.

from fluentassertions.

I'm not sure this is an issue with the process not ignoring members. Based on the code for the BeXmlSerializable method, it looks like this is working as expected and equivalency is failing:

According to line 107 there, the deserialized object should be equivalent to the subject of the assertion. This, combined with my (albeit severely lacking) understanding of how the equivalency test works, should (currently) fail because the ignored members won't be set in the deserialized object.

@dennisdoomen, can this be assigned to me?

from fluentassertions.

I've defined a unit test to cover this scenario and opened a draft PR to begin working this in full. I simply need guidance on the following to prove as effective as possible on the issue:

• Should this be added to the EquivalencyOptions model?
  • If so:
    • What should the name of the method be?
      • I recommend ExcludeXmlIgnoredMembers.
    • Are there any niche references of this that I need to be careful about?
  • If not:
    • How should this be accounted for?
• Are there any ecosystem-wide scenarios or goals I need to keep in mind?
• Are there any known gotchas with the BeXmlSerializable method?
• Is there anything else I should be aware of prior to proceeding?

from fluentassertions.

Is it possible to choose an implementation that also enables BeEquivalentTo to optionally ignore properties with the XmlIgnoreAttribute?

from fluentassertions.