Fix AIX filesystems resolver name

All other filesystems resolvers are named "filesystems" (plural), so match the same name because all these resolvers have the same purpose.
smortex · Sep 11, 2024 · ab513fc · ab513fc
1 parent 7818e00
commit ab513fc
Show file tree

Hide file tree

Showing 5 changed files with 8 additions and 8 deletions.
diff --git a/lib/facter/facts/aix/filesystems.rb b/lib/facter/facts/aix/filesystems.rb
@@ -6,7 +6,7 @@ class Filesystems
       FACT_NAME = 'filesystems'
 
       def call_the_resolver
-        fact_value = Facter::Resolvers::Aix::Filesystem.resolve(:file_systems)
+        fact_value = Facter::Resolvers::Aix::Filesystems.resolve(:file_systems)
         Facter::ResolvedFact.new(FACT_NAME, fact_value)
       end
     end

diff --git a/lib/facter/framework/core/file_loader.rb b/lib/facter/framework/core/file_loader.rb
@@ -212,7 +212,7 @@
 
     require_relative '../../resolvers/aix/architecture'
     require_relative '../../resolvers/aix/disks'
-    require_relative '../../resolvers/aix/filesystem'
+    require_relative '../../resolvers/aix/filesystems'
     require_relative '../../resolvers/aix/hardware'
     require_relative '../../resolvers/aix/load_averages'
     require_relative '../../resolvers/aix/memory'

diff --git a/lib/facter/resolvers/aix/filesystem.rb → lib/facter/resolvers/aix/filesystems.rb b/lib/facter/resolvers/aix/filesystem.rb → lib/facter/resolvers/aix/filesystems.rb
@@ -3,7 +3,7 @@
 module Facter
   module Resolvers
     module Aix
-      class Filesystem < BaseResolver
+      class Filesystems < BaseResolver
         init_resolver
 
         class << self

diff --git a/spec/facter/facts/aix/filesystems_spec.rb b/spec/facter/facts/aix/filesystems_spec.rb
@@ -7,7 +7,7 @@
     let(:files) { 'apfs,autofs,devfs' }
 
     before do
-      allow(Facter::Resolvers::Aix::Filesystem).to \
+      allow(Facter::Resolvers::Aix::Filesystems).to \
         receive(:resolve).with(:file_systems).and_return(files)
     end
 

diff --git a/spec/facter/resolvers/aix/filesystem_spec.rb → .../facter/resolvers/aix/filesystems_spec.rb b/spec/facter/resolvers/aix/filesystem_spec.rb → .../facter/resolvers/aix/filesystems_spec.rb
@@ -1,10 +1,10 @@
 # frozen_string_literal: true
 
-describe Facter::Resolvers::Aix::Filesystem do
+describe Facter::Resolvers::Aix::Filesystems do
   let(:filesystems) { 'ahafs,cdrfs,namefs,procfs,sfs' }
 
   after do
-    Facter::Resolvers::Aix::Filesystem.invalidate_cache
+    Facter::Resolvers::Aix::Filesystems.invalidate_cache
   end
 
   context 'when vfs file is readable' do
@@ -15,7 +15,7 @@
     end
 
     it 'returns filesystems' do
-      result = Facter::Resolvers::Aix::Filesystem.resolve(:file_systems)
+      result = Facter::Resolvers::Aix::Filesystems.resolve(:file_systems)
 
       expect(result).to eq(filesystems)
     end
@@ -29,7 +29,7 @@
     end
 
     it 'returns nil' do
-      result = Facter::Resolvers::Aix::Filesystem.resolve(:file_systems)
+      result = Facter::Resolvers::Aix::Filesystems.resolve(:file_systems)
 
       expect(result).to be(nil)
     end