[openstreetmap/openstreetmap-website] Improving test harness run time (PR #4708)

[Josh Thompson]

Doing a quick audit of the test harness, to see if there's ways to make it run faster/use less compute time.

(I'd like to to run _faster_ before simply throwing more cycles at it, but 'more cycles' is dramatically cheaper than 'developers twiddling thumbs waiting for tests to finish', and each test harness gets run against many different versions of Ruby, so speeding up the tests at all does cool compounding things for everyone.

This is not ready to merge, perhaps that is obvious, but I wanted to compare the changes I'm making (one at a time) to the Circle CI test run time.

![2024-04-16 at 10 24 AM](https://github.com/openstreetmap/openstreetmap-website/assets/5198260/618f204c-bd97-44d5-a569-21e2bac96ad7)

here's the tests as being run on another PR: https://github.com/openstreetmap/openstreetmap-website/pull/4706

You can drill into the details, and see the tests take times like:

- 7m 0s
- 8m 0s
- 7m 26s
- 8m 7s
- 8m 31s

So, lets see if we can nudge that down. I expect 'turning off logging' to lead to a 6% decrease in test time. That would be each of these figures going down by about 30 seconds.
You can view, comment on, or merge this pull request online at:

  https://github.com/openstreetmap/openstreetmap-website/pull/4708

-- Commit Summary --

  * Disable logging in tests for speed benefits?

-- File Changes --

    M config/environments/test.rb (3)

-- Patch Links --

https://github.com/openstreetmap/openstreetmap-website/pull/4708.patch
https://github.com/openstreetmap/openstreetmap-website/pull/4708.diff

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/4708
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/4708 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240416/2bbb8a19/attachment.htm>