-
Notifications
You must be signed in to change notification settings - Fork 8
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
Написать регулярный тест работоспособности пауков #18
Comments
Можно для начала написать скрипт для разовой проверки. Сейчас пауков всё равно никто не запускает. Можно проверять работоспособность после изменений. |
Можно уточнить: скрипт для запуска команды "scrapy crawl ..." и проверки, что какие-то статьи собрались, или в pytest запускать через что-то вроде CrawlerProcess с ассертами? |
@pacifikus оба варианта лучше, чем сейчас, так что на своё усмотрение. Я за то, чтобы обойтись минимальными изменениями/добавлениями в коде. |
@iggisv9t а что если добавить запуск тестов в actions? |
@pacifikus можно добавить, но могут быть нюансы. Кровлеры -- это всегда на грани добра и зла, и если мы тут будем мощностями гитхаба дёргать сайты, то не может ли нам за это от гитхаба прилететь? |
Хм, в целом по количеству запросов - там немного, но, наверно, лучше не рисковать. Можно тогда просто в readme добавить, что есть тест и как его запускать у себя |
Например раз в неделю запускать все пауки и проверять, что статьи собираются.
The text was updated successfully, but these errors were encountered: