Skip to content

chore: Enable OOM crash capturing tests for native backend#1330

Merged
tustanivsky merged 1 commit intomainfrom
chore/oom-e2e-tests
Apr 2, 2026
Merged

chore: Enable OOM crash capturing tests for native backend#1330
tustanivsky merged 1 commit intomainfrom
chore/oom-e2e-tests

Conversation

@tustanivsky
Copy link
Copy Markdown
Collaborator

Key changes:

  • Remove the filter that excluded OutOfMemory and MemoryCorruption crash types from integration tests when using the native backend
  • Skip the CrashType tag assertion for OutOfMemory and MemoryCorruption crashes since these crash types don't reliably produce the tag in different engine versons
  • Skip the CrashType tag assertion on macOS since it's not supported on this platform due to current Cocoa SDK limitations (no on_crash hook)

Related items:

#skip-changelog

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 2, 2026

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


This PR will not appear in the changelog.


🤖 This preview updates automatically when you update the PR.

@tustanivsky tustanivsky marked this pull request as ready for review April 2, 2026 12:15
Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

@tustanivsky tustanivsky requested review from jpnurmi and mujacica April 2, 2026 12:29
@tustanivsky tustanivsky merged commit 3049524 into main Apr 2, 2026
145 of 147 checks passed
@tustanivsky tustanivsky deleted the chore/oom-e2e-tests branch April 2, 2026 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants