Skip to content

Commit 91545de

Browse files
fix: remove duplicate 1280x800@60 viewport entry
The viewport '1280x800@60' was added twice: - Once in commit 899caee (SDK v0.29.0 update) - Again in commit 4d9565b (feat: add 1280x800 viewport support #97) This resulted in 8 items in getAvailableViewports() while the test expected 7 items, causing test failures. Also removes the duplicate assertion in the test file. Co-authored-by: mason <mason@onkernel.com>
1 parent 440c557 commit 91545de

2 files changed

Lines changed: 0 additions & 2 deletions

File tree

cmd/browsers.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -127,7 +127,6 @@ func getAvailableViewports() []string {
127127
"1280x800@60",
128128
"1024x768@60",
129129
"1200x800@60",
130-
"1280x800@60",
131130
}
132131
}
133132

cmd/browsers_test.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1153,7 +1153,6 @@ func TestGetAvailableViewports_ReturnsExpectedOptions(t *testing.T) {
11531153
assert.Contains(t, viewports, "1440x900@25")
11541154
assert.Contains(t, viewports, "1280x800@60")
11551155
assert.Contains(t, viewports, "1200x800@60")
1156-
assert.Contains(t, viewports, "1280x800@60")
11571156
assert.Contains(t, viewports, "1024x768@60")
11581157
}
11591158

0 commit comments

Comments
 (0)