puppet-retrospec 0.4.0 → 0.5.0

data/spec/unit/resource_spec.rb

@@ -0,0 +1,62 @@
+require 'spec_helper'
+require 'retrospec/resource'
+
+describe "resource" do
+  after :all do
+    # enabling the removal slows down tests, but from time to time we may need to
+    FileUtils.rm_rf(fixture_modules_path) if ENV['RETROSPEC_CLEAN_UP_TEST_MODULES'] =~ /true/
+  end
+
+  before :all do
+    # enabling the removal of real modules slows down tests, but from time to time we may need to
+    FileUtils.rm_rf(fixture_modules_path) if ENV['RETROSPEC_CLEAN_UP_TEST_MODULES'] =~ /true/
+    @path = File.join(fixture_modules_path, 'tomcat')
+
+
+  end
+
+  before :each do
+    clean_up_spec_dir(@path)
+    @opts = {:module_path => @path, :enable_beaker_tests => false,
+             :enable_user_templates => false, :template_dir => nil }
+
+  end
+
+  it 'should initialize with one resource' do
+    my_path = File.expand_path(File.join('spec', 'fixtures', 'fixture_modules', 'one_resource_module'))
+    my_retro = Retrospec.new(my_path)
+    r = Resource.all(my_retro.types.find {|x| x.name == 'one_resource'})
+    expect(r.length).to eq(1)
+    expect(r[0].parameters).to eq({"ensure"=>"present"})
+    expect(r[0].title).to eq("/tmp/test")
+    expect(r[0].type).to eq("file")
+  end
+
+  it 'should initialize with two resources' do
+    my_path = File.expand_path(File.join('spec', 'fixtures', 'fixture_modules', 'one_resource_module'))
+    my_retro = Retrospec.new(my_path)
+    test_type = my_retro.types.find {|x| x.name == 'one_resource::another_resource'}
+    VariableStore.populate(test_type)
+    r = Resource.all(test_type)
+    expect(r.length).to eq(2)
+    expect(r[0].parameters).to eq({"ensure"=>"present"})
+    expect(r[0].title).to eq("/tmp/test2")
+    expect(r[0].type).to eq("file")
+    expect(r[1].parameters).to eq({"ensure"=>"present","content" => "/tmp/test3/test3183/oohhhh"})
+    expect(r[1].title).to eq("/tmp/test3")
+    expect(r[1].type).to eq("file")
+  end
+
+  it 'should return resources' do
+    test_module = Retrospec.new(@opts[:module_path], @opts)
+    test_type = test_module.types[0]
+    expect(Resource.all(test_type.code)).to eq([])
+  end
+
+  it 'can process an empty class' do
+    my_path = File.expand_path(File.join('spec', 'fixtures', 'fixture_modules', 'zero_resource_module'))
+    my_retro = Retrospec.new(my_path)
+    r = Resource.all(my_retro.types.find {|x| x.name == 'empty_class'})
+    expect(r.size).to eq(0)
+  end
+end

data/spec/unit/type_code_spec.rb

@@ -0,0 +1,39 @@
+require 'spec_helper'
+require 'retrospec/type_code'
+
+describe "type_code" do
+  after :all do
+    # enabling the removal slows down tests, but from time to time we may need to
+    FileUtils.rm_rf(fixture_modules_path) if ENV['RETROSPEC_CLEAN_UP_TEST_MODULES'] =~ /true/
+  end
+
+  before :all do
+    # enabling the removal of real modules slows down tests, but from time to time we may need to
+    FileUtils.rm_rf(fixture_modules_path) if ENV['RETROSPEC_CLEAN_UP_TEST_MODULES'] =~ /true/
+    @path = File.join(fixture_modules_path, 'tomcat')
+
+
+  end
+
+  before :each do
+    my_path = File.expand_path(File.join('spec', 'fixtures', 'fixture_modules', 'one_resource_module'))
+    @my_retro = Retrospec.new(my_path)
+    @test_type = @my_retro.types.find {|x| x.name == 'one_resource::another_resource'}
+
+  end
+  it 'should initialize correctly scope name' do
+    expect(TypeCode.new(@test_type).scope_name).to eq('one_resource::another_resource')
+    expect(TypeCode.new(@test_type).type).to eq(@test_type)
+  end
+
+  it 'should find all the variables' do
+    expect(TypeCode.new(@test_type).variables.size).to eq(3)
+  end
+
+  it 'should respond correctly when class is empty' do
+    my_path = File.expand_path(File.join('spec', 'fixtures', 'fixture_modules', 'zero_resource_module'))
+    my_retro = Retrospec.new(my_path)
+    test_type = my_retro.types.find {|x| x.name == 'empty_class'}
+    expect{TypeCode.new(test_type).variables}.to_not raise_error
+  end
+end

data/spec/unit/variable_store_spec.rb

