Skip to content

chore(tests): skip tests relying on unsupported APIs on NX and PS platforms#1517

Merged
tustanivsky merged 2 commits intomasterfrom
fix/consoles-cache
Feb 13, 2026
Merged

chore(tests): skip tests relying on unsupported APIs on NX and PS platforms#1517
tustanivsky merged 2 commits intomasterfrom
fix/consoles-cache

Conversation

@tustanivsky
Copy link
Collaborator

Bumping sentry-native to 0.12.7 for Switch and PlayStation introduced the following issues:

  • The cache tests added in feat: offline caching (inproc & breakpad) #1490 and feat(crashpad): offline caching #1493 rely on utime() to modify file timestamps. This POSIX function is not available in either the Nintendo Switch or PlayStation SDKs which causes build failures when compiling sentry_test_unit for these platforms (see the linked Switch and PlayStation CI failures).

  • A new path_rename test exercises sentry__path_rename which ultimately calls rename() on Unix. The NX filesystem does not support rename() so this test also fails on Switch.

Key changes:

  • Guard <utime.h> include and set_file_mtime() implementation with platform checks to avoid linker errors
  • Skip all test_cache tests on NX and PlayStation platforms where utime() is unavailable
  • Skip path_rename test on NX, following the same pattern already used for other NX-unsupported tests in that file

#skip-changelog

Copy link
Collaborator

@jpnurmi jpnurmi left a comment

Choose a reason for hiding this comment

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

Do we run integration tests on consoles? Will test_integration_cache.py (using os.utime) also create issues?

@tustanivsky
Copy link
Collaborator Author

Will test_integration_cache.py (using os.utime) also create issues?

No - to run integration tests on consoles we have a completely different setup that does not rely on the .py scripts used by sentry-native.

@tustanivsky tustanivsky merged commit 76fbfd6 into master Feb 13, 2026
45 checks passed
@tustanivsky tustanivsky deleted the fix/consoles-cache branch February 13, 2026 10:41
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