Skip to content

Conversation

JosephSanjaya
Copy link
Collaborator

📱 Description

This PR fixes SLF4J and Byte Buddy agent warnings that occur when running tests in Android Studio or via Gradle.
These warnings do not affect production builds or the APK. They are specific to the test environment only.

  • SLF4J warnings happen because no logging backend was configured; adding slf4j-android resolves this by routing logs to Logcat.
  • Byte Buddy agent warnings occur due to dynamic Java agent loading during tests; configuring JVM args (-XX:+EnableDynamicAgentLoading) suppresses the warning.
    ℹ️ Note on EnableDynamicAgentLoading
    Suppressing the warning by enabling dynamic agent loading is safe in our case. It only affects the test JVM environment and has no impact on runtime behavior or production APKs. Disabling the suppression would simply keep printing the warning, but still would not break our tests.

Platform

  • Android
  • iOS
  • Games-Unity
  • DevOps (AWS)
  • Website
  • C/Golang

🎯 Type of Change

  • 🐛 Bug fix
  • ✨ New feature
  • 🔧 Refactoring (code change that neither fixes a bug nor adds a feature)
  • 📚 Documentation update
  • 🎨 UI/UX improvement
  • ⚡ Performance improvement
  • 🧪 Test addition or improvement

📋 Changes

  • Added slf4j-android as a test dependency to provide a proper SLF4J binding and avoid "StaticLoggerBinder" warnings.
  • Configured JVM args for Gradle test tasks to enable dynamic agent loading, suppressing Byte Buddy agent warnings during test runs.

🔗 Related Issues

🧪 Tests Status

  • Tests ran successfully locally?
  • Added more tests? How many?
  • Code coverage percentage of the codebase: __%

📸 Screenshots/Videos

Before After
{A58DA21F-C1E3-40A7-8866-414471BCE8BE} {687724C6-ED3C-45A8-AF9A-4D6B6EEAAED7}

🎯 Reviewers

@kcw-grunt, @josikie

…st env

- Added `slf4j-android` as a test dependency to provide a proper SLF4J binding
  and avoid "StaticLoggerBinder" warnings.
- Configured JVM args for Gradle test tasks to enable dynamic agent loading,
  suppressing Byte Buddy agent warnings during test runs.
@JosephSanjaya JosephSanjaya self-assigned this Aug 25, 2025
@JosephSanjaya JosephSanjaya added the bug Something isn't working label Aug 25, 2025
@kcw-grunt kcw-grunt added this to the Sprint 33 milestone Aug 26, 2025
Copy link
Collaborator

@kcw-grunt kcw-grunt left a comment

Choose a reason for hiding this comment

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

LGTM

@kcw-grunt kcw-grunt merged commit 9b191b7 into develop Aug 26, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants