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

bind server address once #21

Merged
merged 1 commit into from
Jul 26, 2017
Merged

Conversation

estin
Copy link
Contributor

@estin estin commented Jul 26, 2017

Hello!
It's me again.

In my test all mocks are unique and I want run tests without --test-threads=1

$ cargo test --all
...
running 3 tests
thread '<unnamed>' panicked at 'called `Result::unwrap()` on an `Err` value: Error { repr: Os { code: 98, message: "Address already in use" } }', /checkout/src/libcore/result.rs:860:4
note: Run with `RUST_BACKTRACE=1` for a backtrace.
thread '<unnamed>' panicked at 'called `Result::unwrap()` on an `Err` value: Error { repr: Os { code: 98, message: "Address already in use" } }', /checkout/src/libcore/result.rs:860:4
test test_1... ok
test test_2 ... ok
test test_3 ... ok

With this PR tests runs without noise about paniced threads.

$ cargo test --all
...
running 3 tests                    
test test_1 ... ok                             
test test_2 ... ok       
test test_3 ... ok       

test result: ok. 3 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out 

Note this PR does't solve problem testings when mocks are duplicated without --test-threads=1

Sorry about my poor English.

@lipanski lipanski merged commit a7443e4 into lipanski:master Jul 26, 2017
@lipanski
Copy link
Owner

0.8.2 is out. thanks 🎉

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.

2 participants