chore: Update tests for structured logs

[tustanivsky]

The existing unit tests for BeforeLogHandler and log attributes had two major issues:

• Test stability and isolation: The tests required subsystem re-initialization mid-test (InitializeWithSettings), which caused flakiness and interfered with subsequent tests. In some cases, running these tests left the editor session in a broken state where the sample app became unusable in Play Mode (PIE).

• Lack of end-to-end validation: The tests were shallow: they only verified that the handler was invoked and received the correct data in-process. They never validated that the modified log actually reached Sentry with the expected attributes. As a result, a passing unit test did not guarantee the feature worked end-to-end.

Key changes:

• Removed problematic unit tests for BeforeLogHandler and global log attributes
• Added e2e coverage for the same scenarios in desktop and Android integration tests:
  • BeforeLogHandler adding and removing attributes
  • Global attributes set via SentrySubsystem->SetAttribute() attached to logs
  • Global attributes removed via SentrySubsystem->RemoveAttribute() not appearing on logs
• Added CppBeforeLogHandler to the sample app, wired via -ini: settings override in tests
• Extended RunLogTest() to exercise global attribute set/remove before sending the log

Depends on:

• Add optional Fields parameter to log query functions app-runner#37

Related items:

• Replace unit tests that rely on subsystem re-initialization with e2e integration tests #1225

#skip-changelog

[github-actions]

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.}