Update JSONArray.java for #1007

[eleumik]

fix array content starting with ',' in strict mode

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[stleary]

Please add at least one test case to JSONArrayTest.java to provide coverage for the new code
For example, this tests both jsonobject and jsonarray, with and without strict mode. It also handles the 'TestWithStrictMode' gradle task where strict mode is always set.

    @Test
    public void foo() {
        String s = "{\"a\":[ ,4,null,8]}";
        JSONParserConfiguration jsonParserConfiguration = new JSONParserConfiguration();
        if (jsonParserConfiguration.isStrictMode()) {
            // expect an error during strict mode testing
            assertThrows(JSONException.class, () -> { new JSONObject(s); });
        } else {
            new JSONObject(s);
        }
        assertThrows(JSONException.class, () -> { new JSONObject(s, new JSONParserConfiguration().withStrictMode(true)); });
        String s2 = "[,4,null,8]";
        if (jsonParserConfiguration.isStrictMode()) {
            // expect an error during strict mode testing
            assertThrows(JSONException.class, () -> { new JSONArray(s2); });
        } else {
            new JSONArray(s2);
        }
        assertThrows(JSONException.class, () -> { new JSONArray(s2, new JSONParserConfiguration().withStrictMode(true)); });
    }

[stleary]

What problem does this code solve?
Strict mode was not checking for leadihng ',' in JSONArray

Does the code still compile with Java6?
Yes

Risks
Low

Changes to the API?
No

Will this require a new release?
No

Should the documentation be updated?
No

Does it break the unit tests?
No, but new unit tests are needed. Code review action item left

Was any code refactored in this commit?
No

Review status
APPROVED

Starting 3-day comment window

[stleary]

@eleumik PR is approved, but please add unit test coverage if you get a chance. Otherwise, I will add it after the merge.

[stleary]

Open comments will be addressed in a future commit