next-exp/IC

Reorganize, simplify and clean up fixtures

Open

#913 opened on 2024年10月30日

GitHub で見る
 (0 comments) (0 reactions) (1 assignee)Python (77 forks)auto 404
easygood first issuehelp wantedtesting

Repository metrics

Stars
 (10 stars)
PR merge metrics
 (PR metrics pending)

説明

We use many fixtures. Most of them are thrown into invisible_cities/conftest.py without thinking much. Let's reorganize them following the following logic:

  • If they are used only in one file, move them to that file
  • If they are used only in one folder, move them to the conftest.py file in that folder
  • Otherwise, keep them in invisible_cities/conftest.py

Moreover, some of those fixtures are monstrous. Let's try to simplify them or replace them with test files whenever possible to keep things readable.

コントリビューターガイド