Skip to content

test(export): set creation date on comments in UserExportTest - #8462

Merged
mejo- merged 1 commit into
mainfrom
test/fix_user_export_test
Sep 29, 2026
Merged

mejo- merged 1 commit into
mainfrom
test/fix_user_export_test

Conversation

@mejo-

@mejo- mejo- commented Sep 29, 2026

Copy link
Copy Markdown
Member

Comment::jsonSerialize() now includes creation_time and throws when no creation date is set, which broke both UserExportTest cases.

Assisted-by: ClaudeCode:claude-opus-5.5

Checklist

  • Code is properly formatted
  • Sign-off message is added to all commits

馃 AI (if applicable)

  • The content of this PR was partly or fully generated using AI (N/A)

Comment::jsonSerialize() now includes creation_time and throws when no
creation date is set, which broke both UserExportTest cases.

Signed-off-by: Jonas <jonas@freesources.org>
Assisted-by: ClaudeCode:claude-opus-5.5
@github-actions

Copy link
Copy Markdown
Contributor

馃悽 Performance warning.
It looks like the query count of the integration tests increased with this PR.
Database query count is now 206505 was 198420 (+4.07%)
Please check your code again. If you added a new test this can be expected and the base value in tests/integration/base-query-count.txt can be increased.

@mejo-
mejo- merged commit c06b76a into main Sep 29, 2026
41 checks passed
@mejo-
mejo- deleted the test/fix_user_export_test branch September 29, 2026 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: 鈽戯笍 Done

Development

Successfully merging this pull request may close these issues.

2 participants