ugeneunipro / ugene

UGENE is free open-source cross-platform bioinformatics software
http://ugene.net
GNU General Public License v2.0
207 stars 60 forks source link

Fix windows tests #1651

Closed EvelinaBiserova closed 2 months ago

EvelinaBiserova commented 2 months ago

В #1645 упали 2 виндовых теста:

Ни один из них к предложенным в том пр правкам отношения не имеет.

yalgaer commented 2 months ago

Что хочу добавить про флаки: чинить их нужно фундаментально, например заменив двойной клик на другое действие просто потому что в текущем контексте двойной клик приводит к флаку.

Сдвиг таймина на 5% не стоит ресурсов и вот почему:

  1. Показывать что флак починен нужно не 1-2 прогонами а десятком прогонов проблемного теста. Это можно сделать через запуск по списку в Teamcity: написать имя теста 50 раз, например.

  2. Если мы изменили базовую утилиту которая используется всеми тестами - то флаки могут полезть в неожиданных местах. Надо прогнать абсолютно все раз 5 чтобы показать что новых флаков нет.

Как результат, если решение не точное, то лучше ограничить фикс влиянием только на проблемный тест, а не на все тесты и легче всего починить флак полной заменой действия (если это не меняет цель сценария).

EvelinaBiserova commented 2 months ago

Согласна с тем, как проверять починку теста