1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
|
require 'spec_helper'
require 'puppet_spec/compiler'
require 'matchers/resource'
describe 'node statements' do
include PuppetSpec::Compiler
include Matchers::Resource
context 'nodes' do
it 'selects a node where the name is just a number' do
# Future parser doesn't allow a number in this position
catalog = compile_to_catalog(<<-MANIFEST, Puppet::Node.new("5"))
node 5 { notify { 'matched': } }
MANIFEST
expect(catalog).to have_resource('Notify[matched]')
end
it 'selects the node with a matching name' do
catalog = compile_to_catalog(<<-MANIFEST, Puppet::Node.new("nodename"))
node noden {}
node nodename { notify { matched: } }
node name {}
MANIFEST
expect(catalog).to have_resource('Notify[matched]')
end
it 'prefers a node with a literal name over one with a regex' do
catalog = compile_to_catalog(<<-MANIFEST, Puppet::Node.new("nodename"))
node /noden.me/ { notify { ignored: } }
node nodename { notify { matched: } }
MANIFEST
expect(catalog).to have_resource('Notify[matched]')
end
it 'selects a node where one of the names matches' do
catalog = compile_to_catalog(<<-MANIFEST, Puppet::Node.new("nodename"))
node different, nodename, other { notify { matched: } }
MANIFEST
expect(catalog).to have_resource('Notify[matched]')
end
it 'arbitrarily selects one of the matching nodes' do
catalog = compile_to_catalog(<<-MANIFEST, Puppet::Node.new("nodename"))
node /not/ { notify { 'is not matched': } }
node /name.*/ { notify { 'could be matched': } }
node /na.e/ { notify { 'could also be matched': } }
MANIFEST
expect([catalog.resource('Notify[could be matched]'), catalog.resource('Notify[could also be matched]')].compact).to_not be_empty
end
it 'selects a node where one of the names matches with a mixture of literals and regex' do
catalog = compile_to_catalog(<<-MANIFEST, Puppet::Node.new("nodename"))
node different, /name/, other { notify { matched: } }
MANIFEST
expect(catalog).to have_resource('Notify[matched]')
end
it 'that have regex names should not collide with matching class names' do
catalog = compile_to_catalog(<<-MANIFEST, Puppet::Node.new("foo"))
class foo {
$bar = 'one'
}
node /foo/ {
$bar = 'two'
include foo
notify{"${::foo::bar}":}
}
MANIFEST
expect(catalog).to have_resource('Notify[one]')
end
it 'does not raise an error with regex and non-regex node names are the same' do
expect do
compile_to_catalog(<<-MANIFEST)
node /a.*(c)?/ { }
node 'a.c' { }
MANIFEST
end.not_to raise_error
end
it 'errors when two nodes with regexes collide after some regex syntax is removed' do
expect do
compile_to_catalog(<<-MANIFEST)
node /a.*(c)?/ { }
node /a.*c/ { }
MANIFEST
end.to raise_error(Puppet::Error, /Node '__node_regexp__a.c' is already defined/)
end
it 'provides captures from the regex in the node body' do
catalog = compile_to_catalog(<<-MANIFEST, Puppet::Node.new("nodename"))
node /(.*)/ { notify { "$1": } }
MANIFEST
expect(catalog).to have_resource('Notify[nodename]')
end
it 'selects the node with the matching regex' do
catalog = compile_to_catalog(<<-MANIFEST, Puppet::Node.new("nodename"))
node /node.*/ { notify { matched: } }
MANIFEST
expect(catalog).to have_resource('Notify[matched]')
end
it 'selects a node that is a literal string' do
catalog = compile_to_catalog(<<-MANIFEST, Puppet::Node.new("node.name"))
node 'node.name' { notify { matched: } }
MANIFEST
expect(catalog).to have_resource('Notify[matched]')
end
it 'selects a node that is a prefix of the agent name' do
Puppet[:strict_hostname_checking] = false
catalog = compile_to_catalog(<<-MANIFEST, Puppet::Node.new("node.name.com"))
node 'node.name' { notify { matched: } }
MANIFEST
expect(catalog).to have_resource('Notify[matched]')
end
it 'does not treat regex symbols as a regex inside a string literal' do
catalog = compile_to_catalog(<<-MANIFEST, Puppet::Node.new("nodexname"))
node 'node.name' { notify { 'not matched': } }
node 'nodexname' { notify { 'matched': } }
MANIFEST
expect(catalog).to have_resource('Notify[matched]')
end
it 'errors when two nodes have the same name' do
expect do
compile_to_catalog(<<-MANIFEST)
node name { }
node 'name' { }
MANIFEST
end.to raise_error(Puppet::Error, /Node 'name' is already defined/)
end
it 'is unable to parse a name that is an invalid number' do
expect do
compile_to_catalog('node 5name {} ')
end.to raise_error(Puppet::Error, /Illegal number '5name'/)
end
it 'parses a node name that is dotted numbers' do
catalog = compile_to_catalog(<<-MANIFEST, Puppet::Node.new("1.2.3.4"))
node 1.2.3.4 { notify { matched: } }
MANIFEST
expect(catalog).to have_resource('Notify[matched]')
end
it 'raises error for node inheritance' do
expect do
compile_to_catalog(<<-MANIFEST, Puppet::Node.new("nodename"))
node default {}
node nodename inherits default { }
MANIFEST
end.to raise_error(/Node inheritance is not supported in Puppet >= 4\.0\.0/)
end
end
end
|
