Add unit tests for RequestBuilder listeners, Display EDT exception, BrowserWindow fallback, and Purchase sync

[shai-almog]

Motivation

• Increase unit-test coverage for several internal listener/helper classes used by the REST stack, UI EDT exception handling and BrowserWindow fallback, and the Purchase receipts synchronization path.
• Exercise real API flows (no reflection) so the asynchronous listeners and runnable synchronization paths are exercised via the test harness.

Description

• Added RequestBuilderActionListenerTest which exercises async/sync JSON map callbacks, properties→business object parsing (fetchAsProperties / fetchAsPropertyList), and the legacy async Callback paths for bytes and strings to hit GetAsBytesAsyncImplActionListener and GetAsStringAsyncImplActionListener.
• Added SampleItem test business object used by the properties tests (implements PropertyBusinessObject).
• Extended DisplayTest with testEdtExceptionCapturedByCrashReporter to enable async stack traces and verify Display$EdtException propagation to a CrashReport.
• Extended BrowserWindowTest with fallbackWindowUsesBrowserForm to exercise the BrowserWindow.BrowserForm fallback code path when native windows are not available.
• Extended PurchaseTest with testSynchronizeReceiptsSyncWaitsForAsyncFetch to exercise the SynchronizeReceiptsSyncRunnable / async receipt fetch path used by synchronizeReceiptsSync.
• Note: CSSBorder.LinearGradient could not be reached via public APIs without reflection, so no direct test was added for it.

Testing

• No CI or Maven tests were executed as part of this change in this PR; the tests were added to maven/core-unittests.
• To run the module tests locally use the project-recommended command: cd maven && mvn -pl core-unittests -am -DunitTests=true -Dmaven.javadoc.skip=true -Plocal-dev-javase test.
• To run the full CI-like verification use: cd maven && mvn clean verify -DunitTests=true -pl core-unittests -am -Dmaven.javadoc.skip=true -Plocal-dev-javase.
• If you want a quick smoke loop while iterating use the helper script: ./scripts/fast-core-unit-smoke.sh.

---

Codex Task

[github-actions]

✅ Continuous Quality Report

Test & Coverage

• ✅ Tests: 2300 total, 0 failed, 0 skipped
• 📊 Line coverage: 52.99% [HTML preview] [Download]
  • Lowest covered classes
    • com.codename1.ui.Display$EdtException – 0.00%
    • com.codename1.ui.plaf.CSSBorder$LinearGradient – 0.00%
    • com.codename1.util.EasyThread$InQueueRunnable – 0.00%
    • com.codename1.components.FloatingActionButton$ReleaseActionListener – 0.00%
    • com.codename1.io.Oauth2$ShowAuthenticationActionListener – 0.00%
    • com.codename1.io.Oauth2$RefreshTokenActionListener – 0.00%
    • com.codename1.util.EasyThread$RunAndWaitRunnable – 0.00%
    • com.codename1.components.ToastBar$FlushAnimationCallback – 0.00%
    • com.codename1.ui.plaf.CSSBorder$ColorStop – 0.00%
    • com.codename1.ui.ElevationComparator – 0.00%

Static Analysis

• SpotBugs [HTML preview] [Download]
  • ❌ ByteCodeTranslator: 106 findings (High: 6, Normal: 70, Low: 30)
    • Top findings
      • High: com/codename1/tools/translator/ByteCodeTranslator.java:525 – Found reliance on default encoding in com.codename1.tools.translator.ByteCodeTranslator.writeCmakeProject(File, File, String): new java.io.FileWriter(File) (rule: DM_DEFAULT_ENCODING)
      • High: com/codename1/tools/translator/ByteCodeTranslator.java:603 – Found reliance on default encoding in com.codename1.tools.translator.ByteCodeTranslator.readFileAsStringBuilder(File): new String(byte[]) (rule: DM_DEFAULT_ENCODING)
      • High: com/codename1/tools/translator/ByteCodeTranslator.java:632 – Found reliance on default encoding in com.codename1.tools.translator.ByteCodeTranslator.replaceInFile(File, String[]): new java.io.FileWriter(File) (rule: DM_DEFAULT_ENCODING)
      • …and 103 more
  • ❌ android: 420 findings (High: 32, Normal: 196, Low: 192)
    • Top findings
      • High: CodenameOne/src/com/codename1/impl/CodenameOneThread.java:48 – com.codename1.impl.CodenameOneThread.STACK_FRAME_SIZE isn't final but should be (rule: MS_SHOULD_BE_FINAL)
      • High: com/codename1/impl/android/AndroidImplementation.java:1065 – Write to static field com.codename1.impl.android.AndroidImplementation.context from instance method com.codename1.impl.android.AndroidImplementation.setContext(Context) (rule: ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD)
      • High: com/codename1/impl/android/AndroidImplementation.java:1266 – com.codename1.impl.android.AndroidImplementation.isDarkMode() has Boolean return type and returns explicit null (rule: NP_BOOLEAN_RETURN_NULL)
      • …and 417 more
  • ❌ codenameone-maven-plugin: 531 findings (High: 33, Normal: 326, Low: 172)
    • Top findings
      • High: com/codename1/builders/AndroidGradleBuilder.java:1552 – Found reliance on default encoding in com.codename1.builders.AndroidGradleBuilder.build(File, BuildRequest): String.getBytes() (rule: DM_DEFAULT_ENCODING)
      • High: com/codename1/builders/AndroidGradleBuilder.java:1982 – Found reliance on default encoding in com.codename1.builders.AndroidGradleBuilder.build(File, BuildRequest): new java.io.FileReader(File) (rule: DM_DEFAULT_ENCODING)
      • High: com/codename1/builders/AndroidGradleBuilder.java:3989 – Found reliance on default encoding in com.codename1.builders.AndroidGradleBuilder.extract(InputStream, File, String): String.getBytes() (rule: DM_DEFAULT_ENCODING)
      • …and 528 more
  • ✅ core-unittests: 0 findings (no issues)
  • ❌ ios: 125 findings (High: 4, Normal: 75, Low: 46)
    • Top findings
      • High: com/codename1/impl/ios/IOSImplementation.java:1218 – com.codename1.impl.ios.IOSImplementation.isDarkMode() has Boolean return type and returns explicit null (rule: NP_BOOLEAN_RETURN_NULL)
      • High: com/codename1/impl/ios/IOSImplementation.java:127 – com.codename1.impl.ios.IOSImplementation.nativeInstance isn't final but should be (rule: MS_SHOULD_BE_FINAL)
      • High: com/codename1/impl/ios/IOSImplementation.java:212 – Write to static field com.codename1.impl.ios.IOSImplementation.instance from instance method com.codename1.impl.ios.IOSImplementation.init(Object) (rule: ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD)
      • …and 122 more
• ✅ PMD: 0 findings (no issues) [Report archive]
• ❌ Checkstyle: 33 findings (Error: 33) [HTML preview] [Download]
  • Top findings
    • Error: CodenameOne/src/com/codename1/ui/Image.java:429:9 – Line contains a tab character. (rule: FileTabCharacterCheck)
    • Error: CodenameOne/src/com/codename1/ui/Image.java:430:9 – Line contains a tab character. (rule: FileTabCharacterCheck)
    • Error: CodenameOne/src/com/codename1/ui/Image.java:431:9 – Line contains a tab character. (rule: FileTabCharacterCheck)
    • Error: CodenameOne/src/com/codename1/ui/Image.java:432:9 – Line contains a tab character. (rule: FileTabCharacterCheck)
    • Error: CodenameOne/src/com/codename1/ui/Image.java:433:9 – Line contains a tab character. (rule: FileTabCharacterCheck)
    • …and 28 more

Generated automatically by the PR CI workflow.