FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Enhance reliability of HTTPTokener through edge-case input validation by surajdm123 · Pull Request #992 · stleary/JSON-java · GitHub

/ JSON-java Public

Enhance reliability of HTTPTokener through edge-case input validation - #992

Merged
stleary merged 1 commit into
stleary:masterfrom
surajdm123:add-tests
Jul 6, 2025
ConversationCommits (1)ChecksFiles changed
Merged

Enhance reliability of HTTPTokener through edge-case input validation#992
stleary merged 1 commit into
stleary:masterfrom
surajdm123:add-tests

Conversation

surajdm123 commented Jul 3, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

Context

This PR improves the reliability of HTTPTokener through edge-case input validation.

Test Plan

mvn -Dtest=HTTPTokenerTest test

...
[INFO] Running org.json.junit.HTTPTokenerTest
[INFO] Tests run: 9, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.040 s -- in org.json.junit.HTTPTokenerTest
[INFO] 
[INFO] Results:
[INFO] 
[INFO] Tests run: 9, Failures: 0, Errors: 0, Skipped: 0
[INFO] 
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------

Next Steps

  1. Add more missing test cases to make this codebase more resilient.

sonarqubecloud Bot commented Jul 3, 2025

Copy link
Copy Markdown

Quality Gate passed

Issues
1 New issue
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 commented Jul 4, 2025

Copy link
Copy Markdown
Owner

What problem does this code solve?
Improves test coverage from 93% to 100% for HttpTokener

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, this PR consists of new unit tests

Was any code refactored in this commit?
No

Review status
APPROVED

Starting 3-day comment window

stleary merged commit d7593fb into stleary:master Jul 6, 2025
surajdm123 changed the title Added JUnit test cases for HTTPTokener Enhance reliability of HTTPTokener through edge-case input validation Apr 24, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

stleary stleary approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

Back | FazBrowse Home | New Git URL