Skip to content
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

Дымовые тесты форм документов: ввод на основании #693

Merged
merged 1 commit into from
Dec 3, 2017

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