Update build optimizations

[david-livefront]

🎟️ Tracking

N/A

📔 Objective

A minor update to simplify parts of the build scripts and improve performance in builds.

⏰ Reminders before review

• Contributor guidelines followed
• All formatters and local linters executed and passed
• Written new unit and / or integration tests where applicable
• Protected functional changes with optionality (feature flags)
• Used internationalization (i18n) for all UI strings
• CI builds passed
• Communicated to DevOps any deployment requirements
• Updated any necessary documentation (Confluence, contributing docs) or informed the documentation team

🦮 Reviewer guidelines

• 👍 (:+1:) or similar for great changes
• 📝 (:memo:) or ℹ️ (:information_source:) for notes or general info
• ❓ (:question:) for questions
• 🤔 (:thinking:) or 💭 (:thought_balloon:) for more open inquiry that's not quite a confirmed issue and could potentially benefit from discussion
• 🎨 (:art:) for suggestions / improvements
• ❌ (:x:) or ⚠️ (:warning:) for more significant problems or concerns needing attention
• 🌱 (:seedling:) or ♻️ (:recycle:) for future improvements or indications of technical debt
• ⛏ (:pick:) for minor or nitpick changes

[claude]

Claude finished @david-livefront's task in 2m 4s —— View job

---

Code Review Summary

Status: Approved

Overview

This PR consolidates duplicate test task configuration from 9 individual module build.gradle.kts files into a single centralized block in the root build.gradle.kts under subprojects { }. This is a good maintainability improvement that follows the DRY principle.

Changes Reviewed

File	Changes
build.gradle.kts	Added centralized test configuration in subprojects block
gradle.properties	Added -XX:+HeapDumpOnOutOfMemoryError JVM flag
9 module build files	Removed duplicate test configuration blocks

Key Details

1. Test Configuration Centralization: All common test settings (useJUnitPlatform, forkEvery, maxHeapSize, maxParallelForks, jvmArgs) are now defined once in the root build.gradle.kts.

2. forkEvery Optimization: Increased from 100 to 500 to reduce JVM restarts during test execution, which should improve test performance.

3. maxHeapSize: The 2g heap size setting is preserved in the centralized configuration.

4. Heap Dump Flag: Added -XX:+HeapDumpOnOutOfMemoryError to gradle.properties for better debugging of OOM issues.

5. Module-Specific Config: The ui module correctly retains its test task block containing only the module-specific android.sourceSets configuration.

Verification

• All common test settings are properly centralized
• Module-specific configurations are preserved where needed
• The @Suppress("UselessCallOnNotNull") annotation on jvmArgs.orEmpty() is appropriate

No issues identified.

---

Generated with Claude Code

[david-livefront]

I have moved the test configs to this singular spot for convenience but there are also some small changes too.

• I have increased the forkEvery count to avoid spinning up so many JVM instances
• I have completely removed the maxHeapSize property to allow the jvm args from the gradle.properties to handle it (-Xmx4g).

[SaintPatrck]

Nice! @LRNcardozoWDF this may help the extra resource consumption you were noticing.

[david-livefront]

I ended up rolling back the maxHeapSize change as we were hitting OOM exceptions

[david-livefront]

Required for some warnings coming from the configuration on release builds.

Thoughts on if we should just remove the configuration cache completely?

[SaintPatrck]

Removing the configuration cache completely will probably increase build times, so I'm hesitant to go that route. What are the warnings you are seeing?

[david-livefront]

You can see the issues here: https://github.com/bitwarden/android/actions/runs/21452527392/job/61788569410

[github-actions]

Checkmarx One – Scan Summary & Details – d80ec312-67f2-43a5-96a9-a00c04d741e4

Great job! No new security vulnerabilities introduced in this pull request

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.02%. Comparing base (92ba38c) to head (ba2eda1).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #6433   +/-   ##
=======================================
  Coverage   86.02%   86.02%           
=======================================
  Files         775      775           
  Lines       56224    56224           
  Branches     8132     8132           
=======================================
  Hits        48366    48366           
  Misses       5026     5026           
  Partials     2832     2832           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.