Possibly unintended breaking change between 2.0.0 and 4.2.1

[asgerhallas]

The following test passed in 2.0.0 but fails in 4.2.1 as it returns a Pass now. Is this intended?

    [Fact]
    public void ComparingObjectsWithNoProperties_DifferentType()
    {
        var comparer = (IComparison)new ComparisonBuilder().Create();
        var context = new ComparisonContext();

        Assert.Equal(
            ComparisonResult.Inconclusive, 
            comparer.Compare(context, new object(), new Blah()).result);
    }
    
    public class Blah { }

[asgerhallas]

I suspect it is this addition that makes the difference:

DeepEqual/src/DeepEqual/ComplexObjectComparer.cs

Line 33 in dbec89d

if (propertyMap.Count == 0) return (ComparisonResult.Pass, context);

[asgerhallas]

Allow me to bump this question :)

Is it intentional that two classes of different types, but both with no properties, should return a Pass?

I'd say it would be more natural with Inconclusive, to be able to fall back to the DefaultComparer.