hammer_cli_foreman 0.14.0 → 0.15.0

data/test/functional/host_test.rb

@@ -53,6 +53,42 @@ describe 'host enc-dump' do
   end
 end
 
+describe 'host boot' do
+  let(:cmd) { ['host', 'boot'] }
+  let(:params) { ['--id=1', '--device=pxe'] }
+  let(:success_boot) do
+    JSON.dump(:action => 'pxe', :result => true)
+  end
+
+  it 'boots the host' do
+    expected_result = success_result("The host is booting.\n")
+    api_expects(:hosts, :boot, 'Interface boots from specified device')
+      .with_params('id' => '1', 'device' => 'pxe')
+      .returns(success_boot)
+
+    result = run_cmd(cmd + params)
+    assert_cmd(expected_result, result)
+  end
+end
+
+describe 'host reset' do
+  let(:cmd) { ['host', 'reset'] }
+  let(:params) { ['--id=1'] }
+  let(:success_cycle) do
+    JSON.dump(:power => true)
+  end
+
+  it 'boots the host' do
+    expected_result = success_result("Host reset started.\n")
+    api_expects(:hosts, :power, 'Run power operation on interface.')
+      .with_params('id' => '1', 'power_action' => :cycle)
+      .returns(success_cycle)
+
+    result = run_cmd(cmd + params)
+    assert_cmd(expected_result, result)
+  end
+end
+
 describe "host create" do
   let(:cmd) { ["host", "create"] }
   let(:minimal_params_without_hostgroup) { ['--location-id=1', '--organization-id=1', '--name=test'] }
@@ -87,6 +123,19 @@ describe "host create" do
     assert_cmd(expected_result, result)
   end
 
+  it "accepts host parameter attributes" do
+    params = ['--parameters', 'param1=value1']
+
+    api_expects(:hosts, :create, 'Create host with parameters') do |par|
+      par['host']['host_parameters_attributes'][0]['name'] == 'param1' &&
+      par['host']['host_parameters_attributes'][0]['value'] == 'value1'
+    end
+
+    expected_result = success_result("Host created.\n")
+    result = run_cmd(cmd + minimal_params + params)
+    assert_cmd(expected_result, result)
+  end
+
   it "sends empty hash for no interfaces" do
     # For some reason the Foreman replaces empty interfaces_attributes to nil,
     # which causes failure in nested attributes assignment in the host model

data/test/functional/subnet/create_test.rb

@@ -0,0 +1,131 @@
+require_relative '../test_helper'
+require 'hammer_cli_foreman/subnet'
+
+module HammerCLIForeman
+  describe Subnet do
+    describe CreateCommand do
+      def subnet_params(additional_params = {})
+        params = {
+          :subnet => {
+            :name => 'net1'
+          }
+        }
+        params[:subnet].merge!(additional_params)
+        params
+      end
+
+      it 'allows minimal options' do
+        api_expects(:subnets, :create) do |par|
+          par['subnet']['name'] == 'net1'
+        end
+        run_cmd(%w(subnet create --name net1 --network=192.168.122.0 --mask=255.255.255.0))
+      end
+
+      it 'allows dhcp id' do
+        api_expects(:subnets, :create).with_params(subnet_params(:dhcp_id => '1'))
+        run_cmd(%w(subnet create --name net1 --network=192.168.122.0 --mask=255.255.255.0 --dhcp-id 1))
+      end
+
+      it 'allows dhcp name' do
+        api_expects_search(:smart_proxies, { :name => 'sp1' })
+        .returns(index_response([{'id' => 1}]))
+        api_expects(:subnets, :create).with_params(subnet_params(:dhcp_id => 1))
+        run_cmd(%w(subnet create --name net1 --network=192.168.122.0 --mask=255.255.255.0 --dhcp sp1))
+      end
+
+      it 'allows dns id' do
+        api_expects(:subnets, :create).with_params(subnet_params(:dns_id => '1'))
+        run_cmd(%w(subnet create --name net1 --network=192.168.122.0 --mask=255.255.255.0 --dns-id 1))
+      end
+
+      it 'allows dns name' do
+        api_expects_search(:smart_proxies, { :name => 'sp1' })
+        .returns(index_response([{'id' => 1}]))
+        api_expects(:subnets, :create).with_params(subnet_params(:dns_id => 1))
+        run_cmd(%w(subnet create --name net1 --network=192.168.122.0 --mask=255.255.255.0 --dns sp1))
+      end
+
+      it 'allows tftp id' do
+        api_expects(:subnets, :create).with_params(subnet_params(:tftp_id => '1'))
+        run_cmd(%w(subnet create --name net1 --network=192.168.122.0 --mask=255.255.255.0 --tftp-id 1))
+      end
+
+      it 'allows tftp name' do
+        api_expects_search(:smart_proxies, { :name => 'sp1' })
+        .returns(index_response([{'id' => 1}]))
+        api_expects(:subnets, :create).with_params(subnet_params(:tftp_id => 1))
+        run_cmd(%w(subnet create --name net1 --network=192.168.122.0 --mask=255.255.255.0 --tftp sp1))
+      end
+
+      it 'allows domain ids' do
+        api_expects(:subnets, :create).with_params(subnet_params(:domain_ids => ['1', '4']))
+        run_cmd(%w(subnet create --name net1 --network=192.168.122.0 --mask=255.255.255.0 --domain-ids 1,4))
+      end
+
+      it 'allows domain names' do
+        api_expects(:domains, :index) do |p|
+          p[:search] == "name = \"d1\" or name = \"d2\""
+        end.returns(index_response([{'id' => 1}, {'id' => 2}]))
+        api_expects(:subnets, :create).with_params(subnet_params(:domain_ids => [1, 2]))
+        run_cmd(%w(subnet create --name net1 --network=192.168.122.0 --mask=255.255.255.0 --domains d1,d2))
+      end
+
+      it 'allows location ids' do
+        api_expects(:subnets, :create).with_params(subnet_params(:location_ids => ['1', '4']))
+        run_cmd(%w(subnet create --name net1 --network=192.168.122.0 --mask=255.255.255.0 --location-ids 1,4))
+      end
+
+      it 'allows location names' do
+        api_expects(:locations, :index) do |p|
+          p[:search] == "name = \"loc1\" or name = \"loc2\""
+        end.returns(index_response([{'id' => 1}, {'id' => 2}]))
+        api_expects(:subnets, :create).with_params(subnet_params(:location_ids => [1, 2]))
+        run_cmd(%w(subnet create --name net1 --network=192.168.122.0 --mask=255.255.255.0 --locations loc1,loc2))
+      end
+
+      it 'allows organization ids' do
+        api_expects(:subnets, :create).with_params(subnet_params(:organization_ids => ['1', '4']))
+        run_cmd(%w(subnet create --name net1 --network=192.168.122.0 --mask=255.255.255.0 --organization-ids 1,4))
+      end
+
+      it 'allows organization names' do
+        api_expects(:organizations, :index) do |p|
+          p[:search] == "name = \"org1\" or name = \"org2\""
+        end.returns(index_response([{'id' => 1}, {'id' => 2}])) 
+        api_expects(:subnets, :create).with_params(subnet_params(:organization_ids => [1, 2]))
+        run_cmd(%w(subnet create --name net1 --network=192.168.122.0 --mask=255.255.255.0 --organizations org1,org2))
+      end
+
+      it 'allows primary dns' do
+        api_expects(:subnets, :create).with_params(subnet_params(:dns_primary => '192.168.122.2'))
+        run_cmd(%w(subnet create --name net1 --network=192.168.122.0 --mask=255.255.255.0 --dns-primary 192.168.122.2))
+      end
+
+      it 'allows secondary dns' do
+        api_expects(:subnets, :create).with_params(subnet_params(:dns_secondary => '192.168.122.2'))
+        run_cmd(%w(subnet create --name net1 --network=192.168.122.0 --mask=255.255.255.0 --dns-secondary 192.168.122.2))
+      end
+
+      it 'allows ip address range FROM' do
+        api_expects(:subnets, :create).with_params(subnet_params(:from => '192.168.122.3'))
+        run_cmd(%w(subnet create --name net1 --network=192.168.122.0 --mask=255.255.255.0 --from 192.168.122.3))
+      end
+
+      it 'allows ip address range TO' do
+        api_expects(:subnets, :create).with_params(subnet_params(:to => '192.168.122.60'))
+        run_cmd(%w(subnet create --name net1 --network=192.168.122.0 --mask=255.255.255.0 --to 192.168.122.60))
+      end
+
+      it 'allows gateway' do
+        api_expects(:subnets, :create).with_params(subnet_params(:gateway => '192.168.122.1'))
+        run_cmd(%w(subnet create --name net1 --network=192.168.122.0 --mask=255.255.255.0 --gateway 192.168.122.1))
+      end
+
+      it 'allows network type' do
+        api_expects(:subnets, :create).with_params(subnet_params(:network_type => 'IPv4'))
+        run_cmd(%w(subnet create --name net1 --network=192.168.122.0 --mask=255.255.255.0 --network-type IPv4))
+      end
+
+    end
+  end
+end

