Development mode warning printed when in test mode

[nwjsmith]

Summary

Steps to reproduce

Run GRAPHILE_ENV=test node -e 'require("postgraphile")' or NODE_ENV=test node -e 'require("postgraphile")'

Expected results

Since we're explicitly in test mode, I would expect to see no warnings meant to catch production deployments of Grafast's development mode.

Actual results

When running GRAPHILE_ENV=test node -e 'require("postgraphile")':

Grafast is running in development mode due to `GRAPHILE_ENV=test`; this is recommended for development environments (and strongly discouraged in production), but will impact on performance - in particular, planning will be significantly more expensive.

When running NODE_ENV=test node -e 'require("postgraphile")'

Grafast is running in development mode due to `NODE_ENV=test`; this is recommended for development environments (and strongly discouraged in production), but will impact on performance - in particular, planning will be significantly more expensive.

Additional context

Filed as suggested in the Discord forum.

[benjie]

Can confirm this was fixed in Feb (but currently unreleased): 4d9f83d

[nwjsmith]

Thank you!