license-acceptance 1.0.18 → 1.0.19

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 857078f9f2d4c9d74fcb5a08876039fa5930c3f7286b1a7d03e2a84bda4d229b
-  data.tar.gz: c7c4d34ac946733f0f980f33d1b5fd0dd518e1c8fc1927a9a406580f3ac491c0
+  metadata.gz: 8383c0bafd2717c7ff9f2af1b0cc6a28b83acb5001150249734e4194a927297a
+  data.tar.gz: 798ba0cdf60a7bd4e6ca8b680eeec5039eeca8caff3a993e48d3fded2875fa37
 SHA512:
-  metadata.gz: c813cf96544c839b605631e587cad4ee3e798d6950022f454e417ac5f83234e969418f0d500f69ee730a102a046e5f20bf97782582fde203d58815d9dae30c9a
-  data.tar.gz: 6b497916918c9537c1cc234e6abace8ed448713640ac937b34593a8ac69b93bec72cfe4cca184514b4fe667203c5ec9af1546b1dcaf696a23d3318a47c37b4dd
+  metadata.gz: df6e3ccef39dcfef85a6928954cd76d2b9e3ce574291a92f368080fe1594a56b9e30fe5a43ad224994b17ce930bc21cf21a5b2dbdfb3e877e267715ef6ac200e
+  data.tar.gz: 98a5f5e51bf72821399458d4c07f9acc6c0ba69eafe748c8d0bb34c60fec62fe06582430a4cce829fdf6bbef9c1a21b3c7ddb9df4367645410ff6af8fa5134f2

data/Gemfile

@@ -1,6 +1,6 @@
 source "https://rubygems.org"
 
-git_source(:github) {|repo_name| "https://github.com/#{repo_name}" }
+git_source(:github) { |repo_name| "https://github.com/#{repo_name}" }
 
 # Specify your gem's dependencies in license-acceptance.gemspec
 gemspec

data/lib/license_acceptance/acceptor.rb

@@ -11,9 +11,9 @@ require "license_acceptance/strategy/provided_value"
 
 module LicenseAcceptance
 
-  ACCEPT = "accept"
-  ACCEPT_SILENT = "accept-silent"
-  ACCEPT_NO_PERSIST = "accept-no-persist"
+  ACCEPT = "accept".freeze
+  ACCEPT_SILENT = "accept-silent".freeze
+  ACCEPT_NO_PERSIST = "accept-no-persist".freeze
 
   class Acceptor
     extend Forwardable
@@ -21,7 +21,7 @@ module LicenseAcceptance
 
     attr_reader :config, :product_reader, :env_strategy, :file_strategy, :arg_strategy, :prompt_strategy, :provided_strategy
 
-    def initialize(opts={})
+    def initialize(opts = {})
       @config = Config.new(opts)
       Logger.initialize(config.logger)
       @product_reader = ProductReader.new
@@ -74,28 +74,28 @@ module LicenseAcceptance
           end
         end
         @acceptance_value = accepted_silent? ? ACCEPT_SILENT : ACCEPT
-        return true
+        true
       elsif config.output.isatty && prompt_strategy.request(missing_licenses) do
           # We have to infer the acceptance value if they use the prompt to accept
-          if config.persist
-            @acceptance_value = ACCEPT
-            file_strategy.persist(product_relationship, missing_licenses)
-          else
-            @acceptance_value = ACCEPT_NO_PERSIST
-            []
-          end
+        if config.persist
+          @acceptance_value = ACCEPT # rubocop: disable Lint/AssignmentInCondition
+          file_strategy.persist(product_relationship, missing_licenses)
+        else
+          @acceptance_value = ACCEPT_NO_PERSIST # rubocop: disable Lint/AssignmentInCondition
+          []
         end
-        return true
+      end
+        true
       else
         raise LicenseNotAcceptedError.new(product_relationship.parent, missing_licenses)
       end
     end
 
-    def self.check_and_persist!(product_id, version, opts={})
+    def self.check_and_persist!(product_id, version, opts = {})
       new(opts).check_and_persist!(product_id, version)
     end
 
-    def self.check_and_persist(product_id, version, opts={})
+    def self.check_and_persist(product_id, version, opts = {})
       new(opts).check_and_persist(product_id, version)
     end
 
@@ -105,7 +105,8 @@ module LicenseAcceptance
       return false if product.nil?
       # If they don't pass a version we assume they want latest
       # All versions in all channels require license acceptance