data/test/functional/subnet/update_test.rb

@@ -0,0 +1,130 @@
+require_relative '../test_helper'
+require 'hammer_cli_foreman/subnet'
+
+module HammerCLIForeman
+  describe Subnet do
+    describe UpdateCommand do
+      def subnet_params(additional_params = {})
+        params = {
+          :id => '1',
+          :subnet => {}
+        }
+        params[:subnet].merge!(additional_params)
+        params
+      end
+
+      it 'allows minimal options' do
+        api_expects(:subnets, :update) do |par|
+          par['id'] == '1'
+        end
+        run_cmd(%w(subnet update --id 1))
+      end
+
+      it 'allows dhcp id' do
+        api_expects(:subnets, :update).with_params(subnet_params(:dhcp_id => '1'))
+        run_cmd(%w(subnet update --id 1 --dhcp-id 1))
+      end
+
+      it 'allows dhcp name' do
+        api_expects_search(:smart_proxies, { :name => 'sp1' })
+        .returns(index_response([{'id' => 1}]))
+        api_expects(:subnets, :update).with_params(subnet_params(:dhcp_id => 1))
+        run_cmd(%w(subnet update --id 1 --dhcp sp1))
+      end
+
+      it 'allows dns id' do
+        api_expects(:subnets, :update).with_params(subnet_params(:dns_id => '1'))
+        run_cmd(%w(subnet update --id 1 --dns-id 1))
+      end
+
+      it 'allows dns name' do
+        api_expects_search(:smart_proxies, { :name => 'sp1' })
+        .returns(index_response([{'id' => 1}]))
+        api_expects(:subnets, :update).with_params(subnet_params(:dns_id => 1))
+        run_cmd(%w(subnet update --id 1 --dns sp1))
+      end
+
+      it 'allows tftp id' do
+        api_expects(:subnets, :update).with_params(subnet_params(:tftp_id => '1'))
+        run_cmd(%w(subnet update --id 1 --tftp-id 1))
+      end
+
+      it 'allows tftp name' do
+        api_expects_search(:smart_proxies, { :name => 'sp1' })
+        .returns(index_response([{'id' => 1}]))
+        api_expects(:subnets, :update).with_params(subnet_params(:tftp_id => 1))
+        run_cmd(%w(subnet update --id 1 --tftp sp1))
+      end
+
+      it 'allows domain ids' do
+        api_expects(:subnets, :update).with_params(subnet_params(:domain_ids => ['1', '4']))
+        run_cmd(%w(subnet update --id 1 --domain-ids 1,4))
+      end
+
+      it 'allows domain names' do
+        api_expects(:domains, :index) do |p|
+          p[:search] == "name = \"d1\" or name = \"d2\""
+        end.returns(index_response([{'id' => 1}, {'id' => 2}]))
+        api_expects(:subnets, :update).with_params(subnet_params(:domain_ids => [1, 2]))
+        run_cmd(%w(subnet update --id 1 --domains d1,d2))
+      end
+
+      it 'allows location ids' do
+        api_expects(:subnets, :update).with_params(subnet_params(:location_ids => ['1', '4']))
+        run_cmd(%w(subnet update --id 1 --location-ids 1,4))
+      end
+
+      it 'allows location names' do
+        api_expects(:locations, :index) do |p|
+          p[:search] == "name = \"loc1\" or name = \"loc2\""
+        end.returns(index_response([{'id' => 1}, {'id' => 2}]))
+        api_expects(:subnets, :update).with_params(subnet_params(:location_ids => [1, 2]))
+        run_cmd(%w(subnet update --id 1 --locations loc1,loc2))
+      end
+
+      it 'allows organization ids' do
+        api_expects(:subnets, :update).with_params(subnet_params(:organization_ids => ['1', '4']))
+        run_cmd(%w(subnet update --id 1 --organization-ids 1,4))
+      end
+
+      it 'allows organization names' do
+        api_expects(:organizations, :index) do |p|
+          p[:search] == "name = \"org1\" or name = \"org2\""
+        end.returns(index_response([{'id' => 1}, {'id' => 2}]))
+        api_expects(:subnets, :update).with_params(subnet_params(:organization_ids => [1, 2]))
+        run_cmd(%w(subnet update --id 1 --organizations org1,org2))
+      end
+
+      it 'allows primary dns' do
+        api_expects(:subnets, :update).with_params(subnet_params(:dns_primary => '192.168.122.2'))
+        run_cmd(%w(subnet update --id 1 --dns-primary 192.168.122.2))
+      end
+
+      it 'allows secondary dns' do
+        api_expects(:subnets, :update).with_params(subnet_params(:dns_secondary => '192.168.122.2'))
+        run_cmd(%w(subnet update --id 1 --dns-secondary 192.168.122.2))
+      end
+
+      it 'allows ip address range FROM' do
+        api_expects(:subnets, :update).with_params(subnet_params(:from => '192.168.122.3'))
+        run_cmd(%w(subnet update --id 1 --from 192.168.122.3))
+      end
+
+      it 'allows ip address range TO' do
+        api_expects(:subnets, :update).with_params(subnet_params(:to => '192.168.122.60'))
+        run_cmd(%w(subnet update --id 1 --to 192.168.122.60))
+      end
+
+      it 'allows gateway' do
+        api_expects(:subnets, :update).with_params(subnet_params(:gateway => '192.168.122.1'))
+        run_cmd(%w(subnet update --id 1 --gateway 192.168.122.1))
+      end
+
+      it 'allows network type' do
+        api_expects(:subnets, :update).with_params(subnet_params(:network_type => 'IPv4'))
+        run_cmd(%w(subnet update --id 1 --network-type IPv4))
+      end
+
+    end
+  end
+end

