r10k 3.15.0 → 3.15.2

Sign up to get free protection for your applications and to get access to all the features.
Files changed (160) hide show
  1. checksums.yaml +4 -4
  2. data/CHANGELOG.mkd +9 -0
  3. data/CODEOWNERS +2 -2
  4. data/README.mkd +11 -0
  5. data/doc/dynamic-environments/configuration.mkd +65 -1
  6. data/lib/r10k/action/puppetfile/install.rb +2 -1
  7. data/lib/r10k/cli/puppetfile.rb +1 -0
  8. data/lib/r10k/module_loader/puppetfile.rb +10 -1
  9. data/lib/r10k/version.rb +1 -1
  10. data/r10k.gemspec +1 -1
  11. metadata +2 -151
  12. data/spec/fixtures/empty/.empty +0 -0
  13. data/spec/fixtures/integration/git/puppet-boolean-bare.tar +0 -0
  14. data/spec/fixtures/module/forge/bad_module/metadata.json +0 -1
  15. data/spec/fixtures/module/forge/eight_hundred/Modulefile +0 -8
  16. data/spec/fixtures/module/forge/eight_hundred/metadata.json +0 -19
  17. data/spec/fixtures/tarball/tarball.tar.gz +0 -0
  18. data/spec/fixtures/unit/action/r10k.yaml +0 -5
  19. data/spec/fixtures/unit/action/r10k_cachedir.yaml +0 -2
  20. data/spec/fixtures/unit/action/r10k_creds.yaml +0 -9
  21. data/spec/fixtures/unit/action/r10k_forge_auth.yaml +0 -4
  22. data/spec/fixtures/unit/action/r10k_forge_auth_no_url.yaml +0 -3
  23. data/spec/fixtures/unit/action/r10k_generate_types.yaml +0 -3
  24. data/spec/fixtures/unit/action/r10k_logging.yaml +0 -12
  25. data/spec/fixtures/unit/action/r10k_puppet_path.yaml +0 -3
  26. data/spec/fixtures/unit/puppetfile/argument-error/Puppetfile +0 -1
  27. data/spec/fixtures/unit/puppetfile/default-branch-override/Puppetfile +0 -5
  28. data/spec/fixtures/unit/puppetfile/duplicate-module-error/Puppetfile +0 -10
  29. data/spec/fixtures/unit/puppetfile/forge-override/Puppetfile +0 -8
  30. data/spec/fixtures/unit/puppetfile/invalid-syntax/Puppetfile +0 -1
  31. data/spec/fixtures/unit/puppetfile/load-error/Puppetfile +0 -1
  32. data/spec/fixtures/unit/puppetfile/name-error/Puppetfile +0 -1
  33. data/spec/fixtures/unit/puppetfile/valid-forge-with-version/Puppetfile +0 -1
  34. data/spec/fixtures/unit/puppetfile/valid-forge-without-version/Puppetfile +0 -1
  35. data/spec/fixtures/unit/puppetfile/various-modules/Puppetfile +0 -10
  36. data/spec/fixtures/unit/puppetfile/various-modules/Puppetfile.new +0 -10
  37. data/spec/fixtures/unit/puppetfile/various-modules/modules/apt/.gitkeep +0 -1
  38. data/spec/fixtures/unit/puppetfile/various-modules/modules/baz/.gitkeep +0 -1
  39. data/spec/fixtures/unit/puppetfile/various-modules/modules/buzz/.gitkeep +0 -1
  40. data/spec/fixtures/unit/puppetfile/various-modules/modules/canary/.gitkeep +0 -1
  41. data/spec/fixtures/unit/puppetfile/various-modules/modules/fizz/.gitkeep +0 -1
  42. data/spec/fixtures/unit/puppetfile/various-modules/modules/rpm/.gitkeep +0 -1
  43. data/spec/fixtures/unit/util/purgeable/managed_one/expected_1 +0 -0
  44. data/spec/fixtures/unit/util/purgeable/managed_one/managed_subdir_1/managed_symlink_file +0 -1
  45. data/spec/fixtures/unit/util/purgeable/managed_one/managed_subdir_1/subdir_allowlisted_2/ignored_1 +0 -0
  46. data/spec/fixtures/unit/util/purgeable/managed_one/managed_subdir_1/subdir_expected_1 +0 -0
  47. data/spec/fixtures/unit/util/purgeable/managed_one/managed_subdir_1/subdir_unmanaged_1 +0 -0
  48. data/spec/fixtures/unit/util/purgeable/managed_one/managed_subdir_1/unmanaged_symlink_dir +0 -1
  49. data/spec/fixtures/unit/util/purgeable/managed_one/managed_symlink_dir +0 -1
  50. data/spec/fixtures/unit/util/purgeable/managed_one/unmanaged_1 +0 -0
  51. data/spec/fixtures/unit/util/purgeable/managed_one/unmanaged_symlink_file +0 -1
  52. data/spec/fixtures/unit/util/purgeable/managed_two/.hidden/unmanaged_3 +0 -0
  53. data/spec/fixtures/unit/util/purgeable/managed_two/expected_2 +0 -0
  54. data/spec/fixtures/unit/util/purgeable/managed_two/unmanaged_2 +0 -0
  55. data/spec/fixtures/unit/util/subprocess/runner/no-execute.sh +0 -3
  56. data/spec/integration/git/rugged/bare_repository_spec.rb +0 -13
  57. data/spec/integration/git/rugged/cache_spec.rb +0 -33
  58. data/spec/integration/git/rugged/thin_repository_spec.rb +0 -14
  59. data/spec/integration/git/rugged/working_repository_spec.rb +0 -48
  60. data/spec/integration/git/shellgit/bare_repository_spec.rb +0 -13
  61. data/spec/integration/git/shellgit/thin_repository_spec.rb +0 -14
  62. data/spec/integration/git/shellgit/working_repository_spec.rb +0 -13
  63. data/spec/integration/git/stateful_repository_spec.rb +0 -175
  64. data/spec/integration/util/purageable_spec.rb +0 -41
  65. data/spec/matchers/exit_with.rb +0 -28
  66. data/spec/matchers/match_realpath.rb +0 -18
  67. data/spec/r10k-mocks/mock_config.rb +0 -33
  68. data/spec/r10k-mocks/mock_env.rb +0 -18
  69. data/spec/r10k-mocks/mock_source.rb +0 -17
  70. data/spec/r10k-mocks.rb +0 -3
  71. data/spec/shared-contexts/git-fixtures.rb +0 -55
  72. data/spec/shared-contexts/tarball.rb +0 -32
  73. data/spec/shared-examples/deploy-actions.rb +0 -69
  74. data/spec/shared-examples/git/bare_repository.rb +0 -132
  75. data/spec/shared-examples/git/thin_repository.rb +0 -26
  76. data/spec/shared-examples/git/working_repository.rb +0 -207
  77. data/spec/shared-examples/git-repository.rb +0 -38
  78. data/spec/shared-examples/puppetfile-action.rb +0 -39
  79. data/spec/shared-examples/settings/ancestry.rb +0 -44
  80. data/spec/shared-examples/subprocess-runner.rb +0 -89
  81. data/spec/spec_helper.rb +0 -41
  82. data/spec/unit/action/cri_runner_spec.rb +0 -72
  83. data/spec/unit/action/deploy/deploy_helpers_spec.rb +0 -38
  84. data/spec/unit/action/deploy/display_spec.rb +0 -61
  85. data/spec/unit/action/deploy/environment_spec.rb +0 -640
  86. data/spec/unit/action/deploy/module_spec.rb +0 -476
  87. data/spec/unit/action/puppetfile/check_spec.rb +0 -53
  88. data/spec/unit/action/puppetfile/cri_runner_spec.rb +0 -47
  89. data/spec/unit/action/puppetfile/install_spec.rb +0 -112
  90. data/spec/unit/action/puppetfile/purge_spec.rb +0 -60
  91. data/spec/unit/action/runner_spec.rb +0 -407
  92. data/spec/unit/action/visitor_spec.rb +0 -39
  93. data/spec/unit/cli_spec.rb +0 -9
  94. data/spec/unit/deployment/config_spec.rb +0 -33
  95. data/spec/unit/deployment_spec.rb +0 -162
  96. data/spec/unit/environment/bare_spec.rb +0 -13
  97. data/spec/unit/environment/base_spec.rb +0 -122
  98. data/spec/unit/environment/git_spec.rb +0 -114
  99. data/spec/unit/environment/name_spec.rb +0 -181
  100. data/spec/unit/environment/plain_spec.rb +0 -8
  101. data/spec/unit/environment/svn_spec.rb +0 -146
  102. data/spec/unit/environment/tarball_spec.rb +0 -45
  103. data/spec/unit/environment/with_modules_spec.rb +0 -122
  104. data/spec/unit/errors/formatting_spec.rb +0 -84
  105. data/spec/unit/feature_spec.rb +0 -50
  106. data/spec/unit/forge/module_release_spec.rb +0 -213
  107. data/spec/unit/git/alternates_spec.rb +0 -116
  108. data/spec/unit/git/cache_spec.rb +0 -66
  109. data/spec/unit/git/rugged/cache_spec.rb +0 -48
  110. data/spec/unit/git/rugged/credentials_spec.rb +0 -215
  111. data/spec/unit/git/shellgit/cache_spec.rb +0 -27
  112. data/spec/unit/git/stateful_repository_spec.rb +0 -45
  113. data/spec/unit/git_spec.rb +0 -102
  114. data/spec/unit/initializers_spec.rb +0 -68
  115. data/spec/unit/instance_cache_spec.rb +0 -78
  116. data/spec/unit/keyed_factory_spec.rb +0 -51
  117. data/spec/unit/logging/terminaloutputter_spec.rb +0 -53
  118. data/spec/unit/logging_spec.rb +0 -68
  119. data/spec/unit/module/base_spec.rb +0 -118
  120. data/spec/unit/module/forge_spec.rb +0 -271
  121. data/spec/unit/module/git_spec.rb +0 -387
  122. data/spec/unit/module/metadata_file_spec.rb +0 -68
  123. data/spec/unit/module/svn_spec.rb +0 -208
  124. data/spec/unit/module/tarball_spec.rb +0 -70
  125. data/spec/unit/module_loader/puppetfile_spec.rb +0 -421
  126. data/spec/unit/module_spec.rb +0 -114
  127. data/spec/unit/puppetfile_spec.rb +0 -304
  128. data/spec/unit/settings/collection_spec.rb +0 -123
  129. data/spec/unit/settings/container_spec.rb +0 -92
  130. data/spec/unit/settings/definition_spec.rb +0 -79
  131. data/spec/unit/settings/enum_definition_spec.rb +0 -20
  132. data/spec/unit/settings/inheritance_spec.rb +0 -38
  133. data/spec/unit/settings/list_spec.rb +0 -88
  134. data/spec/unit/settings/loader_spec.rb +0 -110
  135. data/spec/unit/settings/uri_definition_spec.rb +0 -23
  136. data/spec/unit/settings_spec.rb +0 -303
  137. data/spec/unit/source/base_spec.rb +0 -31
  138. data/spec/unit/source/exec_spec.rb +0 -81
  139. data/spec/unit/source/git_spec.rb +0 -233
  140. data/spec/unit/source/hash_spec.rb +0 -54
  141. data/spec/unit/source/svn_spec.rb +0 -196
  142. data/spec/unit/source/yaml_spec.rb +0 -42
  143. data/spec/unit/source_spec.rb +0 -10
  144. data/spec/unit/svn/remote_spec.rb +0 -21
  145. data/spec/unit/svn/working_dir_spec.rb +0 -56
  146. data/spec/unit/tarball_spec.rb +0 -57
  147. data/spec/unit/util/attempt_spec.rb +0 -82
  148. data/spec/unit/util/cacheable_spec.rb +0 -23
  149. data/spec/unit/util/commands_spec.rb +0 -61
  150. data/spec/unit/util/downloader_spec.rb +0 -98
  151. data/spec/unit/util/exec_env_spec.rb +0 -56
  152. data/spec/unit/util/purgeable_spec.rb +0 -267
  153. data/spec/unit/util/setopts_spec.rb +0 -83
  154. data/spec/unit/util/subprocess/result_spec.rb +0 -36
  155. data/spec/unit/util/subprocess/runner/posix_spec.rb +0 -7
  156. data/spec/unit/util/subprocess/runner/pump_spec.rb +0 -79
  157. data/spec/unit/util/subprocess/runner/windows_spec.rb +0 -7
  158. data/spec/unit/util/subprocess/subprocess_error_spec.rb +0 -26
  159. data/spec/unit/util/subprocess_spec.rb +0 -65
  160. data/spec/unit/util/symbolize_keys_spec.rb +0 -67
@@ -1,162 +0,0 @@
1
- require 'spec_helper'
2
- require 'r10k/deployment'
3
- require 'tmpdir'
4
-
5
- describe R10K::Deployment do
6
-
7
- let(:confdir) { Dir.mktmpdir }
8
-
9
- let(:config) do
10
- R10K::Deployment::MockConfig.new(
11
- :sources => {
12
- :control => {
13
- :type => :mock,
14
- :basedir => File.join(confdir, 'environments'),
15
- :environments => %w[first second third],
16
- },
17
- :hiera => {
18
- :type => :mock,
19
- :basedir => File.join(confdir, 'hiera'),
20
- :environments => %w[fourth fifth sixth],
21
- }
22
- }
23
- )
24
- end
25
-
26
- subject(:deployment) { described_class.new(config) }
27
-
28
- let(:control) { deployment.sources.find { |source| source.name == :control } }
29
- let(:hiera) { deployment.sources.find { |source| source.name == :hiera } }
30
-
31
- describe "loading" do
32
- describe "sources" do
33
- it "creates a source for each key in the ':sources' config entry" do
34
- expect(control.basedir).to eq(File.join(confdir, 'environments'))
35
- expect(hiera.basedir).to eq(File.join(confdir, 'hiera'))
36
- end
37
- end
38
-
39
- describe "loading environments" do
40
- it "loads environments from each source" do
41
- %w[first second third fourth fifth sixth].each do |env|
42
- expect(deployment.environments.map(&:name)).to include(env)
43
- end
44
- end
45
- end
46
- end
47
-
48
- describe "preloading" do
49
- it "invokes #preload! on each source" do
50
- deployment.sources.each do |source|
51
- expect(source).to receive(:preload!)
52
- end
53
- deployment.preload!
54
- end
55
- end
56
-
57
- describe "paths" do
58
- it "retrieves the path for each source" do
59
- expect(deployment.paths).to include(File.join(confdir, 'environments'))
60
- expect(deployment.paths).to include(File.join(confdir, 'hiera'))
61
- end
62
- end
63
-
64
- describe "paths and sources" do
65
- it "retrieves the path for each source" do
66
- p_a_s = deployment.paths_and_sources
67
-
68
- expect(p_a_s[File.join(confdir, 'environments')]).to eq([control])
69
- expect(p_a_s[File.join(confdir, 'hiera')]).to eq([hiera])
70
- end
71
- end
72
-
73
- describe "purging" do
74
- it "purges each managed directory" do
75
- env_basedir = double("basedir environments")
76
- hiera_basedir = double("basedir hiera")
77
-
78
- expect(env_basedir).to receive(:purge!)
79
- expect(hiera_basedir).to receive(:purge!)
80
-
81
- expect(R10K::Util::Basedir).to receive(:new).with(File.join(confdir, 'environments'), [control]).and_return(env_basedir)
82
- expect(R10K::Util::Basedir).to receive(:new).with(File.join(confdir, 'hiera'), [hiera]).and_return(hiera_basedir)
83
-
84
- deployment.purge!
85
- end
86
- end
87
-
88
- describe "accepting a visitor" do
89
- it "passes itself to the visitor" do
90
- visitor = spy('visitor')
91
- expect(visitor).to receive(:visit).with(:deployment, subject)
92
- subject.accept(visitor)
93
- end
94
-
95
- it "passes the visitor to each environment if the visitor yields" do
96
- visitor = spy('visitor')
97
- expect(visitor).to receive(:visit) do |type, other, &block|
98
- expect(type).to eq :deployment
99
- expect(other).to eq subject
100
- block.call
101
- end
102
-
103
- source1 = spy('source')
104
- expect(source1).to receive(:accept).with(visitor)
105
- source2 = spy('source')
106
- expect(source2).to receive(:accept).with(visitor)
107
-
108
- expect(subject).to receive(:sources).and_return([source1, source2])
109
- subject.accept(visitor)
110
- end
111
- end
112
- end
113
-
114
- describe R10K::Deployment, "with environment collisions" do
115
-
116
- let(:confdir) { Dir.mktmpdir }
117
-
118
- let(:config) do
119
- R10K::Deployment::MockConfig.new(
120
- :sources => {
121
- :s1 => {
122
- :type => :mock,
123
- :basedir => File.join(confdir, 'environments'),
124
- :environments => %w[first second third],
125
- },
126
- :s2 => {
127
- :type => :mock,
128
- :basedir => File.join(confdir, 'environments'),
129
- :environments => %w[third fourth fifth],
130
- }
131
- }
132
- )
133
- end
134
-
135
- subject(:deployment) { described_class.new(config) }
136
-
137
- it "raises an error when validating" do
138
- expect {
139
- deployment.validate!
140
- }.to raise_error(R10K::Error, /Environment collision at .* between s\d:third and s\d:third/)
141
- end
142
- end
143
-
144
- describe R10K::Deployment, "checking the 'sources' key" do
145
-
146
- {
147
- "when missing" => {},
148
- "when empty" => {:sources => []},
149
- }.each_pair do |desc, config_hash|
150
- describe desc do
151
-
152
- let(:config) { R10K::Deployment::MockConfig.new(config_hash) }
153
- subject(:deployment) { described_class.new(config) }
154
-
155
- it "raises an error when enumerating sources" do
156
- expect {
157
- deployment.sources
158
- }.to raise_error(R10K::Error, "Unable to load sources; the supplied configuration does not define the 'sources' key")
159
- end
160
- end
161
- end
162
- end
@@ -1,13 +0,0 @@
1
- require 'spec_helper'
2
- require 'r10k/environment'
3
-
4
- describe R10K::Environment::Bare do
5
- it "warns on initialization" do
6
- logger_spy = spy('logger')
7
- allow_any_instance_of(described_class).to receive(:logger).and_return(logger_spy)
8
-
9
- described_class.new('envname', '/basedir', 'dirname', {})
10
-
11
- expect(logger_spy).to have_received(:warn).with(%r{deprecated.*envname})
12
- end
13
- end
@@ -1,122 +0,0 @@
1
- require 'spec_helper'
2
- require 'r10k/environment'
3
-
4
- describe R10K::Environment::Base do
5
-
6
- let(:basepath) { '/some/imaginary/path' }
7
- let(:envname) { 'env_name' }
8
- let(:path) { File.join(basepath, envname) }
9
- subject(:environment) { described_class.new('envname', basepath, envname, {}) }
10
-
11
- it "can return the fully qualified path" do
12
- expect(environment.path).to eq(Pathname.new(path))
13
- end
14
-
15
- it "raises an exception when #sync is called" do
16
- expect { environment.sync }.to raise_error(NotImplementedError)
17
- end
18
-
19
- describe "accepting a visitor" do
20
- it "passes itself to the visitor" do
21
- visitor = spy('visitor')
22
- expect(visitor).to receive(:visit).with(:environment, subject)
23
- subject.accept(visitor)
24
- end
25
-
26
- it "passes the visitor to the puppetfile if the visitor yields" do
27
- visitor = spy('visitor')
28
- expect(visitor).to receive(:visit) do |type, other, &block|
29
- expect(type).to eq :environment
30
- expect(other).to eq subject
31
- block.call
32
- end
33
-
34
- pf = spy('puppetfile')
35
- expect(pf).to receive(:accept).with(visitor)
36
-
37
- expect(subject).to receive(:puppetfile).and_return(pf)
38
- subject.accept(visitor)
39
- end
40
- end
41
-
42
- describe "#whitelist" do
43
- let(:whitelist) do
44
- ['**/*.xpp', 'custom', '*.tmp']
45
- end
46
-
47
- it "combines given patterns with full_path to env" do
48
- expect(subject.whitelist(whitelist)).to all(start_with(subject.path.to_s))
49
- end
50
- end
51
-
52
- describe "#purge_exclusions" do
53
- let(:mock_env) { instance_double("R10K::Environment::Base") }
54
- let(:mock_puppetfile) { instance_double("R10K::Puppetfile", :environment= => true, :environment => mock_env) }
55
- let(:loader) do
56
- instance_double("R10K::ModuleLoader::Puppetfile",
57
- :environment= => nil,
58
- :load => { :modules => @modules,
59
- :managed_directories => @managed_dirs,
60
- :desired_contents => @desired_contents,
61
- :purge_exclusions => @purge_ex })
62
- end
63
-
64
- before(:each) do
65
- @modules = []
66
- @managed_dirs = []
67
- @desired_contents = []
68
- @purge_exclusions = []
69
- end
70
-
71
- it "excludes .r10k-deploy.json" do
72
- allow(R10K::ModuleLoader::Puppetfile).to receive(:new).and_return(loader)
73
- subject.deploy
74
-
75
- expect(subject.purge_exclusions).to include(/r10k-deploy\.json/)
76
- end
77
-
78
- it "excludes puppetfile managed directories" do
79
- @managed_dirs = [
80
- '/some/imaginary/path/env_name/modules',
81
- '/some/imaginary/path/env_name/data',
82
- ]
83
-
84
- allow(R10K::ModuleLoader::Puppetfile).to receive(:new).and_return(loader)
85
- subject.deploy
86
-
87
- exclusions = subject.purge_exclusions
88
-
89
- @managed_dirs.each do |dir|
90
- expect(exclusions).to include(dir)
91
- end
92
- end
93
-
94
- describe "puppetfile desired contents" do
95
-
96
- before(:each) do
97
- @desired_contents = [ 'modules/apache', 'data/local/site' ].collect do |c|
98
- File.join(path, c)
99
- end
100
-
101
- allow(File).to receive(:directory?).and_return true
102
- allow(R10K::ModuleLoader::Puppetfile).to receive(:new).and_return(loader)
103
- subject.deploy
104
- end
105
-
106
- it "excludes desired directory contents with glob" do
107
- exclusions = subject.purge_exclusions
108
-
109
- expect(exclusions).to include(/#{Regexp.escape(File.join('apache', '**', '*'))}$/)
110
- expect(exclusions).to include(/#{Regexp.escape(File.join('site', '**', '*'))}$/)
111
- end
112
-
113
- it "excludes ancestors of desired directories" do
114
- exclusions = subject.purge_exclusions
115
-
116
- expect(exclusions).to include(/modules$/)
117
- expect(exclusions).to include(/data\/local$/)
118
- expect(exclusions).to include(/data$/)
119
- end
120
- end
121
- end
122
- end
@@ -1,114 +0,0 @@
1
- require 'spec_helper'
2
- require 'r10k/environment'
3
-
4
- describe R10K::Environment::Git do
5
-
6
- subject do
7
- described_class.new(
8
- 'myenv',
9
- '/some/nonexistent/environmentdir',
10
- 'gitref',
11
- {
12
- :remote => 'https://git-server.site/my-repo.git',
13
- :ref => 'd026ea677116424d2968edb9cee8cbc24d09322b',
14
- }
15
- )
16
- end
17
-
18
- describe "initializing" do
19
- subject do
20
- described_class.new('name', '/dir', 'ref', {
21
- :remote => 'url',
22
- :ref => 'value',
23
- :puppetfile_name => 'Puppetfile',
24
- :moduledir => 'modules',
25
- :modules => { },
26
- })
27
- end
28
-
29
- it "accepts valid base class initialization arguments" do
30
- expect(subject.name).to eq 'name'
31
- end
32
- end
33
-
34
- describe "storing attributes" do
35
- it "can return the environment name" do
36
- expect(subject.name).to eq 'myenv'
37
- end
38
-
39
- it "can return the environment basedir" do
40
- expect(subject.basedir).to eq '/some/nonexistent/environmentdir'
41
- end
42
-
43
- it "can return the environment dirname" do
44
- expect(subject.dirname).to eq 'gitref'
45
- end
46
-
47
- it "can return the environment remote" do
48
- expect(subject.remote).to eq 'https://git-server.site/my-repo.git'
49
- end
50
-
51
- it "can return the environment ref" do
52
- expect(subject.ref).to eq 'd026ea677116424d2968edb9cee8cbc24d09322b'
53
- end
54
- end
55
-
56
- describe "synchronizing the environment" do
57
- it "syncs the git repository" do
58
- expect(subject.repo).to receive(:sync)
59
- subject.sync
60
- end
61
- end
62
-
63
- describe "generating a puppetfile for the environment" do
64
- let(:puppetfile) { subject.puppetfile }
65
-
66
- it "creates a puppetfile at the full path to the environment" do
67
- expect(puppetfile.basedir).to eq '/some/nonexistent/environmentdir/gitref'
68
- end
69
-
70
- it "sets the moduledir to 'modules' relative to the environment path" do
71
- expect(puppetfile.moduledir).to eq '/some/nonexistent/environmentdir/gitref/modules'
72
- end
73
-
74
- it "sets the puppetfile path to 'Puppetfile' relative to the environment path" do
75
- expect(puppetfile.puppetfile_path).to eq '/some/nonexistent/environmentdir/gitref/Puppetfile'
76
- end
77
- end
78
-
79
- describe "enumerating modules" do
80
- it "loads the Puppetfile and returns modules in that puppetfile" do
81
- loaded = { desired_contents: [], managed_directories: [], purge_exclusions: [] }
82
- mod = double('A module', :name => 'dbl')
83
- expect(subject.loader).to receive(:load).and_return(loaded.merge(modules: [mod]))
84
- expect(subject.modules).to eq([mod])
85
- end
86
- end
87
-
88
- describe "determining the status" do
89
- it "delegates to the repo" do
90
- expect(subject.repo).to receive(:status).and_return :some_status
91
- expect(subject.status).to eq :some_status
92
- end
93
- end
94
-
95
- describe "environment signature" do
96
- it "returns the git sha of HEAD" do
97
- expect(subject.repo).to receive(:head).and_return 'f00b00'
98
- expect(subject.signature).to eq 'f00b00'
99
- end
100
- end
101
-
102
- describe "info hash" do
103
- let(:info_hash) { subject.info }
104
-
105
- before(:each) do
106
- allow(subject.repo).to receive(:head).and_return 'f00b00'
107
- end
108
-
109
- it "includes name and signature" do
110
- expect(info_hash.keys).to include :name, :signature
111
- expect(info_hash).not_to have_value(nil)
112
- end
113
- end
114
- end
@@ -1,181 +0,0 @@
1
- require 'spec_helper'
2
- require 'r10k/environment/name'
3
-
4
- describe R10K::Environment::Name do
5
- describe "strip_component" do
6
- it "does not modify the given name when no strip_component is given" do
7
- bn = described_class.new('myenv', source: 'source', prefix: false)
8
- expect(bn.dirname).to eq 'myenv'
9
- expect(bn.name).to eq 'myenv'
10
- expect(bn.original_name).to eq 'myenv'
11
- end
12
-
13
- it "removes the first occurance of a regex match when a regex is given" do
14
- bn = described_class.new('myenv', source: 'source', prefix: false, strip_component: '/env/')
15
- expect(bn.dirname).to eq 'my'
16
- expect(bn.name).to eq 'my'
17
- expect(bn.original_name).to eq 'myenv'
18
- end
19
-
20
- it "does not modify the given name when there is no regex match" do
21
- bn = described_class.new('myenv', source: 'source', prefix: false, strip_component: '/bar/')
22
- expect(bn.dirname).to eq 'myenv'
23
- expect(bn.name).to eq 'myenv'
24
- expect(bn.original_name).to eq 'myenv'
25
- end
26
-
27
- it "removes the given name's prefix when it matches strip_component" do
28
- bn = described_class.new('env/prod', source: 'source', prefix: false, strip_component: 'env/')
29
- expect(bn.dirname).to eq 'prod'
30
- expect(bn.name).to eq 'prod'
31
- expect(bn.original_name).to eq 'env/prod'
32
- end
33
-
34
- it "raises an error when given an integer" do
35
- expect {
36
- described_class.new('env/prod', source: 'source', prefix: false, strip_component: 4)
37
- }.to raise_error(%r{Improper.*"4"})
38
- end
39
- end
40
-
41
- describe "prefixing" do
42
- it "uses the branch name as the dirname when prefixing is off" do
43
- bn = described_class.new('mybranch', :source => 'source', :prefix => false)
44
- expect(bn.dirname).to eq 'mybranch'
45
- expect(bn.name).to eq 'mybranch'
46
- expect(bn.original_name).to eq 'mybranch'
47
- end
48
-
49
- it "prepends the source name when prefixing is on" do
50
- bn = described_class.new('mybranch', :source => 'source', :prefix => true)
51
- expect(bn.dirname).to eq 'source_mybranch'
52
- expect(bn.name).to eq 'mybranch'
53
- expect(bn.original_name).to eq 'mybranch'
54
- end
55
-
56
- it "prepends the prefix name when prefixing is overridden" do
57
- bn = described_class.new('mybranch', {:prefix => "bar", :sourcename => 'foo'})
58
- expect(bn.dirname).to eq 'bar_mybranch'
59
- expect(bn.name).to eq 'mybranch'
60
- expect(bn.original_name).to eq 'mybranch'
61
- end
62
-
63
- it "uses the branch name as the dirname when prefixing is nil" do
64
- bn = described_class.new('mybranch', {:prefix => nil, :sourcename => 'foo'})
65
- expect(bn.dirname).to eq 'mybranch'
66
- expect(bn.name).to eq 'mybranch'
67
- expect(bn.original_name).to eq 'mybranch'
68
- end
69
- end
70
-
71
- describe "determining the validate behavior with :invalid" do
72
- [
73
- ['correct_and_warn', {:validate => true, :correct => true}],
74
- ['correct', {:validate => false, :correct => true}],
75
- ['error', {:validate => true, :correct => false}],
76
- ].each do |(setting, outcome)|
77
- it "treats #{setting} as #{outcome.inspect}" do
78
- bn = described_class.new('mybranch', :source => 'source', :invalid => setting)
79
- expect(bn.validate?).to eq outcome[:validate]
80
- expect(bn.correct?).to eq outcome[:correct]
81
- end
82
- end
83
- end
84
-
85
- describe "determining if a branch is a valid environment name" do
86
- invalid_cases = [
87
- 'hyphenated-branch',
88
- 'dotted.branch',
89
- 'slashed/branch',
90
- 'at@branch',
91
- 'http://branch'
92
- ]
93
-
94
- valid_cases = [
95
- 'my_branchname',
96
- 'my_issue_346',
97
- ]
98
-
99
- describe "and validate is false" do
100
- invalid_cases.each do |branch|
101
- it "is valid if the branch is #{branch}" do
102
- bn = described_class.new(branch, {:validate => false})
103
- expect(bn).to be_valid
104
- end
105
- end
106
-
107
- valid_cases.each do |branch|
108
- it "is valid if the branch is #{branch}" do
109
- bn = described_class.new(branch, {:validate => false})
110
- expect(bn).to be_valid
111
- end
112
- end
113
- end
114
-
115
- describe "and validate is true" do
116
- invalid_cases.each do |branch|
117
- it "is invalid if the branch is #{branch}" do
118
- bn = described_class.new(branch, {:validate => true})
119
- expect(bn).to_not be_valid
120
- end
121
- end
122
-
123
- valid_cases.each do |branch|
124
- it "is valid if the branch is #{branch}" do
125
- bn = described_class.new(branch, {:validate => true})
126
- expect(bn).to be_valid
127
- end
128
- end
129
-
130
- end
131
- end
132
-
133
- describe "correcting branch names" do
134
- invalid_cases = [
135
- 'hyphenated-branch',
136
- 'dotted.branch',
137
- 'slashed/branch',
138
- 'at@branch',
139
- 'http://branch'
140
- ]
141
-
142
- valid_cases = [
143
- 'my_branchname',
144
- 'my_issue_346',
145
- ]
146
-
147
- describe "and correct is false" do
148
- invalid_cases.each do |branch|
149
- it "doesn't modify #{branch}" do
150
- bn = described_class.new(branch.dup, {:correct => false})
151
- expect(bn.dirname).to eq branch
152
- end
153
- end
154
-
155
- valid_cases.each do |branch|
156
- it "doesn't modify #{branch}" do
157
- bn = described_class.new(branch.dup, {:correct => false})
158
- expect(bn.dirname).to eq branch
159
- end
160
- end
161
- end
162
-
163
- describe "and correct is true" do
164
- invalid_cases.each do |branch|
165
- it "replaces invalid characters in #{branch} with underscores" do
166
- bn = described_class.new(branch.dup, {:correct => true})
167
- expect(bn.dirname).to eq branch.gsub(/\W/, '_')
168
- expect(bn.name).to eq branch
169
- expect(bn.original_name).to eq branch
170
- end
171
- end
172
-
173
- valid_cases.each do |branch|
174
- it "doesn't modify #{branch}" do
175
- bn = described_class.new(branch.dup, {:correct => true})
176
- expect(bn.dirname).to eq branch
177
- end
178
- end
179
- end
180
- end
181
- end
@@ -1,8 +0,0 @@
1
- require 'spec_helper'
2
- require 'r10k/environment'
3
-
4
- describe R10K::Environment::Plain do
5
- it "initializes successfully" do
6
- expect(described_class.new('envname', '/basedir', 'dirname', {})).to be_a_kind_of(described_class)
7
- end
8
- end