test(helpers): pass client instead of pool; fix typing

graphile / starter

Opinionated SaaS quick-start with pre-built user account and organization system for full-stack application development in React, Node.js, GraphQL and PostgreSQL. Powered by PostGraphile, TypeScript, Apollo Client, Graphile Worker, Graphile Migrate, GraphQL Code Generator, Ant Design and Next.js

Other

1.75k stars 219 forks source link

Description

This fixes passing the client to test helpers instead of the pool. It also adds typings to the pools cache to allow TS to catch future mismatches.

Performance impact

Presumably none.

Security impact

Presumably none.

Checklist

[x] My code matches the project's code style and yarn lint:fix passes.
[x] I've added tests for the new feature, and yarn test passes.
[ ] I have detailed the new feature in the relevant documentation.
[ ] I have added this feature to 'Pending' in the RELEASE_NOTES.md file (if one exists).
[ ] If this is a breaking change I've explained why.

graphile / starter