tag1consulting / goose

Load testing framework, inspired by Locust
https://tag1.com/goose
Apache License 2.0
813 stars 71 forks source link

Issue #531: Fix missing impl for TransactionError #579

Closed alecsmrekar closed 10 months ago

alecsmrekar commented 10 months ago

This PR addressed issue https://github.com/tag1consulting/goose/issues/531.

The original bug report provided a minimum reproducible code, but I think we're able to shrink that even further to:

async fn test_case(user: &mut GooseUser) -> TransactionResult {
    let goose = user
        .get("my/url")
        .await?;
    let _ = goose.response?;
    Ok(())
}

Trying to compile a loadtest with this code should give the following error:

the trait `From<reqwest::error::Error>` is not implemented for `Box<TransactionError>`

With this PR, it should compile cleanly.