Skip to content

Commit 79d5b4f

Browse files
committed
Fix unit test stubs
1 parent e951bbf commit 79d5b4f

File tree

3 files changed

+41
-41
lines changed

3 files changed

+41
-41
lines changed

spec/unit/puppet/provider/mysql_database/mysql_spec.rb

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -30,17 +30,17 @@
3030
Facter.stubs(:value).with(:root_home).returns('/root')
3131
Puppet::Util.stubs(:which).with('mysql').returns('/usr/bin/mysql')
3232
File.stubs(:file?).with('/root/.my.cnf').returns(true)
33-
provider.class.stubs(:mysql).with('show databases', 'regular').returns('new_database')
34-
provider.class.stubs(:mysql).with(["show variables like '%_database'", 'new_database'], 'regular').returns("character_set_database latin1\ncollation_database latin1_swedish_ci\nskip_show_database OFF") # rubocop:disable Metrics/LineLength
33+
provider.class.stubs(:mysql_caller).with('show databases', 'regular').returns('new_database')
34+
provider.class.stubs(:mysql_caller).with(["show variables like '%_database'", 'new_database'], 'regular').returns("character_set_database latin1\ncollation_database latin1_swedish_ci\nskip_show_database OFF") # rubocop:disable Metrics/LineLength
3535
end
3636

3737
let(:instance) { provider.class.instances.first }
3838

3939
describe 'self.instances' do
4040
it 'returns an array of databases' do
41-
provider.class.stubs(:mysql).with('show databases', 'regular').returns(raw_databases)
41+
provider.class.stubs(:mysql_caller).with('show databases', 'regular').returns(raw_databases)
4242
raw_databases.each_line do |db|
43-
provider.class.stubs(:mysql).with(["show variables like '%_database'", db.chomp], 'regular').returns("character_set_database latin1\ncollation_database latin1_swedish_ci\nskip_show_database OFF") # rubocop:disable Metrics/LineLength
43+
provider.class.stubs(:mysql_caller).with(["show variables like '%_database'", db.chomp], 'regular').returns("character_set_database latin1\ncollation_database latin1_swedish_ci\nskip_show_database OFF") # rubocop:disable Metrics/LineLength
4444
end
4545
databases = provider.class.instances.map { |x| x.name }
4646
expect(parsed_databases).to match_array(databases)
@@ -56,15 +56,15 @@
5656

5757
describe 'create' do
5858
it 'makes a database' do
59-
provider.expects(:mysql).with("create database if not exists `#{resource[:name]}` character set `#{resource[:charset]}` collate `#{resource[:collate]}`", 'regular')
59+
provider.class.expects(:mysql_caller).with("create database if not exists `#{resource[:name]}` character set `#{resource[:charset]}` collate `#{resource[:collate]}`", 'regular')
6060
provider.expects(:exists?).returns(true)
6161
expect(provider.create).to be_truthy
6262
end
6363
end
6464

6565
describe 'destroy' do
6666
it 'removes a database if present' do
67-
provider.expects(:mysql).with("drop database if exists `#{resource[:name]}`", 'regular')
67+
provider.class.expects(:mysql_caller).with("drop database if exists `#{resource[:name]}`", 'regular')
6868
provider.expects(:exists?).returns(false)
6969
expect(provider.destroy).to be_truthy
7070
end
@@ -95,7 +95,7 @@
9595

9696
describe 'charset=' do
9797
it 'changes the charset' do
98-
provider.expects(:mysql).with("alter database `#{resource[:name]}` CHARACTER SET blah", 'regular').returns('0')
98+
provider.class.expects(:mysql_caller).with("alter database `#{resource[:name]}` CHARACTER SET blah", 'regular').returns('0')
9999

100100
provider.charset = 'blah'
101101
end
@@ -109,7 +109,7 @@
109109

110110
describe 'collate=' do
111111
it 'changes the collate' do
112-
provider.expects(:mysql).with("alter database `#{resource[:name]}` COLLATE blah", 'regular').returns('0')
112+
provider.class.expects(:mysql_caller).with("alter database `#{resource[:name]}` COLLATE blah", 'regular').returns('0')
113113

114114
provider.collate = 'blah'
115115
end

spec/unit/puppet/provider/mysql_plugin/mysql_spec.rb

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
Facter.stubs(:value).with(:root_home).returns('/root')
1818
Puppet::Util.stubs(:which).with('mysql').returns('/usr/bin/mysql')
1919
File.stubs(:file?).with('/root/.my.cnf').returns(true)
20-
provider.class.stubs(:mysql).with('show plugins', 'regular').returns('auth_socket ACTIVE AUTHENTICATION auth_socket.so GPL')
20+
provider.class.stubs(:mysql_caller).with('show plugins', 'regular').returns('auth_socket ACTIVE AUTHENTICATION auth_socket.so GPL')
2121
end
2222

2323
let(:instance) { provider.class.instances.first }
@@ -31,15 +31,15 @@
3131

3232
describe 'create' do
3333
it 'loads a plugin' do
34-
provider.expects(:mysql).with("install plugin #{resource[:name]} soname '#{resource[:soname]}'", 'regular')
34+
provider.class.expects(:mysql_caller).with("install plugin #{resource[:name]} soname '#{resource[:soname]}'", 'regular')
3535
provider.expects(:exists?).returns(true)
3636
expect(provider.create).to be_truthy
3737
end
3838
end
3939

4040
describe 'destroy' do
4141
it 'unloads a plugin if present' do
42-
provider.expects(:mysql).with("uninstall plugin #{resource[:name]}", 'regular')
42+
provider.class.expects(:mysql_caller).with("uninstall plugin #{resource[:name]}", 'regular')
4343
provider.expects(:exists?).returns(false)
4444
expect(provider.destroy).to be_truthy
4545
end

0 commit comments

Comments
 (0)