diff options
author | Jay Mundrawala <jdmundrawala@gmail.com> | 2015-05-13 09:08:00 -0500 |
---|---|---|
committer | Jay Mundrawala <jdmundrawala@gmail.com> | 2015-05-13 09:08:00 -0500 |
commit | 38b381e46236e020348b9b38d7310cec65b80545 (patch) | |
tree | 6e420718506df9e353bea0c0832d716f0e231c23 /lib/chef/formatters | |
parent | 5b60038b0ca2636aeb425d51582dacc4905363ae (diff) | |
download | chef-38b381e46236e020348b9b38d7310cec65b80545.tar.gz |
Refactor inline regexes into method callsjdm/3055
Diffstat (limited to 'lib/chef/formatters')
-rw-r--r-- | lib/chef/formatters/error_inspectors/resource_failure_inspector.rb | 12 |
1 files changed, 11 insertions, 1 deletions
diff --git a/lib/chef/formatters/error_inspectors/resource_failure_inspector.rb b/lib/chef/formatters/error_inspectors/resource_failure_inspector.rb index eeff5f10d2..6e4d9322f9 100644 --- a/lib/chef/formatters/error_inspectors/resource_failure_inspector.rb +++ b/lib/chef/formatters/error_inspectors/resource_failure_inspector.rb @@ -63,7 +63,7 @@ class Chef def recipe_snippet return nil if dynamic_resource? @snippet ||= begin - if file = resource.source_line[/^(([\w]:)?[^:]+):([\d]+)/,1] and line = resource.source_line[/^#{Regexp.escape(file)}:([\d]+)/,1].to_i + if file = parse_source and line = parse_line(file) return nil unless ::File.exists?(file) lines = IO.readlines(file) @@ -111,6 +111,16 @@ class Chef line_nr_string + line end + def parse_source + resource.source_line[/^(([\w]:)?[^:]+):([\d]+)/,1] + end + + def parse_line(source) + resource.source_line[/^#{Regexp.escape(source)}:([\d]+)/,1].to_i + end + + + end end end |