Closed kyle-winkelman closed 11 months ago
I would be willing to create further unit tests to prove out that this removes the described limitations, but it would be helpful to know what those limitations are.
Can we trust AstSorterTest to be extensive enough to rely on or should we add more?
@kyle-winkelman Thank you for the PR! I wasn't aware of AstSorter and AstComparator, but I think they are great ideas and definitely worth using. AstSorterTest seems to be sufficiently tested as it is.
After this change, we will can remove the limitations and release this extension as 1.0 soon!
Currently, custom logic occurs within GraphqlQueryNormalizer to normalize the order of the fields within the query. I believe this leads to the described limitations.
References
Submitter checklist
#help-contributing
or a project-specific channel like#wiremock-java