buildASTSchema ignores schema extensions

[mike-marcacci]

While using graphql-js to check a sample schema for a blog post, I discovered that buildASTSchema completely ignores extension logic. I haven't had a chance to fix this yet, but just added failing test to show the current behavior.

I'll create a fix for this when I have time.

[mike-marcacci]

Doh! It looks like this is already being discussed in #922 which I somehow missed... and @IvanGoncharov now has a WIP with #2248. Closing as duplicate.

[IvanGoncharov]

@mike-marcacci Thanks for reporting 👍
It's one last issue I wanted to fix before feature freeze: #2199 (comment)
I did most of the preparation job in #2235 and just pushed WIP I have at the moment as #2248 (still missing some of the tests).
Also, it has a bunch any conversions but its something we can latter fix in the separate PRs.

[IvanGoncharov]

@mike-marcacci Sorry for duplicating work.
Also, I need to figure out how to better distribute work in this repo in general.

[mike-marcacci]

@IvanGoncharov - oh no worries at all! I didn't put much time into it at all... clearly not enough to even find the existing issue, haha.

Let me know if you want a second set of eyes on #2248!