From ad10aa3381f26b63f548e0156aa9f3b2dd03e679 Mon Sep 17 00:00:00 2001 From: Jason Novinger Date: Fri, 11 Sep 2026 01:45:11 -0500 Subject: [PATCH] Fixes #23046: Install dulwich in CI so Git backend tests run dulwich is the optional 'git' extra and is not in requirements.txt, so CI never had it installed. The four GitBackendCredentialIntegrationTestCase tests added in #21252 have therefore been skipped since they were written, leaving the Git data source credential handling without CI coverage. These tests mock dulwich.porcelain.clone, so they only need the module to be importable. Installing it in the test environment does not change what a NetBox deployment pulls in: dulwich stays optional at runtime, as decided in #12906. --- .github/workflows/ci.yml | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 9a7220c21..340c2f4f3 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -122,6 +122,10 @@ jobs: python -m pip install --upgrade pip pip install -r requirements.txt pip install coverage tblib + # dulwich is the optional 'git' extra, so it is absent from requirements.txt. + # The Git data backend tests mock dulwich.porcelain.clone and only need the + # module to be importable; without it they skip silently. + pip install dulwich - name: Check for missing migrations run: python netbox/manage.py makemigrations --check