foreman-tasks 0.9.6 → 0.10.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: e1e9246d6fdeb8fc99d6e42f7ed41e87e398341a
-  data.tar.gz: a67dd1ec1cbad7e37a168b8495ed48242a862bf2
+  metadata.gz: acd7b0adebf587957c53c75ff415ae872d5b5995
+  data.tar.gz: e682ae90a94614bae6d1cf7b1014e39bce5fa82e
 SHA512:
-  metadata.gz: d3f758c1445380c7b28ee3e132fdfc2f467c1c7b907286e18a56b6fdfeb62abcb0f81753e9c3809f7d8c8342ff7e35860dbdb34d491acc71b7107aac10fbde79
-  data.tar.gz: c814a2d7dd11fadd76cce7a11a7f38e469529301ab6debd6e3fdc7763a5fc8793acc20291bf11c69ba65ad837670f8a3cc027d320641f83e7b5633e9384df03b
+  metadata.gz: bd0a1edea81a4b634fea1f4e81ab7b2778a11d68cb5cc2d8c35974c60db2d9cc58ac12b85904fb0943ba22a1540b19d513504aefac6f285f1703cba8f4ce3217
+  data.tar.gz: 036bb166af540a7509a590fa8f5de2d056d93c8c77bf8311e92705d3e8936097365449b60b4f025fe6814df9ca8f6e5fc3e70d58fdc6a6ee74e9682a52351e13

data/app/controllers/foreman_tasks/tasks_controller.rb

@@ -36,16 +36,6 @@ module ForemanTasks
       redirect_to :back
     end
 
-    def abort
-      task = find_dynflow_task
-      if task.abort
-        flash[:notice] = _('Trying to abort the task')
-      else
-        flash[:warning] = _('The task cannot be aborted at the moment.')
-      end
-      redirect_to :back
-    end
-
     def resume
       task = find_dynflow_task
       if task.resumable?
@@ -96,7 +86,7 @@ module ForemanTasks
       case params[:action]
       when 'sub_tasks'
         :view
-      when 'resume', 'unlock', 'force_unlock', 'cancel_step', 'cancel', 'abort'
+      when 'resume', 'unlock', 'force_unlock', 'cancel_step', 'cancel'
         :edit
       else
         super

data/app/lib/actions/proxy_action.rb

@@ -31,8 +31,6 @@ module Actions
           # do nothing
         when ::Dynflow::Action::Cancellable::Cancel
           cancel_proxy_task
-        when ::Dynflow::Action::Cancellable::Abort
-          abort_proxy_task
         when CallbackData
           on_data(event.data)
         when ::Dynflow::Action::Timeouts::Timeout
@@ -80,11 +78,6 @@ module Actions
       end
     end
 
-    def abort_proxy_task
-      proxy.cancel_task(output[:proxy_task_id])
-      error! ForemanTasks::Task::TaskCancelledException.new(_('Task aborted: the task might be still running on the proxy'))
-    end
-
     def on_resume
       # TODO: add logic to load the data from the external action
       suspend

data/app/models/foreman_tasks/task.rb

@@ -7,11 +7,8 @@ module ForemanTasks
     def check_permissions_after_save
       # there's no create_tasks permission, tasks are created as a result of internal actions, in such case we
       # don't do authorization, that should have been performed on wrapping action level
-      if id_changed?
-        true
-      else
-        super
-      end
+      # the same applies for updating
+      true
     end
 
     # TODO: missing validation of states

data/app/models/foreman_tasks/task/dynflow_task.rb

@@ -9,8 +9,8 @@ module ForemanTasks
       self.external_id    = data[:id]
       self.started_at     = utc_zone.parse(data[:started_at]) unless data[:started_at].nil?
       self.ended_at       = utc_zone.parse(data[:ended_at]) unless data[:ended_at].nil?
-      self.result         = map_result(data).to_s
       self.state          = data[:state].to_s
+      self.result         = map_result(data[:result])
       self.start_at       = utc_zone.parse(data[:start_at]) if data[:start_at]
       self.start_before   = utc_zone.parse(data[:start_before]) if data[:start_before]
       self.parent_task_id ||= begin
@@ -32,10 +32,6 @@ module ForemanTasks
       execution_plan!.cancel.any?
     end
 
-    def abort
-      execution_plan!.cancel(true).any?
-    end
-
     def resumable?
       execution_plan.try(:state) == :paused
     end
