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

Статусы выполнения юнит-тестов не различимы для дальтоников #448

Closed
karnilaev opened this issue Mar 19, 2015 · 7 comments

Comments

@karnilaev
Copy link
Contributor

Я, как дальтоник, не могу понять, выполнился юнит-тест с ошибкой, или без, из-за того, что красный кружок и зеленый кружок для меня практически идентичны.
Предлагаю красный кружок исправить на красный крестик. Так дальтоники смогут отличить "хорошие" тесты от "плохих"

@artbear
Copy link
Member

artbear commented Mar 19, 2015

На крестик согласен.
Я давно хочу немного доработать интерфейс в плане повышения юзабилити.
Для тестов, которые были пропущены, можно еще какой-нибудь значок поставить (например, пауза или квадрат/стоп)

@artbear
Copy link
Member

artbear commented Mar 19, 2015

Жду пулл-реквест

@artbear artbear modified the milestone: 2.8.1.0 Mar 19, 2015
@artbear
Copy link
Member

artbear commented Mar 19, 2015

Выполнено.

@karnilaev
Copy link
Contributor Author

Спасибо! Я пока не понял как правильно пул-реквесты делать. Точнее не понял, как обновить свой форк до состояния оригинального репозитария. Как только разберусь, буду присылать правильные пул-реквесты )

pumbaEO pushed a commit that referenced this issue Mar 20, 2015
* remotes/upstream/develop:
  2.8.0.21 Добавил возможность поиска элементов по ключевым полям (новая колонка Ключ) #380
  ОФ+УФ: Для падающих тестов новая картинка - красный крестик (вместо кружка) увеличенного размера fixed #448
  Вернулся к хранению настроек в хранилище 1С при ошибочной загрузке тестов fixed #442
  Дополнение к вопросу об отключении Информатора для повышения юзабельности #442
  Исправлена ошибка в УФ: метод СоздатьДанныеПоТабличномуДокументу имел сигнатуру вызова, отличающуюся от ОФ.
  Добавлена проверка и выдача юзабельного сообщения об ошибке, если 1С:Предприятие 8 не было запущено в режиме Менеджера тестирования
  ОФ: Если при загрузке тестов при включенном Информаторе падала 1С, при следующей загрузке пользователю будет дана возможность отключить и Информатора, и загрузку тестов и загрузить браузер тестов в нужном режиме. fixed #442 fixed #412 Также добавлена форма настройки для ОФ
  ОФ+УФ: Если при загрузке тестов падала 1С, при следующей загрузке пользователю будет дана возможность отключить загрузку и загрузить браузер тестов для ручного выбора тестов. #442

Conflicts:
	src/xddTestRunner/Form/Форма/form
	src/xddTestRunner/Form/ФормаВыбораПодсистемы/form
	src/xddTestRunner/und/7e542ac7-435a-4390-85c5-964b3a399bee.0
	src/xddTestRunner/und/versions
	xddTestRunner.epf
@artbear
Copy link
Member

artbear commented Mar 20, 2015

Тебе нужно добавить ещё один remote/origin для своего локального репо. 1й
remote указывает на твой форк, второй на основной репо хЮнит на Гитхабе.
Затем делаешь pull со второго репо и получаешь к себе наши изменения

@silverbulleters-github-service

Если изменения небольшие можешь просто у себя fork удалить и потом сделать
форк заново. Обработку при этом можешь сохранить и потом сделать
сравнение/объединение внешних обработок.

Можно по инструкции github
http://stackoverflow.com/questions/7244321/how-to-update-github-forked-repository

20 марта 2015 г., 9:18 пользователь karnilaev [email protected]
написал:

Спасибо! Я пока не понял как правильно пул-реквесты делать. Точнее не
понял, как обновить свой форк до состояния оригинального репозитария. Как
только разберусь, буду присылать правильные пул-реквесты )


Reply to this email directly or view it on GitHub
#448 (comment)
.

Сосна Евгений Викторович
mail:[email protected]

@karnilaev
Copy link
Contributor Author

Спасибо большое. Разобрался! :)

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