search

OpenRIAServices / OpenRiaServices

The Open RIA Services project continues what was previously known as WCF RIA Services.
https://openriaservices.gitbook.io/openriaservices/
Apache License 2.0
54 stars 47 forks source link

Add new test that validate how object level validation interact with property level validation #455

Closed Daniel-Svensson closed 8 months ago

Daniel-Svensson commented 8 months ago

There is a change that #453 changed behaviour for properties which are only validated using object level validation (either IValidatableObject or Validation attributes on the type) but does not have any property level validation attributes.

Previously such object level validation errors where cleared when if the property was modified.

sonarcloud[bot] commented 8 months ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

Daniel-Svensson commented 8 months ago

@SandstromErik/ @erikoijwall please merge if you think this is a good solution and the test seems OK.

I have validated that the test passed against 5.4.0, but failed in main without the fix here.

Normally object validation is triggered when submitting changes as well