@@ -152,20 +148,9 @@ module ForemanTasks
 
     private
 
-    def map_result(data)
-      if state_result_transitioned?(%w[planned pending], %w[stopped error], data) ||
-         (data[:result] == :error && cancelled?)
-        :cancelled
-      else
-        data[:result]
-      end
-    end
-
-    def state_result_transitioned?(from, to, data)
-      oldstate, oldresult = from
-      newstate, newresult = to
-      state == oldstate && data[:state].to_s == newstate &&
-        result == oldresult && data[:result].to_s == newresult
+    def map_result(result)
+      result = :cancelled if result == :error && cancelled?
+      result.to_s
     end
 
     def cancelled?

data/app/views/foreman_tasks/tasks/_details.html.erb

@@ -3,7 +3,7 @@
   <% if authorized_for(:permission => :edit_foreman_tasks, :auth_object => @task) %>
     <%= link_to(_('Dynflow console'),
                 format((defined?(Rails) ? request.script_name : '') + '/foreman_tasks/dynflow/%s', @task.external_id),
-                class: %w(btn btn-sm btn-default)) if Setting['dynflow_enable_console'] %>
+                class: %w(btn btn-sm btn-default), target: '_blank') if Setting['dynflow_enable_console'] %>
 
     <%= link_to(_('Resume'),
                 resume_foreman_tasks_task_path(@task),

data/app/views/foreman_tasks/tasks/dashboard/_latest_tasks_in_error_warning.html.erb

@@ -1,15 +1,17 @@
 <h4><%= link_to _("Latest Warning/Error Tasks"), foreman_tasks_tasks_path(:order=>'started_at DESC') %></h4>
-<table class="table table-striped">
+<table class="table table-striped table-fixed">
   <tr>
-    <th><%= _("Name") %></th>
-    <th><%= _("State") %></th>
-    <th><%= _("Result") %></th>
+    <th class="col-md-5"><%= _("Name") %></th>
+    <th class="col-md-2"><%= _("State") %></th>
+    <th class="col-md-2"><%= _("Result") %></th>
+    <th class="col-md-3"><%= _("Started") %></th>
   </tr>
   <% ForemanTasks::Task::Summarizer.new.latest_tasks_in_errors_warning.each do |task| %>
     <tr class="<%= ForemanTasks::Task::StatusExplicator.new.is_erroneous(task) ? 'text-danger' : '' %>">
-      <td><%= link_to task.humanized[:action], defined?(main_app) ? main_app.foreman_tasks_task_path(task.id) : foreman_tasks_task_path(task.id) %></td>
+      <td class="ellipsis"><%= link_to task.humanized[:action], defined?(main_app) ? main_app.foreman_tasks_task_path(task.id) : foreman_tasks_task_path(task.id) %></td>
       <td><%= task.state %></td>
       <td><%= task.result %></td>
+      <td><%= _('%s ago') % time_ago_in_words(task.started_at) %></td>
     </tr>
   <% end %>
 </table>

data/config/foreman-tasks.yaml.example

@@ -9,43 +9,18 @@
 #     :action:
 #       :enabled: true
 
-  # Task backup configuration can be changed by altering the values in
-  # the backup section
-  #
-  :backup:
-    #
-    # Whether to back up tasks when they are removed
-    #
-    :backup_deleted_tasks: true
-    #
-    # Where to put the tasks which were backed up
-    #
-    :backup_dir: /var/lib/foreman/tasks-backup
 
 # Cleaning configuration: how long should the actions be kept before deleted
 # by `rake foreman_tasks:clean` task
 #
-  :cleanup:
+#  :cleanup:
 #
 # the period after which to delete all the tasks (by default all tasks are not being deleted after some period)
-# will be deprecated in Foreman 1.18 and the use of rules is recommended.
 #
-#     :after: 30d
+#    :after: 365d
 #
 # per action settings to override the default defined in the actions (self.cleanup_after method)
 #
 #    :actions:
 #      - :name: Actions::Foreman::Host::ImportFacts
 #        :after: 10d
-#
-# Rules defined in this section by default don't operate
-# on tasks specified in the actions section. This behavior
-# can be overriden by setting the override_actions to true
-   :rules:
-     # Delete successful tasks after a month
-     - :filter: result = success
-       :after: 30d
-     # Delete everything (any action, any state) after one year
-     - :states: all # Either list of state names or all
-       :after: 1y
-       :override_actions: true

data/config/routes.rb

@@ -12,7 +12,6 @@ Foreman::Application.routes.draw do
       end
       member do
         get :sub_tasks
-        post :abort
         post :cancel
         post :resume
         post :unlock

data/foreman-tasks.gemspec

@@ -29,7 +29,7 @@ same resource. It also optionally provides Dynflow infrastructure for using it f
   s.extra_rdoc_files = Dir['README*', 'LICENSE']
 
   s.add_dependency "foreman-tasks-core"
-  s.add_dependency "dynflow", '~> 0.8.29'
+  s.add_dependency "dynflow", '~> 0.8.24'
   s.add_dependency "sequel" # for Dynflow process persistence
   s.add_dependency "sinatra" # for Dynflow web console
   s.add_dependency "daemons" # for running remote executor

data/lib/foreman_tasks/cleaner.rb

@@ -1,5 +1,3 @@
-require 'csv'
-
 module ForemanTasks
   # Represents the cleanup mechanism for tasks
   class Cleaner
@@ -13,16 +11,11 @@ module ForemanTasks
           end
         end
         if cleanup_settings[:after]
-          Foreman::Deprecation.deprecation_warning('1.18', _(':after setting in tasks cleanup section is deprecated, use :after in :rules section to set the value. to cleanup rules'))
           new(options.merge(:filter => '', :after => cleanup_settings[:after])).delete
         end
-        with_periods = actions_with_default_cleanup
-        with_periods.each do |action_class, period|
+        actions_with_default_cleanup.each do |action_class, period|
           new(options.merge(:filter => "label = #{action_class.name}", :after => period)).delete
         end
-        actions_by_rules(with_periods).each do |hash|
-          new(options.merge(hash)).delete
-        end
       end
     end
 
@@ -51,19 +44,6 @@ module ForemanTasks
       @cleanup_settings = SETTINGS[:'foreman-tasks'] && SETTINGS[:'foreman-tasks'][:cleanup] || {}
     end
 
-    def self.actions_by_rules(actions_with_periods)
-      disable_actions_with_periods = "label !^ (#{actions_with_periods.keys.join(', ')})"
-      cleanup_settings.fetch(:rules, []).map do |hash|
-        next if hash[:after].nil?
-        conditions = []
-        conditions << disable_actions_with_periods unless hash[:override_actions]
-        conditions << hash[:filter] if hash[:filter]
-        hash[:states] = [] if hash[:states] == 'all'
-        hash[:filter] = conditions.map { |condition| "(#{condition})" }.join(' AND ')
-        hash
-      end.compact
-    end
-
     attr_reader :filter, :after, :states, :verbose, :batch_size, :noop, :full_filter
 
     # @param filter [String] scoped search matching the tasks to be deleted
@@ -76,8 +56,7 @@ module ForemanTasks
                           :verbose      => false,
                           :batch_size   => 1000,
                           :noop         => false,
-                          :states       => ['stopped'],
-                          :backup_dir   => ForemanTasks.dynflow.world.persistence.current_backup_dir }
+                          :states       => ['stopped'] }
       options         = default_options.merge(options)
 
       @filter         = options[:filter]
