Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Build ndc-postgres and v3-engine binaries using Nix in e2e tests #592

Merged
merged 1 commit into from
Aug 23, 2024

Conversation

danieljharvey
Copy link
Contributor

@danieljharvey danieljharvey commented Aug 23, 2024

What

Speed up e2e tests by building the v3-engine and ndc-postgres binaries with Nix.

How

Mostly copied from the equivalent v3-engine CI job.

@danieljharvey danieljharvey changed the title Binary ndc-postgres and v3-engine binaries using Nix in e2e tests Build ndc-postgres and v3-engine binaries using Nix in e2e tests Aug 23, 2024
@danieljharvey danieljharvey force-pushed the djh/copy-e2e-testing-from-engine branch from ba4f2b5 to a50a75d Compare August 23, 2024 13:05
@danieljharvey danieljharvey marked this pull request as ready for review August 23, 2024 13:05
@danieljharvey danieljharvey added this pull request to the merge queue Aug 23, 2024
Comment on lines +65 to +74
env:
ENGINE_BINARY: >-
${{ github.workspace }}/v3-engine/result/bin/engine
NDC_POSTGRES_BINARY: >-
${{ github.workspace }}/ndc-postgres/result/bin/ndc-postgres
run: |
npm install
just start-postgres-dependencies start-apollo-subgraph
trap 'just stop-everything' EXIT
RUST_LOG=DEBUG ./crates/postgres/static/run-postgres-tests.sh '.*' "$NDC_POSTGRES_BINARY" "$ENGINE_BINARY"
Copy link
Contributor

@soupi soupi Aug 23, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That doesn't look right.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah sorry, this is e2e-tests.

Merged via the queue into main with commit f9b942b Aug 23, 2024
30 checks passed
@danieljharvey danieljharvey deleted the djh/copy-e2e-testing-from-engine branch August 23, 2024 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants