Skip to content
This repository has been archived by the owner on Aug 10, 2023. It is now read-only.

Test for 'metrics' not show existing bug #86

Closed
KostyaTretyak opened this issue Jan 19, 2017 · 2 comments
Closed

Test for 'metrics' not show existing bug #86

KostyaTretyak opened this issue Jan 19, 2017 · 2 comments

Comments

@KostyaTretyak
Copy link

If we write this:

console.log('****************************** before metrics.latency:', metrics.latency);
assert.isAtLeast(metrics.latency, 200);
console.log('****************************** after metrics.latency:', metrics.latency);

here, and run test, we will see before metrics.latency but not after metrics.latency.

It seems that here bug in mocha and restify-plugins.

@KostyaTretyak
Copy link
Author

KostyaTretyak commented Jan 19, 2017

Now I see this is not bug restify-plugin or mocha, but script is failed and test not show this.

@retrohacker
Copy link
Member

Hey @KostyaTretyak,

Thank you for opening this issue. From my understanding it looks like you found this wasn't a bug in either the restify-plugin or mocha repos.

I'm going to close this for now as resolved. If I'm misunderstanding, please comment and I would be happy to re-open and discuss ❤️

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

No branches or pull requests

2 participants