-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
Trim down test harness #8146
Trim down test harness #8146
Conversation
a8c897d
to
e032387
Compare
c18bc3d
to
6a659af
Compare
6a659af
to
53aa6d2
Compare
}); | ||
}); | ||
// Setup auto releasing stubs and spys | ||
setupAutoRelease(sinon, window.afterEach); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see setupAutoRelease(sinon, window.afterEach);
on line 12 and line 16. Is it needed in both places?
// chrome expects to be loaded first, let it get its way | ||
import chrome from 'ui/chrome'; | ||
|
||
import Nonsense from 'Nonsense'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I couldn't find any other usages of Nonsense. Think it's safe to remove the dependency from package.json?
LGTM after changes above. |
53aa6d2
to
ae4dded
Compare
Backports PR elastic#8146 **Commit 1:** trim down test harness, remove StackTraceMapper * Original sha: ae4dded * Authored by spalger <email@spalger.com> on 2016-09-01T16:11:45Z Former-commit-id: 2991900
Relies on #8150
While investigating potential memory leaks, discovered some unnecessary stuff in the test harness.
ui/stack_trace_mapper
, it didn't work anywayNonsense
, it also didn't work