Skip to content

Make xref archive download test hermetic - #11092

Merged
vicancy merged 1 commit into
mainfrom
vicancy-fix-hermetic-xref-download-test
Aug 11, 2026
Merged

vicancy merged 1 commit into
mainfrom
vicancy-fix-hermetic-xref-download-test

Conversation

@vicancy

@vicancy vicancy commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Summary

  • replace the live dotnet.github.io xrefmap request with a minimal local fixture
  • isolate the fixture and generated archive in a unique temporary directory and always clean it up
  • remove the HTTP-to-HTTPS redirect dependency that causes the required macOS CI test to fail

Validation

dotnet test test\Docfx.Build.Tests\Docfx.Build.Tests.csproj --no-build --no-restore --framework net8.0 --filter "FullyQualifiedName=Docfx.Build.Engine.Tests.XRefArchiveBuilderTest.TestDownload" --verbosity minimal (1 passed)

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@vicancy
vicancy merged commit 794b5d6 into main Aug 11, 2026
10 checks passed
@vicancy
vicancy deleted the vicancy-fix-hermetic-xref-download-test branch August 11, 2026 03:10
vicancy added a commit to sergimos/docfx that referenced this pull request Sep 18, 2026
Reuse the existing main-branch fix from dotnet#11092 on this older PDF branch. Keep real local xref loading, archive generation, and parsed-content assertions while removing the live HTTP dependency. Full main synchronization could not be pushed because the OAuth credential lacks workflow scope.

(cherry picked from commit 794b5d6)

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 37887675-2b86-4eb0-b0f4-94f8ed759c8c
vicancy added a commit that referenced this pull request Sep 18, 2026
* feat(pdf): allow footer and header to be printed on the cover page and toc pages

* Update heading size in documentation

* Revert package-lock.json

* test(pdf): verify cover and TOC header/footer rendering

Render real PDFs through Docset.Build and Docset.Pdf and inspect page text, margin glyphs, numbering and bookmarks for default and independent opt-in behavior. Document defaults and numbering semantics.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 37887675-2b86-4eb0-b0f4-94f8ed759c8c

* test: inherit hermetic xref archive download fixture

Reuse the existing main-branch fix from #11092 on this older PDF branch. Keep real local xref loading, archive generation, and parsed-content assertions while removing the live HTTP dependency. Full main synchronization could not be pushed because the OAuth credential lacks workflow scope.

(cherry picked from commit 794b5d6)

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 37887675-2b86-4eb0-b0f4-94f8ed759c8c

* fix(pdf): cache headers and footers by page geometry

Keep static templates rendered for differently sized or oriented pages separate, including a landscape cover followed by portrait TOC and article pages. Add real PDF margin-position regressions for standard and custom cover sizes and preserve all existing flag/default cases.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 37887675-2b86-4eb0-b0f4-94f8ed759c8c

---------

Co-authored-by: Ferreira Sergio <sergio.ferreira@skan.ch>
Co-authored-by: Liangying.Wei <lianwei@microsoft.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 37887675-2b86-4eb0-b0f4-94f8ed759c8c
This was referenced Sep 25, 2026
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.

1 participant