Dark Mode

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

tests: add tests for the current state of "Checking a Local Folder with URL Remapping"#1965

Open
katrinafyi wants to merge 16 commits intolycheeverse:masterfrom
rina-forks:mapping-tests
Open

tests: add tests for the current state of "Checking a Local Folder with URL Remapping"#1965
katrinafyi wants to merge 16 commits intolycheeverse:masterfrom
rina-forks:mapping-tests

Conversation

Copy link
Member

katrinafyi commented Dec 21, 2025 *
edited
Loading

this adds tests for the use case of "Checking a Local Folder with URL Remapping", as motivated in lycheeverse/lycheeverse.github.io#140. it makes no behaviour changes.

to review this PR, i suggest first understanding the use-case, then you can read the input files and expected URLs to check that they make sense. there are two new .md files inside the fixtures folder, one for whole domain mapping and one for subpath mapping. the markdown text explains the reasoning for each URL.

the remaps are deeply unpleasant and you shouldn't try too hard to understand them. they're also different from lycheeverse/lycheeverse.github.io#140 because i found new edge cases and new bugs. i'll update the docs pr after...

issues arising:

Copy link
Member Author

katrinafyi commented Feb 6, 2026

Hi can this PR be checked? It is useful tests to have, especially with #2005. Let me know if the PR is too big or hard to understand.

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant