diff --git a/internal/resolution/variablesources/bundle.go b/internal/resolution/variablesources/bundle.go index eebc308d9..0ba3cb7ae 100644 --- a/internal/resolution/variablesources/bundle.go +++ b/internal/resolution/variablesources/bundle.go @@ -94,7 +94,7 @@ func MakeBundleVariables( // get bundle dependencies dependencies, err := filterBundleDependencies(allBundles, head) if err != nil { - return nil, fmt.Errorf("could not determine dependencies for bundle with id %q: %w", id, err) + return nil, fmt.Errorf("could not determine dependencies for bundle %q from package %q in catalog %q: %s", head.Name, head.Package, head.CatalogName, err) } // add bundle dependencies to queue for processing @@ -119,7 +119,7 @@ func filterBundleDependencies(allBundles []*catalogmetadata.Bundle, bundle *cata catalogfilter.InBlangSemverRange(requiredPackage.SemverRange), )) if len(packageDependencyBundles) == 0 { - return nil, fmt.Errorf("could not find package dependencies for bundle %q", bundle.Name) + return nil, fmt.Errorf("no bundles found matching required package %q in range %q", requiredPackage.PackageName, requiredPackage.VersionRange) } for i := 0; i < len(packageDependencyBundles); i++ { bundle := packageDependencyBundles[i] diff --git a/internal/resolution/variablesources/bundle_test.go b/internal/resolution/variablesources/bundle_test.go index 32da44fbd..bad8b7fdd 100644 --- a/internal/resolution/variablesources/bundle_test.go +++ b/internal/resolution/variablesources/bundle_test.go @@ -194,7 +194,7 @@ func TestMakeBundleVariables_NonExistentDepedencies(t *testing.T) { installedPackages := []*olmvariables.InstalledPackageVariable{} bundles, err := variablesources.MakeBundleVariables(allBundles, requiredPackages, installedPackages) - assert.ErrorContains(t, err, `could not determine dependencies for bundle with id "fake-catalog-test-package-test-package.v1.0.0"`) + assert.ErrorContains(t, err, `could not determine dependencies for bundle "test-package.v1.0.0" from package "test-package" in catalog "fake-catalog"`) assert.Nil(t, bundles) } @@ -550,7 +550,7 @@ var _ = Describe("BundlesAndDepsVariableSource", func() { ) _, err := bdvs.GetVariables(context.TODO()) Expect(err).To(HaveOccurred()) - Expect(err.Error()).To(ContainSubstring(`could not determine dependencies for bundle with id "fake-catalog-test-package-bundle-2": could not find package dependencies for bundle "bundle-2"`)) + Expect(err.Error()).To(ContainSubstring(`could not determine dependencies for bundle "bundle-2" from package "test-package" in catalog "fake-catalog": no bundles found matching required package "some-package" in range ">=1.0.0 <2.0.0"`)) }) It("should return error if an inner variable source returns an error", func() {