foreman-tasks 2.0.3 → 3.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e37924776f38fa00d98c1389b22baff43347f4f4c28c0d3848de93951b5cd1d9
-  data.tar.gz: 49c49253f6cd157b6c602bb1ceb5bee9b249aee207939501b60e273b4fce1734
+  metadata.gz: 24da58e2837688da28a18832b72b154ac74868157f6176f942119b8b856987c8
+  data.tar.gz: 3031305c5f86f3b03aed95a4501670cbd2dbd595db38f638354ff046f416823a
 SHA512:
-  metadata.gz: 53aebec3deef276e131483dc20f5ce3753a9bf8b7d64e9a443af9a0284e9749a657c97e03923034db81e1a698aa24d0de120fe3530b5768725e9a910f8b34e52
-  data.tar.gz: ae5931295bc54e9bfcf3b48278c52724265ddc0e902d39be2d05897f1231b5c29291810f7e638d83f0a436a6f0a414453692158ad9b93c2445db038e87872168
+  metadata.gz: a6a21ff8c4b9336f63118c2cdf6a4f11bf25ff882fe1dbd1b17d05f9722705c9cebabb522792416758c3fede1a8507d720cac0317076d8e833bfbb95a83fa8a4
+  data.tar.gz: bef5dd6cac70ac43d53e83950668e930883649d42966b10ff6303e0d581431d3f11028ae24c7ecc9ed0e2875214f4a512cfbaa46f1da81fd912d6a6f25da2db2

data/.github/workflows/js_tests.yml

@@ -0,0 +1,27 @@
+name: JavaScript Testing
+on:
+  pull_request:
+    paths:
+      - 'webpack/**'
+      - 'package.json'
+      - '.github/workflows/js_tests.yml'
+
+jobs:
+  test_js:
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        node-version: [10, 12]
+    steps:
+      - uses: actions/checkout@v2
+      - name: Setup Node
+        uses: actions/setup-node@v1
+        with:
+          node-version:  ${{ matrix.node-version }}
+      - name: Npm install
+        run: npm install
+      - name: Run plugin linter
+        run: npm run lint
+      - name: Run plugin tests
+        run: npm run test

data/.github/workflows/ruby_tests.yml

@@ -0,0 +1,74 @@
+name: Ruby Testing
+on: [pull_request]
+env:
+  RAILS_ENV: test
+  DATABASE_URL: postgresql://postgres:@localhost/test
+  DATABASE_CLEANER_ALLOW_REMOTE_DATABASE_URL: true
+jobs:
+  rubocop:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+      - name: Setup Ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: 2.6
+      - name: Setup
+        run: |
+          gem install bundler
+          bundle install --jobs=3 --retry=3
+      - name: Run rubocop
+        run: bundle exec rubocop
+  test_ruby:
+    runs-on: ubuntu-latest
+    needs: rubocop
+    services:
+      postgres:
+        image: postgres:12.1
+        ports: ['5432:5432']
+        options: --health-cmd pg_isready --health-interval 10s --health-timeout 5s --health-retries 5
+    strategy:
+      fail-fast: false
+      matrix:
+        foreman-core-branch: [develop]
+        ruby-version: [2.5, 2.6]
+        node-version: [12]
+    steps:
+      - run: sudo apt-get update
+      - run: sudo apt-get install build-essential libcurl4-openssl-dev zlib1g-dev libpq-dev
+      - uses: actions/checkout@v2
+        with:
+          repository: theforeman/foreman
+          ref: ${{ matrix.foreman-core-branch }}
+      - uses: actions/checkout@v2
+        with:
+          path: foreman-tasks
+      - name: Setup Ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.ruby-version }}
+      - name: Setup Node
+        uses: actions/setup-node@v1
+        with:
+          node-version:  ${{ matrix.node-version }}
+      - uses: actions/cache@v2
+        with:
+          path: vendor/bundle
+          key: ${{ runner.os }}-fgems-${{ matrix.ruby-version }}-${{ hashFiles('Gemfile.lock') }}
+          restore-keys: |
+            ${{ runner.os }}-fgems-${{ matrix.ruby-version }}-
+      - name: Setup Bundler
+        run: |
+          echo "gem 'foreman-tasks', path: './foreman-tasks'" > bundler.d/foreman-tasks.local.rb
+          gem install bundler
+          bundle config set without journald development console libvirt
+          bundle config set path vendor/bundle
+      - name: Prepare test env
+        run: |
+          bundle install --jobs=3 --retry=3
+          bundle exec rake db:create
+          bundle exec rake db:migrate
+      - name: Run plugin tests
+        run: |
+          bundle exec rake test:foreman-tasks
+          bundle exec rake test TEST="test/unit/foreman/access_permissions_test.rb"

