Skip to content

Commit

Permalink
(maint) resolve test changes from updating postgresql module
Browse files Browse the repository at this point in the history
  • Loading branch information
jonathannewman authored and h0tw1r3 committed Feb 15, 2024
1 parent 8934822 commit ee431f3
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 17 deletions.
3 changes: 1 addition & 2 deletions spec/acceptance/standalone_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,7 @@
let(:puppetdb_params) {}
let(:puppetdb_master_config_params) {}

# TODO: temporary work-around for EL installs
let(:postgres_version) { "(getvar('facts.os.family') == 'RedHat') ? { true => '12', default => undef }" }
let(:postgres_version) { 'undef' } # default
let(:manage_firewall) { "(getvar('facts.os.family') == 'RedHat' and Integer(getvar('facts.os.release.major')) > 7)" }

describe 'with defaults' do
Expand Down
12 changes: 0 additions & 12 deletions spec/support/acceptance/shared/puppetdb.rb
Original file line number Diff line number Diff line change
Expand Up @@ -3,19 +3,7 @@
shared_examples 'puppetdb' do
let(:pp) do
<<~PP
# FIXME: temporary work-around for EL installs
if $facts['os']['family'] == 'RedHat' {
$gpg_key_file = $facts['os']['release']['major'] ? {
'7' => 'PGDG-RPM-GPG-KEY-RHEL7',
default => 'PGDG-RPM-GPG-KEY-RHEL',
}
file { "/etc/pki/rpm-gpg/${gpg_key_file}":
source => "https://download.postgresql.org/pub/repos/yum/keys/${gpg_key_file}",
}
-> Yumrepo <| tag == 'postgresql::repo' |> {
gpgkey => "file:///etc/pki/rpm-gpg/${gpg_key_file}",
}
# Work-around EL systemd in docker bug affecting forked services
file_line { 'puppetdb-unit-remove-pidfile':
path => '/lib/systemd/system/puppetdb.service',
Expand Down
4 changes: 2 additions & 2 deletions spec/support/unit/shared/database.rb
Original file line number Diff line number Diff line change
Expand Up @@ -220,14 +220,14 @@
end

shared_examples 'puppetdb::database::postgresql_ssl_rules' do |error|
let(:defaults) { {} }
let(:defaults) { { postgres_version: '14' } }
let(:with) { defined?(args) ? defaults.merge(args) : defaults }

if error
it { is_expected.to raise_error(error) }
else
let(:identity_map_key) { "#{with[:database_name]}-#{with[:database_username]}-map" }
let(:client_cert) { (with[:postgres_version].to_f >= 12.0) ? 'verify-full' : '1' }
let(:client_cert) { with[:postgres_version].to_f < 12 ? '1' : 'verify-full' }

Check failure on line 230 in spec/support/unit/shared/database.rb

View workflow job for this annotation

GitHub Actions / Spec / Test Puppet ~> 7.24 with ruby 2.7

Style/TernaryParentheses: Use parentheses for ternary expressions with complex conditions.

Check failure on line 230 in spec/support/unit/shared/database.rb

View workflow job for this annotation

GitHub Actions / Spec / Test Puppet ~> 8.0 with ruby 3.2

Style/TernaryParentheses: Use parentheses for ternary expressions with complex conditions.

it { is_expected.to contain_puppetdb__database__postgresql_ssl_rules(name).with(with) }

Expand Down
2 changes: 1 addition & 1 deletion spec/unit/classes/database/postgresql_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,7 @@
{
read_database_username: params[:read_database_username],
database_name: params[:database_name],
password_hash: 'md588e898a4bade3fe1c9b96f650ec85900', # TODO: mock properly
password_hash: %r{^(md5|SCRAM)}, # TODO: mock properly
database_owner: params[:database_username],
}
end
Expand Down

0 comments on commit ee431f3

Please sign in to comment.