<p></p>
<p><b>@tomhughes</b> requested changes on this pull request.</p>

<p dir="auto">In addition to the inline comments, please reorder the commits so we have Xvfb support before the commit where we start needed it.</p><hr>

<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/6504#discussion_r2554260206">.github/workflows/tests.yml</a>:</p>
<pre style='color:#555'>>        with:
         ruby-version: ${{ matrix.ruby }}
         rubygems: 3.4.10
         bundler-cache: true
+    - name: Enable virtual display & WebGL
+      run: |
+        sudo apt-get install -y xvfb mesa-utils libgl1-mesa-dri
</pre>
<p dir="auto">This should just be added to the existing package install rather than running <code class="notranslate">apt-get</code> twice.</p>

<hr>

<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/6504#discussion_r2554260847">.github/workflows/tests.yml</a>:</p>
<pre style='color:#555'>>        with:
         ruby-version: ${{ matrix.ruby }}
         rubygems: 3.4.10
         bundler-cache: true
+    - name: Enable virtual display & WebGL
+      run: |
+        sudo apt-get install -y xvfb mesa-utils libgl1-mesa-dri
+        Xvfb :99 -screen 0 1024x768x24 &
+        export DISPLAY=:99
+        echo "Using virtual display $DISPLAY"
+        glxinfo | grep "OpenGL renderer" || echo "GL info not available"
</pre>
<p dir="auto">These three lines are just debugging and should just be removed now you've got it working.</p>
<p dir="auto">This step, which will then be just the one line to start <code class="notranslate">Xvfb</code> should be moved down just before the test steps that use it, and after the package install.</p>

<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/6504#pullrequestreview-3497939227">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLPG6KNL77DMMYEAVZT36H7ZRAVCNFSM6AAAAACLKBT5BCVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZTIOJXHEZTSMRSG4">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLL7Y5M32EQT6I5FW7T36H7ZRA5CNFSM6AAAAACLKBT5BCWGG33NNVSW45C7OR4XAZNRKB2WY3CSMVYXKZLTORJGK5TJMV32UY3PNVWWK3TUL5UWJTWQPZIRW.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/6504/review/3497939227</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/6504#pullrequestreview-3497939227",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/6504#pullrequestreview-3497939227",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>