Add tests for symbol conversion

lib/facterdb.rb

@@ -140,7 +140,7 @@ def self.valid_filters?(filters)
   # @param symbolize_keys [Boolean]
   #   Whether to symbolize the keys. Note this is only on the top level and not
   #   on nested values.
-  def self.get_facts(filter = nil, cache = true, symbolize_keys: true)
+  def self.get_facts(filter = nil, cache = true, symbolize_keys = true)
     if cache && filter && filter == Thread.current[:facterdb_last_filter_seen]
       return Thread.current[:facterdb_last_facts_seen]
     end

spec/facter_db_spec.rb

@@ -319,14 +319,36 @@
   end
 
   describe '.get_facts' do
-    subject(:result) { FacterDB.get_facts(filter) }
+    subject(:result) { FacterDB.get_facts(filter, cache, symbolize_keys) }
 
     let(:filter) { nil }
+    let(:cache) { nil }
+    let(:symbolize_keys) { nil }
 
     context 'without parameters' do
       include_examples 'returns a result'
     end
 
+    context 'with stringified output' do
+      let(:symbolize_keys) { false }
+
+      it 'returns strings as keys in factsets' do
+        result.each do |factset|
+          expect(factset.keys).to all(be_an_instance_of(String))
+        end
+      end
+    end
+
+    context 'with symbolized output' do
+      let(:symbolize_keys) { true }
+
+      it 'returns strings as keys in factsets' do
+        result.each do |factset|
+          expect(factset.keys).to all(be_an_instance_of(Symbol))
+        end
+      end
+    end
+
     context 'with an Array filter' do
       let(:filter) { [osfamily: 'Debian'] }