summaryrefslogtreecommitdiff
path: root/spec/functions/chop_spec.rb
diff options
context:
space:
mode:
authorMark Chappell <mchappel@redhat.com>2014-04-22 09:36:28 +0200
committerMark Chappell <mchappel@redhat.com>2014-09-22 19:49:50 +0200
commite2d7f3bb89a91d3aff6f9810d69bd84bc82ffb29 (patch)
treee98f261a7fdafde7d4612f0770e20bbb832bf4a6 /spec/functions/chop_spec.rb
parentb347cc83e24e7ef51dc340bc753b96af026050f8 (diff)
(MODULES-707) chomp() fails because generate() no longer returns a string
We need to use unless value.is_a?(String) || value.is_a?(Array) rather than klass = value.class unless [String, Array].include?(klass) because the klass version enforces type checking which is too strict, and does not allow us to accept objects wich have extended String (or Array). For example, generate() function now returns Puppet::Util::Execution::ProcessOutput which is just a very simple extension of String. While this in it's self was not intentional (PUP-2306) it is not unreasonable to cope with objects which extend Strings
Diffstat (limited to 'spec/functions/chop_spec.rb')
-rwxr-xr-xspec/functions/chop_spec.rb9
1 files changed, 9 insertions, 0 deletions
diff --git a/spec/functions/chop_spec.rb b/spec/functions/chop_spec.rb
index d9dbb88..c8a1951 100755
--- a/spec/functions/chop_spec.rb
+++ b/spec/functions/chop_spec.rb
@@ -16,4 +16,13 @@ describe "the chop function" do
result = scope.function_chop(["asdf\n"])
expect(result).to(eq("asdf"))
end
+
+ it "should accept objects which extend String" do
+ class AlsoString < String
+ end
+
+ value = AlsoString.new("abc\n")
+ result = scope.function_chop([value])
+ result.should(eq('abc'))
+ end
end