(GH-166) Add tests for definition provider

This commit adds basic integration tests for the definition provider.

Author: glennsarti

server/spec/languageserver/fixtures/environments/.gitkeep

@@ -0,0 +1,4 @@
+define deftypeone(
+  $ensure       = 'UNSET',
+) {
+}

server/spec/languageserver/fixtures/environments/test-fixtures/modules/definedtypes/manifests/deftypeone.pp

@@ -0,0 +1,9 @@
+{
+  "name": "testfixture-definedtypes",
+  "version": "0.1.0",
+  "author": "testfixture",
+  "summary": "Test fixture with defined types",
+  "license": "Apache-2.0",
+  "source": "http://localhost",
+  "dependencies": []
+}

server/spec/languageserver/fixtures/environments/test-fixtures/modules/definedtypes/metadata.json

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

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

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

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

@@ -0,0 +1,9 @@
+{
+  "name": "testfixture-testclasses",
+  "version": "0.1.0",
+  "author": "testfixture",
+  "summary": "Test fixture with puppet classes",
+  "license": "Apache-2.0",
+  "source": "http://localhost",
+  "dependencies": []
+}

server/spec/languageserver/fixtures/environments/test-fixtures/modules/testclasses/metadata.json

@@ -0,0 +1,144 @@
+require 'spec_helper'
+
+RSpec.shared_examples "a single definition result" do |filename_regex|
+  it "should return a single definition result which matches #{filename_regex.to_s}" do
+    result = subject.find_definition(content, line_num, char_num)
+
+    expect(result).to be_a(Array)
+    expect(result.count).to eq(1)
+    expect(result[0]['uri']).to match(filename_regex)
+    expect(result[0]['range']['start']['line']).to_not be_nil
+    expect(result[0]['range']['start']['character']).to_not be_nil
+    expect(result[0]['range']['end']['line']).to_not be_nil
+    expect(result[0]['range']['end']['character']).to_not be_nil
+  end
+end
+
+describe 'definition_provider' do
+  let(:subject) { PuppetLanguageServer::DefinitionProvider }
+
+  describe '#find_defintion' do
+    before(:all) do
+      # Ensure the functions are loaded so that defintion information is available
+      PuppetLanguageServer::PuppetHelper.load_functions unless PuppetLanguageServer::PuppetHelper.functions_loaded?
+
+      # Ensure the types are loaded so that defintion information is available
+      PuppetLanguageServer::PuppetHelper.load_types unless PuppetLanguageServer::PuppetHelper.types_loaded?
+
+      # Ensure the classes are loaded so that defintion information is available
+      PuppetLanguageServer::PuppetHelper.load_classes unless PuppetLanguageServer::PuppetHelper.classes_loaded?
+    end
+
+    context 'When cursor is on a function name' do
+      let(:content) { <<-EOT
+class Test::NoParams {
+  alert('This is an alert message')
+}
+EOT
+      }
+      let(:line_num) { 1 }
+      let(:char_num) { 5 }
+
+      it_should_behave_like "a single definition result", /functions\.rb/
+    end
+
+    context 'When cursor is on a custom puppet type' do
+      let(:content) { <<-EOT
+class Test::NoParams {
+  user { 'foo':
+    ensure => 'present',
+    name   => 'name',
+  }
+}
+EOT
+      }
+      let(:line_num) { 1 }
+      let(:char_num) { 5 }
+
+      it_should_behave_like "a single definition result", /user\.rb/
+    end
+
+    context 'When cursor is on a defined type' do
+      let(:content) { <<-EOT
+class Test::NoParams {
+  deftypeone { 'foo':
+    ensure => 'present',
+  }
+}
+EOT
+      }
+      let(:line_num) { 1 }
+      let(:char_num) { 5 }
+
+      it_should_behave_like "a single definition result", /deftypeone\.pp/
+    end
+
+    context 'When cursor is on a puppet class' do
+      let(:content) { <<-EOT
+class Test::NoParams {
+  puppetclassone { 'foo':
+    ensure => 'present',
+  }
+}
+EOT
+      }
+      let(:line_num) { 1 }
+      let(:char_num) { 5 }
+
+      it_should_behave_like "a single definition result", /puppetclassone\.pp/
+    end
+
+    context 'When cursor is on a classname for an include statement' do
+      let(:content) { <<-EOT
+class Test::NoParams {
+  include puppetclassone
+}
+EOT
+      }
+      let(:line_num) { 1 }
+      let(:char_num) { 14 }
+
+      it_should_behave_like "a single definition result", /puppetclassone\.pp/
+    end
+
+    context 'When cursor is on a fully qualified classname for an include statement' do
+      let(:content) { <<-EOT
+class Test::NoParams {
+  include testclasses::nestedclass
+}
+EOT
+      }
+      let(:line_num) { 1 }
+      let(:char_num) { 14 }
+
+      it_should_behave_like "a single definition result", /nestedclass\.pp/
+    end
+
+    context 'When cursor is on a root classname for an include statement' do
+      let(:content) { <<-EOT
+class Test::NoParams {
+  include ::puppetclassone
+}
+EOT
+      }
+      let(:line_num) { 1 }
+      let(:char_num) { 14 }
+
+      it_should_behave_like "a single definition result", /puppetclassone\.pp/
+    end
+
+    context 'When cursor is on a function name for an include statement' do
+      let(:content) { <<-EOT
+class Test::NoParams {
+  include puppetclassone
+}
+EOT
+      }
+      let(:line_num) { 1 }
+      let(:char_num) { 5 }
+
+      it_should_behave_like "a single definition result", /include\.rb/
+    end
+
+  end
+end