Skip to content

test: cover fetcher gaps#37

Merged
wiebe-vandendriessche merged 3 commits into
idlab-discover:mainfrom
beanscg:beanlabs/fetcher-test-coverage-issue-25
Jun 10, 2026
Merged

test: cover fetcher gaps#37
wiebe-vandendriessche merged 3 commits into
idlab-discover:mainfrom
beanscg:beanlabs/fetcher-test-coverage-issue-25

Conversation

@beanscg

@beanscg beanscg commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add httptest coverage for dataset API fetching, dataset README parsing, dataset/model tree pagination, and model search
  • add table-driven tests for markdown front-matter extraction and string-slice normalization
  • fix splitFrontMatter so the closing --- marker is stripped from the returned body

Issue

Addresses part of #25.

Verification

go test ./internal/fetcher
go test ./...

Both pass locally.

@wiebe-vandendriessche wiebe-vandendriessche merged commit 52b534d into idlab-discover:main Jun 10, 2026
15 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