detached 7839b6b

test(ui): skip the allocation guard under the race detector

commit
7839b6b350ade1912b051902c1dd72cdf2b2f140
author
brice · · 2026-10-01 16:33 UTC
parent
ee44fbd
signature
GPG VerifiedCD88F9E564E0102646FB6077A1A07C7C7C3E349D · arch-gpg Verified for the committer email
3 files · +14 -0
A internal/ui/race_off_test.go +5 -0
@@ -0,0 +1,5 @@
1+//go:build !race
2+
3+package ui
4+
5+const raceEnabled = false
@@ -0,0 +1,5 @@
1+//go:build !race
2+
3+package ui
4+
5+const raceEnabled = false
A internal/ui/race_on_test.go +6 -0
@@ -0,0 +1,6 @@
1+//go:build race
2+
3+package ui
4+
5+// raceEnabled: the race detector adds allocations, so alloc guards skip.
6+const raceEnabled = true
@@ -0,0 +1,6 @@
1+//go:build race
2+
3+package ui
4+
5+// raceEnabled: the race detector adds allocations, so alloc guards skip.
6+const raceEnabled = true
M internal/ui/renderer_gpu_test.go +3 -0
@@ -189,6 +189,9 @@ const allocBaselineRenderer = 244
189189 // changes on every frame and an unchanged structure. It runs on the real GPU
190190 // path because testify mocks would allocate.
191191 func TestAllocRendererSteadyFrame(t *testing.T) {
192+ if raceEnabled {
193+ t.Skip("race instrumentation changes allocation counts")
194+ }
192195 r, tg := gpuRenderer(t)
193196 var out Output
194197 n := 0
@@ -189,6 +189,9 @@ const allocBaselineRenderer = 244
189 // changes on every frame and an unchanged structure. It runs on the real GPU189 // changes on every frame and an unchanged structure. It runs on the real GPU
190 // path because testify mocks would allocate.190 // path because testify mocks would allocate.
191 func TestAllocRendererSteadyFrame(t *testing.T) {191 func TestAllocRendererSteadyFrame(t *testing.T) {
192+ if raceEnabled {
193+ t.Skip("race instrumentation changes allocation counts")
194+ }
192 r, tg := gpuRenderer(t)195 r, tg := gpuRenderer(t)
193 var out Output196 var out Output
194 n := 0197 n := 0

Keyboard shortcuts

Single-key shortcuts are on. ctrl k and esc work whatever this says.

ctrl k :
go to a section, branch, tag, file or repository
?
this list
j k
move the row cursor down / up
enter
open the row under the cursor; on Files, a directory opens or closes in place
↑ ↓ ← → home end
in the file tree (focus on a row): previous / next row, close / open a directory or go to its parent / first entry, first / last row. They never open a file
l h
on Files: open a directory or a file / close a directory or go up to its parent, then to the parent page. Elsewhere: into the side panel (on a merge request, its files) / back to the content; there j k move between its controls
g g G
first / last row
/
filter the list; on Files, find a file anywhere in the repository (esc clears, enter opens the best match)
b
switch branch or tag
g s g f g c g b g m g i g r g a
tabs: summary, files, commits, branches & tags, merge requests, issues, releases, actions (when available)
g h
home
w
wide mode: use the whole width of the screen, or not (remembered)
y
copy a permalink pinned to the commit
d u x
diff: split / unified / collapse or expand all
e E
diff: the whole file in view / every file, with the lines the hunks hide
n
merge request: next note in the diff
esc
close a dialog, leave a field

Branches 1

Tags 12

↑↓ moveenter openesc close