-      return true if ["latest", "unstable", "current", "stable"].include?(version.to_s) || version.nil?
+      return true if %w{latest unstable current stable}.include?(version.to_s) || version.nil?
+
       Gem::Version.new(version) >= Gem::Version.new(product.license_required_version)
     end
 
@@ -114,6 +115,7 @@ module LicenseAcceptance
     def id_from_mixlib(mixlib_name)
       product = product_reader.lookup_by_mixlib(mixlib_name)
       return nil if product.nil?
+
       product.id
     end
 
@@ -141,20 +143,20 @@ module LicenseAcceptance
     # In the case where users accept with a command line argument or environment variable
     # we still want to output the fact that the filesystem was changed.
     def output_num_persisted(count)
-      s = count > 1 ? "s": ""
+      s = count > 1 ? "s" : ""
       output.puts <<~EOM
-      #{Strategy::Prompt::BORDER}
-      #{Strategy::Prompt::CHECK} #{count} product license#{s} accepted.
-      #{Strategy::Prompt::BORDER}
+        #{Strategy::Prompt::BORDER}
+        #{Strategy::Prompt::CHECK} #{count} product license#{s} accepted.
+        #{Strategy::Prompt::BORDER}
       EOM
     end
 
     def output_persist_failed(errs)
       output.puts <<~EOM
-      #{Strategy::Prompt::BORDER}
-      #{Strategy::Prompt::CHECK} Product license accepted.
-      Could not persist acceptance:\n\t* #{errs.map(&:message).join("\n\t* ")}
-      #{Strategy::Prompt::BORDER}
+        #{Strategy::Prompt::BORDER}
+        #{Strategy::Prompt::CHECK} Product license accepted.
+        Could not persist acceptance:\n\t* #{errs.map(&:message).join("\n\t* ")}
+        #{Strategy::Prompt::BORDER}
       EOM
     end
 

data/lib/license_acceptance/cli_flags/mixlib_cli.rb

@@ -1,5 +1,5 @@
 begin
-  require 'mixlib/cli'
+  require "mixlib/cli"
 rescue => exception
   raise "Must have mixlib-cli gem installed to use this mixin"
 end

data/lib/license_acceptance/cli_flags/thor.rb

@@ -1,5 +1,5 @@
 begin
-  require 'thor'
+  require "thor"
 rescue => exception
   raise "Must have thor gem installed to use this mixin"
 end
@@ -12,7 +12,7 @@ module LicenseAcceptance
       def self.included(klass)
         klass.class_option :chef_license,
           type: :string,
-          desc: 'Accept the license for this product and any contained products: accept, accept-no-persist, accept-silent'
+          desc: "Accept the license for this product and any contained products: accept, accept-no-persist, accept-silent"
       end
 
     end

data/lib/license_acceptance/config.rb

@@ -1,10 +1,10 @@
-require 'logger'
+require "logger"
 
 module LicenseAcceptance
   class Config
     attr_accessor :output, :logger, :license_locations, :persist_location, :persist
 
-    def initialize(opts={})
+    def initialize(opts = {})
       @output = opts.fetch(:output, $stdout)
       @logger = opts.fetch(:logger, ::Logger.new(IO::NULL))
       @license_locations = opts.fetch(:license_locations, default_license_locations)
@@ -21,7 +21,7 @@ module LicenseAcceptance
 
     def default_license_locations
       if windows?
-        root = ENV.fetch("SYSTEMDRIVE","C:")
+        root = ENV.fetch("SYSTEMDRIVE", "C:")
         l = [ File.join(root, "chef/accepted_licenses/") ]
         unless is_root?
           # Look through a list of possible user locations and pick the first one that exists
@@ -32,6 +32,7 @@ module LicenseAcceptance
           possible_dirs << ENV["HOMESHARE"] + ENV["HOMEPATH"] if ENV["HOMESHARE"] && ENV["HOMEPATH"]
           possible_dirs << ENV["USERPROFILE"] if ENV["USERPROFILE"]
           raise NoValidEnvironmentVar if possible_dirs.empty?
+
           possible_dirs.each do |possible_dir|
             if Dir.exist?(possible_dir)
               full_possible_dir = File.join(possible_dir, ".chef/accepted_licenses/")
@@ -42,7 +43,7 @@ module LicenseAcceptance
         end
       else
         l = [ "/etc/chef/accepted_licenses/" ]