data/.rubocop.yml

@@ -6,10 +6,10 @@ require:
 inherit_from: .rubocop_todo.yml
 
 AllCops:
-  Include:
-    - 'Rakefile'
-    - 'app/views/foreman_tasks/api/**/*.rabl'
-    - lib/**/*.rake
+  Exclude:
+    - 'node_modules/**/*'
+    - 'locale/*'
+  TargetRubyVersion: 2.5
 
 Lint/ShadowingOuterLocalVariable:
   Enabled: false
@@ -68,6 +68,8 @@ Naming/FileName:
     - Gemfile
     - db/seeds.d/*
     - lib/foreman-tasks.rb
+    - gemfile.d/foreman-tasks.rb
+
 
 Layout/HeredocIndentation:
   Exclude:
@@ -115,3 +117,9 @@ Naming/RescuedExceptionsVariableName:
 
 Style/WordArray:
   Enabled: false
+
+Style/TrailingCommaInArrayLiteral:
+  EnforcedStyleForMultiline: comma
+
+Style/TrailingCommaInHashLiteral:
+  EnforcedStyleForMultiline: comma

data/.rubocop_todo.yml

@@ -1,21 +1,34 @@
-# Offense count: 1
-Lint/EmptyWhen:
+# This configuration was generated by
+# `rubocop --auto-gen-config`
+# on 2020-08-07 08:31:32 UTC using RuboCop version 0.89.0.
+# The point is for the user to remove these configuration records
+# one by one as the offenses are removed from the code base.
+# Note that changes in the inspected code, or installation of new
+# versions of RuboCop, may require this file to be generated again.
+
+# Offense count: 2
+# Configuration parameters: Include.
+# Include: **/*.gemspec
+Gemspec/RequiredRubyVersion:
   Exclude:
-    - 'app/lib/actions/proxy_action.rb'
+    - 'foreman-tasks-core.gemspec'
+    - 'foreman-tasks.gemspec'
 
 # Offense count: 1
 Lint/UselessAssignment:
   Exclude:
     - 'lib/foreman_tasks/tasks/export_tasks.rake'
 
-Minitest/AssertEmpty:
-  Enabled: false
-
-Minitest/AssertEqual:
-  Enabled: false
-
-Minitest/GlobalExpectations:
-  Enabled: false
+# Offense count: 1
+# Configuration parameters: ExpectMatchingDefinition, CheckDefinitionPathHierarchy, Regex, IgnoreExecutableScripts, AllowedAcronyms.
+# AllowedAcronyms: CLI, DSL, ACL, API, ASCII, CPU, CSS, DNS, EOF, GUID, HTML, HTTP, HTTPS, ID, IP, JSON, LHS, QPS, RAM, RHS, RPC, SLA, SMTP, SQL, SSH, TCP, TLS, TTL, UDP, UI, UID, UUID, URI, URL, UTF8, VM, XML, XMPP, XSRF, XSS
+Naming/FileName:
+  Exclude:
+    - '*.gemspec'
+    - 'Gemfile'
+    - 'db/seeds.d/*'
+    - 'lib/foreman-tasks.rb'
+    - 'gemfile.d/foreman-tasks.rb'
 
 # Offense count: 3
 # Configuration parameters: EnforcedStyleForLeadingUnderscores.
@@ -28,7 +41,7 @@ Naming/MemoizedInstanceVariableName:
 
 # Offense count: 11
 # Configuration parameters: MinNameLength, AllowNamesEndingInNumbers, AllowedNames, ForbiddenNames.
-# AllowedNames: io, id, to, by, on, in, at, ip, db, os, pp
+# AllowedNames: at, by, db, id, in, io, ip, of, on, os, pp, to
 Naming/MethodParameterName:
   Exclude:
     - 'app/helpers/foreman_tasks/foreman_tasks_helper.rb'
@@ -67,11 +80,13 @@ Rails/OutputSafety:
   Exclude:
     - 'app/helpers/foreman_tasks/foreman_tasks_helper.rb'
 
-# Offense count: 1
+# Offense count: 2
 # Configuration parameters: Include.
 # Include: **/Rakefile, **/*.rake
 Rails/RakeEnvironment:
   Exclude:
+    - 'lib/capistrano/tasks/**/*.rake'
+    - 'lib/foreman_tasks/tasks/dynflow.rake'
     - 'lib/foreman_tasks/tasks/test.rake'
 
 # Offense count: 12
