foreman_ansible 2.2.5 → 2.2.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: e462c3b9312bf42cb1f7a267edbe619aae3849dc
-  data.tar.gz: 1cd77d5a1ab0047c0b25108a995c73790f92e774
+SHA256:
+  metadata.gz: 4d5259c34f569a52fce421e5fea3256094bb90c70ca595c0b4aa483ec3374666
+  data.tar.gz: efa7a4f4b6525b9f3a95e8656139d398971eb181ead180c9e73c51ff335c0fdf
 SHA512:
-  metadata.gz: e77318b028e35757faa9bc78b523edfba003bf15c4a687f1592c6d16e0191d1d2fa913da461e757f95118bc6bf0132a78f58381c1c10bcf9c2bd9ab94ee65da3
-  data.tar.gz: d381adc5b3aaa880a0e71da579bb2f7f063bd76e8542500de92ed53ccb06443974e168e31a5588cb51f351c96416805f47146c17c406c75d0f249996ea327d71
+  metadata.gz: 6efb74bceb03772f6882aee0231e5dc1b7a5d8f8806b4b60c5918aa0dbc26108a01de3eb3fab9d8c34cafd6d8448590ee7d361234d772bd21c3530798de83099
+  data.tar.gz: '08596fb8fb3847f2c744cc5aa08552fd25769199d9ef1c202a9e38d24c06c10e014522c00346bb83b8dfd7bc125793f91ae106a5900f5b57c07dd5fcf9156d4e'

data/app/controllers/ansible_roles_controller.rb

@@ -25,7 +25,7 @@ class AnsibleRolesController < ::ApplicationController
   def import
     changed = @importer.import!
     if changed.values.all?(&:empty?)
-      notice no_changed_roles_message
+      success no_changed_roles_message
       redirect_to ansible_roles_path
     else
       render :locals => { :changed => changed }
@@ -33,8 +33,8 @@ class AnsibleRolesController < ::ApplicationController
   end
 
   def confirm_import
-    @importer.finish_import(params[:changed])
-    notice _('Import of roles successfully finished.')
+    @importer.finish_import(params[:changed].to_unsafe_h)
+    success _('Import of roles successfully finished.')
     redirect_to ansible_roles_path
   end
 

data/app/services/foreman_ansible/ui_roles_importer.rb

@@ -12,13 +12,13 @@ module ForemanAnsible
     end
 
     def create_new_roles(changes)
-      changes.values.each do |new_role|
+      changes.each_value do |new_role|
         ::AnsibleRole.create(JSON.parse(new_role))
       end
     end
 
     def delete_old_roles(changes)
-      changes.values.each do |old_role|
+      changes.each_value do |old_role|
         ::AnsibleRole.find(JSON.parse(old_role)['id']).destroy
       end
     end

data/lib/foreman_ansible/version.rb

@@ -2,5 +2,5 @@
 # This way other parts of Foreman can just call ForemanAnsible::VERSION
 # and detect what version the plugin is running.
 module ForemanAnsible
-  VERSION = '2.2.5'.freeze
+  VERSION = '2.2.6'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: foreman_ansible
 version: !ruby/object:Gem::Version
-  version: 2.2.5
+  version: 2.2.6
 platform: ruby
 authors:
 - Daniel Lobato Garcia
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-07-25 00:00:00.000000000 Z
+date: 2018-08-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -265,43 +265,43 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.8
+rubygems_version: 2.7.6
 signing_key: 
 specification_version: 4
 summary: Ansible integration with Foreman (theforeman.org)
 test_files:
-- test/unit/host_ansible_role_test.rb
-- test/unit/helpers/ansible_reports_helper_test.rb
-- test/unit/ansible_role_test.rb
+- test/fixtures/report.json
+- test/fixtures/sample_facts.json
+- test/fixtures/insights_playbook.yaml
 - test/unit/concerns/config_reports_extensions_test.rb
-- test/unit/concerns/host_managed_extensions_test.rb
 - test/unit/concerns/hostgroup_extensions_test.rb
+- test/unit/concerns/host_managed_extensions_test.rb
+- test/unit/services/ansible_report_importer_test.rb
+- test/unit/services/inventory_creator_test.rb
+- test/unit/services/api_roles_importer_test.rb
+- test/unit/services/fact_parser_test.rb
+- test/unit/services/roles_importer_test.rb
+- test/unit/services/fact_sparser_test.rb
+- test/unit/services/structured_fact_importer_test.rb
+- test/unit/services/ui_roles_importer_test.rb
+- test/unit/services/insights_plan_runner_test.rb
+- test/unit/services/fact_importer_test.rb
+- test/unit/lib/proxy_api/ansible_test.rb
 - test/unit/lib/foreman_ansible_core/command_creator_test.rb
 - test/unit/lib/foreman_ansible_core/roles_reader_test.rb
 - test/unit/lib/foreman_ansible_core/playbook_runner_test.rb
-- test/unit/lib/proxy_api/ansible_test.rb
-- test/unit/ansible_provider_test.rb
+- test/unit/host_ansible_role_test.rb
 - test/unit/hostgroup_ansible_role_test.rb
-- test/unit/services/fact_importer_test.rb
-- test/unit/services/ui_roles_importer_test.rb
-- test/unit/services/inventory_creator_test.rb
-- test/unit/services/roles_importer_test.rb
-- test/unit/services/insights_plan_runner_test.rb
-- test/unit/services/fact_parser_test.rb
-- test/unit/services/api_roles_importer_test.rb
-- test/unit/services/structured_fact_importer_test.rb
-- test/unit/services/fact_sparser_test.rb
-- test/unit/services/ansible_report_importer_test.rb
 - test/unit/actions/run_proxy_ansible_command_test.rb
 - test/unit/actions/run_ansible_job_test.rb
-- test/test_plugin_helper.rb
-- test/functional/hosts_controller_test.rb
-- test/functional/api/v2/hosts_controller_test.rb
+- test/unit/ansible_provider_test.rb
+- test/unit/ansible_role_test.rb
+- test/unit/helpers/ansible_reports_helper_test.rb
 - test/functional/api/v2/ansible_roles_controller_test.rb
 - test/functional/api/v2/hostgroups_controller_test.rb
+- test/functional/api/v2/hosts_controller_test.rb
 - test/functional/ansible_roles_controller_test.rb
+- test/functional/hosts_controller_test.rb
 - test/factories/ansible_roles.rb
 - test/factories/ansible_proxy.rb
-- test/fixtures/report.json
-- test/fixtures/sample_facts.json
-- test/fixtures/insights_playbook.yaml
+- test/test_plugin_helper.rb