-        l << File.join(ENV['HOME'], ".chef/accepted_licenses/") unless is_root?
+        l << File.join(ENV["HOME"], ".chef/accepted_licenses/") unless is_root?
       end
       l
     end

data/lib/license_acceptance/product.rb

@@ -14,14 +14,15 @@ module LicenseAcceptance
 
     def ==(other)
       return false if other.class != Product
+
       if other.id == id &&
-         other.pretty_name == pretty_name &&
-         other.filename == filename &&
-         other.mixlib_name == mixlib_name
-         other.license_required_version == license_required_version
-         return true
+          other.pretty_name == pretty_name &&
+          other.filename == filename &&
+          other.mixlib_name == mixlib_name
+        other.license_required_version == license_required_version
+        return true
       end
-      return false
+      false
     end
 
   end

data/lib/license_acceptance/product_reader.rb

@@ -18,7 +18,7 @@ module LicenseAcceptance
       toml = Tomlrb.load_file(location, symbolize_keys: false)
       raise InvalidProductInfo.new(location) if toml.empty? || toml["products"].nil? || toml["relationships"].nil?
 
-      for product in toml["products"]
+      toml["products"].each do |product|
         products[product["id"]] = Product.new(
           product["id"], product["pretty_name"],
           product["filename"], product["mixlib_name"],
@@ -26,7 +26,7 @@ module LicenseAcceptance
         )
       end
 
-      for parent_id, children in toml["relationships"]
+      toml["relationships"].each do |parent_id, children|
         parent = products[parent_id]
         raise UnknownParent.new(parent_id) if parent.nil?
         # Its fine to not have a relationship entry, but not fine to have
@@ -34,9 +34,11 @@ module LicenseAcceptance
         if children.nil? || children.empty? || !children.is_a?(Array)
           raise NoChildRelationships.new(parent)
         end
+
         children.map! do |child_id|
           child = products[child_id]
           raise UnknownChild.new(child_id) if child.nil?
+
           child
         end
         relationships[parent] = children
@@ -52,6 +54,7 @@ module LicenseAcceptance
       if ENV["CHEF_LICENSE_PRODUCT_INFO"]
         return ENV["CHEF_LICENSE_PRODUCT_INFO"]
       end
+
       File.absolute_path(File.join(__FILE__, "../../../config/product_info.toml"))
     end
 
@@ -60,9 +63,10 @@ module LicenseAcceptance
         raise UnknownProduct.new(parent_id)
       end
       children = relationships.fetch(parent_product, [])
-      if !parent_version.is_a? String
+      unless parent_version.is_a? String
         raise ProductVersionTypeError.new(parent_version)
       end
+
       ProductRelationship.new(parent_product, children, parent_version)
     end
 

data/lib/license_acceptance/strategy/argument.rb

@@ -30,14 +30,15 @@ module LicenseAcceptance
         if argv.include?("--chef-license=#{sought}")
           return true
         end
+
         i = argv.index("--chef-license")
         unless i.nil?
-          val = argv[i+1]
-          if val != nil && val.downcase == sought
+          val = argv[i + 1]
+          if !val.nil? && val.downcase == sought
             return true
           end
         end
-        return false
+        false
       end
     end
   end

data/lib/license_acceptance/strategy/environment.rb

@@ -27,10 +27,11 @@ module LicenseAcceptance
       private
 
       def look_for_value(sought)
-        if env['CHEF_LICENSE'] && env['CHEF_LICENSE'].downcase == sought
+        if env["CHEF_LICENSE"] && env["CHEF_LICENSE"].downcase == sought
           return true
         end
-        return false
+
+        false
       end
 
     end

data/lib/license_acceptance/strategy/file.rb

@@ -1,7 +1,7 @@
-require 'date'
-require 'yaml'
-require 'fileutils'
-require 'etc'
+require "date"
+require "yaml"
+require "fileutils"
+require "etc"
 require "license_acceptance/logger"
 require "license_acceptance/strategy/base"
 
@@ -49,7 +49,7 @@ module LicenseAcceptance
         parent_version = product_relationship.parent_version
         root_dir = config.persist_location
 
-        if !Dir.exist?(root_dir)
+        unless Dir.exist?(root_dir)
           begin
             FileUtils.mkdir_p(root_dir)
           rescue StandardError => e
@@ -70,7 +70,7 @@ module LicenseAcceptance
             errs << err unless err.nil?
           end
         end
