feat: web progress bar#423
Conversation
|
|
| test_runs_count: int = 1, | ||
| config: dict[str, Any] | None = None, | ||
| ) -> None: | ||
| from imgtests.suites.map import ALL_SUITES # noqa: PLC0415 |
There was a problem hiding this comment.
Лучше по возможности вообще не делать так импорты без крайней необходимости, а если таким образом попытка избежать циклических импортов -> скорее всего требуется провести рефакторинг кода или поместить логику в другой модуль.
There was a problem hiding this comment.
Аналогичная ситуация имеется здесь
https://github.com/moevm/OS-Image-Testing/blob/web-prog-ui/src/imgtests/runner.py#L553
Поэтому предлагаю сделать небольшой рефакторинг. Займусь этим в отдельной ветке
There was a problem hiding this comment.
Хорошо, в любом случае рефакторинг было бы не плохо сделать.
| with Path.open(PROG_LOG_PATH, encoding="utf-8") as file: | ||
| data = json.load(file) | ||
| return JsonResponse(data) | ||
| except json.decoder.JSONDecodeError: |
There was a problem hiding this comment.
Тут скорее 238 и 239 строки нужно обернуть, 240 уже не вернёт такую ошибку.
| progress_file = Path(PROG_LOG_PATH) | ||
| if progress_file.exists(): | ||
| try: | ||
| with Path.open(PROG_LOG_PATH, encoding="utf-8") as file: |
There was a problem hiding this comment.
А если параллельно запускать это всё, то не будет ли гонки при записи в один и тот же файл и потом как отличить где какой запуск?
2 participants
#408