data/test/unit/apipie_resource_mock.rb

@@ -45,6 +45,10 @@ module ResourceMocks
     ResourceMocks.mock_action_call(:compute_resources, :available_networks, [])
   end
 
+  def self.compute_resources_available_clusters
+    ResourceMocks.mock_action_call(:compute_resources, :available_clusters, [])
+  end
+
   def self.organizations_index
     ResourceMocks.mock_action_call(:organizations, :index, {
      "results" => [

data/test/unit/compute_resource_test.rb

@@ -94,6 +94,28 @@ describe HammerCLIForeman::ComputeResource do
 
   end
 
+  context "AvailableClustersCommand" do
+    before do
+      ResourceMocks.compute_resources_available_clusters
+    end
+
+    let(:cmd) { HammerCLIForeman::ComputeResource::AvailableClustersCommand.new("", ctx) }
+
+    context "parameters" do
+      it_should_accept "id", ["--id=1"]
+      it_should_accept "name", ["--name=domain-c7"]
+    end
+
+    context "output" do
+      let(:expected_record_count) { count_records(cmd.resource.call(:available_clusters)) }
+
+      with_params ["--name=testcr"] do
+        it_should_print_n_records
+        it_should_print_columns ["Name", "Id"]
+      end
+    end
+  end
+
   context "AvailableNetworksCommand" do
     before do
       ResourceMocks.compute_resources_available_networks

data/test/unit/host_test.rb

@@ -23,7 +23,7 @@ describe HammerCLIForeman::Host do
       let(:expected_record_count) { count_records(cmd.resource.call(:index)) }
 
       it_should_print_n_records
-      it_should_print_columns ["Id", "Name", "Operating System", "Host Group", "IP", "MAC"]
+      it_should_print_columns ["Id", "Name", "Operating System", "Host Group", "IP", "MAC", "Global Status"]
     end
 
   end
@@ -47,7 +47,7 @@ describe HammerCLIForeman::Host do
         it_should_print_columns ["Id", "Name", "Organization", "Location"]
         it_should_print_columns ["Host Group", "Compute Resource", "Compute Profile", "Puppet Environment"]
         it_should_print_columns ["Puppet CA Proxy", "Puppet Master Proxy", "Cert name"]
-        it_should_print_columns ["Managed", "Installed at", "Last report"]
+        it_should_print_columns ["Managed", "Status", "Installed at", "Last report"]
         it_should_print_columns ["Network", "Network interfaces", "Operating system", "Parameters", "All parameters", "Additional info"]
       end
     end

data/test/unit/output/formatters_test.rb

@@ -1,9 +1,8 @@
 require File.join(File.dirname(__FILE__), '../test_helper')
 
+describe HammerCLIForeman::Output::Formatters::StructuredReferenceFormatter do
 
-describe HammerCLIForeman::Output::Formatters::SingleReferenceFormatter do
-
-  let(:formatter) { HammerCLIForeman::Output::Formatters::SingleReferenceFormatter.new }
+  let(:formatter) { HammerCLIForeman::Output::Formatters::StructuredReferenceFormatter.new }
   let(:reference) do
     {
       :server_id => 1,
@@ -17,30 +16,32 @@ describe HammerCLIForeman::Output::Formatters::SingleReferenceFormatter do
     end
   end
 
-  context "with symbol keys" do
-    it "formats name" do
-      options = {:key => :server}
-      formatter.format(reference, options).must_equal 'Server'
+  context 'with symbol keys' do
+    it 'formats name' do
+      options = {:display_field_key => :server_name}
+      formatter.format(reference, options).must_equal('Name' => "#{reference_str_keys['server_name']}")
     end
 
-    it "formats id" do
-      options = {:key => :server, :context => {:show_ids => true}}
-      formatter.format(reference, options).must_equal 'Server (id: 1)'
+    it 'formats id' do
+      options = {:display_field_key => :server_name,
+                 :details => {:structured_label => 'Id', :key => :server_id}}
+      formatter.format(reference, options)
+        .must_equal('Name' => "#{reference_str_keys['server_name']}", 'Id' => reference_str_keys['server_id'])
     end
   end
 
-  context "with string keys" do
-    it "formats name" do
-      options = {:key => :server}
-      formatter.format(reference_str_keys, options).must_equal 'Server'
+  context 'with string keys' do
+    it 'formats name' do
+      options = {:display_field_key => :server_name}
+      formatter.format(reference_str_keys, options).must_equal('Name' => "#{reference_str_keys['server_name']}")
     end
 
-    it "formats id" do
-      options = {:key => :server, :context => {:show_ids => true}}
-      formatter.format(reference_str_keys, options).must_equal 'Server (id: 1)'
+    it 'formats id' do
+      options = {:display_field_key => :server_name,
+                 :details => {:structured_label => 'Id', :key => :server_id}}
+      formatter.format(reference_str_keys, options).must_equal('Name' => "#{reference_str_keys['server_name']}", 'Id' => reference_str_keys['server_id'])
     end
   end
-
 end
 
 describe HammerCLIForeman::Output::Formatters::ReferenceFormatter do
@@ -57,94 +58,102 @@ describe HammerCLIForeman::Output::Formatters::ReferenceFormatter do
     }
   end
 
-
-  it "recovers when the resource is missing" do
+  it 'recovers when the resource is missing' do
     formatter.format(nil).must_equal ''
   end
 
-  context "with symbol keys" do
+  context 'with symbol keys' do
     let(:reference_sym_keys) do
       reference
     end
 
-    it "formats name" do
-      formatter.format(reference_sym_keys, {}).must_equal 'Server'
-    end
-
-    it "can override name key" do
-      options = {:name_key => :another_name}
-      formatter.format(reference_sym_keys, options).must_equal 'SERVER'
-    end
-
-    it "formats id" do
-      options = {:context => {:show_ids => true}}
-      formatter.format(reference_sym_keys, options).must_equal 'Server (id: 1)'
+    it 'formats name' do
+      formatter.format(reference_sym_keys, {}).must_equal("#{reference_sym_keys[:name]}")
     end
 
-    it "can override id key" do
-      options = {:id_key => :another_id, :context => {:show_ids => true}}
-      formatter.format(reference_sym_keys, options).must_equal 'Server (id: 2)'
+    it 'can override name key' do
+      options = {:display_field_key => :another_name}
+      formatter.format(reference_sym_keys, options).must_equal("#{reference_sym_keys[:another_name]}")
     end
 
-    it "formats details" do
-      options = {:details => :url}
-      formatter.format(reference_sym_keys, options).must_equal 'Server (URL)'
+    it 'formats id' do
+      options = {:details => {:key => :id, :label => 'Id'}, :context => {:show_ids => true}}
+      formatter.format(reference_sym_keys, options)
+        .must_equal("Server (Id: #{reference_sym_keys[:id]})")
     end
 
-    it "formats multiple details" do
-      options = {:details => [:url, :desc]}
-      formatter.format(reference_sym_keys, options).must_equal 'Server (URL, Description)'
+    it 'formats details' do
+      options = { :details => { :label => _('url'), :key => :url } }
+      formatter.format(reference_sym_keys, options).must_equal("Server (url: #{reference_sym_keys[:url]})")
     end
 
-    it "formats details and id" do
-      options = {:context => {:show_ids => true}, :details => [:url, :desc]}
-      formatter.format(reference_sym_keys, options).must_equal 'Server (URL, Description, id: 1)'
+    it 'formats multiple details' do
+      options = { :details => [{ :label => _('url'), :key => :url },
+                               {:label => _('desc'), :key => :desc }]
+                }
+      formatter.format(reference_sym_keys, options)
+        .must_equal("Server (url: #{reference_sym_keys[:url]}, desc: #{reference_sym_keys[:desc]})")
     end
 
-    it "handles empty string properly" do
-      formatter.format("", {}).must_equal ""
+    it 'formats details and id' do
+      options = {:context => {:show_ids => true},
+                 :details => [{ :label => _('url'), :key => :url },
+                              {:label => _('desc'), :key => :desc },
+                              {:label => _('Id'), :key => :id }]
+                }
+      formatter.format(reference_sym_keys, options)
+        .must_equal("Server (url: #{reference_sym_keys[:url]}, desc: #{reference_sym_keys[:desc]}, Id: #{reference_sym_keys[:id]})")
     end
   end
 
-  context "with string keys" do
+  context 'with string keys' do
     let(:reference_str_keys) do
       reference.inject({}) do |new_ref, (key, value)|
         new_ref.update(key.to_s => value)
       end
     end
 
-    it "formats name" do
-      formatter.format(reference_str_keys, {}).must_equal 'Server'
+    it 'formats name' do
+      formatter.format(reference_str_keys, {}).must_equal("#{reference_str_keys['name']}")
     end
 
-    it "can override name key" do
-      options = {:name_key => :another_name}
-      formatter.format(reference_str_keys, options).must_equal 'SERVER'
+    it 'can override name key' do
+      options = {:display_field_key => :another_name}
+      formatter.format(reference_str_keys, options).must_equal("#{reference_str_keys['another_name']}")
     end
 
-    it "formats id" do
-      options = {:context => {:show_ids => true}}
-      formatter.format(reference_str_keys, options).must_equal 'Server (id: 1)'
+    it 'formats id when show_ids is true' do
+      options = {:details => {:label => 'Id', :key => :id, :id => 1}, :context => {:show_ids => true}}
+      formatter.format(reference_str_keys, options)
+               .must_equal("Server (Id: #{reference_str_keys['id']})")
     end
 
-    it "can override id key" do
-      options = {:id_key => :another_id, :context => {:show_ids => true}}
-      formatter.format(reference_str_keys, options).must_equal 'Server (id: 2)'
+    it 'does not formats id when show_ids is false' do
+      options = {:details => {:label => 'Id', :key => :id,  :id => 1}, :context => {:show_ids => false}}
+      formatter.format(reference_str_keys, options)
+               .must_equal("#{reference_str_keys['name']}")
     end
 
-    it "formats details" do
-      options = {:details => :url}
-      formatter.format(reference_str_keys, options).must_equal 'Server (URL)'
+    it 'formats details' do
+      options = { :details => { :label => _('url'), :key => :url } }
+      formatter.format(reference_str_keys, options)
+               .must_equal("Server (url: #{reference_str_keys['url']})")
     end
 
-    it "formats multiple details" do
-      options = {:details => [:url, :desc]}
-      formatter.format(reference_str_keys, options).must_equal 'Server (URL, Description)'
+    it 'formats multiple details' do
+      options = { :details => [{ :label => _('url'), :key => :url },
+                               { :label => _('desc'), :key => :desc }] }
+      formatter.format(reference_str_keys, options)
+               .must_equal("Server (url: #{reference_str_keys['url']}, desc: #{reference_str_keys['desc']})")
     end
 
-    it "formats details and id" do
-      options = {:context => {:show_ids => true}, :details => [:url, :desc]}
-      formatter.format(reference_str_keys, options).must_equal 'Server (URL, Description, id: 1)'
+    it 'formats details and id' do
+      options = {:context => { :show_ids => true },
+                 :details => [{ :label => _('url'), :key => :url },
+                              { :label => _('desc'), :key => :desc },
+                              { :label => _('Id'), :key => :id }] }
+      formatter.format(reference_str_keys, options)
+               .must_equal("Server (url: #{reference_str_keys['url']}, desc: #{reference_str_keys['desc']}, Id: #{reference_str_keys['id']})")
     end
   end