Skip to content

Commit ade0a1c

Browse files
KoditkarVedantjoemcbride
authored andcommitted
160 add overlapping fields can be merged validation rule (graphql-dotnet#1005)
* first try * update OverlappingFieldsCanBeMerged rule * add test cases for OverlappingFieldsCanBeMerged rule * add more test cases * fix Conflicting args should fail test case * Fix test Encounters_conflict_in_fragments_should_fail * Add more tests and fix the logic * Add toString override to print the GraphType * make schema configurable in ValidationTest Config * Add more test cases and fixed the logic * change rule error code from 5.3.2.2 to 5.3.2 * update readme.md
1 parent 757486f commit ade0a1c

File tree

6 files changed

+2328
-3
lines changed

6 files changed

+2328
-3
lines changed

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -181,7 +181,7 @@ var json = schema.Execute(_ =>
181181
- [x] No undefined variables
182182
- [x] No unused fragments
183183
- [x] No unused variables
184-
- [ ] Overlapping fields can be merged ([help with this rule](https://github.com/graphql-dotnet/graphql-dotnet/issues/160))
184+
- [x] Overlapping fields can be merged
185185
- [x] Possible fragment spreads
186186
- [x] Provide non-null arguments
187187
- [x] Scalar leafs

0 commit comments

Comments
 (0)