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

добавлено unit тестирование #424

Merged
merged 2 commits into from Jun 12, 2020

Conversation

matsievskiysv
Copy link

PR checklist

Пожалуйста, убедитесь, что Ваш PR удовлетворяет требованиям:

  • Изменения были протестированы (make examples)
  • Изменения отражены в документации к шаблону
  • Код соответствует правилам индентации шаблона (make indent)

Тип PR

Отметьте графы, которые относятся к данному PR:

  • Bugfix
  • Feature
  • Форматирование стиля кода
  • Изменение текста шаблона
  • Изменение документации
  • Другое:

Описание PR

Добавлены скрипты для проведения простейших тестов.

  • test_src -- проверка исходников:
    • кодировка файлов
    • стиль разделителей строк (LF vs CRLF)
    • длина строк в файлах
    • проверка логов сборки
  • test_pdf -- проверка pdf файлов:
    • отсутствие аннотаций
    • наличие TOC
    • размер страниц
    • размер полей страниц
    • нумерация списка литературы в диссертации
    • простая проверка ссылок в списках рисунков и таблиц

Note

  • В данный момент весь код не соответствует правилам индентации.
  • При проверке файлов, сгенерированных при помощи make examples возникает множетво ошибок. Например, в файлах -draft отсутствуют закладки. Можно доставать информацию о pdf файле из его названия, и применять соответствующие настройки к тестам.
  • Нужно больше тестов.

Смежные обсуждения

Обсуждение unit-тестирования в #364, CI - в #152

Тестирование шаблона

Тестирование производилось в среде

  • Windows
  • Windows Cygwin
  • GNU/Linux (на основе Debian)
  • GNU/Linux (на основе RedHat)
  • Arch Linux
  • Docker (ссылка на контейнер):

Другая информация

Сделано на основе https://blog.martisak.se/2020/05/16/latex-test-cases/

@matsievskiysv
Copy link
Author

@Lenchik этот PR не готов для ветки master. Может стоит создать отдельную ветку для работы над тестами?

@Lenchik
Copy link
Collaborator

Lenchik commented Jun 10, 2020

Чтобы туда пулл реквесты добавлять? Так удобней будет?

@matsievskiysv
Copy link
Author

Так будет удобнее, если другие участники обсуждения #364 присоединятся к написанию тестов.

@Lenchik
Copy link
Collaborator

Lenchik commented Jun 10, 2020

Заведена отдельная ветка для проб по юнит тестам - feature/unit-tests -
https://github.com/AndreyAkinshin/Russian-Phd-LaTeX-Dissertation-Template/tree/feature/unit-tests

@matsievskiysv matsievskiysv marked this pull request as ready for review June 11, 2020 13:19
@matsievskiysv matsievskiysv changed the base branch from master to feature/unit-tests June 11, 2020 13:21
@Lenchik Lenchik merged commit 1cb890f into AndreyAkinshin:feature/unit-tests Jun 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants