Skip to content

Commit f817e0d

Browse files
committed
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.
1 parent 6af75fc commit f817e0d

File tree

5 files changed

+8
-8
lines changed

5 files changed

+8
-8
lines changed

lib/facter/facts/aix/filesystems.rb

+1-1
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ class Filesystems
66
FACT_NAME = 'filesystems'
77

88
def call_the_resolver
9-
fact_value = Facter::Resolvers::Aix::Filesystem.resolve(:file_systems)
9+
fact_value = Facter::Resolvers::Aix::Filesystems.resolve(:file_systems)
1010
Facter::ResolvedFact.new(FACT_NAME, fact_value)
1111
end
1212
end

lib/facter/framework/core/file_loader.rb

+1-1
Original file line numberDiff line numberDiff line change
@@ -212,7 +212,7 @@
212212

213213
require_relative '../../resolvers/aix/architecture'
214214
require_relative '../../resolvers/aix/disks'
215-
require_relative '../../resolvers/aix/filesystem'
215+
require_relative '../../resolvers/aix/filesystems'
216216
require_relative '../../resolvers/aix/hardware'
217217
require_relative '../../resolvers/aix/load_averages'
218218
require_relative '../../resolvers/aix/memory'

lib/facter/resolvers/aix/filesystem.rb renamed to lib/facter/resolvers/aix/filesystems.rb

+1-1
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
module Facter
44
module Resolvers
55
module Aix
6-
class Filesystem < BaseResolver
6+
class Filesystems < BaseResolver
77
init_resolver
88

99
class << self

spec/facter/facts/aix/filesystems_spec.rb

+1-1
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
let(:files) { 'apfs,autofs,devfs' }
88

99
before do
10-
allow(Facter::Resolvers::Aix::Filesystem).to \
10+
allow(Facter::Resolvers::Aix::Filesystems).to \
1111
receive(:resolve).with(:file_systems).and_return(files)
1212
end
1313

spec/facter/resolvers/aix/filesystem_spec.rb renamed to spec/facter/resolvers/aix/filesystems_spec.rb

+4-4
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
# frozen_string_literal: true
22

3-
describe Facter::Resolvers::Aix::Filesystem do
3+
describe Facter::Resolvers::Aix::Filesystems do
44
let(:filesystems) { 'ahafs,cdrfs,namefs,procfs,sfs' }
55

66
after do
7-
Facter::Resolvers::Aix::Filesystem.invalidate_cache
7+
Facter::Resolvers::Aix::Filesystems.invalidate_cache
88
end
99

1010
context 'when vfs file is readable' do
@@ -15,7 +15,7 @@
1515
end
1616

1717
it 'returns filesystems' do
18-
result = Facter::Resolvers::Aix::Filesystem.resolve(:file_systems)
18+
result = Facter::Resolvers::Aix::Filesystems.resolve(:file_systems)
1919

2020
expect(result).to eq(filesystems)
2121
end
@@ -29,7 +29,7 @@
2929
end
3030

3131
it 'returns nil' do
32-
result = Facter::Resolvers::Aix::Filesystem.resolve(:file_systems)
32+
result = Facter::Resolvers::Aix::Filesystems.resolve(:file_systems)
3333

3434
expect(result).to be(nil)
3535
end

0 commit comments

Comments
 (0)