@@ -0,0 +1,82 @@
+require 'spec_helper'
+
+describe "variable_store" do
+  after :all do
+    # enabling the removal slows down tests, but from time to time we may need to
+    FileUtils.rm_rf(fixture_modules_path) if ENV['RETROSPEC_CLEAN_UP_TEST_MODULES'] =~ /true/
+  end
+
+  before :all do
+    # enabling the removal of real modules slows down tests, but from time to time we may need to
+    FileUtils.rm_rf(fixture_modules_path) if ENV['RETROSPEC_CLEAN_UP_TEST_MODULES'] =~ /true/
+  end
+
+  before :each do
+    @my_path = File.expand_path(File.join('spec', 'fixtures', 'fixture_modules', 'one_resource_module'))
+  end
+
+  it 'should initialize' do
+    expect(VariableStore.instance.class).to eq(VariableStore)
+  end
+
+  it 'should resolve a string' do
+    my_retro = Retrospec.new(@my_path)
+    test_type = my_retro.types.find {|x| x.name == 'one_resource::another_resource'}
+    r = Resource.all(test_type).first
+    string1 = ObjectSpace.each_object(Puppet::Parser::AST::String).find {|x| x.to_s == "\"value1\"" }
+    expect(VariableStore.resolve(string1)).to eq("\"value1\"")
+  end
+
+  it 'should resolve a variable' do
+    my_retro = Retrospec.new(@my_path)
+    test_type = my_retro.types.find {|x| x.name == 'one_resource::another_resource'}
+    r = Resource.all(test_type).first
+    var1 = ObjectSpace.each_object(Puppet::Parser::AST::Variable).find {|x| x.to_s == '$file_name' }
+    expect(VariableStore.resolve(var1)).to eq("/tmp/test3")
+  end
+
+  it 'should resolve a basic vardef type' do
+    my_retro = Retrospec.new(@my_path)
+    test_type = my_retro.types.find {|x| x.name == 'one_resource::another_resource'}
+    VariableStore.populate(test_type)
+    vardef1 =  ObjectSpace.each_object(Puppet::Parser::AST::VarDef).find {|x| x.name.value == 'some_var'}
+    expect(VariableStore.resolve(vardef1)).to eq("oohhhh")
+  end
+
+  it 'should resolve a concat vardef type' do
+    my_retro = Retrospec.new(@my_path)
+    test_type = my_retro.types.find {|x| x.name == 'one_resource::another_resource'}
+    VariableStore.populate(test_type)
+    vardef2 =  ObjectSpace.each_object(Puppet::Parser::AST::VarDef).find {|x| x.name.value == 'concat_var'}
+    expect(VariableStore.resolve(vardef2)).to eq("/tmp/test3/test3183/oohhhh")
+  end
+
+  it 'should resolve vardef/concat with parameter value' do
+    my_retro = Retrospec.new(@my_path)
+    test_type = my_retro.types.find {|x| x.name == 'one_resource::another_resource'}
+    VariableStore.populate(test_type)
+    vardef3 =  ObjectSpace.each_object(Puppet::Parser::AST::VarDef).find {|x| x.name.value == 'cli_modules'}
+    expect(VariableStore.resolve(vardef3)).to eq("/etc/hammer/cli.modules.d")
+  end
+
+  describe '#add' do
+    it 'should store the value correctly' do
+      my_retro = Retrospec.new(@my_path)
+      test_type = my_retro.types.find {|x| x.name == 'one_resource::another_resource'}
+      string1 = ObjectSpace.each_object(Puppet::Parser::AST::String).find {|x| x.to_s == "\"value1\"" }
+      expect(VariableStore.add(string1, 'blah')).to eq("blah")
+      expect(VariableStore.lookup(string1)).to eq('blah')
+    end
+
+    it 'variable name should be stored in key with $' do
+      keys = VariableStore.instance.store.keys
+      expect(keys.include?('$var1')).to eq(true)
+    end
+
+    it 'should store vardef as is' do
+      my_retro = Retrospec.new(@my_path)
+      test_type = my_retro.types.find {|x| x.name == 'one_resource::another_resource'}
+      VariableStore.populate(test_type)
+    end
+  end
+end

metadata

@@ -1,139 +1,181 @@
 --- !ruby/object:Gem::Specification
 name: puppet-retrospec
 version: !ruby/object:Gem::Version
-  version: 0.4.0
+  version: 0.5.0
 platform: ruby
 authors:
 - Corey Osman
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-11-21 00:00:00.000000000 Z
+date: 2015-02-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: puppet
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - '='
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 3.7.3
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - '='
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 3.7.3
 - !ruby/object:Gem::Dependency
   name: trollop
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '2.14'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '2.14'
 - !ruby/object:Gem::Dependency
   name: yard
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '0.7'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '0.7'
 - !ruby/object:Gem::Dependency
   name: rdoc
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '3.12'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '3.12'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.0'
 - !ruby/object:Gem::Dependency
   name: jeweler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: pry
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: fakefs
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: pry-rescue
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: pry-stack_explorer
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: pry-coolline
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 description: Retrofits and generates valid puppet rspec test code to existing modules
@@ -145,46 +187,58 @@ extra_rdoc_files:
 - LICENSE
 - README.md
 files:
