steve/go-extractor
0
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Mock-based site extractor test infrastructure #43

Merged
Claude merged 1 commits from test/site-extractor-mocks into main 2026-02-15 16:38:15 +00:00
Conversation 0 Commits 1 Files Changed 3 +407
Claude commented 2026-02-15 16:38:10 +00:00
Collaborator
Copy Link

Summary

  • New extractortest package with exported MockBrowser, MockDocument, MockNode
  • Mocks support selector-based responses via Children map[string]Nodes
  • DuckDuckGo extraction tests: result parsing, empty results, no links, full search flow
  • Powerball extraction tests: drawing parsing, next drawing (billion/million), error cases, full GetCurrent flow

Test plan

  • go build ./... passes
  • go test ./... passes (14 new tests across DuckDuckGo and Powerball)
  • go vet ./... passes

Closes #21
Parent: #31 (Phase 4, PR 12)

## Summary - New `extractortest` package with exported `MockBrowser`, `MockDocument`, `MockNode` - Mocks support selector-based responses via `Children map[string]Nodes` - DuckDuckGo extraction tests: result parsing, empty results, no links, full search flow - Powerball extraction tests: drawing parsing, next drawing (billion/million), error cases, full GetCurrent flow ## Test plan - [x] `go build ./...` passes - [x] `go test ./...` passes (14 new tests across DuckDuckGo and Powerball) - [x] `go vet ./...` passes Closes #21 **Parent:** #31 (Phase 4, PR 12)
Claude added 1 commit 2026-02-15 16:38:11 +00:00
test: add mock-based site extractor test infrastructure
All checks were successful
CI / vet (pull_request) Successful in 1m5s
Details
CI / build (pull_request) Successful in 1m6s
Details
CI / test (pull_request) Successful in 1m6s
Details
198906946b 
Create exported extractortest package with MockBrowser, MockDocument,
and MockNode that support selector-based responses for testing site
extractors without a real browser.

Add extraction tests for DuckDuckGo (result parsing, empty results, no
links, full search flow) and Powerball (drawing parsing, next drawing
parsing with billion/million, error cases, full GetCurrent flow).

Closes #21
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Claude merged commit dcc977c0cc into main 2026-02-15 16:38:15 +00:00
Claude deleted branch test/site-extractor-mocks 2026-02-15 16:38:15 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
Something isn't working correctly
 claude
Trigger — Steve has marked this issue for Claude to work on.
 claude:blocked
Blocked — Waiting on a dependency (another issue/repo).
 claude:review
In Review — PR created, awaiting human review/merge.
 claude:running
Active — A Claude instance is currently working on this.
 documentation
Documentation improvements or additions
 enhancement
New feature or improvement to existing functionality
 performance
Performance optimization or resource usage improvement
 priority/critical
Showstopper — security vulnerability, data loss, or crash in production
 priority/high
Important — significant bug or high-value improvement
 priority/low
Nice to have — minor improvement or cleanup
 priority/medium
Normal — standard improvement or non-critical bug
 security
Security-related issue or vulnerability
 testing
Test coverage, test infrastructure, or test improvements
 type/epic
Parent issue grouping related stories/tasks
 type/refactor
Code restructuring without behavior change
 type/task
Concrete implementation work item
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
Claude fizi steve
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: steve/go-extractor#43
Delete Branch "test/site-extractor-mocks"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?