Add comprehensive tests for JavaScript context and date formatting

[shai-almog]

Summary

• add unit tests for JavascriptContext covering listener lifecycle, value conversion, async callbacks, and cleanup
• add unit tests for JSObject exercising typed accessors, callback registration, and call delegation
• add localization-aware SimpleDateFormat tests for formatting, timezone parsing, and pattern handling

Testing

• mvn -f maven/core-unittests/pom.xml test (fails: missing provided dependency com.codenameone:codenameone-factory:8.0-SNAPSHOT)

---

https://chatgpt.com/codex/tasks/task_e_68f5946b63c48331ad03ab0d319c356b

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[github-actions]

✅ Continuous Quality Report

Test & Coverage

• ✅ Tests: 252 total, 0 failed, 0 skipped
• 📊 Line coverage: 10.83% [HTML preview] [Download]
  • Lowest covered classes
    • com.codename1.io.services.RSSService$1 – 0.00%
    • com.codename1.io.services.ImageDownloadService – 0.00%
    • com.codename1.io.services.ImageDownloadService$6 – 0.00%
    • com.codename1.io.services.ImageDownloadService$1 – 0.00%
    • com.codename1.io.services.ImageDownloadService$3 – 0.00%
    • com.codename1.io.services.ImageDownloadService$2 – 0.00%
    • com.codename1.io.services.ImageDownloadService$5 – 0.00%
    • com.codename1.io.services.ImageDownloadService$4 – 0.00%
    • com.codename1.io.services.TwitterRESTService$1 – 0.00%
    • com.codename1.io.services.RSSService$1FinishParsing – 0.00%

Static Analysis

• ❌ SpotBugs: 993 findings (High: 92, Normal: 484, Low: 417) [HTML preview] [Download]
  • Top findings
    • High: CodenameOne/src/com/codename1/charts/util/ColorUtil.java:38 – Field isn't final but should be (rule: MS_SHOULD_BE_FINAL)
    • High: CodenameOne/src/com/codename1/charts/util/ColorUtil.java:30 – Field isn't final but should be (rule: MS_SHOULD_BE_FINAL)
    • High: CodenameOne/src/com/codename1/charts/util/ColorUtil.java:33 – Field isn't final but should be (rule: MS_SHOULD_BE_FINAL)
    • High: CodenameOne/src/com/codename1/charts/views/XYChart.java:128 – Call to equals() comparing different types (rule: EC_UNRELATED_TYPES)
    • High: CodenameOne/src/com/codename1/compat/java/util/Objects.java:51 – Redundant nullcheck of value known to be non-null (rule: RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE)
    • …and 988 more
• ❌ PMD: 2726 findings (P2: 1, P3: 649, P4: 2076) [HTML preview] [Download]
  • Top findings
    • P2: CodenameOne/src/com/codename1/ui/Display.java:1012 – Avoid using a branching statement as the last in a loop. (rule: AvoidBranchingStatementAsLastInLoop)
    • P3: CodenameOne/src/com/codename1/facebook/FacebookRESTService.java:76 – Avoid unused constructor parameters such as 'token'. (rule: UnusedFormalParameter)
    • P3: CodenameOne/src/com/codename1/impl/CodenameOneImplementation.java:8380 – These nested if statements could be combined (rule: CollapsibleIfStatements)
    • P3: CodenameOne/src/com/codename1/impl/FullScreenAdService.java:103 – These nested if statements could be combined (rule: CollapsibleIfStatements)
    • P3: CodenameOne/src/com/codename1/io/CSVParser.java:134 – These nested if statements could be combined (rule: CollapsibleIfStatements)
    • …and 2721 more
• ❌ Checkstyle: 183940 findings (Warning: 183940) [HTML preview] [Download]
  • Top findings
    • Warning: CodenameOne/src/com/codename1/charts/models/XYSeries.java:3 –

      tag should be placed immediately before the first word, with no space after. (rule: JavadocParagraphCheck)

    • Warning: CodenameOne/src/com/codename1/charts/models/XYSeries.java:3 –

      tag should be preceded with an empty line. (rule: JavadocParagraphCheck)

    • Warning: CodenameOne/src/com/codename1/charts/models/XYSeries.java:7 –

      tag should be placed immediately before the first word, with no space after. (rule: JavadocParagraphCheck)

    • Warning: CodenameOne/src/com/codename1/charts/models/XYSeries.java:7 –

      tag should be preceded with an empty line. (rule: JavadocParagraphCheck)

    • Warning: CodenameOne/src/com/codename1/charts/models/XYSeries.java:9 –

      tag should be placed immediately before the first word, with no space after. (rule: JavadocParagraphCheck)

    • …and 183935 more

Generated automatically by the PR CI workflow.