foreman_remote_execution 0.2.3 → 0.3.0

data/test/unit/remote_execution_feature_test.rb

@@ -0,0 +1,42 @@
+require 'test_plugin_helper'
+
+describe RemoteExecutionFeature do
+
+  let(:install_feature) do
+    RemoteExecutionFeature.register(:katello_install_package, N_('Katello: Install package'),
+                                    :description => 'Install package via Katello user interface',
+                                    :provided_inputs => ['package'])
+  end
+
+  let(:package_template) do
+    FactoryGirl.create(:job_template).tap do |job_template|
+      job_template.job_category = 'Package Action'
+      job_template.name = 'Package Action - SSH Default'
+      job_template.template_inputs.create(:name => 'package', :input_type => 'user')
+    end
+  end
+
+  let(:host) { FactoryGirl.create(:host) }
+
+  before do
+    User.current = users :admin
+    install_feature.update_attributes!(:job_template_id => package_template.id)
+  end
+
+  describe 'composer' do
+    it 'prepares composer for given feature based on the mapping' do
+      composer = JobInvocationComposer.for_feature(:katello_install_package, host, :package => 'zsh')
+      assert composer.valid?
+      composer.pattern_template_invocations.size.must_equal 1
+      template_invocation = composer.pattern_template_invocations.first
+      template_invocation.template.must_equal package_template
+      template_invocation.input_values.size.must_equal 1
+
+      input_value = template_invocation.input_values.first
+      input_value.value.must_equal 'zsh'
+      input_value.template_input.name.must_equal 'package'
+
+      composer.targeting.search_query.must_equal "name = #{host.name}"
+    end
+  end
+end

data/test/unit/targeting_test.rb

@@ -16,7 +16,7 @@ describe Targeting do
     end
 
     context 'resolved_at is set' do
-      before { targeting.resolved_at = Time.now }
+      before { targeting.resolved_at = Time.now.getlocal }
       it { assert targeting.resolved? }
     end
 
@@ -86,7 +86,7 @@ describe Targeting do
     end
 
     context 'for two hosts' do
-      let(:query) { targeting.build_query_from_hosts([ host.id, second_host.id ]) }
+      let(:query) { Targeting.build_query_from_hosts([ host.id, second_host.id ]) }
 
       it 'builds query using host names joining with or' do
         query.must_include "name = #{host.name}"
@@ -99,7 +99,7 @@ describe Targeting do
     end
 
     context 'for one host' do
-      let(:query) { targeting.build_query_from_hosts([ host.id ]) }
+      let(:query) { Targeting.build_query_from_hosts([ host.id ]) }
 
       it 'builds query using host name' do
         query.must_equal "name = #{host.name}"
@@ -109,7 +109,7 @@ describe Targeting do
     end
 
     context 'for no id' do
-      let(:query) { targeting.build_query_from_hosts([]) }
+      let(:query) { Targeting.build_query_from_hosts([]) }
 
       it 'builds query to find all hosts' do
         Host.search_for(query).must_include host

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: foreman_remote_execution
 version: !ruby/object:Gem::Version
-  version: 0.2.3
+  version: 0.3.0
 platform: ruby
 authors:
 - Foreman Remote Execution team
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-02-11 00:00:00.000000000 Z
+date: 2016-02-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: deface
@@ -30,14 +30,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.8.8
+        version: 0.8.10
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.8.8
+        version: 0.8.10
 - !ruby/object:Gem::Dependency
   name: foreman-tasks
   requirement: !ruby/object:Gem::Requirement
@@ -143,9 +143,11 @@ files:
 - app/controllers/api/v2/foreign_input_sets_controller.rb
 - app/controllers/api/v2/job_invocations_controller.rb
 - app/controllers/api/v2/job_templates_controller.rb
+- app/controllers/api/v2/remote_execution_features_controller.rb
 - app/controllers/api/v2/template_inputs_controller.rb
 - app/controllers/job_invocations_controller.rb
 - app/controllers/job_templates_controller.rb
+- app/controllers/remote_execution_features_controller.rb
 - app/controllers/template_invocations_controller.rb
 - app/helpers/concerns/foreman_remote_execution/hosts_helper_extensions.rb
 - app/helpers/concerns/foreman_remote_execution/job_templates_extensions.rb
@@ -176,6 +178,7 @@ files:
 - app/models/job_invocation_task_group.rb
 - app/models/job_template.rb
 - app/models/job_template_effective_user.rb
+- app/models/remote_execution_feature.rb
 - app/models/remote_execution_provider.rb
 - app/models/setting/remote_execution.rb
 - app/models/ssh_execution_provider.rb
@@ -204,13 +207,16 @@ files:
 - app/views/api/v2/job_templates/main.json.rabl
 - app/views/api/v2/job_templates/show.json.rabl
 - app/views/api/v2/job_templates/update.json.rabl
+- app/views/api/v2/remote_execution_features/base.json.rabl
+- app/views/api/v2/remote_execution_features/index.json.rabl
+- app/views/api/v2/remote_execution_features/main.json.rabl
+- app/views/api/v2/remote_execution_features/show.json.rabl
 - app/views/api/v2/template_inputs/base.json.rabl
 - app/views/api/v2/template_inputs/create.json.rabl
 - app/views/api/v2/template_inputs/index.json.rabl
 - app/views/api/v2/template_inputs/main.json.rabl
 - app/views/api/v2/template_inputs/show.json.rabl
 - app/views/dashboard/.gitkeep
-- app/views/job_invocation_task_groups/_job_invocation_task_group.html.erb
 - app/views/job_invocation_task_groups/_job_invocation_task_groups.html.erb
 - app/views/job_invocations/_description_fields.html.erb
 - app/views/job_invocations/_form.html.erb
@@ -234,17 +240,21 @@ files:
 - app/views/overrides/nics/_execution_interface.html.erb
 - app/views/overrides/subnets/_rex_tab.html.erb
 - app/views/overrides/subnets/_rex_tab_pane.html.erb
+- app/views/remote_execution_features/_form.html.erb
+- app/views/remote_execution_features/index.html.erb
+- app/views/remote_execution_features/show.html.erb
 - app/views/template_inputs/_foreign_input_set_form.html.erb
 - app/views/template_inputs/_form.html.erb
+- app/views/template_inputs/_invocation_form.html.erb
 - app/views/template_invocations/_output_line_set.html.erb
 - app/views/template_invocations/_refresh.js.erb
 - app/views/template_invocations/show.html.erb
 - app/views/template_invocations/show.js.erb
 - app/views/templates/package_action.erb
+- app/views/templates/power_action.erb
 - app/views/templates/puppet_run_once.erb
 - app/views/templates/run_command.erb
 - app/views/templates/service_action.erb
-- app/views/unattended/snippets/_remote_execution_ssh_keys.erb
 - config/routes.rb
 - db/migrate/20150612121541_add_job_template_to_template.rb
 - db/migrate/20150616080015_create_template_input.rb
@@ -273,9 +283,13 @@ files:
 - db/migrate/20160113162007_expand_all_template_invocations.rb
 - db/migrate/20160114120200_rename_job_categories.rb
 - db/migrate/20160114125628_rename_job_name_to_job_category.rb
+- db/migrate/20160118124600_create_remote_execution_features.rb
+- db/migrate/20160125155108_make_job_template_name_unique.rb
+- db/migrate/20160127134031_add_advanced_to_template_input.rb
+- db/migrate/20160127162711_reword_puppet_template_description.rb
+- db/migrate/20160203104056_add_concurrency_options_to_job_invocation.rb
 - db/seeds.d/60-ssh_proxy_feature.rb
 - db/seeds.d/70-job_templates.rb
-- db/seeds.d/80-provision_templates.rb
 - db/seeds.d/90-bookmarks.rb
 - doc/.bin/.gitkeep
 - doc/.gitignore
@@ -320,6 +334,7 @@ files:
 - lib/foreman_remote_execution/version.rb
 - lib/tasks/foreman_remote_execution_tasks.rake
 - locale/Makefile
+- locale/action_names.rb
 - locale/en/foreman_remote_execution.po
 - locale/foreman_remote_execution.pot
 - locale/gemspec.rb
@@ -327,6 +342,7 @@ files:
 - test/functional/api/v2/foreign_input_sets_controller_test.rb
 - test/functional/api/v2/job_invocations_controller_test.rb
 - test/functional/api/v2/job_templates_controller_test.rb
+- test/functional/api/v2/remote_execution_features_controller_test.rb
 - test/functional/api/v2/template_inputs_controller_test.rb
 - test/test_plugin_helper.rb
 - test/unit/actions/run_hosts_job_test.rb
