Display an error when a package is not found

src/lib/registration/clients/online_search.rb

@@ -18,6 +18,7 @@
 # find current contact information at www.suse.com.
 
 require "yast"
+require "yast2/popup"
 require "registration/dialogs/online_search"
 require "registration/addon"
 require "registration/registration"
@@ -131,7 +132,7 @@ def register_addons
       def select_packages
         ::Registration::SwMgmt.select_addon_products
         package_search_dialog.selected_packages.each do |pkg|
-          Yast::Pkg.PkgInstall(pkg.name)
+          pkg_install_error_message(pkg.name) unless Yast::Pkg.PkgInstall(pkg.name)
         end
         :next
       end
@@ -159,6 +160,17 @@ def selected_addons
       def reset_selected_addons_cache!
         @selected_addons = nil
       end
+
+      def pkg_install_error_message(name)
+        Yast2::Popup.show(
+          format(
+            # TRANSLATORS: 'name' is the package's name
+            _("Package %{name} could not be selected for installation."),
+            name: name
+          ),
+          headline: :error
+        )
+      end
     end
   end
 end

test/registration/clients/online_search_test.rb

@@ -44,6 +44,7 @@
       allow(Registration::SwMgmt).to receive(:select_addon_products)
       allow(Registration::UrlHelpers).to receive(:registration_url)
         .and_return("https://scc.suse.com") # speed up the test
+      allow(Yast::Pkg).to receive(:PkgInstall).and_return(true)
     end
 
     context "when an addon is selected" do
@@ -110,9 +111,20 @@
 
     context "when a package is selected" do
       it "selects the package for installation" do
-        expect(Yast::Pkg).to receive(:PkgInstall).with(package.name)
+        expect(Yast::Pkg).to receive(:PkgInstall).with(package.name).and_return(true)
         subject.run
       end
+
+      context "but the package is not found" do
+        before do
+          allow(Yast::Pkg).to receive(:PkgInstall).and_return(false)
+        end
+
+        it "warns the user" do
+          expect(Yast2::Popup).to receive(:show).with(/could not be selected/, headline: :error)
+          subject.run
+        end
+      end
     end
 
     context "when the user aborts the search" do