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

Ошибка при копирование основного клиента тестирования #652

Closed
GenVP opened this issue Jan 29, 2020 · 12 comments
Labels
bug Something isn't working

Comments

@GenVP
Copy link
Contributor

GenVP commented Jan 29, 2020

При копировании основного клиента тестирования "Этот клиент" в таблице клиентов сохраняется невидимый флаг "ЭтотКлиент" (у него то же отображается звездочка в первой колонке) и поэтому новый клиент не выгружается в файл настроек JSON.

Исправлю.

@GenVP
Copy link
Contributor Author

GenVP commented Jan 29, 2020

Заодно вопрос: должен ли выгружаться в настройки основной клиент с заданными доп. параметрами? А то получается, что для него можно задать пользователя и пароль или win-авторизацию, но он не выгрузится в файл для пакетного запуска?

@Pr-Mex
Copy link
Owner

Pr-Mex commented Jan 30, 2020

@GenVP
На CI сервере путь скорее будет всего другой, а запись "Этот клиент" всегда соответствует базе TestManager. Поэтому было принято решение не выгружать эту строку.
Считаешь что её надо выгружать?

@Pr-Mex Pr-Mex added the bug Something isn't working label Jan 30, 2020
@GenVP
Copy link
Contributor Author

GenVP commented Jan 30, 2020

@Pr-Mex
Если клиент один, то да. Раньше колонка ЭтотКлиент ("звездочка") в таблице клиентов использовалась для идентификации "Этот клиент" или для исключения клиентов из выгрузки?

Сейчас колонка "ЭтоКлиент" в таблице клиентов используется:

  1. При добавлении клиента "Этот клиент" - заполняется Истина
  2. При выгрузке для исключения "ЭтотКлиент" = Истина.
  3. В "РаботаСTestClient.ЯКопируюТекущийПрофильTestClientСУстановкойПараметров" - сбрасывается, если не "Этот клиент".

В остальных случаях проверка "Этот клиент" выполняется по имени.

@GenVP
Copy link
Contributor Author

GenVP commented Jan 30, 2020

Можно оставить на выбор пользователя, переключая звездочку по двойному щелчку

@Pr-Mex
Copy link
Owner

Pr-Mex commented Jan 30, 2020

@fenixnow
Напомни, плиз, ты какой смысл закладывал в поле со звёздочкой? Чтобы отказаться от поиска по имени?

@fenixnow
Copy link
Contributor

Если не ошибаюсь, то мы не выгружаем "этот клиент". Это предопределенный элемент списка.

@fenixnow
Copy link
Contributor

@fenixnow
Напомни, плиз, ты какой смысл закладывал в поле со звёздочкой? Чтобы отказаться от поиска по имени?

Поле со звездочкой это для красоты, оно сокращает ширину колонки....

@fenixnow
Copy link
Contributor

Можно оставить на выбор пользователя, переключая звездочку по двойному щелчку

Не врубился, что это нам даст? Признак вызгузки в json?
Если идея сводится к тому, что в json должен быть "Этот клиент" доп настроками авторизации и тд, то думаю это лучше добавить руками в json
Могу ошибаться.

@GenVP
Copy link
Contributor Author

GenVP commented Jan 30, 2020

@fenixnow
Если это признак только для исключения выгрузки в json, то лучше сделать его выбираемым, а по-умолчанию установленным. Если можно не редактировать файл настроек вручную - это, по-моему, хорошо.
Если это признак предопределеннного клиента, тогда пусть остается как есть. Только убирать при копировании. Можно только переделать в коде проверку клиента по имени на признак при рефакторинге.

@fenixnow
Copy link
Contributor

fenixnow commented Jan 30, 2020

Можно только переделать в коде проверку клиента по имени на признак при рефакторинге.

Не возражаю, если Леонид не против.

@Pr-Mex
Copy link
Owner

Pr-Mex commented Jan 31, 2020

Можно только переделать в коде проверку клиента по имени на признак при рефакторинге.

Не возражаю, если Леонид не против.

@GenVP @fenixnow
Я за.

@Pr-Mex
Copy link
Owner

Pr-Mex commented Mar 8, 2020

Решено.

@Pr-Mex Pr-Mex closed this as completed Mar 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants