Make transport tests less racy

lightstep / lightstep-tracer-go

The Lightstep distributed tracing library for Go

https://lightstep.com

MIT License

98 stars 53 forks source link

Closed tedsuo closed 6 years ago

tedsuo commented 6 years ago

use an explicit Flush instead of a small reporting period
remove unnecessary options for clarity
one exception is CloseTracer tests, they still need to test with a short reporting period instead of Flush.

tedsuo commented 6 years ago

@jmacd reassigning because #NoDolan. This PR is no biggie (just tests) so I'll just merge if you don't have time.

jmacd commented 6 years ago

FYI I've created a branch named "v0.16.0" for post-0.15 feature development.

iredelmeier commented 6 years ago

Closing because it's stale. Happy to reopen if desired!