Skip to content

test action fixes and other test fixes for feat: add support for parsing local sitemap files#193

Open
tylertech-lee-hazlett wants to merge 14 commits intoseantomburke:claude/issue-15-20250531_060038from
tylertech-lee-hazlett:claude/issue-15-20250531_060038
Open

test action fixes and other test fixes for feat: add support for parsing local sitemap files#193
tylertech-lee-hazlett wants to merge 14 commits intoseantomburke:claude/issue-15-20250531_060038from
tylertech-lee-hazlett:claude/issue-15-20250531_060038

Conversation

@tylertech-lee-hazlett
Copy link

@tylertech-lee-hazlett tylertech-lee-hazlett commented Jan 14, 2026

  • GitHub Actions were failing to build because bad import syntax on core modules
  • Existing tests were broken because emerald.com doesn't have a sitemap.xml any more
  • Coverage tests were running successfully because an improper mock value set on Priority
  • Local file tests were failing because xml was not copied to lib

Note

Ensures local-file parsing works end-to-end and CI passes.

  • Build: Update compile to cp src/tests/*.xml lib/tests/; switch to ESM namespace imports for fs/zlib in src/assets/sitemapper.js and tests; minor example formatting.
  • Tests: Replace emerald.com with google.com; adjust expectations to numeric priority; add/strengthen local file tests (fields, lastmod filtering, exclusions, gzipped files); fix imports; minor cleanup.
  • Examples: Add src/examples/local-file.js and update example.js local file snippet formatting.

Written by Cursor Bugbot for commit db5c5d3. This will update automatically on new commits. Configure here.

@tylertech-lee-hazlett
Copy link
Author

@seantomburke this fixes #181

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant