From 99dfc80003cd3b0e89775b559091003fd6384ff1 Mon Sep 17 00:00:00 2001 From: Tai Groot Date: Thu, 26 Feb 2026 01:55:24 +0000 Subject: [PATCH] fix(ci): pin fedora:39 for dnf4, fix test paths for debian/alpine MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Fedora 40+ ships dnf5 (not 41 as assumed) — pin to 39 - Debian container: /usr/bin/bash doesn't exist, use /usr/bin/dpkg - Alpine: use busybox for LatestVersion test (always available) --- .github/workflows/integration.yml | 2 +- apk/apk_integration_test.go | 2 +- dpkg/dpkg_integration_test.go | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/.github/workflows/integration.yml b/.github/workflows/integration.yml index aa2ba9e..50ccdec 100644 --- a/.github/workflows/integration.yml +++ b/.github/workflows/integration.yml @@ -55,7 +55,7 @@ jobs: fedora: name: Fedora (dnf) runs-on: ubuntu-latest - container: fedora:41 # pin to dnf4; dnf5 (fedora 42+) has different output format + container: fedora:39 # last release with dnf4; dnf5 (fedora 40+) needs separate parser steps: - uses: actions/checkout@v4 - uses: actions/setup-go@v5 diff --git a/apk/apk_integration_test.go b/apk/apk_integration_test.go index 460021d..048d0d5 100644 --- a/apk/apk_integration_test.go +++ b/apk/apk_integration_test.go @@ -83,7 +83,7 @@ func TestIntegration_Apk(t *testing.T) { t.Run("Capabilities", func(t *testing.T) { if vq, ok := mgr.(snack.VersionQuerier); ok { - ver, err := vq.LatestVersion(ctx, "tree") + ver, err := vq.LatestVersion(ctx, "busybox") require.NoError(t, err) assert.NotEmpty(t, ver) diff --git a/dpkg/dpkg_integration_test.go b/dpkg/dpkg_integration_test.go index 59d969b..3ac0414 100644 --- a/dpkg/dpkg_integration_test.go +++ b/dpkg/dpkg_integration_test.go @@ -53,7 +53,7 @@ func TestIntegration_Dpkg(t *testing.T) { t.Run("FileOwner", func(t *testing.T) { if fo, ok := mgr.(snack.FileOwner); ok { - owner, err := fo.Owner(ctx, "/usr/bin/bash") + owner, err := fo.Owner(ctx, "/usr/bin/dpkg") require.NoError(t, err) assert.NotEmpty(t, owner)