Skip to content

chore: switch stub tests - #2795

Open
bitsandfoxes wants to merge 1 commit into
mainfrom
chore/switch-stub-tests
Open

chore: switch stub tests#2795
bitsandfoxes wants to merge 1 commit into
mainfrom
chore/switch-stub-tests

Conversation

@bitsandfoxes

Copy link
Copy Markdown
Contributor

So the native bindings and what we have on the stubs does no longer drift.

#skip-changelog

@cursor cursor Bot left a comment

Copy link
Copy Markdown

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.

Reviewed by Cursor Bugbot for commit ec44e82. Configure here.

{
Assert.That(Regex.IsMatch(stubContent, $@"\b{Regex.Escape(entryPoint)}\s*\("), Is.True,
$"Switch binding '{entryPoint}' not found in {stubPath}");
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

New test fails against current stubs

Medium Severity

Stub_ContainsEverySwitchNativeBinding requires every Switch __Internal P/Invoke to appear in sentry_native_stubs.c, but the stubs omit bindings that the Switch assembly still exports, including sentry_attach_file, sentry_attach_bytes, sentry_clear_attachments, sentry_options_set_logger_enabled_when_crashed, and sentry_app_hang_pause. The new EditMode test fails immediately.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit ec44e82. Configure here.

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.

1 participant