Skip to content

test(quality): expand feature validation coverage #84

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
DavidBabinec merged 10 commits into from
Jun 26, 2026
Merged

test(quality): expand feature validation coverage #84

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
c7281e9
test(quality): expand feature validation coverage
DavidBabinec Jun 23, 2026
ef7155b
fix(settings): contain preferences modal on mobile
DavidBabinec Jun 23, 2026
10eb2e8
test(users): cover non-owner lifecycle validation
DavidBabinec Jun 23, 2026
af6d516
test(admin): cover global toolbar live targets
DavidBabinec Jun 23, 2026
4dd3abc
fix(admin): harden workspace shell recovery
DavidBabinec Jun 23, 2026
896bf79
fix(admin): contain right sidebar on mobile
DavidBabinec Jun 23, 2026
7fd476f
test(server): clarify audit SQL null bindings
DavidBabinec Jun 24, 2026
e2fd7b6
test(quality): clear code quality blockers
DavidBabinec Jun 24, 2026
ecc347d
test(quality): avoid CodeQL step-up false positive
DavidBabinec Jun 24, 2026
26da03a
test(quality): resolve feature validation rebase drift
DavidBabinec Jun 26, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
test(server): clarify audit SQL null bindings
  • Loading branch information
@DavidBabinec
DavidBabinec committed Jun 26, 2026
commit 7fd476fd9fb1d4bf37281875622dd6628be78573
9 changes: 6 additions & 3 deletions src/__tests__/server/auditLogEdges.test.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -105,6 +105,9 @@ describe('audit log edge semantics', () => {

for (let i = 0; i < 105; i += 1) {
const metadata = i === 104 ? { nested: { unsupported: true } } : { sequence: i }
const actorUserId: string | null = null
const ipAddress: string | null = null
const userAgent: string | null = null
await harness.db`
insert into audit_events (
id,
Expand All @@ -119,13 +122,13 @@ describe('audit log edge semantics', () => {
)
values (
${`audit-edge-${i.toString().padStart(3, '0')}`},
${null},
${actorUserId},
Comment thread
github-code-quality[bot] marked this conversation as resolved.
Fixed
${'publish'},
${'site'},
${'site'},
${metadata},
${null},
${null},
${ipAddress},
Comment thread
github-code-quality[bot] marked this conversation as resolved.
Fixed
${userAgent},
Comment thread
github-code-quality[bot] marked this conversation as resolved.
Fixed
${new Date(baseTime + i * 1000).toISOString()}
)
`
Expand Down