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

Загрузка глобальных переменных из внешнего файла #346

Merged
merged 7 commits into from Nov 2, 2018

Conversation

Projects
None yet
4 participants
@ripreal

ripreal commented Oct 25, 2018

реализация #343

сделанные изменения

  • Новый плагин ЗагрузчикПользовательскихНастроек, который загружает сохраняемый контекст из json файла
  • Автоматическая загрузка настроек после открытия формы bddRunner
  • Загрузка настроек при изменении каталога проекта на вкладке Сервис

@silverbulleters/Collaborators - просьба прокомментировать и проверить

@ripreal ripreal changed the title from Feature/issue 343 to Загрузка глобальных переменных из внешнего файла Oct 25, 2018

@artbear artbear self-assigned this Oct 25, 2018

@artbear artbear added this to the next milestone Oct 25, 2018

@pumbaEO

This comment has been minimized.

Collaborator

pumbaEO commented Oct 26, 2018

Воу, воу, воу хлопцы. Не спешите:

  1. Первое, что мешает пользователю сохранить путь к файлу глоб.переменных в хранилище системных настроек?
  2. Через гит определять не получится, т.к. при запуске мы не знаем откуда запустились то, может из ProgrammFiles, зачем привязывается к git, если за нас это делает runner.

Имхо алгоритм такой: в файл settings вынесли настройку, если есть путь с макросом, тогда разворачиваем и проверяем, если пути нет, тогда пытаемся прочитать из хранилища системных настроек, вдруг сам пользователь его указал. В остальных случаях не загружаем настройки, а оставляем пользователю возможность указать в диалоге и сохранить.

@ripreal

This comment has been minimized.

ripreal commented Oct 26, 2018

@pumbaEO
Твой вариант не подходит.

  • Во-первых тесты, например у нас, проводятся интерактивно на чистой копии базы, на которой не сохранены каталог проекта и прочее, включая хранилище системных настроек.
  • Во-вторых при интерактивном выполнении раннер не используется.

Ванесса у нас запускается изнутри репозитория проекта, поэтому вариант @artbear нам бы подошел.

@ripreal

This comment has been minimized.

ripreal commented Oct 26, 2018

@pumbaEO
Указывать перед каждым запуском настройки в диалоге было бы очень неудобно (т.к. все тесты проводятся на чистой копии базы)

@pumbaEO

This comment has been minimized.

Collaborator

pumbaEO commented Oct 26, 2018

Интерактивно - это пользователь что-ли сам загружает базу и запускает ее и потом делает Файл - Открыть указывает путь к bddRunner.epf указывает путь к фичам и потом "play"?

@ripreal

This comment has been minimized.

ripreal commented Oct 26, 2018

@pumbaEO Да, именно так. Простому пользователю нужно видеть результат выполнения всех тестов и возможность пощупать их, прокрутить заново частями. Я знаю, что с помощью враннера можно добиться такого же эффекта, но этот вариант нам не подходит, т.к. тесты пишут еще и аналитики - они не должны копаться в конфигах

ripreal and others added some commits Oct 30, 2018

@artbear

artbear approved these changes Nov 2, 2018

Я исправил неверный ключ.
Сейчас приму PR

На след.неделе выпущу релиз.

@artbear artbear merged commit 7ddb2c5 into silverbulleters:develop Nov 2, 2018

1 check was pending

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
@artbear

This comment has been minimized.

Collaborator

artbear commented Nov 2, 2018

@ripreal Большое спасибо за доработку!
И поздравляю с первым контрибьютингом в наш проект!

@silverbulleters-ci-admin

This comment has been minimized.

Collaborator

silverbulleters-ci-admin commented Nov 2, 2018

This pull request has been mentioned on Automation Driven Development Forum. There might be relevant details there:

https://xdd.silverbulleters.org/t/novye-kontribyutory-v-add/2440/1

@ripreal

This comment has been minimized.

ripreal commented Nov 3, 2018

@artbear Спасибо. У меня вот как раз только сегодня дошли руки, чтобы проверить как работает без параметра "-c". Проверил - все отлично

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