Switch from puppet releases to openvox releases

Author: bastelfreak

README.md

@@ -572,9 +572,15 @@ When no facts are available for the specific facter/operating system combination
 ## Maintenance
 
 This gem uses information about puppet AIO component versions to build/test.
-They are stored at `ext/puppet_agent_facter_versions.json`. If they are outdated,
-the `puppet_versions:test` rake task will fail and they need to be updated.
-This is as easy as running: `bundle exec rake puppet_versions:update`
+They are stored at `ext/puppet_agent_facter_versions.json`.
+
+In the past we used `bundle exec rake puppet_versions:update` to update the file and `puppet_versions:test` to validate it.
+This relied on information from the puppet forge.
+With the 6.0.0 release we're switching to openvox.
+The file is currently maintained by hand, because openvox doesn't offer an API yet to map agent releases to openfact releases.
+It got also renamed from `ext/puppet_agent_facter_versions.json` -> `ext/openvox_agent_facter_versions.json`.
+
+**`ext/openvox_agent_facter_versions.json`now lists a mapping of openvox releases to openfact releases**
 
 ## License
 

ext/openvox_agent_facter_versions.json

@@ -0,0 +1,17 @@
+{
+  "8.23.1": "5.1.0",
+  "8.23.0": "5.1.0",
+  "8.22.1": "5.0.0",
+  "8.19.2": "4.11.0",
+  "8.19.1": "4.11.0",
+  "8.19.0": "4.11.0",
+  "8.18.1": "4.11.0",
+  "8.17.0": "4.11.0",
+  "8.16.0": "4.11.0",
+  "8.15.0": "4.11.0",
+  "8.14.0": "4.11.0",
+  "8.13.0": "4.11.0",
+  "8.12.1": "4.11.0",
+  "8.12.0": "4.11.0",
+  "8.11.0": "4.11.0"
+}

ext/puppet_agent_facter_versions.json

@@ -403,7 +403,7 @@ def self.facter_version_to_loose_requirement_string(version)
   def self.facter_version_for_puppet_version(puppet_version)
     return Facter.version if puppet_version.nil?
 
-    json_path = File.expand_path(File.join(__dir__, '..', 'ext', 'puppet_agent_facter_versions.json'))
+    json_path = File.expand_path(File.join(__dir__, '..', 'ext', 'openvox_agent_facter_versions.json'))
     unless File.file?(json_path) && File.readable?(json_path)
       warning "#{json_path} does not exist or is not readable, defaulting to Facter #{Facter.version}"
       return Facter.version

lib/rspec-puppet-facts.rb

@@ -21,7 +21,7 @@
     end
 
     let(:component_json_path) do
-      File.expand_path(File.join(__dir__, '..', 'ext', 'puppet_agent_facter_versions.json'))
+      File.expand_path(File.join(__dir__, '..', 'ext', 'openvox_agent_facter_versions.json'))
     end
 
     let(:puppet_version) { Puppet.version }
@@ -98,18 +98,21 @@
     end
 
     context 'when passed a known Puppet version' do
-      let(:puppet_version) { '5.2.0' }
+      let(:puppet_version) { '8.11.0' }
 
       it 'returns the Facter version for that Puppet version' do
-        expect(facter_version).to eq('3.9.0')
+        expect(facter_version).to eq('4.11.0')
       end
     end
 
     context 'when passed a Puppet version between two known versions' do
-      let(:puppet_version) { '5.2.5' }
+      # openvox 8.19.0 has facter 4.11.0
+      # openvox 8.22.1 has openfact 5.0.0
+      # openvox 9.18.5 doesn't exist
+      let(:puppet_version) { '8.19.5' }
 
       it 'returns the Facter version for the lower Puppet version' do
-        expect(facter_version).to eq('3.9.0')
+        expect(facter_version).to eq('4.11.0')
       end
     end