Skip to content

vdirsyncer: generate man page, disable broken tests - #62420

Open
rookiejet wants to merge 3 commits into
void-linux:masterfrom
rookiejet:submit/vdirsyncer_manpage
Open

vdirsyncer: generate man page, disable broken tests#62420
rookiejet wants to merge 3 commits into
void-linux:masterfrom
rookiejet:submit/vdirsyncer_manpage

Conversation

@rookiejet

Copy link
Copy Markdown
Contributor

Testing the changes

  • I tested the changes in this PR: briefly

Notes

The HTTP storage tests mock requests via aioresponses, which does not pass the stream_writer keyword argument required by aiohttp >= 3.10.2, so every mocked response fails with:

TypeError: ClientResponse.__init__() missing 1 required
keyword-only argument: 'stream_writer'

aioresponses upstream has no release with the fix yet (pnuckowski/aioresponses#288). Re-enable these tests once a compatible aioresponses release is packaged (or if vdirsyncer upstream switches to aioresponses-ng).

@ahesford

ahesford commented Sep 6, 2026

Copy link
Copy Markdown
Member
  • Please ping template maintainers (me, in this case) when you modify their packages.
  • Please combine this with an update to the latest version.

@rookiejet
rookiejet force-pushed the submit/vdirsyncer_manpage branch from ad73203 to f0a89fa Compare September 6, 2026 20:38
@rookiejet

Copy link
Copy Markdown
Contributor Author

Noted. PR scope expanded a bit from the vdirsyncer version bump:

  • Added missing tenacity dependency to vdirsyncer and updated tenacity to 9.1.4 (tenacity>=9.0.0 required since 0.21.0).
  • Moved test-only deps out of python3-tenacity depends into checkdepends, dropped python3-Sphinx as that was an obsolete runtime dependency.
  • Checked python3-plotly, python3-stamina as they depend on python3-tenacity
    • python3-stamina test suite runs and completes (311 passed, 2 skipped) with minversion bypass
    • python3-plotly-5.24.1 has no test suite, so would need a user to test (@pfpulux). During pkg build verify-python-deps flagged missing dependency python3-packaging, added to template.

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