Skip to content

Commit

Permalink
Test adding nextest
Browse files Browse the repository at this point in the history
  • Loading branch information
AurelienFT committed Oct 28, 2024
1 parent f333ae8 commit 17adede
Show file tree
Hide file tree
Showing 3 changed files with 19 additions and 15 deletions.
27 changes: 16 additions & 11 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -118,20 +118,20 @@ jobs:
args: --all-features --workspace --no-deps
- command: make
args: check --locked
- command: test
- command: nextest
args: --workspace
- command: test
args: --all-features --workspace
- command: test
args: -p fuel-core --no-default-features
- command: test
args: -p fuel-core --lib executor --features wasm-executor
- command: nextest
args: run --retries 10 --all-features --workspace
- command: nextest
args: run --retries 10 -p fuel-core --no-default-features
- command: nextest
args: run --retries 10 -p fuel-core --lib executor --features wasm-executor
env:
FUEL_ALWAYS_USE_WASM=true
- command: test
args: -p fuel-core-client --no-default-features
- command: test
args: -p fuel-core-chain-config --no-default-features
- command: nextest
args: run --retries 10 -p fuel-core-client --no-default-features
- command: nextest
args: run --retries 10 -p fuel-core-chain-config --no-default-features
# Don't split this command; this is a workaround.
# We need to run `cargo check` first to fetch the locked dependencies
# for `fuel-core 0.26.0`(because of the bug with `--offline`
Expand Down Expand Up @@ -177,6 +177,11 @@ jobs:
~/.cargo/git/db/
target/
key: ${{ matrix.command }}-${{ matrix.args }}-${{ runner.os }}-cargo-${{ hashFiles('**/Cargo.lock') }}
- uses: actions-rs/cargo@v1
if: ${{ matrix.command == 'nextest' }}
with:
command: install
args: cargo-nextest --locked
- name: ${{ matrix.command }} ${{ matrix.args }}
run: ${{ matrix.env }} cargo ${{ matrix.command }} ${{ matrix.args }}
- uses: FuelLabs/.github/.github/actions/slack-notify-template@master
Expand Down
6 changes: 3 additions & 3 deletions bin/e2e-test-client/src/tests/script.rs
Original file line number Diff line number Diff line change
Expand Up @@ -137,7 +137,7 @@ pub async fn run_contract_large_state(ctx: &TestContext) -> Result<(), Failed> {
if result?.is_none() {
let deployment_request = ctx.bob.deploy_contract(contract_config, salt);

timeout(Duration::from_secs(20), deployment_request).await??;
timeout(Duration::from_secs(90), deployment_request).await??;
}

_dry_runs(ctx, &[dry_run], 100, DryRunResult::MayFail).await
Expand Down Expand Up @@ -199,9 +199,9 @@ async fn _dry_runs(
});
}

// All queries should be resolved for 60 seconds.
// All queries should be resolved for 90 seconds.
let queries =
tokio::time::timeout(Duration::from_secs(60), futures::future::join_all(queries))
tokio::time::timeout(Duration::from_secs(90), futures::future::join_all(queries))
.await?;

let chain_info = ctx.alice.client.chain_info().await?;
Expand Down
1 change: 0 additions & 1 deletion tests/tests/poa.rs
Original file line number Diff line number Diff line change
Expand Up @@ -259,7 +259,6 @@ mod p2p {
// Then starts second_producer that uses the first one as a reserved peer.
// second_producer should not produce blocks while the first one is producing
// after the first_producer stops, second_producer should start producing blocks
#[ignore = "seems to be flaky, issue: https://github.com/FuelLabs/fuel-core/issues/2351"]
#[tokio::test(flavor = "multi_thread")]
async fn test_poa_multiple_producers() {
const SYNC_TIMEOUT: u64 = 30;
Expand Down

0 comments on commit 17adede

Please sign in to comment.