foreman_remote_execution 0.0.4 → 0.0.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: c8ad2a2f768b914a79f715a13d1a653092d6c4a0
-  data.tar.gz: fe10900091a3fd6817f59bba6b1548cd97cb90ef
+  metadata.gz: 66bde99bb7667a338eddcafc4ace45a5a033b8bd
+  data.tar.gz: d55a2a2d6789076c01948424f8a9548e7ed1d2de
 SHA512:
-  metadata.gz: af70b52d0cf04a10ae1da4992b65c767e2e97b00ed7cb0f59404f0801c3b98c0c4a7188d0d7ee8fec02034d49d4dea7165621af94626d4123f1f0462e62ea277
-  data.tar.gz: 544b92e0a6d36e2553bd9005d57c6cb37e71a2069951a93f5346cba8b395d85dfcac75abbd368280970cc4170807ae0904da6dc2140588710ca940391b279c16
+  metadata.gz: b99fca6b8d6d9f576cc42414f6c37e91b2121887cf696692d2beec4264ac776c4d2f16c983366c914917023724ce2e8548b1ef6caeef7b7883f10380f8bba344
+  data.tar.gz: 85dd26a1fbbc873a0a0493672922d73067d5d73cf33d610115357a3abd59570776bb6cc06ea38b32986f08b64c914db146a61561c81718e5c1b056989dc1342c

data/app/assets/javascripts/template_invocation.js

@@ -19,6 +19,11 @@ function refresh_execution_form() {
   });
 }
 