@@ -86,7 +65,6 @@ module ForemanTasks
       @verbose        = options[:verbose]
       @batch_size     = options[:batch_size]
       @noop           = options[:noop]
-      @backup_dir     = options[:backup_dir]
 
       raise ArgumentError, 'filter not speficied' if @filter.nil?
 
@@ -113,33 +91,12 @@ module ForemanTasks
     end
 
     def delete_tasks(chunk)
-      tasks = ForemanTasks::Task.where(:id => chunk.map(&:id))
-      tasks_to_csv(tasks, @backup_dir, 'foreman_tasks.csv') if @backup_dir
-      tasks.delete_all
-    end
-
-    def tasks_to_csv(dataset, backup_dir, file_name)
-      with_backup_file(backup_dir, file_name) do |csv, appending|
-        csv << ForemanTasks::Task.attribute_names.to_csv unless appending
-        dataset.each do |row|
-          csv << row.attributes.values.to_csv
-        end
-      end
-      dataset
-    end
-
-    def with_backup_file(backup_dir, file_name)
-      FileUtils.mkdir_p(backup_dir) unless File.directory?(backup_dir)
-      csv_file = File.join(backup_dir, file_name)
-      appending = File.exist?(csv_file)
-      File.open(csv_file, 'a') do |f|
-        yield f, appending
-      end
+      ForemanTasks::Task.where(:id => chunk.map(&:id)).delete_all
     end
 
     def delete_dynflow_plans(chunk)
       dynflow_ids = chunk.find_all { |task| task.is_a? Task::DynflowTask }.map(&:external_id)
-      ForemanTasks.dynflow.world.persistence.delete_execution_plans({ 'uuid' => dynflow_ids }, batch_size, @backup_dir)
+      ForemanTasks.dynflow.world.persistence.delete_execution_plans({ 'uuid' => dynflow_ids }, batch_size)
     end
 
     def prepare_filter

data/lib/foreman_tasks/dynflow/configuration.rb

@@ -103,8 +103,6 @@ module ForemanTasks
         config.transaction_adapter = transaction_adapter
         config.executor            = ->(world, _) { initialize_executor(world) }
         config.connector           = ->(world, _) { initialize_connector(world) }
-        config.backup_deleted_plans = backup_settings[:backup_deleted_plans]
-        config.backup_dir           = backup_settings[:backup_dir]
 
         # we can't do any operation until the ForemanTasks.dynflow.world is set
         config.auto_execute        = false
@@ -146,29 +144,5 @@ module ForemanTasks
     def initialize_persistence
       ForemanTasks::Dynflow::Persistence.new(default_sequel_adapter_options)
     end
-
-    def backup_settings
-      return @backup_settings if @backup_settings
-      backup_options = {
-        :backup_deleted_plans => true,
-        :backup_dir => default_backup_dir
-      }
-      settings = SETTINGS[:'foreman-tasks'] && SETTINGS[:'foreman-tasks'][:backup]
-      backup_options.merge!(settings) if settings
-      @backup_settings = with_environment_override backup_options
-    end
-
-    def default_backup_dir
-      File.join(Rails.root, 'tmp', 'task-backup')
-    end
-
-    def with_environment_override(options)
-      env_var = ENV['TASK_BACKUP']
-      unless env_var.nil?
-        # Everything except 0, n, no, false is considered to be a truthy value
-        options[:backup_deleted_plans] = !%w[0 n no false].include?(env_var.downcase)
-      end
-      options
-    end
   end
 end

data/lib/foreman_tasks/engine.rb

@@ -58,7 +58,7 @@ module ForemanTasks
         security_block :foreman_tasks do |_map|
           permission :view_foreman_tasks, { :'foreman_tasks/tasks' => [:auto_complete_search, :sub_tasks, :index, :show],
                                             :'foreman_tasks/api/tasks' => [:bulk_search, :show, :index, :summary] }, :resource_type => ForemanTasks::Task.name
