Skip to content

Commit eedccb3

Browse files
committed
(bugfix) Fix tests to account for changes
1 parent 8a80d11 commit eedccb3

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

spec/unit/puppet/provider/firewall/firewall_public_spec.rb

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -156,6 +156,7 @@
156156
{ is_hash: { state: 'NEW' }, should_hash: { state: ['NEW', 'INVALID'] }, result: nil },
157157
{ is_hash: { state: ['INVALID', 'NEW'] }, should_hash: { state: ['NEW', 'INVALID'] }, result: true },
158158
{ is_hash: { state: ['! INVALID', 'NEW'] }, should_hash: { state: ['! NEW', 'INVALID'] }, result: true },
159+
{ is_hash: { state: ['! INVALID', 'NEW'] }, should_hash: { state: ['! NEW', '! INVALID'] }, result: true },
159160
{ is_hash: { state: ['! INVALID', 'NEW'] }, should_hash: { state: ['! NEW', 'INVALID', 'UNTRACKED'] }, result: false },
160161
] },
161162
{ testing: 'icmp', property_name: :icmp, comparisons: [
@@ -200,12 +201,11 @@
200201
{ is_hash: { jump: 'accept' }, should_hash: { jump: 'drop' }, result: false },
201202
] },
202203
{ testing: 'dport/sport', property_name: :dport, comparisons: [
203-
{ is_hash: { dport: '! 50' }, should_hash: { dport: '! 50' }, result: true },
204-
{ is_hash: { dport: '50:60' }, should_hash: { dport: '50-60' }, result: true },
205-
{ is_hash: { dport: ['50:60'] }, should_hash: { dport: '50-60' }, result: true },
204+
{ is_hash: { dport: '50' }, should_hash: { dport: '50' }, result: nil },
205+
{ is_hash: { dport: ['50:60'] }, should_hash: { dport: '50-60' }, result: nil },
206206
{ is_hash: { dport: ['50:60'] }, should_hash: { dport: ['50-60'] }, result: true },
207-
{ is_hash: { dport: ['! 50:60', '90'] }, should_hash: { dport: ['! 90', '50-60'] }, result: true },
208-
{ is_hash: { dport: '50' }, should_hash: { dport: '90' }, result: false },
207+
{ is_hash: { dport: ['! 50:60', '90'] }, should_hash: { dport: ['! 90', '! 50-60'] }, result: true },
208+
{ is_hash: { dport: ['! 50:60', '90'] }, should_hash: { dport: ['! 100', '! 60-70'] }, result: false },
209209
] },
210210
{ testing: 'string_hex', property_name: :string_hex, comparisons: [
211211
{ is_hash: { string_hex: '! |f4 6d 04 25 b2 02 00 0a|' }, should_hash: { string_hex: '! |f46d0425b202000a|' }, result: true },

0 commit comments

Comments
 (0)