ci: add 11 missing packages with tests to CI workflow #635
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Re-implements the changes from PR #633 which was closed due to test hanging issues (now fixed in PR #634). This adds 11 packages with tests that were missing from the CI test matrix:
graphql/server-testgraphql/envgraphql/servergraphql/testgraphql/playwright-testpackages/csv-to-pgpackages/smtppostmasterpostgres/drizzle-orm-testuploads/etag-hashuploads/etag-streamuploads/stream-to-etagAlso updates the
graphql/envtest snapshot to include newsmtpconfiguration fields that were added to the codebase.Note:
jobs/knative-job-workeris commented out (temporarily disabled per user request).Updates since last revision
localhostwhich resolves to::1(IPv6) on some systems, but supertest connects to127.0.0.1(IPv4), causingECONNREFUSEDerrors. Now explicitly binds to127.0.0.1.'listening'event before accessinghttpServer.address()(which was returningnull).allUsers/allPoststousers/poststo match the actual PostGraphile-generated schema.127.0.0.1instead oflocalhost.pgclient (superuser) doesn't participate in thedbclient's transaction rollback - test now correctly demonstrates this behavior.Review & Testing Checklist for Human
localhostto127.0.0.1is the core fix; confirm tests pass in CIgraphql/server-test/src/server.ts- verify the listening event wait logic and IPv4 binding are correctusersvsallUsers) match the actual schema generated by PostGraphilepgclient vsdbclient differencesSuggested test plan: Wait for CI to complete and verify all 11 new jobs pass, particularly
graphql/server-test. If tests fail with connection errors, the IPv6/IPv4 fix may need adjustment for that environment.Notes
Link to Devin run: https://app.devin.ai/sessions/3cb2368e9a0d4516a01fbf7cbcd33e94
Requested by: Dan Lynch (@pyramation)