Skip to content

Add comprehensive test coverage improvements#21

Merged
marevol merged 1 commit intomasterfrom
claude/analyze-test-coverage-682Dr
Jan 10, 2026
Merged

Add comprehensive test coverage improvements#21
marevol merged 1 commit intomasterfrom
claude/analyze-test-coverage-682Dr

Conversation

@marevol
Copy link
Collaborator

@marevol marevol commented Jan 10, 2026

  • Add RawTextElementsTest.java for script/style/textarea/title element handling
  • Add StrictModeTest.java for nekohtml.dom.strict system property testing
  • Add BrowserQuirksIntegrationTest.java for browser quirks and implied elements
  • Extend SimpleHTMLScannerEnhancementsTest with entity and encoding tests
  • Extend AdoptionAgencyAlgorithmExtendedTest with AAA loop limit, formatting marker, and table context tests
  • Extend HTMLSAXConfigurationTest with feature combination and property tests
  • Extend PerformanceStressTest with concurrent parsing, stress patterns, and real-world simulation tests

- Add RawTextElementsTest.java for script/style/textarea/title element handling
- Add StrictModeTest.java for nekohtml.dom.strict system property testing
- Add BrowserQuirksIntegrationTest.java for browser quirks and implied elements
- Extend SimpleHTMLScannerEnhancementsTest with entity and encoding tests
- Extend AdoptionAgencyAlgorithmExtendedTest with AAA loop limit, formatting marker, and table context tests
- Extend HTMLSAXConfigurationTest with feature combination and property tests
- Extend PerformanceStressTest with concurrent parsing, stress patterns, and real-world simulation tests
@marevol marevol requested a review from Copilot January 10, 2026 08:54
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR significantly expands test coverage across multiple test suites to validate comprehensive HTML parsing behavior, edge cases, and stress scenarios in the NekoHTML library.

Changes:

  • Adds new test files for raw text elements, strict mode handling, and browser quirks compatibility
  • Extends existing test suites with entity handling, encoding variations, handler management, feature combinations, and concurrent/stress testing scenarios
  • Introduces tests for the Adoption Agency Algorithm's loop limits, formatting markers, and complex nesting patterns

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Show a summary per file
File Description
SimpleHTMLScannerEnhancementsTest.java Adds tests for entity handling (named, numeric, incomplete), encoding variations (ISO-8859-1, UTF-16, UTF-8), input source handling, and SAX handler management
RawTextElementsTest.java New file testing raw text element handling for script, style, textarea, title, and deprecated elements with various content patterns
HTMLSAXConfigurationTest.java Extends with feature combination tests, property management, pipeline configuration, handler getters/setters, and complex feature interactions
AdoptionAgencyAlgorithmExtendedTest.java Adds comprehensive tests for AAA outer loop limits, formatting markers in table contexts, special scope elements, and stress scenarios
StrictModeTest.java New file testing nekohtml.dom.strict system property behavior across default, lenient, and strict modes with edge cases
BrowserQuirksIntegrationTest.java New file testing browser quirks compatibility including implied elements, table quirks, form handling, list patterns, and DOCTYPE variations
PerformanceStressTest.java Extends with deep nesting tests, concurrent parsing validation, misnesting patterns, and complex real-world page simulations
Comments suppressed due to low confidence (1)

src/test/java/org/codelibs/nekohtml/sax/SimpleHTMLScannerEnhancementsTest.java:2

  • The copyright year is set to 2025, which is in the future. The current date is January 10, 2026, so the copyright notice should be updated to include 2026.
 * Copyright 2012-2025 CodeLibs Project and the Others.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@marevol marevol merged commit 5e4f8ae into master Jan 10, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants