From fa7bd7a79b0bd1c8e7c9d1f59910128d1f927310 Mon Sep 17 00:00:00 2001
From: Tai Groot <tai@taigrr.com>
Date: Thu, 26 Feb 2026 01:47:25 +0000
Subject: [PATCH] fix: use snack.Manager interface type for capability
 assertions in integration tests

Type assertions require an interface type, not a concrete struct pointer.
Changed all integration tests from 'mgr := pkg.New()' to
'var mgr snack.Manager = pkg.New()' so capability checks like
mgr.(snack.VersionQuerier) compile correctly.
---
 apk/apk_integration_test.go         | 2 +-
 apt/apt_integration_test.go         | 2 +-
 dnf/dnf_integration_test.go         | 2 +-
 dpkg/dpkg_integration_test.go       | 2 +-
 flatpak/flatpak_integration_test.go | 2 +-
 pacman/pacman_integration_test.go   | 2 +-
 rpm/rpm_integration_test.go         | 2 +-
 snap/snap_integration_test.go       | 2 +-
 8 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/apk/apk_integration_test.go b/apk/apk_integration_test.go
index f80fc4f..88da244 100644
--- a/apk/apk_integration_test.go
+++ b/apk/apk_integration_test.go
@@ -13,7 +13,7 @@ import (
 )
 
 func TestIntegration_Apk(t *testing.T) {
-	mgr := apk.New()
+	var mgr snack.Manager = apk.New()
 	if !mgr.Available() {
 		t.Skip("apk not available")
 	}
diff --git a/apt/apt_integration_test.go b/apt/apt_integration_test.go
index b123a68..2379235 100644
--- a/apt/apt_integration_test.go
+++ b/apt/apt_integration_test.go
@@ -13,7 +13,7 @@ import (
 )
 
 func TestIntegration_Apt(t *testing.T) {
-	mgr := apt.New()
+	var mgr snack.Manager = apt.New()
 	if !mgr.Available() {
 		t.Skip("apt not available")
 	}
diff --git a/dnf/dnf_integration_test.go b/dnf/dnf_integration_test.go
index 8c60429..c491006 100644
--- a/dnf/dnf_integration_test.go
+++ b/dnf/dnf_integration_test.go
@@ -13,7 +13,7 @@ import (
 )
 
 func TestIntegration_DNF(t *testing.T) {
-	mgr := dnf.New()
+	var mgr snack.Manager = dnf.New()
 	if !mgr.Available() {
 		t.Skip("dnf not available")
 	}
diff --git a/dpkg/dpkg_integration_test.go b/dpkg/dpkg_integration_test.go
index 9308734..9c99b18 100644
--- a/dpkg/dpkg_integration_test.go
+++ b/dpkg/dpkg_integration_test.go
@@ -13,7 +13,7 @@ import (
 )
 
 func TestIntegration_Dpkg(t *testing.T) {
-	mgr := dpkg.New()
+	var mgr snack.Manager = dpkg.New()
 	if !mgr.Available() {
 		t.Skip("dpkg not available")
 	}
diff --git a/flatpak/flatpak_integration_test.go b/flatpak/flatpak_integration_test.go
index c92d4d3..c6d4710 100644
--- a/flatpak/flatpak_integration_test.go
+++ b/flatpak/flatpak_integration_test.go
@@ -13,7 +13,7 @@ import (
 )
 
 func TestIntegration_Flatpak(t *testing.T) {
-	mgr := flatpak.New()
+	var mgr snack.Manager = flatpak.New()
 	if !mgr.Available() {
 		t.Skip("flatpak not available — install it first")
 	}
diff --git a/pacman/pacman_integration_test.go b/pacman/pacman_integration_test.go
index 7f968df..0a8ab9f 100644
--- a/pacman/pacman_integration_test.go
+++ b/pacman/pacman_integration_test.go
@@ -13,7 +13,7 @@ import (
 )
 
 func TestIntegration_Pacman(t *testing.T) {
-	mgr := pacman.New()
+	var mgr snack.Manager = pacman.New()
 	if !mgr.Available() {
 		t.Skip("pacman not available")
 	}
diff --git a/rpm/rpm_integration_test.go b/rpm/rpm_integration_test.go
index 7853afa..27e3c3e 100644
--- a/rpm/rpm_integration_test.go
+++ b/rpm/rpm_integration_test.go
@@ -13,7 +13,7 @@ import (
 )
 
 func TestIntegration_RPM(t *testing.T) {
-	mgr := rpm.New()
+	var mgr snack.Manager = rpm.New()
 	if !mgr.Available() {
 		t.Skip("rpm not available")
 	}
diff --git a/snap/snap_integration_test.go b/snap/snap_integration_test.go
index 23ae4b8..6d555f1 100644
--- a/snap/snap_integration_test.go
+++ b/snap/snap_integration_test.go
@@ -13,7 +13,7 @@ import (
 )
 
 func TestIntegration_Snap(t *testing.T) {
-	mgr := snap.New()
+	var mgr snack.Manager = snap.New()
 	if !mgr.Available() {
 		t.Skip("snap not available")
 	}