-          permission :edit_foreman_tasks, { :'foreman_tasks/tasks' => [:resume, :unlock, :force_unlock, :cancel_step, :cancel, :abort],
+          permission :edit_foreman_tasks, { :'foreman_tasks/tasks' => [:resume, :unlock, :force_unlock, :cancel_step, :cancel],
                                             :'foreman_tasks/api/tasks' => [:bulk_resume] }, :resource_type => ForemanTasks::Task.name
 
           permission :create_recurring_logics, {}, :resource_type => ForemanTasks::RecurringLogic.name

data/lib/foreman_tasks/tasks/cleanup.rake

@@ -5,11 +5,10 @@ namespace :foreman_tasks do
 
         * TASK_SEARCH : scoped search filter (example: 'label = "Actions::Foreman::Host::ImportFacts"')
         * AFTER       : delete tasks created after *AFTER* period. Expected format is a number followed by the time unit (s,h,m,y), such as '10d' for 10 days
-        * STATES      : comma separated list of task states to touch with the cleanup, by default only stopped tasks are covered, special value all can be used to clean the tasks, disregarding their states
-        * NOOP        : set to "true" if the task should not actually perform the deletion
+        * STATES      : comma separated list of task states to touch with the cleanup, by default only stopped tasks are covered
+        * NOOP        : set to "true" if the task should not actuall perform the deletion
         * VERBOSE     : set to "true" for more verbose output
         * BATCH_SIZE  : the size of batches the tasks get processed in (1000 by default)
-        * TASK_BACKUP : set to "true" or "false" to enable/disable task backup
 
       If none of TASK_SEARCH, BEFORE, STATES is specified, the tasks will be cleaned based
       configuration in settings
@@ -22,7 +21,6 @@ namespace :foreman_tasks do
       options[:after] = ENV['AFTER'] if ENV['AFTER']
 
       options[:states] = ENV['STATES'].to_s.split(',') if ENV['STATES']
-      options[:states] = [] if options[:states] == ['all']
 
       options[:noop] = true if ENV['NOOP']
 
@@ -54,24 +52,6 @@ namespace :foreman_tasks do
           printf("%-50s %s\n", action.name, after)
         end
       end
-      puts
-      by_rules = ForemanTasks::Cleaner.actions_by_rules(ForemanTasks::Cleaner.actions_with_default_cleanup)
-      if by_rules.empty?
-        puts _('No cleanup rules are configured')
-      else
-        printf("%-50s %-15s %s\n", _('states'), _('delete after'), _('filter'))
-        by_rules.each do |hash|
-          state = case hash[:states]
-                  when []
-                    _('ANY')
-                  when nil
-                    'stopped'
-                  else
-                    hash[:states]
-                  end
-          printf("%-50s %-15s %s\n", state, hash[:after], hash[:filter])
-        end
-      end
     end
   end
 

data/lib/foreman_tasks/version.rb

@@ -1,3 +1,3 @@
 module ForemanTasks
-  VERSION = '0.9.6'.freeze
+  VERSION = '0.10.0'.freeze
 end

data/locale/action_names.rb

@@ -1,4 +1,6 @@
 # Autogenerated!
-_('Import Puppet classes')
-_('Import facts')
-_('Action with sub plans')
+_("Action with sub plans")
+_("Import facts")
+_("Import Puppet classes")
+_("Remote action:")
+_("Play Ansible roles")

data/locale/en/foreman_tasks.po

@@ -5,7 +5,7 @@
 #
 msgid ""
 msgstr ""
-"Project-Id-Version: foreman_tasks 0.1.3\n"
+"Project-Id-Version: foreman_tasks 0.1.2\n"
 "Report-Msgid-Bugs-To: \n"
 "PO-Revision-Date: 2016-02-03 09:55-0500\n"
 "Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
@@ -370,6 +370,9 @@ msgstr ""
 msgid "Start before"
 msgstr ""
 
+msgid "Started"
+msgstr ""
+
 msgid "Started at"
 msgstr ""
 

data/locale/foreman_tasks.pot

@@ -8,8 +8,8 @@ msgid ""
 msgstr ""
 "Project-Id-Version: foreman_tasks 1.0.0\n"
 "Report-Msgid-Bugs-To: \n"
-"POT-Creation-Date: 2017-07-11 18:00+0200\n"
-"PO-Revision-Date: 2017-07-11 18:00+0200\n"
+"POT-Creation-Date: 2017-07-19 17:30+0200\n"
+"PO-Revision-Date: 2017-07-19 17:30+0200\n"
 "Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
 "Language-Team: LANGUAGE <LL@li.org>\n"
 "Language: \n"
@@ -146,11 +146,12 @@ msgid "N/A"
 msgstr ""
 
 #: ../app/helpers/foreman_tasks/foreman_tasks_helper.rb:40
-msgid "%s ago"
+msgid "in %s"
 msgstr ""
 
 #: ../app/helpers/foreman_tasks/foreman_tasks_helper.rb:40
-msgid "in %s"
+#: ../app/views/foreman_tasks/tasks/dashboard/_latest_tasks_in_error_warning.html.erb:14
+msgid "%s ago"
 msgstr ""
 
 #: ../app/helpers/foreman_tasks/foreman_tasks_helper.rb:50
@@ -394,11 +395,11 @@ msgstr ""
 msgid "Finished"
 msgstr ""
 
-#: ../app/models/foreman_tasks/task.rb:87
+#: ../app/models/foreman_tasks/task.rb:84
 msgid "Delayed"
 msgstr ""
 
-#: ../app/models/foreman_tasks/task.rb:87
+#: ../app/models/foreman_tasks/task.rb:84
 msgid "Immediate"
 msgstr ""
 
@@ -703,6 +704,11 @@ msgstr ""
 msgid "Latest Warning/Error Tasks"
 msgstr ""
 
+#:
+#: ../app/views/foreman_tasks/tasks/dashboard/_latest_tasks_in_error_warning.html.erb:7
+msgid "Started"
+msgstr ""
+
 #: ../app/views/foreman_tasks/tasks/dashboard/_tasks_status.html.erb:1
 #: ../lib/foreman_tasks/engine.rb:81
 msgid "Task Status"

data/test/unit/cleaner_test.rb

@@ -1,11 +1,6 @@
 require 'foreman_tasks_test_helper'
 
 class TasksTest < ActiveSupport::TestCase
-  before do
-    # To stop dynflow from backing up actions, execution_plans and steps
-    ForemanTasks.dynflow.world.persistence.adapter.stubs(:backup_to_csv)
-  end
-
   describe ForemanTasks::Cleaner do
     it 'is able to delete tasks (including the dynflow plans) based on filter' do
       cleaner = ForemanTasks::Cleaner.new(:filter => 'label = "Actions::User::Create"', :after => '10d')
@@ -17,7 +12,6 @@ class TasksTest < ActiveSupport::TestCase
                            task.save
                          end,
                          FactoryGirl.create(:dynflow_task, :product_create_task)]
