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

Create a Mock tracer implementation #40

Closed
yurishkuro opened this issue Nov 12, 2017 · 2 comments
Closed

Create a Mock tracer implementation #40

yurishkuro opened this issue Nov 12, 2017 · 2 comments

Comments

@yurishkuro
Copy link
Member

Most other language implementations support a Mock tracer. Having a mock tracer is useful for two reasons:

  • it allows unit testing of the instrumentation with only the opentracing dependency
  • implementing a mock tracer helps with ironing out the issues in the API itself, like Removes carriers. #38, as it serves as a reference implementation
@jcchavezs
Copy link
Collaborator

jcchavezs commented Nov 12, 2017

Great feedback @yurishkuro however there is already an issue for this: #21. Let's continue the discussion there.

@yurishkuro
Copy link
Member Author

sorry, should've checked the issues

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants