Some refactoring to run grep before tests run #146
Some refactoring to run grep before tests run #146
Conversation
1 similar comment
1451f67
to
f8e1a42
Compare
1 similar comment
f8e1a42
to
61ede4f
Compare
1 similar comment
@@ -28,123 +29,125 @@ function requireWithNoCache(moduleName) { | |||
return result; | |||
} | |||
|
|||
module.exports = inherit(ChainedEmitter, { | |||
module.exports = inherit(EventEmitter, { |
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.
Why?
ChainedEmmiter
was there for the plugins needs. Basically, it allows to return a promise from event handler and delay the further execution until that promise resolves. This allows, for example, to launch SauceConnect
or ssh-tunnel before any tests are launched.
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.
По коду это совершенно непонятно, а то единственное использование ChainedReporter, что я нашел, выглядело очень натянуто.
Либо как-то странно реализована поддержка плагинов, либо нужно более явно в коде объяснить, почему нужно делать именно так, а не иначе.
Плохое архитектурное решение, кмк.
61ede4f
to
4ae5670
Compare
11be320
to
64ae9e1
Compare
64ae9e1
to
1a92664
Compare
1 similar comment
1a92664
to
111fcf5
Compare
Some refactoring to run grep before tests run
No description provided.