Merge pull request #172 from beergeek/hash_checking

Check Node/Classes hashes
voxpupuli · May 13, 2018 · 3146e90 · 3146e90
2 parents c902b4f + 8b95b2a
commit 3146e90
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/lib/onceover/testconfig.rb b/lib/onceover/testconfig.rb
@@ -126,9 +126,13 @@ def self.subtractive_to_list(subtractive_hash)
       # { 'include' => 'somegroup'
       #   'exclude' => 'other'}
       # and return a list of classes/nodes
-      include_list = Onceover::TestConfig.find_list(subtractive_hash['include']).flatten
-      exclude_list = Onceover::TestConfig.find_list(subtractive_hash['exclude']).flatten
-      include_list - exclude_list
+      if subtractive_hash.has_key?('include') && subtractive_hash.has_key?('exclude')
+        include_list = Onceover::TestConfig.find_list(subtractive_hash['include']).flatten
+        exclude_list = Onceover::TestConfig.find_list(subtractive_hash['exclude']).flatten
+        include_list - exclude_list
+      else
+        raise "The classes/nodes hash must have an `exclude` if using an `include`"
+      end
     end
 
     def verify_spec_test(controlrepo, test)