Skip to content

Refreshing test_equations[docling] VCR cassette#1264

Merged
jamesbraza merged 1 commit intomainfrom
refreshing-test-equations
Jan 5, 2026
Merged

Refreshing test_equations[docling] VCR cassette#1264
jamesbraza merged 1 commit intomainfrom
refreshing-test-equations

Conversation

@jamesbraza
Copy link
Copy Markdown
Collaborator

@jamesbraza jamesbraza commented Jan 5, 2026

Seen in this CI run:

FAILED tests/test_paperqa.py::test_equations[docling] - vcr.errors.CannotOverwriteExistingCassetteException: Can't overwrite existing cassette ('/home/runner/work/paper-qa/paper-qa/tests/cassettes/test_equations[docling].yaml') in your current record mode ('none').
No match for the request (<Request (HEAD) https://huggingface.co/api/resolve-cache/models/docling-project/docling-layout-heron/bdb7099d742220552d703932cc0ce0a26a7a8da8/.gitattributes>) was found.
Found 2 similar requests with 2 different matcher(s) :

1 - (<Request (GET) https://huggingface.co/api/models/docling-project/docling-layout-heron/revision/main>).
Matchers succeeded : ['scheme', 'host', 'port', 'query']
Matchers failed :
method - assertion failure :
HEAD != GET
path - assertion failure :
/api/resolve-cache/models/docling-project/docling-layout-heron/bdb7099d742220552d703932cc0ce0a26a7a8da8/.gitattributes != /api/models/docling-project/docling-layout-heron/revision/main

2 - (<Request (GET) https://huggingface.co/api/models/docling-project/docling-models/revision/v2.3.0>).
Matchers succeeded : ['scheme', 'host', 'port', 'query']
Matchers failed :
method - assertion failure :
HEAD != GET
path - assertion failure :
/api/resolve-cache/models/docling-project/docling-layout-heron/bdb7099d742220552d703932cc0ce0a26a7a8da8/.gitattributes != /api/models/docling-project/docling-models/revision/v2.3.0

I don't feel like figuring out why the VCR cassette went stale.


Note

Updates the recorded HTTP interactions to match current upstream API responses, unblocking the failing test.

  • Re-records tests/cassettes/test_equations[docling].yaml with up-to-date Crossref, Semantic Scholar (still 404), and Hugging Face responses
  • Refreshes response bodies/headers: new dates/ETags, content lengths, rate limit headers, user-agent (torch/2.9.1), and request IDs
  • Hugging Face model metadata updated (new sha, downloads/likes, added docling_heron_400.png, updated tags)
  • Interaction order adjusted (Crossref first), keeping request URIs and methods aligned with current behavior

Written by Cursor Bugbot for commit 0b1afdd. Configure here.

@jamesbraza jamesbraza self-assigned this Jan 5, 2026
Copilot AI review requested due to automatic review settings January 5, 2026 19:46
@jamesbraza jamesbraza added the bug Something isn't working label Jan 5, 2026
@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Jan 5, 2026
@dosubot
Copy link
Copy Markdown

dosubot bot commented Jan 5, 2026

Related Documentation

Checked 1 published document(s) in 0 knowledge base(s). No updates required.

How did I do? Any feedback?  Join Discord

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jan 5, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR refreshes the VCR cassette file for the test_equations[docling] test, which was causing test failures due to mismatched HTTP requests. The cassette records HTTP interactions with external APIs (Semantic Scholar and HuggingFace) to enable deterministic testing without making actual network calls.

Key changes:

  • Updated API interaction timestamps from November 2025 to January 5, 2026
  • Reordered the two Semantic Scholar API requests in the recording sequence
  • Refreshed HuggingFace model metadata to reflect current state (download counts, SHA hashes, model details)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@jamesbraza jamesbraza merged commit cb346e5 into main Jan 5, 2026
14 checks passed
@jamesbraza jamesbraza deleted the refreshing-test-equations branch January 5, 2026 20:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants