<p dir="auto">Doing a quick audit of the test harness, to see if there's ways to make it run faster/use less compute time.</p>
<p dir="auto">(I'd like to to run <em>faster</em> 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.</p>
<p dir="auto">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.</p>
<p dir="auto"><a href="https://github.com/openstreetmap/openstreetmap-website/assets/5198260/618f204c-bd97-44d5-a569-21e2bac96ad7">2024-04-16.at.10.24.AM.png (view on web)</a></p>
<p dir="auto">here's the tests as being run on another PR: <a class="issue-link js-issue-link" data-error-text="Failed to load title" data-id="2246307625" data-permission-text="Title is private" data-url="https://github.com/openstreetmap/openstreetmap-website/issues/4706" data-hovercard-type="pull_request" data-hovercard-url="/openstreetmap/openstreetmap-website/pull/4706/hovercard" href="https://github.com/openstreetmap/openstreetmap-website/pull/4706">#4706</a></p>
<p dir="auto">You can drill into the details, and see the tests take times like:</p>
<ul dir="auto">
<li>7m 0s</li>
<li>8m 0s</li>
<li>7m 26s</li>
<li>8m 7s</li>
<li>8m 31s</li>
</ul>
<p dir="auto">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.</p>

<hr>

<h4>You can view, comment on, or merge this pull request online at:</h4>
<p>  <a href='https://github.com/openstreetmap/openstreetmap-website/pull/4708'>https://github.com/openstreetmap/openstreetmap-website/pull/4708</a></p>

<h4>Commit Summary</h4>
<ul>
  <li><a href="https://github.com/openstreetmap/openstreetmap-website/pull/4708/commits/dd7754117bb913f0b916b04107b3dc3af9bb7e6c" class="commit-link">dd77541</a>  Disable logging in tests for speed benefits?</li>
</ul>

<h4 style="display: inline-block">File Changes </h4> <p style="display: inline-block">(<a href="https://github.com/openstreetmap/openstreetmap-website/pull/4708/files">1 file</a>)</p>
<ul>
  <li>
    <strong>M</strong>
    <a href="https://github.com/openstreetmap/openstreetmap-website/pull/4708/files#diff-77f322f5ab0c54b1f2793b339574333dc55889645d910a93ede0fd7aa13b217a">config/environments/test.rb</a>
    (3)
  </li>
</ul>

<h4>Patch Links:</h4>
<ul>
  <li><a href='https://github.com/openstreetmap/openstreetmap-website/pull/4708.patch'>https://github.com/openstreetmap/openstreetmap-website/pull/4708.patch</a></li>
  <li><a href='https://github.com/openstreetmap/openstreetmap-website/pull/4708.diff'>https://github.com/openstreetmap/openstreetmap-website/pull/4708.diff</a></li>
</ul>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />Reply to this email directly, <a href="https://github.com/openstreetmap/openstreetmap-website/pull/4708">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLKQDAR3HAJM6MP4QEDY5VGRBAVCNFSM6AAAAABGJWCJB6VHI2DSMVQWIX3LMV43ASLTON2WKOZSGI2DMNBVG44TAMA">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLPTAA3E3OBY3LV4QKTY5VGRBA5CNFSM6AAAAABGJWCJB6WGG33NNVSW45C7OR4XAZNFJFZXG5LFVJRW63LNMVXHIX3JMTHILZR2FQ.gif" height="1" width="1" alt="" /><span style="color: transparent; font-size: 0; display: none; visibility: hidden; overflow: hidden; opacity: 0; width: 0; height: 0; max-width: 0; max-height: 0; mso-hide: all">Message ID: <span><openstreetmap/openstreetmap-website/pull/4708</span><span>@</span><span>github</span><span>.</span><span>com></span></span></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/openstreetmap/openstreetmap-website/pull/4708",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/4708",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>