-      cleaner.expects(:tasks_to_csv)
       cleaner.delete
       ForemanTasks::Task.where(id: tasks_to_delete).must_be_empty
       ForemanTasks::Task.where(id: tasks_to_keep).order(:id).map(&:id).must_equal tasks_to_keep.map(&:id).sort
@@ -38,7 +32,6 @@ class TasksTest < ActiveSupport::TestCase
                          end]
 
       tasks_to_keep   = [FactoryGirl.create(:dynflow_task, :product_create_task)]
-      cleaner.expects(:tasks_to_csv)
       cleaner.delete
       ForemanTasks::Task.where(id: tasks_to_delete).must_be_empty
       ForemanTasks::Task.where(id: tasks_to_keep).must_equal tasks_to_keep
@@ -53,32 +46,11 @@ class TasksTest < ActiveSupport::TestCase
                            task.started_at = task.ended_at = Time.zone.now
                            task.save
                          end]
-      cleaner.expects(:tasks_to_csv)
       cleaner.delete
       ForemanTasks::Task.where(id: tasks_to_delete).must_be_empty
       ForemanTasks::Task.where(id: tasks_to_keep).must_equal tasks_to_keep
     end
 
-    it 'backs tasks up before deleting' do
-      dir = '/tmp'
-      cleaner = ForemanTasks::Cleaner.new(:filter => '', :after => '10d', :backup_dir => dir)
-      tasks_to_delete = [FactoryGirl.create(:dynflow_task, :user_create_task),
-                         FactoryGirl.create(:dynflow_task, :product_create_task)]
-
-      r, w = IO.pipe
-      cleaner.expects(:with_backup_file)
-             .with(dir, 'foreman_tasks.csv')
-             .yields(w, false)
-      cleaner.delete
-      w.close
-      header, *data = r.readlines.map(&:chomp)
-      header.must_equal ForemanTasks::Task.attribute_names.join(',')
-      expected_lines = tasks_to_delete.map { |task| task.attributes.values.join(',') }
-      data.count.must_equal expected_lines.count
-      expected_lines.each { |line| data.must_include line }
-      ForemanTasks::Task.where(id: tasks_to_delete).must_be_empty
-    end
-
     class ActionWithCleanup < Actions::Base
       def self.cleanup_after
         '15d'
@@ -96,29 +68,6 @@ class TasksTest < ActiveSupport::TestCase
                                      { :actions => [{ :name => ActionWithCleanup.name, :after => '5d' }] })
         ForemanTasks::Cleaner.actions_with_default_cleanup[ActionWithCleanup].must_equal '5d'
       end
-
-      it 'deprecates the usage of :after' do
-        Foreman::Deprecation.expects(:deprecation_warning)
-        ForemanTasks::Cleaner.any_instance.expects(:delete)
-        ForemanTasks::Cleaner.stubs(:cleanup_settings =>
-                                    { :after => '1d' })
-        ForemanTasks::Cleaner.stubs(:actions_with_default_cleanup).returns({})
-        ForemanTasks::Cleaner.run({})
-      end
-
-      it 'generates filters from rules properly' do
-        actions_with_default = { 'action1' => nil, 'action2' => nil }
-        rules = [{ :after => nil },
-                 { :after => '10d', :filter => 'label = something', :states => %w[stopped paused] },
-                 { :after => '15d', :filter => 'label = something_else',
-                   :override_actions => true, :states => 'all' }]
-        ForemanTasks::Cleaner.stubs(:cleanup_settings).returns(:rules => rules)
-        r1, r2 = ForemanTasks::Cleaner.actions_by_rules actions_with_default
-        r1[:filter].must_equal '(label !^ (action1, action2)) AND (label = something)'
-        r1[:states].must_equal %w[stopped paused]
-        r2[:filter].must_equal '(label = something_else)'
-        r2[:states].must_equal []
-      end
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: foreman-tasks
 version: !ruby/object:Gem::Version
