hammer_cli_foreman 0.1.1 → 0.1.2

data/lib/hammer_cli_foreman/searchables_option_builder.rb

@@ -1,99 +0,0 @@
-
-module HammerCLIForeman
-
-  class SearchablesOptionBuilder < HammerCLI::AbstractOptionBuilder
-
-    def initialize(resource, searchables)
-      @resource = resource
-      @searchables = searchables
-    end
-
-    def build(builder_params={})
-      @searchables.for(@resource).collect do |s|
-        option(
-          optionamize("--#{s.name}"),
-          s.name.upcase,
-          s.description
-        )
-      end
-    end
-  end
-
-
-  class DependentSearchablesOptionBuilder < HammerCLI::AbstractOptionBuilder
-
-    def initialize(dependent_resources, searchables)
-      @dependent_resources = dependent_resources.is_a?(Array) ? dependent_resources : [dependent_resources]
-      @searchables = searchables
-    end
-
-
-    def build(builder_params={})
-      @dependent_resources.collect do |res|
-        dependent_options(res)
-      end.flatten(1)
-    end
-
-    def dependent_options(resource)
-      options = []
-      searchables = @searchables.for(resource)
-      resource_name = resource.singular_name
-
-      unless searchables.empty?
-        first = searchables[0]
-        remaining = searchables[1..-1] || []
-
-        # First option is named by the resource
-        # Eg. --organization with accessor option_organization_name
-        options << option(
-          optionamize("--#{resource_name}"),
-          "#{resource_name}_#{first.name}".upcase,
-          " ",
-          :attribute_name => HammerCLI.option_accessor_name("#{resource_name}_#{first.name}")
-        )
-
-        # Other options are named by the resource plus the searchable name
-        # Eg. --organization-label with accessor option_organization_label
-        remaining.each do |s|
-          options << option(
-            optionamize("--#{resource_name}-#{s.name}"),
-            "#{resource_name}_#{s.name}".upcase,
-            " "
-          )
-        end
-      end
-
-      options << option(
-        optionamize("--#{resource_name}-id"),
-        "#{resource_name}_id".upcase,
-        " "
-      )
-      options
-    end
-
-  end
-
-  class SearchablesUpdateOptionBuilder < HammerCLI::AbstractOptionBuilder
-
-    def initialize(resource, searchables)
-      @resource = resource
-      @searchables = searchables
-    end
-
-    def build(builder_params={})
-
-      @searchables.for(@resource).collect do |s|
-        if s.editable?
-          option(
-            optionamize("--new-#{s.name}"),
-            "NEW_#{s.name.upcase}",
-            " "
-          )
-        end
-      end.compact
-    end
-
-  end
-
-
-end

data/test/unit/searchables_option_builder_test.rb

@@ -1,172 +0,0 @@
-require File.join(File.dirname(__FILE__), 'test_helper')
-require File.join(File.dirname(__FILE__), 'helpers/fake_searchables')
-
-
-
-describe HammerCLIForeman::SearchablesOptionBuilder do
-
-  let(:resource) { HammerCLIForeman.foreman_resource(:architectures) }
-  let(:searchables) { FakeSearchables.new(["name", "label"]) }
-  let(:builder) { HammerCLIForeman::SearchablesOptionBuilder.new(resource, searchables) }
-  let(:options) { builder.build }
-
-  describe "empty searchables" do
-    let(:searchables) { FakeSearchables.new([]) }
-
-    it "builds no options for empty searchables" do
-      options.must_equal []
-    end
-  end
-
-  describe "multiple searchables" do
-
-    it "builds correct switches" do
-      options.map(&:switches).must_equal [["--name"], ["--label"]]
-    end
-
-    it "builds correct descriptions" do
-      options.map(&:description).must_equal ["Search by name", "Search by label"]
-    end
-
-    it "builds correct types" do
-      options.map(&:type).must_equal ["NAME", "LABEL"]
-    end
-
-    it "builds correct attribute readers" do
-      options.map(&:read_method).must_equal [
-        "option_name",
-        "option_label"
-      ]
-    end
-
-    it "none of the options is required" do
-      options.any?{|opt| opt.required? }.must_equal false
-    end
-  end
-
-end
-
-
-describe HammerCLIForeman::DependentSearchablesOptionBuilder do
-
-  let(:resource) { HammerCLIForeman.foreman_resource(:architectures) }
-  let(:searchables) { FakeSearchables.new(["name", "label", "uuid"]) }
-  let(:builder) { HammerCLIForeman::DependentSearchablesOptionBuilder.new(resource, searchables) }
-  let(:options) { builder.build }
-
-  describe "empty searchables" do
-
-    let(:searchables) { FakeSearchables.new([]) }
-
-    it "builds only id options" do
-      options.map(&:switches).must_equal [["--architecture-id"]]
-    end
-
-  end
-
-
-  describe "multiple searchables" do
-
-    it "creates correct switches" do
-      options.map(&:switches).must_equal [
-        ["--architecture"],       # first option does not have the suffix
-        ["--architecture-label"], # other options with suffixes
-        ["--architecture-uuid"],
-        ["--architecture-id"]     # additional id
-      ]
-    end
-
-    it "creates correct option types" do
-      options.map(&:type).must_equal [
-        "ARCHITECTURE_NAME",
-        "ARCHITECTURE_LABEL",
-        "ARCHITECTURE_UUID",
-        "ARCHITECTURE_ID",
-      ]
-    end
-
-    it "creates correct attribute readers" do
-      options.map(&:read_method).must_equal [
-        "option_architecture_name",
-        "option_architecture_label",
-        "option_architecture_uuid",
-        "option_architecture_id",
-      ]
-    end
-
-    it "none of the options is required" do
-      options.any?{|opt| opt.required? }.must_equal false
-    end
-  end
-
-
-  describe "multiple resources" do
-
-    let(:resource) { [
-      HammerCLIForeman.foreman_resource(:architectures),
-      HammerCLIForeman.foreman_resource(:domains)
-    ] }
-    let(:searchables) { FakeSearchables.new(["name", "label"]) }
-
-    it "builds options for all resources" do
-      options.map(&:switches).must_equal [
-        ["--architecture"],       # first option does not have the suffix
-        ["--architecture-label"], # other options with suffixes
-        ["--architecture-id"],    # additional id
-        ["--domain"],       # first option does not have the suffix
-        ["--domain-label"], # other options with suffixes
-        ["--domain-id"]     # additional id
-      ]
-    end
-
-    it "none of the options is required" do
-      options.any?{|opt| opt.required? }.must_equal false
-    end
-
-  end
-
-end
-
-
-describe HammerCLIForeman::SearchablesUpdateOptionBuilder do
-
-  let(:resource) { HammerCLIForeman.foreman_resource(:architectures) }
-  let(:searchables) { FakeSearchables.new(["name"], ["label"]) }
-  let(:builder) { HammerCLIForeman::SearchablesUpdateOptionBuilder.new(resource, searchables) }
-  let(:options) { builder.build }
-
-  describe "empty searchables" do
-    let(:searchables) { FakeSearchables.new([]) }
-
-    it "builds no options for empty searchables" do
-      options.must_equal []
-    end
-  end
-
-  describe "multiple searchables" do
-
-    it "builds correct switches" do
-      options.map(&:switches).must_equal [["--new-label"]]
-    end
-
-    it "builds correct descriptions" do
-      options.map(&:description).must_equal [" "]
-    end
-
-    it "builds correct types" do
-      options.map(&:type).must_equal ["NEW_LABEL"]
-    end
-
-    it "builds correct attribute readers" do
-      options.map(&:read_method).must_equal [
-        "option_new_label"
-      ]
-    end
-
-    it "none of the options is required" do
-      options.any?{|opt| opt.required? }.must_equal false
-    end
-  end
-
-
-end