Skip to content

Memoize ActiveRecord::Migrator.migrations in tests

What does this MR do?

Monkey-patches ActiveRecord so that the list of migrations is only read once in a test run

Are there points in the code the reviewer needs to double check?

Why was this MR needed?

This is free time off every spec with no downside (that I can see). Saves 0.2-0.5 seconds per spec in my local testing (which I wouldn't overly trust).

Screenshots (if relevant)

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Spawned off #23034 (closed)

Merge request reports