-  version: 0.9.6
+  version: 0.10.0
 platform: ruby
 authors:
 - Ivan Nečas
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-09-21 00:00:00.000000000 Z
+date: 2017-07-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: foreman-tasks-core
@@ -30,14 +30,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.8.29
+        version: 0.8.24
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.8.29
+        version: 0.8.24
 - !ruby/object:Gem::Dependency
   name: sequel
   requirement: !ruby/object:Gem::Requirement
@@ -261,7 +261,6 @@ files:
 - test/unit/config/environment.rb
 - test/unit/daemon_test.rb
 - test/unit/dynflow_console_authorizer_test.rb
-- test/unit/otp_manager_test.rb
 - test/unit/proxy_selector_test.rb
 - test/unit/recurring_logic_test.rb
 - test/unit/task_groups_test.rb
@@ -286,7 +285,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.12
+rubygems_version: 2.4.5
 signing_key: 
 specification_version: 4
 summary: Foreman plugin for showing tasks information for resoruces and users
@@ -306,7 +305,6 @@ test_files:
 - test/unit/config/environment.rb
 - test/unit/daemon_test.rb
 - test/unit/dynflow_console_authorizer_test.rb
-- test/unit/otp_manager_test.rb
 - test/unit/proxy_selector_test.rb
 - test/unit/recurring_logic_test.rb
 - test/unit/task_groups_test.rb

data/test/unit/otp_manager_test.rb

@@ -1,70 +0,0 @@
-require 'foreman_tasks_test_helper'
-require 'foreman_tasks_core/otp_manager'
-
-module ForemanTasksCore
-  class OtpManagerTest < ActiveSupport::TestCase
-    class TestOtpManager < OtpManager
-      def self.reset!
-        @passwords = nil
-      end
-    end
-
-    before do
-      TestOtpManager.reset!
-    end
-
-    let(:username) { 'myuser' }
-    let(:password) { '123456789' }
-    let(:base64)   { 'bXl1c2VyOjEyMzQ1Njc4OQ==' }
-
-    it 'it doesn\'t raise when no passwords were generated yet' do
-      assert_nil TestOtpManager.drop_otp('missing', 'password')
-    end
-
-    it 'generates OTPs using SecureRandom.hex and converts them to strings' do
-      otp = 4
-      SecureRandom.stubs(:hex).returns(otp)
-      TestOtpManager.generate_otp(username).must_equal otp.to_s
-    end
-
-    it 'removes OTP only when correct username and password is provided' do
-      otp = TestOtpManager.generate_otp(username)
-      assert_nil TestOtpManager.drop_otp('wrong_username', 'wrong_password')
-      assert_nil TestOtpManager.drop_otp(username, 'wrong_password')
-      assert_nil TestOtpManager.drop_otp('wrong_username', otp)
-      TestOtpManager.drop_otp(username, otp).must_equal otp
-    end
-
-    it 'parses the hash correctly' do
-      SecureRandom.stubs(:hex).returns(password)
-      TestOtpManager.expects(:drop_otp).with(username, password.to_s)
-      TestOtpManager.authenticate(base64)
-    end
-
-    it 'authenticates correctly' do
-      SecureRandom.stubs(:hex).returns(password)
-      generated = TestOtpManager.generate_otp(username)
-
-      TestOtpManager.authenticate(base64).must_equal generated
-    end
-
-    it 'OTPs can be used only once' do
-      SecureRandom.stubs(:hex).returns(password)
-      generated = TestOtpManager.generate_otp(username)
-
-      TestOtpManager.authenticate(base64).must_equal generated
-      assert_nil TestOtpManager.authenticate(base64)
-    end
-
-    it 'creates token from username and password correctly' do
-      TestOtpManager.tokenize(username, password).must_equal base64
-    end
-
-    it 'overwrites old OTP when generating a new one for the same username' do
-      old = TestOtpManager.generate_otp(username)
-      new = TestOtpManager.generate_otp(username)
-      assert_nil TestOtpManager.drop_otp(username, old)
-      TestOtpManager.drop_otp(username, new).must_equal new
-    end
-  end
-end