refactor: introduce adapter pattern for test results #493
Conversation
| @@ -83,7 +83,6 @@ | |||
| "qs": "^6.9.1", | |||
| "signal-exit": "^3.0.2", | |||
| "tmp": "^0.1.0", | |||
| "urijs": "^1.18.12", | |||
There was a problem hiding this comment.
а разве из package-lock.json он не должен был тоже пропасть?
There was a problem hiding this comment.
Да, должен, просто package-lock я обычно обновляю перед влитием. Обязательно обновлю. Ну и не хотел пугать большим диффом в PR )
| @@ -77,11 +81,12 @@ module.exports = (hermione, reportBuilder, client, reportPath) => { | |||
|
|
|||
| hermione.on(hermione.events.TEST_FAIL, (testResult) => { | |||
| queue.add(async () => { | |||
| const formattedResult = reportBuilder.format(testResult, hermione.events.TEST_FAIL); | |||
| const status = hasDiff(testResult.assertViewResults) ? TestStatus.FAIL : TestStatus.ERROR; | |||
There was a problem hiding this comment.
я понимаю, что эта логика была и до этого PR-а, но почему мы выставляем fail именно при наличии диффа? чем это отличается от обычного провалившегося ассерта?
//cc @DudaGod
There was a problem hiding this comment.
Я сам недавно задавался этим вопросом когда интеграцию pwt с testcop пилил. Сходу не помню почему так было сделано.
|
|
||
| return {name, suitePath, browserId}; | ||
| get url(): string | undefined { | ||
| return this._testResult.meta.url as string | undefined; | ||
| } | ||
|
|
||
| get multipleTabs(): boolean { |
There was a problem hiding this comment.
кажется, эта фигня уже никому не нужна
There was a problem hiding this comment.
Согласен. Но на это есть много завязок в html-reporter, многое может сломаться, если сейчас выгасить. Мне кажется это лучше делать в отдельном PR
shadowusr
force-pushed
the
users/shadowusr/HERMIONE-1003.refactoring-part-2
branch
from
9db24c7 to
e10d2fe
Compare
package-lock.json
Outdated
There was a problem hiding this comment.
дифф большой из-за добавления pwt в devDependencies.
shadowusr
force-pushed
the
users/shadowusr/HERMIONE-1003.refactoring-part-2
branch
from
e10d2fe to
a4a6b9b
Compare
shadowusr
force-pushed
the
users/shadowusr/HERMIONE-1003.refactoring-part-2
branch
from
a4a6b9b to
a679d08
Compare
What's done?
ReporterTestResultas a single interface to interact with across all of the html-reporter code. AdjustedTestAdapteraccordingly, renamed it toHermioneTestAdapterand got rid of unnecessary field/methodsSuiteAdapterby partially merging it intoTestAdapterand partially moving the logic to helpersWhy?