@@ -340,6 +356,7 @@ files:
 - test/unit/job_template_effective_user_test.rb
 - test/unit/job_template_test.rb
 - test/unit/proxy_load_balancer_test.rb
+- test/unit/remote_execution_feature_test.rb
 - test/unit/remote_execution_provider_test.rb
 - test/unit/targeting_test.rb
 - test/unit/template_input_test.rb
@@ -373,6 +390,7 @@ test_files:
 - test/functional/api/v2/foreign_input_sets_controller_test.rb
 - test/functional/api/v2/job_invocations_controller_test.rb
 - test/functional/api/v2/job_templates_controller_test.rb
+- test/functional/api/v2/remote_execution_features_controller_test.rb
 - test/functional/api/v2/template_inputs_controller_test.rb
 - test/test_plugin_helper.rb
 - test/unit/actions/run_hosts_job_test.rb
@@ -386,6 +404,7 @@ test_files:
 - test/unit/job_template_effective_user_test.rb
 - test/unit/job_template_test.rb
 - test/unit/proxy_load_balancer_test.rb
+- test/unit/remote_execution_feature_test.rb
 - test/unit/remote_execution_provider_test.rb
 - test/unit/targeting_test.rb
 - test/unit/template_input_test.rb

data/app/views/job_invocation_task_groups/_job_invocation_task_group.html.erb

@@ -1,31 +0,0 @@
-<% job_invocation = task_group.job_invocation %>
-<table class='table table-condensed'>
-  <tr>
-    <th>ID</th>
-    <td><%= link_to(job_invocation.id, job_invocation) %></td>
-  </tr>
-  <tr>
-    <th>Job Name</th>
-    <td><%= job_invocation.job_category %></td>
-  </tr>
-  <tr>
-    <th>Status</th>
-    <td><%= link_to_invocation_task_if_authorized(job_invocation) %></td>
-  </tr>
-  <tr>
-    <th>Succeeded</th>
-    <td><%= invocation_count(job_invocation, :output_key => :success_count) %></td>
-  </tr>
-  <tr>
-    <th>Failed</th>
-    <td><%= invocation_count(job_invocation, :output_key => :failed_count) %></td>
-  </tr>
-  <tr>
-    <th>Pending</th>
-    <td><%= invocation_count(job_invocation, :output_key => :pending_count) %></td>
-  </tr>
-  <tr>
-    <th>Total hosts</th>
-    <td><%= invocation_count(job_invocation, :output_key => :total_count) %></td>
-  </tr>
-</table>

data/app/views/unattended/snippets/_remote_execution_ssh_keys.erb

@@ -1,18 +0,0 @@
-<%#
-  kind: snippet
-  name: remote_execution_ssh_keys
-%>
-
-<% if !@host.params['remote_execution_ssh_keys'].blank? %>
-<% ssh_user = @host.params['remote_execution_ssh_user'] || 'root' %>
-<% ssh_path = "~#{ssh_user}/.ssh" %>
-
-mkdir -p <%= ssh_path %>
-
-cat << EOF >> <%= ssh_path %>/authorized_keys
-<%= @host.params['remote_execution_ssh_keys'].join("\n") %>
-EOF
-
-chmod 700 <%= ssh_path %>
-chmod 600 <%= ssh_path %>/authorized_keys
-<% end %>

data/db/seeds.d/80-provision_templates.rb

@@ -1,21 +0,0 @@
-ProvisioningTemplate.without_auditing do
-  templates = [{:name => 'remote_execution_ssh_keys', :source => 'snippets/_remote_execution_ssh_keys.erb', :snippet => true}]
-
-  defaults = {:vendor => 'Remote Execution', :default => true, :locked => true}
-
-  templates.each do |template|
-    next if ProvisioningTemplate.find_by_name(template[:name])
-
-    template.merge!(defaults)
-
-    t= ProvisioningTemplate.create({
-      :snippet => false,
-      :template => File.read(File.join("#{ForemanRemoteExecution::Engine.root}/app/views/unattended", template.delete(:source)))
-    }.merge(template))
-
-    t.organizations << (Organization.all - t.organizations)
-    t.locations << (Location.all - t.locations)
-
-    raise "Unable to create template #{t.name}: #{format_errors t}" if t.nil? || t.errors.any?
-  end
-end