@@ -92,22 +107,14 @@ Style/ClassAndModuleChildren:
     - 'test/controllers/api/recurring_logics_controller_test.rb'
     - 'test/controllers/api/tasks_controller_test.rb'
 
-# Offense count: 2
+# Offense count: 1
+# Configuration parameters: EnforcedStyle.
+# SupportedStyles: allowed_in_returns, forbidden
 Style/DoubleNegation:
   Exclude:
-    - 'app/models/foreman_tasks/lock.rb'
     - 'app/models/foreman_tasks/recurring_logic.rb'
 
-# Offense count: 36
+# Offense count: 35
 # Configuration parameters: MinBodyLength.
 Style/GuardClause:
   Enabled: false
-
-Style/HashEachMethods:
-  Enabled: false
-
-Style/HashTransformKeys:
-  Enabled: false
-
-Style/HashTransformValues:
-  Enabled: false

data/Gemfile

@@ -13,3 +13,8 @@ gemspec :name => 'foreman-tasks'
 
 # To use debugger
 # gem 'debugger'
+
+gem 'rubocop', '~> 0.87'
+gem 'rubocop-minitest', '~> 0.9.0'
+gem 'rubocop-performance', '~> 1.5.2'
+gem 'rubocop-rails', '~> 2.5.2'

data/app/controllers/foreman_tasks/api/tasks_controller.rb

@@ -31,17 +31,6 @@ module ForemanTasks
       param :id, :identifier, desc: 'UUID of the task'
       def details; end
 
-      api :GET, '/tasks/:id/sub_tasks', 'Show sub_tasks details'
-      param :id, :identifier, desc: 'UUID of the task'
-      def sub_tasks
-        parent_task = resource_scope.find(params[:id])
-        filtered_scope = parent_task.sub_tasks
-        action_name = { "action_name" => parent_task.action }
-
-        filtered_scope = DashboardTableFilter.new(filtered_scope, params).scope
-        render :json => action_name.merge(tasks_list(filtered_scope))
-      end
-
       api :POST, '/tasks/bulk_search', 'List dynflow tasks for uuids'
       param :searches, Array, :desc => 'List of uuids to fetch info about' do
         param :search_id, String, :desc => <<-DESC
@@ -116,7 +105,7 @@ module ForemanTasks
           total: resumed.length + failed.length + skipped.length,
           resumed: resumed,
           failed: failed,
-          skipped: skipped
+          skipped: skipped,
         }
       end
 
@@ -137,7 +126,7 @@ module ForemanTasks
         render :json => {
           total: cancelled.length + skipped.length,
           cancelled: cancelled,
-          skipped: skipped
+          skipped: skipped,
         }
       end
 
@@ -165,22 +154,33 @@ module ForemanTasks
         render :json => {
           total: total_length,
           stopped_length: to_stop_length,
-          skipped_length: skipped_length
+          skipped_length: skipped_length,
         }
       end
 
       api :GET, '/tasks', N_('List tasks')
-      param :search, String, :desc => N_('Search string')
-      param :page, :number, :desc => N_('Page number, starting at 1')
-      param :per_page, :number, :desc => N_('Number of results per page to return')
-      param :order, String, :desc => N_("Sort field and order, e.g. 'name DESC'")
-      param :sort, Hash, :desc => N_("Hash version of 'order' param") do
-        param :by, String, :desc => N_('Field to sort the results on')
-        param :order, String, :desc => N_('How to order the sorted results (e.g. ASC for ascending)')
-      end
+      api :GET, '/tasks/:parent_task_id/sub_tasks', 'Show sub_tasks details'
+      param :parent_task_id, :identifier, desc: 'UUID of the task'
+      param_group :search_and_pagination, ::Api::V2::BaseController
       def index
-        filtered_scope = DashboardTableFilter.new(resource_scope, params).scope
-        render :json => tasks_list(filtered_scope)
+        if params[:sort_by] || params[:sort_order]
+          Foreman::Deprecation.api_deprecation_warning(
+            "The sort params sort_by and sort_order are deprecated.
+            Please use the order param instead as one string 'order=started_at desc'"
+          )
+
+          ordering_params = {
+            sort_by: params[:sort_by] || 'started_at',
+            sort_order: params[:sort_order] || 'DESC',
+          }
+          params[:order] = "#{ordering_params[:sort_by]} #{ordering_params[:sort_order]}"
+        end
+        params[:order] ||= 'started_at DESC'
+        @tasks = DashboardTableFilter.new(resource_scope_for_index, params).scope.order(params[:order].to_s)
+      end
+
+      def search_options
+        [params[:search], {}]
       end
 
       def_param_group :callback_target do
