Skip to content

test: increase test coverage from 84% to 96% - #112

Merged
deviantintegral merged 3 commits into
mainfrom
claude/increase-test-coverage-PcwgQ
Dec 15, 2025
Merged

deviantintegral merged 3 commits into
mainfrom
claude/increase-test-coverage-PcwgQ

Conversation

@deviantintegral

Copy link
Copy Markdown
Owner

Added new test files:

  • BrowserTest: Tests for Browser class (extends Creator)
  • CacheStateTest: Tests for cache state getters/setters
  • LogTest: Tests for Log class including version, creator, browser, pages, and entries
  • Adapter/Psr7/ResponseTest: Tests for PSR-7 Response adapter
  • Repository/HarFileRepositoryTest: Tests for HAR file repository operations
  • Handler/DateFormatInterfaceHandlerTest: Tests for date format interface handler
  • Handler/TruncatingDateTimeHandlerTest: Tests for truncating datetime handler

Enhanced existing test files:

  • RequestTest: Added tests for isResponseCached(), hasPostData(), getUrl/setUrl, getQueryString, and getMethod/setMethod
  • ResponseTest: Added tests for all getters/setters and serialization
  • EntryTest: Added tests for all getters/setters including pageref, time, request, response, cache, timings, serverIPAddress, connection, and initiator
  • HarTest: Added tests for getLog/setLog and splitLogEntries() method
  • PageTest: Added tests for getId/setId, getTitle/setTitle, and getPageTimings/setPageTimings
  • CacheStateTest: Added comprehensive serialization tests

Test count increased from 82 to 148 tests (66 new tests). All tests pass successfully.

@deviantintegral
deviantintegral force-pushed the claude/increase-test-coverage-PcwgQ branch from b78470f to b7587d0 Compare December 15, 2025 01:00
@deviantintegral deviantintegral changed the title test: significantly increase test coverage with comprehensive unit tests test: increase test coverage Dec 15, 2025
@deviantintegral deviantintegral changed the title test: increase test coverage test: increase test coverage from 84% to 96% Dec 15, 2025
@deviantintegral
deviantintegral force-pushed the claude/increase-test-coverage-PcwgQ branch from b7587d0 to 01422bb Compare December 15, 2025 01:12
@deviantintegral
deviantintegral marked this pull request as ready for review December 15, 2025 01:14
@deviantintegral
deviantintegral merged commit ac7133c into main Dec 15, 2025
7 checks 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.

2 participants