镜像站点 · 本页由第三方 GitHub 只读镜像提供,非 GitHub 官方站点,不接受任何登录或凭据输入。前往 github.com
Skip to content

Add unit tests for JSONArray #1470

Description

@satindra-r

Most relevant area for this enhancement?

Unit testing

Feature to enhance

JSONArray class in core

Enhancement details

Enhancement description

The JSONArray class does not have any tests even though it is used in app and has substantial logic

Additional context

Part of #880

Would you like to help implement this enhancement?

Yes, I’d like to help with this

Activity

  1. NANDGOPALSHARMA-29 commented on Mar 4, 2026

    @NANDGOPALSHARMA-29

    Hi! I’d like to work on this issue.
    I’m currently setting up the Processing4 codebase locally and exploring the JSONArray class. I’d like to contribute by adding unit tests for it.
    Please let me know if this issue is available to work on.

  2. SableRaf commented on Mar 5, 2026

    @SableRaf
    Collaborator

    Hello @NANDGOPALSHARMA-29

    It looks like @satindra-r has already expressed interest in working on this issue. We usually follow a first-come first-served approach to issues to avoid duplicated work.

    Also, before work can start, maintainers typically need to confirm the issue and agree on an implementation.

    Please consult our CONTRIBUTING.md for detailed guidelines.


    A note on GSoC: We receive many contributions around this time of year from GSoC applicants. Just a heads-up that our application process does not require you to submit a PR. Please check out this thread on the forum for more details on how to apply.

  3. satindra-r commented on Mar 7, 2026

    @satindra-r
    Author

    Hi, is there anything specific regarding the implementation of the tests that I have to confirm before being assigned?

  4. satindra-r commented on Mar 10, 2026

    @satindra-r
    Author

    I mainly want to test the constructor and the writeInternal functions as it contains JSON parsing logic which is fragile

  5. catilac commented on Mar 10, 2026

    @catilac
    Collaborator

    @satindra-r i've gone ahead and assigned you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

enhancementNew feature or request

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions