Skip to content

Conversation

@Tushkan
Copy link
Contributor

@Tushkan Tushkan commented Mar 25, 2017

сделанные доработки

  • дымовые тесты открытия основной формы документа на основании ДокументаОснования
  • обработка может быть использована и в VanessaBehavior и в xUnit - хорошо, что в данных проектах нет одинаковых экспортных переменных и процедур используемых при инициализации тестов/фич

@Tushkan
Copy link
Contributor Author

Tushkan commented Mar 25, 2017

@artbear @Pr-Mex посмотрите имеет ли право на жизнь такая реализация 2в1.
P.S. В ванессу пока PR не делал, что бы не разносить обсуждение в две ветки

@Pr-Mex
Copy link

Pr-Mex commented Mar 26, 2017

Посмотрел.
Наверно имеет смысл такие универсальные вещи выносить в отдельный проект.
Например SmokeTests.

@artbear
Copy link
Member

artbear commented Mar 26, 2017

@Tushkan Очень интересно, посмотрю.
Здорово, что получилось сделать общий вариант для 2х фреймворков!

я буквально час назад наконец-то смержил свой PR #686
Предложения:

  • смержить мою доку readme.md со своей документацией
  • добавить возможность настройки исключений с помощью json-настройки.
    • удобнее сделать через отдельный внешний файл, например, based-on.json
    • пример настройки посмотри в моем дымовом тесте открытия форм

Сделаешь?

@artbear
Copy link
Member

artbear commented Jun 22, 2017

@Tushkan Сможешь поправить по моим последним предложениям?

@artbear artbear modified the milestone: 4.1.0.0 Jun 26, 2017
@artbear artbear modified the milestones: 4.2.0.0, 4.3.0.0, 4.Х.0.0 Jul 5, 2017
@artbear artbear modified the milestones: 4.Х.0.0, 4.3.0.0 Oct 12, 2017
@artbear artbear self-assigned this Dec 1, 2017
@artbear artbear merged commit 4f62d0f into xDrivenDevelopment:develop Dec 3, 2017
@artbear
Copy link
Member

artbear commented Dec 3, 2017

@Tushkan Большое спасибо.
Наконец-то я смержил этот PR

@kuntashov
Copy link
Member

Наверно имеет смысл такие универсальные вещи выносить в отдельный проект.
Например SmokeTests.

Поддержу @Pr-Mex, #766

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants