Just looking through the test suites, there's obviously heavy SQLAlchemy fixture coverage, but loaders.py and the actual Wiki Commons Toolforge ingestion logic doesn't have any mock testing.
If the Toolforge schema migrates or something drops on Commons' end, import_entries will crash in production and our CI won't blink. We should wire up some basic unittest.mock fixtures around the network calls.
Just looking through the test suites, there's obviously heavy SQLAlchemy fixture coverage, but
loaders.pyand the actual Wiki Commons Toolforge ingestion logic doesn't have any mock testing.If the Toolforge schema migrates or something drops on Commons' end,
import_entrieswill crash in production and our CI won't blink. We should wire up some basicunittest.mockfixtures around the network calls.