Skip to content

ci: Do not use nextest #6266

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.

Sign up for GitHub

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

Open
lutter wants to merge 1 commit into
base: master
Choose a base branch
from
+11 −63
Open

ci: Do not use nextest #6266

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 0 additions & 18 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -59,12 +59,6 @@ jobs:
- name: Setup just
uses: extractions/setup-just@e33e0265a09d6d736e2ee1e0eb685ef1de4669ff # v3

- name: Install cargo-nextest
uses: baptiste0928/cargo-install@e38323ef017552d7f7af73a3f4db467f278310ed # v3
with:
crate: cargo-nextest
version: ^0.9

- name: Run unit tests
run: just test-unit --verbose

Expand Down Expand Up @@ -106,12 +100,6 @@ jobs:
- name: Setup just
uses: extractions/setup-just@e33e0265a09d6d736e2ee1e0eb685ef1de4669ff # v3

- name: Install cargo-nextest
uses: baptiste0928/cargo-install@e38323ef017552d7f7af73a3f4db467f278310ed # v3
with:
crate: cargo-nextest
version: ^0.9

- name: Install pnpm
uses: pnpm/action-setup@a7487c7e89a18df4991f7f222e4898a00d66ddda # v4

Expand Down Expand Up @@ -165,12 +153,6 @@ jobs:
- name: Setup just
uses: extractions/setup-just@e33e0265a09d6d736e2ee1e0eb685ef1de4669ff # v3

- name: Install cargo-nextest
uses: baptiste0928/cargo-install@e38323ef017552d7f7af73a3f4db467f278310ed # v3
with:
crate: cargo-nextest
version: ^0.9

- name: Install Foundry
uses: foundry-rs/foundry-toolchain@82dee4ba654bd2146511f85f0d013af94670c4de # v1
with:
Expand Down
56 changes: 11 additions & 45 deletions justfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,67 +22,33 @@ check-all:
build *EXTRA_FLAGS:
cargo build --bin graph-node {{EXTRA_FLAGS}}

# Run all tests (unit and integration)
test *EXTRA_FLAGS:
_cargo-test *ARGS:
#!/usr/bin/env bash
set -e # Exit on error

# Ensure that the `THEGRAPH_STORE_POSTGRES_DIESEL_URL` environment variable is set.
if [ -z "$THEGRAPH_STORE_POSTGRES_DIESEL_URL" ]; then
echo "Error: THEGRAPH_STORE_POSTGRES_DIESEL_URL is not set"
if [ -z "$THEGRAPH_STORE_POSTGRES_DIESEL_URL" -a -z "$GRAPH_NODE_TEST_CONFIG" ]; then
echo "error: Neither THEGRAPH_STORE_POSTGRES_DIESEL_URL nor GRAPH_NODE_TEST_CONFIG is set"
exit 1
fi

if command -v "cargo-nextest" &> /dev/null; then
cargo nextest run {{EXTRA_FLAGS}} --workspace
else
cargo test {{EXTRA_FLAGS}} --workspace -- --nocapture
fi
cargo test {{ARGS}}

# Run all tests (unit and integration)
test *EXTRA_FLAGS:
@just _cargo-test {{EXTRA_FLAGS}} --workspace -- --nocapture

# Run unit tests
test-unit *EXTRA_FLAGS:
#!/usr/bin/env bash
set -e # Exit on error

# Ensure that the `THEGRAPH_STORE_POSTGRES_DIESEL_URL` environment variable is set.
if [ -z "$THEGRAPH_STORE_POSTGRES_DIESEL_URL" ]; then
echo "Error: THEGRAPH_STORE_POSTGRES_DIESEL_URL is not set"
exit 1
fi

if command -v "cargo-nextest" &> /dev/null; then
cargo nextest run {{EXTRA_FLAGS}} --workspace --exclude graph-tests
else
cargo test {{EXTRA_FLAGS}} --workspace --exclude graph-tests -- --nocapture
fi
@just _cargo-test {{EXTRA_FLAGS}} --workspace --exclude graph-tests -- --nocapture

# Run runner tests
test-runner *EXTRA_FLAGS:
#!/usr/bin/env bash
set -e # Exit on error

# Ensure that the `THEGRAPH_STORE_POSTGRES_DIESEL_URL` environment variable is set.
if [ -z "$THEGRAPH_STORE_POSTGRES_DIESEL_URL" ]; then
echo "Error: THEGRAPH_STORE_POSTGRES_DIESEL_URL is not set"
exit 1
fi

if command -v "cargo-nextest" &> /dev/null; then
cargo nextest run {{EXTRA_FLAGS}} --package graph-tests --test runner_tests
else
cargo test {{EXTRA_FLAGS}} --package graph-tests --test runner_tests -- --nocapture
fi
@just _cargo-test {{EXTRA_FLAGS}} --package graph-tests --test runner_tests -- --nocapture

# Run integration tests
test-integration *EXTRA_FLAGS:
#!/usr/bin/env bash
set -e # Exit on error

if command -v "cargo-nextest" &> /dev/null; then
cargo nextest run {{EXTRA_FLAGS}} --package graph-tests --test integration_tests
else
cargo test {{EXTRA_FLAGS}} --package graph-tests --test integration_tests -- --nocapture
fi
@just _cargo-test {{EXTRA_FLAGS}} --package graph-tests --test integration_tests -- --nocapture

# Clean workspace (cargo clean)
clean:
Expand Down