From fb58e53facf1f47570ec31078e80b696c8e2dbac Mon Sep 17 00:00:00 2001 From: Joe Rafaniello Date: Mon, 24 Apr 2017 13:42:42 -0400 Subject: [PATCH] Remove unneeded join Seen in https://github.com/ManageIQ/manageiq/pull/14852 It's already a single string, there's no need to wrap it in an array and join the single element with a space. --- spec/lib/rbac/filterer_spec.rb | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/spec/lib/rbac/filterer_spec.rb b/spec/lib/rbac/filterer_spec.rb index 03678b2c85b..73813bce6be 100644 --- a/spec/lib/rbac/filterer_spec.rb +++ b/spec/lib/rbac/filterer_spec.rb @@ -909,8 +909,8 @@ def get_rbac_results_for_and_expect_objects(klass, expected_objects) group.entitlement.set_belongsto_filters([]) group.save! - ansible_configuration_script_with_tag.tag_with(['/managed/environment/prod'].join(' '), :ns => '*') - ansible_playbook_with_tag.tag_with(['/managed/environment/prod'].join(' '), :ns => '*') + ansible_configuration_script_with_tag.tag_with('/managed/environment/prod', :ns => '*') + ansible_playbook_with_tag.tag_with('/managed/environment/prod', :ns => '*') end it 'lists only tagged ConfigurationScripts' do @@ -962,7 +962,7 @@ def get_rbac_results_for_and_expect_objects(klass, expected_objects) group.entitlement.set_belongsto_filters([]) group.save! - network_object_with_tag.tag_with(['/managed/environment/prod'].join(' '), :ns => '*') + network_object_with_tag.tag_with('/managed/environment/prod', :ns => '*') end it "lists only tagged #{network_model}" do