This reverts commit f87d458.

This reverts commit 627594b.

The way it was before only would catch errors / print deprecation
messages if the user was using a gateway. Somewhat annoyingly, the
gateway and apollo server use different construction objects, so sharing
the method was cleaner (since we would, in theory, like it to be
possible to create an engine-reporting-agent as a class, it's important
that it handles its own errors itself). This does, however, mean that a
gateway Apollo Server that is using the deprecated key will get two
deprecation messages, but I think it would cause more clutter to resolve
that then the value it would provide to "fix" it.

This also adds tests.

Additionally, since the logging is now performed by a function coming
from apollo-engine-reporting, it seemed out-of-scope to add a pluggable
logger to its feature set. If that's a blocker, please tell me!

I added these to both apollo gateway and the top-level

Co-Authored-By: Jesse Rosenberger <[email protected]>

Co-Authored-By: Jesse Rosenberger <[email protected]>

…t.ts

Co-Authored-By: Jesse Rosenberger <[email protected]>

…t.ts

Co-Authored-By: Jesse Rosenberger <[email protected]>