@@ -312,7 +312,13 @@ module ForemanTasks
       end
 
       def resource_scope(_options = {})
-        @resource_scope ||= ForemanTasks::Task.authorized("#{action_permission}_foreman_tasks")
+        scope = ForemanTasks::Task.authorized("#{action_permission}_foreman_tasks")
+        scope = scope.where(:parent_task_id => params[:parent_task_id]) if params[:parent_task_id]
+        scope
+      end
+
+      def resource_scope_for_index(*args)
+        super.select("DISTINCT foreman_tasks_tasks.*, coalesce(ended_at, current_timestamp) - coalesce(coalesce(started_at, ended_at), current_timestamp) as duration")
       end
 
       def controller_permission
@@ -330,39 +336,6 @@ module ForemanTasks
         end
       end
 
-      def tasks_list(filtered_scope)
-        total = resource_scope.count
-
-        search_scope = filtered_scope.search_for(params[:search])
-        subtotal = search_scope.select('DISTINCT foreman_tasks_tasks.id').count
-        filtered_scope = search_scope.select('DISTINCT foreman_tasks_tasks.*')
-
-        ordering_params = {
-          sort_by: params[:sort_by] || 'started_at',
-          sort_order: params[:sort_order] || 'DESC'
-        }
-        filtered_scope = ordering_scope(filtered_scope, ordering_params)
-
-        pagination_params = {
-          page: params[:page] || 1,
-          per_page: params[:per_page] || Setting[:entries_per_page] || 20
-        }
-        filtered_scope = pagination_scope(filtered_scope, pagination_params)
-        results = filtered_scope.map { |task| task_hash(task) }
-
-        {
-          total: total,
-          subtotal: subtotal,
-          page: pagination_params[:page],
-          per_page: pagination_params[:per_page],
-          sort: {
-            by: ordering_params[:sort_by],
-            order: ordering_params[:sort_order]
-          },
-          results: results
-        }
-      end
-
       def bulk_scope
         scope = resource_scope
         scope = scope.search_for(params[:search]) if params[:search]

data/app/controllers/foreman_tasks/concerns/parameters/triggering.rb

@@ -15,7 +15,7 @@ module ForemanTasks
                   *::ForemanTasks::Triggering::PARAMS,
                   :days_of_week => {},
                   :time => {},
-                  :end_time => {}
+                  :end_time => {},
                 ],
                 :nested => true
               )

data/app/controllers/foreman_tasks/recurring_logics_controller.rb

@@ -1,5 +1,7 @@
 module ForemanTasks
   class RecurringLogicsController < ::ApplicationController
+    include Foreman::Controller::AutoCompleteSearch
+
     before_action :find_recurring_logic, :only => [:show, :cancel, :enable, :disable]
 
     def index
@@ -41,6 +43,11 @@ module ForemanTasks
     def change_enabled(value)
       begin
         @recurring_logic.update!(:enabled => value)
+        flash[:success] = if value
+                            _('The recurring logic was enabled.')
+                          else
+                            _('The recurring logic was disabled.')
+                          end
       rescue RecurringLogicCancelledException => e
         @errors = e.message
       end

data/app/helpers/foreman_tasks/foreman_tasks_helper.rb

@@ -82,7 +82,7 @@ module ForemanTasks
             ' — ',
             content_tag(:span, nil, :class => 'time', :style => 'white-space: nowrap;') do
               fields.time_select(attr, datetime_options.merge(:ignore_date => true), html_options)
-            end
+            end,
           ].join
         end
       end
@@ -139,7 +139,7 @@ module ForemanTasks
         cronline_fieldset(f, triggering),
         monthly_fieldset(f, triggering),
         weekly_fieldset(f, triggering),
-        time_picker_fieldset(f, triggering)
+        time_picker_fieldset(f, triggering),
       ]
 
       content_tag(:fieldset, nil, :id => 'trigger_mode_recurring', :class => "trigger_mode_form #{'hidden' unless triggering.recurring?}") do
@@ -158,7 +158,7 @@ module ForemanTasks
         # TRANSLATORS: this translation is referring to an option which is a time interval
         _('is month (range: 1-12)'),
         # TRANSLATORS: this translation is referring to an option which is a time interval
-        _('is day of week (range: 0-6)')
+        _('is day of week (range: 0-6)'),
       ].map { |opt| content_tag(:li, opt) }.join
 
       help = _("Cron line format 'a b c d e', where: %s") % "<br><ol type=\"a\">#{options}</ol>".html_safe