-- .document
-- .rspec
-- .travis.yml
+- ".document"
+- ".rspec"
+- ".travis.yml"
 - Gemfile
 - LICENSE
 - README.md
 - Rakefile
 - VERSION
 - bin/retrospec
-- lib/helpers.rb
-- lib/puppet-retrospec.rb
-- lib/templates/acceptance_spec_test.erb
-- lib/templates/fixtures_file.erb
-- lib/templates/gemfile.erb
-- lib/templates/nodesets/centos-59-x64.yml
-- lib/templates/nodesets/centos-64-x64-pe.yml
-- lib/templates/nodesets/centos-64-x64.yml
-- lib/templates/nodesets/centos-65-x64.yml
-- lib/templates/nodesets/debian-607-x64.yml
-- lib/templates/nodesets/debian-70rc1-x64.yml
-- lib/templates/nodesets/debian-73-i386.yml
-- lib/templates/nodesets/debian-73-x64.yml
-- lib/templates/nodesets/default.yml
-- lib/templates/nodesets/fedora-18-x64.yml
-- lib/templates/nodesets/sles-11sp1-x64.yml
-- lib/templates/nodesets/ubuntu-server-10044-x64.yml
-- lib/templates/nodesets/ubuntu-server-12042-x64.yml
-- lib/templates/nodesets/ubuntu-server-1310-x64.yml
-- lib/templates/nodesets/ubuntu-server-1404-x64.yml
-- lib/templates/rakefile.erb
-- lib/templates/resource_spec_file.erb
-- lib/templates/shared_context.erb
-- lib/templates/spec_helper_acceptance.rb.erb
-- lib/templates/spec_helper_file.erb
+- lib/retrospec.rb
+- lib/retrospec/conditional.rb
+- lib/retrospec/helpers.rb
+- lib/retrospec/resource.rb
+- lib/retrospec/templates/acceptance_spec_test.erb
+- lib/retrospec/templates/fixtures_file.erb
+- lib/retrospec/templates/gemfile.erb
+- lib/retrospec/templates/nodesets/centos-59-x64.yml
+- lib/retrospec/templates/nodesets/centos-64-x64-pe.yml
+- lib/retrospec/templates/nodesets/centos-64-x64.yml
+- lib/retrospec/templates/nodesets/centos-65-x64.yml
+- lib/retrospec/templates/nodesets/debian-607-x64.yml
+- lib/retrospec/templates/nodesets/debian-70rc1-x64.yml
+- lib/retrospec/templates/nodesets/debian-73-i386.yml
+- lib/retrospec/templates/nodesets/debian-73-x64.yml
+- lib/retrospec/templates/nodesets/default.yml
+- lib/retrospec/templates/nodesets/fedora-18-x64.yml
+- lib/retrospec/templates/nodesets/sles-11sp1-x64.yml
+- lib/retrospec/templates/nodesets/ubuntu-server-10044-x64.yml
+- lib/retrospec/templates/nodesets/ubuntu-server-12042-x64.yml
+- lib/retrospec/templates/nodesets/ubuntu-server-1310-x64.yml
+- lib/retrospec/templates/nodesets/ubuntu-server-1404-x64.yml
+- lib/retrospec/templates/rakefile.erb
+- lib/retrospec/templates/resource_spec_file.erb
+- lib/retrospec/templates/shared_context.erb
+- lib/retrospec/templates/spec_helper_acceptance.rb.erb
+- lib/retrospec/templates/spec_helper_file.erb
+- lib/retrospec/type_code.rb
+- lib/retrospec/variable_store.rb
+- lib/retrospec/version.rb
 - puppet-retrospec.gemspec
+- spec/fixtures/fixture_modules/one_resource_module/manifests/another_resource_class.pp
+- spec/fixtures/fixture_modules/one_resource_module/manifests/one_resource_class.pp
+- spec/fixtures/fixture_modules/zero_resource_module/manifests/empty_class.pp
 - spec/fixtures/fixture_modules/zero_resource_module/manifests/not_a_resource_defination.pp
 - spec/fixtures/manifests/includes-class.pp
 - spec/fixtures/manifests/includes-defines.pp
 - spec/spec_helper.rb
+- spec/unit/conditional_spec.rb
 - spec/unit/puppet-retrospec_spec.rb
+- spec/unit/resource_spec.rb
+- spec/unit/type_code_spec.rb
+- spec/unit/variable_store_spec.rb
 homepage: http://github.com/logicminds/puppet-retrospec
 licenses:
 - MIT
@@ -195,17 +249,17 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.4
+rubygems_version: 2.4.5
 signing_key: 
 specification_version: 4
 summary: Generates puppet rspec test code based on the classes and defines inside