Skip to content

Convert most remote inventory fetch tests to use a local server; bump Pythons and dev Sphinx#325

Merged
bskinn merged 27 commits intomainfrom
local-http-testing
Dec 24, 2025
Merged

Convert most remote inventory fetch tests to use a local server; bump Pythons and dev Sphinx#325
bskinn merged 27 commits intomainfrom
local-http-testing

Conversation

@bskinn
Copy link
Copy Markdown
Owner

@bskinn bskinn commented Dec 24, 2025 •
edited
Loading

  • Add fixture machinery to stand up a local HTTP server pointed at tests/resource.
    • Update all tests reaching for GitHub raw .inv files to use this local server instead.
  • Make a few other small test suite changes to reduce flakiness.
    • Many hosts are now much touchier about what they block, understandably.
  • Shift supported Pythons to 3.10-3.14
  • Adjust mainline CI Python to 3.13
  • Bump dev-pin Sphinx to 8.2.3 8.1.3
    • Can't go to Sphinx v9 yet because sphinx-rtd-theme only supports up to <9.
    • Capped at 8.1.3 as the last version supporting Python 3.10

bskinn added 18 commits December 24, 2025 11:41
@bskinn
Add initial fixtures_http.py
6c47b57
@bskinn
Move conftest.py into tests/
230cd6a
@bskinn
Update paths inside conftest.py
86e8cd2
@bskinn
Adjust tox envs for conftest.py now being inside tests/
7338ba8
@bskinn
Add HTTP/HTTPS comments to test_api_inventory_known_header_required()
892208d 
Specifically, that the HTTP is intentional
@bskinn
Remove extra newline
36d93e5
@bskinn
Replace GitHub remote URL with local-server URL
c7b3a58
@bskinn
Switch resource_url fixture to session-scope
09c9640 
Since it shouldn't change, and we access it at session scope deeper in
@bskinn
Add 'tests' "package" to first-party in flake config
a9e37bf
@bskinn
Update external-remote URL-load check test
f2d56f6 
Add note about how it's now doing the primary lifting of checking
truly-remote URL inventory loads.

Add better in-situ reporting of which project's objects.inv failed,
if it fails.
@bskinn
Convert --testall URL API imports to use local server
f784a6d
@bskinn
Convert nonlocal CLI tests to use local server
b802899
@bskinn
Remove Django-specific URL test
543e092 
Too flaky, breaking regularly even with retries.
@bskinn
Update supported Python range to 3.10-3.14
a07f2a5
@bskinn
Expand on a comment for historical info
551e5a0
@bskinn
Bump active dev Sphinx version
4c91212
@bskinn
Convert docs 'suggest' call from Github to RTD
a42a01b 
Should render more reliably.
@bskinn
Fix inventory filename
f4726e8
@bskinn bskinn added type: maintenance 🔧 Something administrative needs upkeep pr: needs changelog 📍 area: tests 🩺 Focused on the test suite area: infra 🌉 area: meta 🧠 Dealing with the project itself bump: upkeep 🐞 Changes outside the package itself (x.y.z.w) labels Dec 24, 2025
@bskinn bskinn marked this pull request as ready for review December 24, 2025 19:29
@bskinn bskinn merged commit 0d9d181 into main Dec 24, 2025
21 checks passed
@bskinn bskinn deleted the local-http-testing branch December 24, 2025 23:41
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

area: infra 🌉 area: meta 🧠 Dealing with the project itself area: tests 🩺 Focused on the test suite bump: upkeep 🐞 Changes outside the package itself (x.y.z.w) pr: merge ready 👍 type: maintenance 🔧 Something administrative needs upkeep

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@bskinn