+function refresh_search_query(value){
+  id = value.val;
+  $('textarea#targeting_search_query').val($('span#bookmark_query_map span#bookmark-' + id).data('query'));
+}
+
 function job_invocation_form_binds() {
   $('input.job_template_selector').on('click', function () {
     parent_fieldset = $(this).closest('fieldset');
@@ -29,4 +34,8 @@ function job_invocation_form_binds() {
   $('select#job_invocation_job_name').on('change', refresh_execution_form);
 
   $('button#refresh_execution_form').on('click', refresh_execution_form);
+
+  $('textarea#targeting_search_query').on('change', refresh_execution_form);
+
+  $('select#targeting_bookmark_id').on('change', refresh_search_query);
 }

data/app/controllers/job_invocations_controller.rb

@@ -2,16 +2,28 @@ class JobInvocationsController < ApplicationController
   include Foreman::Controller::AutoCompleteSearch
 
   def new
-    @composer = JobInvocationComposer.new(JobInvocation.new,
-                                          :host_ids => params[:host_ids],
-                                          :targeting => {
-                                            :targeting_type => Targeting::STATIC_TYPE,
-                                            :bookmark_id => params[:bookmark_id]
-                                          })
+    @composer = JobInvocationComposer.new.compose_from_params(
+      :host_ids => params[:host_ids],
+      :targeting => {
+        :targeting_type => Targeting::STATIC_TYPE,
+        :bookmark_id => params[:bookmark_id]
+      })
+  end
+
+  def rerun
+    job_invocation = resource_base.find(params[:id])
+    @composer = JobInvocationComposer.new.compose_from_invocation(job_invocation)
+
+    if params[:failed_only]
+      host_ids = job_invocation.failed_host_ids
+      @composer.search_query = @composer.targeting.build_query_from_hosts(host_ids)
+    end
+
+    render :action => 'new'
   end
 
   def create
-    @composer = JobInvocationComposer.new(JobInvocation.new, params)
+    @composer = JobInvocationComposer.new.compose_from_params(params)
     if @composer.save
       @task = ForemanTasks.async_task(::Actions::RemoteExecution::RunHostsJob, @composer.job_invocation)
       redirect_to job_invocation_path(@composer.job_invocation)
@@ -30,6 +42,17 @@ class JobInvocationsController < ApplicationController
 
   # refreshes the form
   def refresh
-    @composer = JobInvocationComposer.new(JobInvocation.new, params)
+    @composer = JobInvocationComposer.new.compose_from_params(params)
+  end
+
+  private
+
+  def action_permission
+    case params[:action]
+      when 'rerun'
+        'create'
+      else
+        super
+    end
   end
 end

data/app/helpers/remote_execution_helper.rb

@@ -57,6 +57,16 @@ module RemoteExecutionHelper
   def job_invocation_task_buttons(task)
     buttons = []
     buttons << link_to(_('Refresh'), {}, :class => 'btn btn-default', :title => _('Refresh this page'))
+    if authorized_for(:permission => :create_job_invocations)
+      buttons << link_to(_("Rerun"), rerun_job_invocation_path(:id => task.locks.where(:resource_type => 'JobInvocation').first.resource),
+                         :class => "btn btn-default",
+                         :title => _('Rerun the job'))
+    end
+    if authorized_for(:permission => :create_job_invocations)
+      buttons << link_to(_("Rerun failed"), rerun_job_invocation_path(:id => task.locks.where(:resource_type => 'JobInvocation').first.resource, :failed_only => 1),
+                         :class => "btn btn-default",
+                         :title => _('Rerun on failed hosts'))
+    end
     if authorized_for(:permission => :view_foreman_tasks, :auth_object => task)
       buttons << link_to(_("Last Job Task"), foreman_tasks_task_path(task),
                          :class => "btn btn-default",

data/app/models/job_invocation.rb

@@ -18,4 +18,28 @@ class JobInvocation < ActiveRecord::Base
   def to_action_input
     { :id => id, :name => job_name }
   end
+
+  def template_invocations_tasks
+    if last_task.present?
+      last_task.sub_tasks.for_action_types('Actions::RemoteExecution::RunHostJob')
+    else
+      ForemanTasks::Task.for_action_types('Actions::RemoteExecution::RunHostJob').where('1=0')
+    end
+  end
+
+  def failed_template_invocation_tasks
+    template_invocations_tasks.where(:result => 'warning')
+  end
+
+  def failed_host_ids
+    locks_for_resource(failed_template_invocation_tasks, 'Host::Managed').map(&:resource_id)
+  end
+
+  def failed_hosts
+    locks_for_resource(failed_template_invocation_tasks, 'Host::Managed').map(&:resource)
+  end
+
+  def locks_for_resource(tasks, resource_type)
+    tasks.map { |task| task.locks.where(:resource_type => resource_type).first }.compact
+  end
 end

data/app/models/job_invocation_composer.rb

@@ -3,8 +3,11 @@ class JobInvocationComposer
   attr_reader :job_template_ids
   delegate :job_name, :targeting, :to => :job_invocation
 
-  def initialize(job_invocation, params)
+  def initialize(job_invocation = JobInvocation.new)
     @job_invocation = job_invocation
+  end
+
+  def compose_from_params(params)
     @params = params
 
     @host_ids = validate_host_ids(params[:host_ids])
@@ -15,6 +18,19 @@ class JobInvocationComposer
     job_invocation.targeting = build_targeting
 
     @job_template_ids = validate_job_template_ids(job_templates_base.keys.compact)
+    self
+  end
+
+  def compose_from_invocation(invocation)
+    @params = {}
+
+    job_invocation.job_name = validate_job_name(invocation.job_name)
+    job_invocation.targeting = invocation.targeting.dup
+    @search_query = targeting.search_query unless targeting.bookmark_id.present?
+
+    @job_template_ids = invocation.template_invocations.map(&:template_id)
+    @template_invocations = dup_template_invocations(invocation)
+    self
   end
 
   def valid?
@@ -117,6 +133,14 @@ class JobInvocationComposer
 
   private
 
+  def dup_template_invocations(job_invocation)
+    job_invocation.template_invocations.map do |template_invocation|
+      duplicate = template_invocation.dup
+      template_invocation.input_values.map { |value| duplicate.input_values.build :value => value.value, :template_input_id => value.template_input_id }
+      duplicate
+    end
+  end
+
   def targeting_base
     @params.fetch(:targeting, {})
   end

data/app/models/job_template.rb

@@ -37,14 +37,53 @@ class JobTemplate < ::Template
   end
   self.table_name = 'templates'
 
+  # Import a template ERB, with metadata in the first YAML comment
+  def self.import(template, options = {})
+    metadata = parse_metadata(template)
+    return if metadata.blank? || metadata.delete('kind') != 'job_template' || self.find_by_name(metadata['name'])
+
+    inputs = metadata.delete('template_inputs')
+
+    # This awkward dance is because you can't instantiate a new template with :locked => true
+    template = self.create(metadata.merge(:template => template.gsub(/<%\#.+?.-?%>\n?/m, '')))
+    template.update_attributes(options)
+    template.assign_taxonomies
+
+    inputs.each do |input|
+      template.template_inputs << TemplateInput.create(input)
+    end
+
+    template
+  end
+
   # Override method in Taxonomix as Template is not used attached to a Host,
   # and matching a Host does not prevent removing a template from its taxonomy.
   def used_taxonomy_ids(type)
     []
   end
 
+  def dup
+    dup = super
+    self.template_inputs.each do |input|
+      dup.template_inputs.build input.attributes.except('template_id', 'id', 'created_at', 'updated_at')
+    end
+    dup
+  end
+
+  def assign_taxonomies
+    if default
+      organizations << Organization.all if SETTINGS[:organizations_enabled]
+      locations << Location.all if SETTINGS[:locations_enabled]
+    end
+  end
+
   private
 
+  def self.parse_metadata(template)
+    match = template.match(/<%\#(.+?).-?%>/m)
+    match.nil? ? {} : YAML.load(match[1])
+  end
+
   # we can't use standard validator, .provider_names output can change but the validator does not reflect it
   def provider_type_whitelist
     errors.add :provider_type, :uniq unless RemoteExecutionProvider.provider_names.include?(self.provider_type)

data/app/models/template_input.rb

@@ -6,7 +6,8 @@ class TemplateInput < ActiveRecord::Base
             :puppet_parameter => N_('Puppet parameter') }.with_indifferent_access
 
   attr_accessible :name, :required, :input_type, :fact_name, :variable_name,
-                  :puppet_class_name, :puppet_parameter_name, :description, :job_template_id
+                  :puppet_class_name, :puppet_parameter_name, :description, :job_template_id,
+                  :options
 
   belongs_to :template
   has_many :template_invocation_input_values, :dependent => :destroy
@@ -42,6 +43,10 @@ class TemplateInput < ActiveRecord::Base
     get_resolver(renderer).value
   end
 
+  def options_array
+    self.options.blank? ? [] : self.options.split(/\r?\n/).map(&:strip)
+  end
+
   private
 
   def get_resolver(renderer)

data/app/models/template_invocation_input_value.rb

@@ -5,4 +5,6 @@ class TemplateInvocationInputValue < ActiveRecord::Base
 
   validates :value, :presence => true, :if => proc { |v| v.template_input.required? }
 
+  validates :value, :inclusion => { :in => proc { |v| v.template_input.options_array } },
+                    :if => proc { |v| v.template_input.input_type == 'user' && v.template_input.options_array.present? }
 end

data/app/views/job_invocations/_form.html.erb

@@ -2,7 +2,12 @@
   <%= selectable_f f, :job_name, @composer.available_job_names %>
 
   <%= fields_for @composer.targeting do |targeting_fields| %>
-    <%= targeting_fields.hidden_field :bookmark_id, :value => params[:bookmark_id] %>
+    <span id="bookmark_query_map" >
+      <% @composer.available_bookmarks.each do |bookmark| %>
+        <span id="bookmark-<%= bookmark.id %>" data-query="<%= bookmark.query %>"></span>
+      <% end %>
+    </span>
+    <%= selectable_f targeting_fields, :bookmark_id, @composer.available_bookmarks.map {|b| [ b.name, b.id ] }, :selected => @composer.targeting.bookmark_id, :include_blank => true %>
     <%= textarea_f targeting_fields, :search_query, :value => @composer.displayed_search_query, :rows => 5 %>
 
     <div class="form-group ">
@@ -49,7 +54,11 @@
                   <%= job_template_fields.fields_for :input_values do |input_values_fields| %>
                     <% job_template.template_inputs.where(:input_type => 'user').each do |input| %>
                       <%= input_values_fields.fields_for input.id.to_s, @composer.template_invocation_input_value_for(input) do |input_fields| %>
-                        <%= text_f input_fields, :value, :label => input.name, :help_inline => input.description, :required => input.required %>
+                        <% unless input.options.blank? %>
+                          <%= selectable_f input_fields, :value, input.options_array, {:include_blank => !input.required }, :require => input.required, :label => input.name, :help_inline => input.description %>
+                        <% else %>
+                          <%= textarea_f input_fields, :value, :label => input.name, :help_inline => input.description, :required => input.required, :rows => 2 %>
+                        <% end %>
                       <% end %>
                     <% end %>
                   <% end %>

data/app/views/job_invocations/_tab_overview.html.erb

@@ -27,7 +27,17 @@
     <h5>
       <b><%= template_invocation.template.name %></b> <%= 'through' %> <%= _(RemoteExecutionProvider.provider_for(template_invocation.template.provider_type)) %>
     </h5>
-    <pre><%= InputTemplateRenderer.new(template_invocation.template, nil, template_invocation).preview %></pre>
+    <% target = template_invocation.targeting.hosts.with_os.first || template_invocation.targeting.hosts.first %>
+    <%= _('Preview for target %s') % target.try(:name) || 'N/A' %>
+
+    <% renderer = InputTemplateRenderer.new(template_invocation.template, target, template_invocation) %>
+    <% if (preview = renderer.preview) %>
+      <pre><%= preview %></pre>
+    <% else %>
+      <%= alert :class => "alert-block alert-danger base in fade has-error",
+                :text => renderer.error_message.html_safe %>
+    <% end %>
+
 
     <% if template_invocation.input_values.present? %>
       <%= _('following user inputs were provided') %>

data/app/views/template_inputs/_form.html.erb

@@ -16,6 +16,9 @@
         <%= text_f f, :puppet_class_name, :class => 'puppet_parameter_input_type', :required => true %>
         <%= text_f f, :puppet_parameter_name, :class => 'puppet_parameter_input_type', :required => true %>
       </div>
+      <div class="user_input_type custom_input_type_fields" style="<%= (f.object.user_template_input? || f.object.new_record?) ? '' : 'display:none' %>">
+        <%= textarea_f f, :options, :rows => 3, :class => 'user_input_type', :help_inline => _("A list of options the user can select from. If not provided, the user will be given a free-form field") %>
+      </div>
       <%= textarea_f f, :description, :rows => 3 %>
     <% end %>
   </div>

data/app/views/templates/package_action.erb

@@ -0,0 +1,52 @@
+<%#
+kind: job_template
+name: Package Action - SSH Default
+job_name: Package Action
+provider_type: Ssh
+template_inputs:
+- name: pre_script
+  description: A script to run prior to the package action
+  input_type: user
+  required: false
+- name: action
+  description: 'The package action: install, update, or remove'
+  input_type: user
+  required: true
+  options: "install\nupdate\nremove"
+- name: package
+  description: The name of the package, if any
+  input_type: user
+  required: false
+- name: post_script
+  description: A script to run after the package action
+  input_type: user
+  required: false
+%>
+
+die() {
+  echo "${1}, exiting..."
+  exit $2
+}
+
+<% unless input("pre_script").blank? -%>
+  # Pre Script
+  <%= input("pre_script") %>
+  RETVAL=$?
+  [ $RETVAL -eq 0 ] || die "Pre script failed" $RETVAL
+<% end -%>
+
+# Action
+<% if @host.operatingsystem.family == 'Redhat' -%>
+  yum -y <%= input("action") %> <%= input("package") %>
+<% elsif @host.operatingsystem.family == 'Debian' -%>
+  apt-get -y <%= input("action") %> <%= input("package") %>
+<% end -%>
+RETVAL=$?
+[ $RETVAL -eq 0 ] || die "Package action failed" $RETVAL
+
+<% unless input("post_script").blank? -%>
+  # Post Script
+  <%= input("post_script") %>
+  RETVAL=$?
+  [ $RETVAL -eq 0 ] || die "Post script failed" $RETVAL
+<% end -%>

data/app/views/templates/puppet_run_once.erb

@@ -0,0 +1,12 @@
+<%#
+kind: job_template
+name: Puppet Run Once - SSH Default
+job_name: Puppet Run Once
+provider_type: Ssh
+template_inputs:
+- name: puppet_options
+  description: Additional options to pass to puppet
+  input_type: user
+  required: false
+%>
+puppet agent --onetime --no-usecacheonfailure <%= input("puppet_options") %>

data/app/views/templates/run_command.erb

@@ -0,0 +1,12 @@
+<%#
+kind: job_template
+name: Run Command - SSH Default
+job_name: Run Command
+provider_type: Ssh
+template_inputs:
+- name: command
+  description: Command to run on the host
+  input_type: user
+  required: true
+%>
+<%= input("command") %>

data/app/views/templates/service_action.erb

@@ -0,0 +1,21 @@
+<%#
+kind: job_template
+name: Service Action - SSH Default
+job_name: Service Action
+provider_type: Ssh
+template_inputs:
+- name: action
+  description: Action to perform on the service
+  input_type: user
+  options: "restart\nstart\nstop\nstatus"
+  required: true
+- name: service
+  description: Name of the service
+  input_type: user
+  required: true
+%>
+<% if @host.operatingsystem.family == "Redhat" && @host.operatingsystem.major.to_i > 6 %>
+systemctl <%= input("action") %> <%= input("service") %>
+<% else %>
+service <%= input("service") %> <%= input("action") %>
+<% end -%>

data/config/routes.rb

@@ -18,6 +18,9 @@ Rails.application.routes.draw do
       post 'refresh'
       get 'auto_complete_search'
     end
+    member do
+      get 'rerun'
+    end
   end
 
   namespace :api, :defaults => {:format => 'json'} do

data/db/migrate/20150827144500_change_targeting_search_query_type.rb

@@ -0,0 +1,5 @@
+class ChangeTargetingSearchQueryType < ActiveRecord::Migration
+  def change
+    change_column :targetings, :search_query, :text
+  end
+end

data/db/migrate/20150827152730_add_options_to_template_input.rb

@@ -0,0 +1,5 @@
+class AddOptionsToTemplateInput < ActiveRecord::Migration
+  def change
+    add_column :template_inputs, :options, :text
+  end
+end

data/db/seeds.d/70-job_templates.rb

@@ -0,0 +1,7 @@
+User.as_anonymous_admin do
+  JobTemplate.without_auditing do
+    Dir[File.join("#{ForemanRemoteExecution::Engine.root}/app/views/templates/**/*.erb")].each do |template|
+      JobTemplate.import(File.read(template), :default => true, :locked => true)
+    end
+  end
+end

data/lib/foreman_remote_execution/engine.rb

@@ -45,7 +45,7 @@ module ForemanRemoteExecution
 
           permission :view_job_invocations, { :job_invocations => [:index, :show, :auto_complete_search] }, :resource_type => 'JobInvocation'
 
-          permission :create_job_invocations, { :job_invocations => [:new, :create, :refresh] }, :resource_type => 'JobInvocation'
+          permission :create_job_invocations, { :job_invocations => [:new, :create, :refresh, :rerun] }, :resource_type => 'JobInvocation'
         end
 
         # Add a new role called 'ForemanRemoteExecution' if it doesn't exist

data/lib/foreman_remote_execution/version.rb

@@ -1,3 +1,3 @@
 module ForemanRemoteExecution
-  VERSION = '0.0.4'
+  VERSION = '0.0.5'
 end

data/test/unit/input_template_renderer_test.rb

@@ -84,6 +84,33 @@ describe InputTemplateRenderer do
         end
       end
     end
+
+    context 'with options specified' do
+
+      let(:job_invocation) { FactoryGirl.create(:job_invocation) }
+      let(:template_invocation) { FactoryGirl.build(:template_invocation, :template => template) }
+      let(:result) { renderer.render }
+
+      before do
+        template.template_inputs << FactoryGirl.build(:template_input, :name => 'service_name', :input_type => 'user', :options => "httpd\nforeman")
+      end
+
+      context 'with a valid input defined' do
+        before do
+          job_invocation.template_invocations << template_invocation
+          renderer.invocation = template_invocation
+
+          FactoryGirl.create(:template_invocation_input_value,
+                             :template_invocation => template_invocation,
+                             :template_input => template.template_inputs.first,
+                             :value => 'foreman')
+        end
+
+        it 'can render with job invocation with corresponding value' do
+          renderer.render.must_equal 'service restart foreman'
+        end
+      end
+    end
   end
 
   context "renderer for template with fact input used" do

data/test/unit/job_invocation_composer_test.rb

@@ -39,7 +39,7 @@ describe JobInvocationComposer do
   context 'with general new invocation and empty params' do
     let(:params) { {} }
     let(:job_invocation) { JobInvocation.new }
-    let(:composer) { JobInvocationComposer.new(job_invocation, params) }
+    let(:composer) { JobInvocationComposer.new(job_invocation).compose_from_params(params) }
 
     describe '#available_templates' do
       it 'obeys authorization' do
@@ -369,7 +369,9 @@ describe JobInvocationComposer do
                 :input_values => { input1.id.to_s => { :value => 'value1' } }
               } } }
         end
-        let(:params) { { :job_invocation => { :providers => { :ssh => ssh_params } }, :targeting => { :search_query => "name = #{host.name}" } }.with_indifferent_access }
+        let(:params) do
+          { :job_invocation => { :providers => { :ssh => ssh_params } }, :targeting => { :search_query => "name = #{host.name}" } }.with_indifferent_access
+        end
 
         it 'validates all associated objects even if some of the is invalid' do
           composer
@@ -410,6 +412,52 @@ describe JobInvocationComposer do
         end
       end
 
+      describe '#compose_from_invocation(existing_invocation)' do
+        let(:host) { FactoryGirl.create(:host) }
+        let(:ssh_params) do
+          { :job_template_id => testing_job_template_1.id.to_s,
+            :job_templates => {
+              testing_job_template_1.id.to_s => {
+                :input_values => { input1.id.to_s => { :value => 'value1' } }
+              } } }
+        end
+        let(:params) do
+          {
+            :job_invocation => {
+              :providers => { :ssh => ssh_params }
+            },
+            :targeting => {
+              :search_query => "name = #{host.name}",
+              :targeting_type => Targeting::STATIC_TYPE
+            }
+          }.with_indifferent_access
+        end
+        let(:existing) { job_invocation.reload }
+        let(:new_job_invocation) { JobInvocation.new }
+        let(:new_composer) { JobInvocationComposer.new(new_job_invocation).compose_from_invocation(job_invocation) }
+
+        before do
+          composer.save
+        end
+
+        it 'sets the same job name' do
+          new_composer.job_name.must_equal existing.job_name
+        end
+
+        it 'builds new targeting object which keeps search query' do
+          new_composer.targeting.wont_equal existing.targeting
+          new_composer.search_query.must_equal existing.targeting.search_query
+        end
+
+        it 'keeps job template ids' do
+          new_composer.job_template_ids.must_equal existing.template_invocations.map(&:template_id)
+        end
+
+        it 'keeps template invocations and their values' do
+          new_composer.template_invocations.size.must_equal existing.template_invocations.size
+        end
+
+      end
     end
   end
 end

data/test/unit/job_template_test.rb

@@ -1,5 +1,54 @@
 require 'test_plugin_helper'
 
 describe JobTemplate do
+  context 'cloning' do
+    let(:job_template) { FactoryGirl.build(:job_template, :with_input) }
 
+    describe '#dup' do
+      it 'duplicates also template inputs' do
+        duplicate = job_template.dup
+        duplicate.wont_equal job_template
+        duplicate.template_inputs.wont_be_empty
+        duplicate.template_inputs.first.wont_equal job_template.template_inputs.first
+        duplicate.template_inputs.first.name.must_equal job_template.template_inputs.first.name
+      end
+    end
+  end
+
+  context 'importing a template' do
+    let(:template) do
+      template = <<-END_TEMPLATE
+      <%#
+      kind: job_template
+      name: Service Restart
+      job_name: Service Restart
+      provider_type: Ssh
+      template_inputs:
+      - name: service_name
+        input_type: user
+        required: true
+      %>
+
+      service <%= input("service_name") %> restart
+      END_TEMPLATE
+
+      JobTemplate.import(template, :default => true)
+    end
+
+    it 'sets the name' do
+      template.name.must_equal 'Service Restart'
+    end
+
+    it 'has a template' do
+      template.template.squish.must_equal 'service <%= input("service_name") %> restart'
+    end
+
+    it 'imports inputs' do
+      template.template_inputs.first.name.must_equal 'service_name'
+    end
+
+    it 'sets additional options' do
+      template.default.must_equal true
+    end
+  end
 end

data/test/unit/template_invocation_input_value_test.rb

@@ -0,0 +1,29 @@
+require 'test_plugin_helper'
+
+describe TemplateInvocationInputValue do
+  let(:template) { FactoryGirl.build(:job_template, :template => 'service restart <%= input("service_name") -%>') }
+  let(:renderer) { InputTemplateRenderer.new(template) }
+  let(:job_invocation) { FactoryGirl.create(:job_invocation) }
+  let(:template_invocation) { FactoryGirl.build(:template_invocation, :template => template) }
+  let(:result) { renderer.render }
+
+  context 'with selectable options' do
+    before do
+      result # let is lazy
+      template.template_inputs << FactoryGirl.build(:template_input, :name => 'service_name', :input_type => 'user',
+                                                                     :required => true, :options => "foreman\nhttpd")
+    end
+
+    it 'fails with an invalid option' do
+      refute_valid FactoryGirl.build(:template_invocation_input_value, :template_invocation => template_invocation,
+                                                                       :template_input => template.template_inputs.first,
+                                                                       :value => 'sendmail')
+    end
+
+    it 'succeeds with valid option' do
+      assert_valid FactoryGirl.build(:template_invocation_input_value, :template_invocation => template_invocation,
+                                                                       :template_input => template.template_inputs.first,
+                                                                       :value => 'foreman')
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: foreman_remote_execution
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: 0.0.5
 platform: ruby
 authors:
 - Foreman Remote Execution team
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-08-26 00:00:00.000000000 Z
+date: 2015-09-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -161,6 +161,10 @@ files:
 - app/views/job_templates/index.html.erb
 - app/views/job_templates/new.html.erb
 - app/views/template_inputs/_form.html.erb
+- app/views/templates/package_action.erb
+- app/views/templates/puppet_run_once.erb
+- app/views/templates/run_command.erb
+- app/views/templates/service_action.erb
 - config/routes.rb
 - db/migrate/20150612121541_add_job_template_to_template.rb
 - db/migrate/20150616080015_create_template_input.rb
@@ -169,7 +173,10 @@ files:
 - db/migrate/20150708133305_add_template_invocation.rb
 - db/migrate/20150812110800_add_resolved_at_to_targeting.rb
 - db/migrate/20150812145900_add_last_task_id_to_job_invocation.rb
+- db/migrate/20150827144500_change_targeting_search_query_type.rb
+- db/migrate/20150827152730_add_options_to_template_input.rb
 - db/seeds.d/60-ssh_proxy_feature.rb
+- db/seeds.d/70-job_templates.rb
 - doc/Gemfile
 - doc/Gemfile.lock
 - doc/Rakefile
@@ -223,6 +230,7 @@ files:
 - test/unit/remote_execution_provider_test.rb
 - test/unit/targeting_test.rb
 - test/unit/template_input_test.rb
+- test/unit/template_invocation_input_value_test.rb
 homepage: https://github.com/theforeman/foreman_remote_execution
 licenses: []
 metadata: {}
@@ -250,6 +258,7 @@ summary: A plugin bringing remote execution to the Foreman, completing the confi
 test_files:
 - test/test_plugin_helper.rb
 - test/unit/job_template_test.rb
+- test/unit/template_invocation_input_value_test.rb
 - test/unit/template_input_test.rb
 - test/unit/job_invocation_test.rb
 - test/unit/targeting_test.rb