Fix documentation

library/packages/src/lib/y2packager/license.rb

@@ -141,6 +141,7 @@ def reject!
     end
 
   private
+
     # @return [Y2Packager::LicensesFetchers::Base] License fetcher object
     attr_reader :fetcher
   end

library/packages/src/lib/y2packager/licenses_fetchers.rb

@@ -14,6 +14,10 @@
 require "y2packager/licenses_fetchers/rpm"
 
 module Y2Packager
+  # This module contains licenses fetchers
+  #
+  # Licenses can be retrieved from different places (libzypp, URLs, etc.). The classes
+  # defined in this module are able to retrieve licenses contents.
   module LicensesFetchers
     # Return the licenses proper fetcher for a given source
     #

library/packages/src/lib/y2packager/licenses_handlers.rb

@@ -13,6 +13,11 @@
 require "y2packager/licenses_handlers/rpm"
 
 module Y2Packager
+  # This module contains licenses handlers
+  #
+  # Licenses can be retrieved from different places (libzypp, URLs, etc.). The
+  # classes defined in this module are able to interact with these sources in
+  # order to find out, for instance, whether a license must be accepted.
   module LicensesHandlers
     # Return the licenses proper fetcher for a given source
     #

library/packages/src/lib/y2packager/licenses_handlers/base.rb

@@ -30,4 +30,3 @@ def initialize(product_name)
     end
   end
 end
-

library/packages/src/lib/y2packager/licenses_handlers/rpm.rb

@@ -16,9 +16,9 @@
 
 module Y2Packager
   module LicensesHandlers
+    # This class is responsible for interacting with libzypp in order to get/set
+    # the license acceptance status for a given product
     class Rpm < Base
-      attr_reader :product_name
-
       # Determine whether the license should be accepted or not
       #
       # @return [Boolean] true if the license acceptance is required

library/packages/src/lib/y2packager/product_license.rb

@@ -16,19 +16,18 @@
 require "y2packager/licenses_handlers"
 
 module Y2Packager
+  # This class holds the license stuff for a given product
+  #
+  # Why a separate ProductLicense class? First of all, we wanted to extract
+  # the license handling from Y2Packager::Product and moving this logic to
+  # Y2Packager::License was not a good idea because different products could
+  # share the same license. Additionally, this class offers an API to work
+  # with licenses when a proper Product or Addon object is not available
+  # (backward compatibility reasons).
+  #
+  # @see Y2Packager::Product
+  # @see Y2Packager::License
   class ProductLicense
-    # This class holds the license stuff for a given product
-    #
-    # Why a separate ProductLicense class? First of all, we wanted to extract
-    # the license handling from Y2Packager::Product and moving this logic to
-    # Y2Packager::License was not a good idea because different products could
-    # share the same license. Additionally, this class offers an API to work
-    # with licenses when a proper Product or Addon object is not available
-    # (backward compatibility reasons).
-    #
-    # @see Y2Packager::Product
-    # @see Y2Packager::License
-
     extend Forwardable
 
     def_delegators :@license, :content_for, :locales, :accept!, :reject!

library/packages/test/y2packager/license_test.rb

@@ -100,7 +100,7 @@
 
       context "and the default language is specified" do
         it "returns the given content" do
-          expect(license.content_for()).to eq("Some content")
+          expect(license.content_for).to eq("Some content")
         end
       end