-        return errs
+        errs
       end
 
       private
@@ -91,11 +91,11 @@ module LicenseAcceptance
           contents = Hash[contents.map { |k, v| [k.to_s, v] }]
           license_file << YAML.dump(contents)
         end
-        return nil
+        nil
       rescue StandardError => e
         msg = "Could not persist license to #{path}"
         logger.info "#{msg}\n\t#{e.message}\n\t#{e.backtrace.join("\n\t")}"
-        return e
+        e
       end
 
     end

data/lib/license_acceptance/strategy/prompt.rb

@@ -1,5 +1,5 @@
-require 'tty-prompt'
-require 'pastel'
+require "tty-prompt"
+require "pastel"
 require "license_acceptance/logger"
 require "license_acceptance/strategy/base"
 require "timeout"
@@ -17,30 +17,30 @@ module LicenseAcceptance
         @output = config.output
       end
 
-      WIDTH = 50.freeze
+      WIDTH = 50
       PASTEL = Pastel.new
       BORDER = "+---------------------------------------------+".freeze
       YES = PASTEL.green.bold("yes")
-      CHECK  = PASTEL.green("✔")
+      CHECK = PASTEL.green("✔")
 
       def request(missing_licenses, &persist_callback)
         logger.debug("Requesting a license for #{missing_licenses.map(&:id)}")
         c = missing_licenses.size
-        s = c > 1 ? "s": ""
+        s = c > 1 ? "s" : ""
 
         acceptance_question = "Do you accept the #{c} product license#{s} (#{YES}/no)?"
         output.puts <<~EOM
-        #{BORDER}
-                    Chef License Acceptance
+          #{BORDER}
+                      Chef License Acceptance
 
-        Before you can continue, #{c} product license#{s}
-        must be accepted. View the license at
-        https://www.chef.io/end-user-license-agreement/
+          Before you can continue, #{c} product license#{s}
+          must be accepted. View the license at
+          https://www.chef.io/end-user-license-agreement/
 
-        License#{s} that need accepting:
-          * #{missing_licenses.map(&:pretty_name).join("\n  * ")}
+          License#{s} that need accepting:
+            * #{missing_licenses.map(&:pretty_name).join("\n  * ")}
 
-        #{acceptance_question}
+          #{acceptance_question}
 
         EOM
 
@@ -51,10 +51,10 @@ module LicenseAcceptance
 
         output.puts <<~EOM
 
-        If you do not accept this license you will
-        not be able to use Chef products.
+          If you do not accept this license you will
+          not be able to use Chef products.
 
-        #{acceptance_question}
+          #{acceptance_question}
 
         EOM
 
@@ -62,7 +62,7 @@ module LicenseAcceptance
         if answer != "yes"
           output.puts BORDER
         end
-        return answer
+        answer
       end
 
       private
@@ -73,7 +73,7 @@ module LicenseAcceptance
 
         answer = "no"
         begin
-          Timeout::timeout(60, PromptTimeout) do
+          Timeout.timeout(60, PromptTimeout) do
             answer = prompt.ask(">") do |q|
               q.modify :down, :trim
               q.required true
@@ -95,13 +95,13 @@ module LicenseAcceptance
             output.puts "#{CHECK} #{c} product license#{s} persisted.\n\n"
           else
             output.puts <<~EOM
-            #{CHECK} #{c} product license#{s} accepted.
-            Could not persist acceptance:\n\t* #{errs.map(&:message).join("\n\t* ")}
+              #{CHECK} #{c} product license#{s} accepted.
+              Could not persist acceptance:\n\t* #{errs.map(&:message).join("\n\t* ")}
             EOM
           end
           return true
         end
-        return false
+        false
       end
     end
 

data/lib/license_acceptance/version.rb

@@ -1,3 +1,3 @@
 module LicenseAcceptance
-  VERSION = "1.0.18"
+  VERSION = "1.0.19".freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: license-acceptance
 version: !ruby/object:Gem::Version
-  version: 1.0.18
+  version: 1.0.19
 platform: ruby
 authors:
 - tyler-ball
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-03-14 00:00:00.000000000 Z
+date: 2020-04-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: pastel
@@ -184,6 +184,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.2'
+- !ruby/object:Gem::Dependency
+  name: chefstyle
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: Chef End User License Agreement Acceptance for Ruby products
 email:
 - tball@chef.io