test: replace sleeps with freezegun time ticks in tests

[bndikt]

Summary

• replace wall-clock sleeps in tests with freezegun time ticks
• wire SQLite test connections to mock_sqlite_time so DATETIME('now') follows frozen Python time
• freeze tests at the current UTC second to match SQLite timestamp precision

[chrisburr]

As this is defined in two places it probably belongs in diracx-testing/src/diracx/testing/time.py

[chrisburr]

Similarly, this is now in three places, maybe it can be simplified with a helper:

# diracx-testing/src/diracx/testing/time.py
def install_sqlite_time_mock(engine) -> None:
    """Make SQLite's date/time functions follow frozen Python time on `engine`.

    Registers `mock_sqlite_time` as a "connect" listener so every new DBAPI
    connection gets the mocked DATETIME()/JULIANDAY()/... functions. Accepts a
    sync Engine or an AsyncEngine.
    """
    sync_engine = getattr(engine, "sync_engine", engine)
    sqlalchemy.event.listen(sync_engine, "connect", mock_sqlite_time)

# utils.py:339  (create_db_schemas)
install_sqlite_time_mock(db.engine)

# test_sandbox_metadata.py:21  (sandbox_metadata_db fixture)
install_sqlite_time_mock(sandbox_metadata_db.engine)

# test_freeze_time.py:43
if with_mock:
    install_sqlite_time_mock(engine)

[read-the-docs-community]

Documentation build overview

📚 diracx | 🛠️ Build #33307984 | 📁 Comparing 9515104 against latest (8b944bf)

  🔍 Preview build  

1 file changed

± dev/reference/pixi-tasks/index.html

[aldbr]

@chrisburr I noticed freezegun and sqlalchemy are used within diracx-testing but not part of the pyproject.toml. I assume this was forgotten, unless I miss something here?

[aldbr]

ddev: usually we add the import at the top of the modules, we should probably add an explicit rule about that (guidelines for devs)