(GH-166) Detect definition for Resources bodies

Previously definition detection would fail on resource bodies as the text comes
in as a LiteralString e.g.

class { 'testclass':    <--- testclass would be the LiteralString inside
} 							 a ResourceBody

This commit updates the definition provider to detect these instances and return
a defintion location.  This commit also adds tests for this scenario.

Author: glennsarti

server/lib/puppet-languageserver/definition_provider.rb

@@ -14,6 +14,21 @@ def self.find_definition(content, line_num, char_num)
         func_name = item.functor_expr.value
         response << function_name(resource_name)
 
+      when 'Puppet::Pops::Model::LiteralString'
+        # LiteralString could be anything.  Context is the key here
+        parent = path.last
+
+        # What if it's a resource name.  Then the Literal String must be the same as the Resource Title
+        # e.g.
+        # class { 'testclass':    <--- testclass would be the LiteralString inside a ResourceBody
+        # }
+        if !parent.nil? &&
+          parent.class.to_s == 'Puppet::Pops::Model::ResourceBody' &&
+          parent.title.value == item.value
+            resource_name = item.value
+            response << type_or_class(resource_name)
+        end
+
       when 'Puppet::Pops::Model::QualifiedName'
         # Qualified names could be anything.  Context is the key here
         parent = path.last

server/spec/languageserver/fixtures/environments/test-fixtures/modules/testclasses/manifests/init.pp

@@ -0,0 +1,3 @@
+class testclasses () {
+  $param1 = 'value2'
+}

server/spec/languageserver/integration/puppet-languageserver/definition_provider_spec.rb

@@ -58,6 +58,51 @@ class Test::NoParams {
       it_should_behave_like "a single definition result", /user\.rb/
     end
 
+    context 'When cursor is on a puppet class' do
+      let(:content) { <<-EOT
+class Test::NoParams {
+  class { 'testclasses':
+    ensure => 'present',
+  }
+}
+EOT
+      }
+      let(:line_num) { 1 }
+      let(:char_num) { 13 }
+
+      it_should_behave_like "a single definition result", /init\.pp/
+    end
+
+    context 'When cursor is on a root puppet class' do
+      let(:content) { <<-EOT
+class Test::NoParams {
+  class { '::testclasses':
+    ensure => 'present',
+  }
+}
+EOT
+      }
+      let(:line_num) { 1 }
+      let(:char_num) { 13 }
+
+      it_should_behave_like "a single definition result", /init\.pp/
+    end
+
+    context 'When cursor is on a fully qualified puppet class' do
+      let(:content) { <<-EOT
+class Test::NoParams {
+  class { 'testclasses::nestedclass':
+    ensure => 'present',
+  }
+}
+EOT
+      }
+      let(:line_num) { 1 }
+      let(:char_num) { 13 }
+
+      it_should_behave_like "a single definition result", /nestedclass\.pp/
+    end
+
     context 'When cursor is on a defined type' do
       let(:content) { <<-EOT
 class Test::NoParams {