Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP: Refactor network_route using resource api #262

Closed
wants to merge 15 commits into from
Closed
Prev Previous commit
Next Next commit
Initial test working but needs mocks
dhollinger authored and KeithWard committed Dec 8, 2019
commit 1f7ea322f942e6689c266148af8fbd76b98a2b4b
89 changes: 62 additions & 27 deletions spec/unit/puppet/provider/network_route/network_route_spec.rb
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
require 'spec_helper'

ensure_module_defined('Puppet::Provider::NetworkRoute')
# nsure_module_defined('Puppet::Provider::NetworkRoute')
module Puppet::Provider::NetworkRoute; end
require 'puppet/provider/network_route/network_route'

RSpec.describe Puppet::Provider::NetworkRoute::NetworkRoute do
@@ -11,39 +12,73 @@
describe '#get' do
it 'processes resources' do
expect(provider.get(context)).to eq [
{
name: 'foo',
ensure: 'present',
},
{
name: 'bar',
ensure: 'present',
},
{:ensure=>"present",
:prefix=>"default",
:default_route=>true,
:gateway=>"10.155.255.1",
:interface=>"wlp3s0",
:metric=>"600",
:protocol=>"dhcp"},
{:ensure=>"present",
:prefix=>"10.155.255.0/24",
:default_route=>false,
:interface=>"wlp3s0",
:metric=>"600",
:source=>"10.155.255.110",
:scope=>"link",
:protocol=>"kernel"},
{:ensure=>"present",
:prefix=>"169.254.0.0/16",
:default_route=>false,
:interface=>"virbr0",
:metric=>"1000",
:scope=>"link"},
{:ensure=>"present",
:prefix=>"172.17.0.0/16",
:default_route=>false,
:interface=>"docker0",
:source=>"172.17.0.1",
:scope=>"link",
:protocol=>"kernel"},
{:ensure=>"present",
:prefix=>"172.18.0.0/16",
:default_route=>false,
:interface=>"br-39a722eeac35",
:source=>"172.18.0.1",
:scope=>"link",
:protocol=>"kernel"},
{:ensure=>"present",
:prefix=>"192.168.122.0/24",
:default_route=>false,
:interface=>"virbr0",
:source=>"192.168.122.1",
:scope=>"link",
:protocol=>"kernel"}
]
end
end

describe 'create(context, name, should)' do
it 'creates the resource' do
expect(context).to receive(:notice).with(%r{\ACreating 'a'})
# describe 'create(context, name, should)' do
# it 'creates the resource' do
# expect(context).to receive(:notice).with(%r{\ACreating 'a'})

provider.create(context, 'a', name: 'a', ensure: 'present')
end
end
# provider.create(context, 'a', name: 'a', ensure: 'present')
# end
# end

describe 'update(context, name, should)' do
it 'updates the resource' do
expect(context).to receive(:notice).with(%r{\AUpdating 'foo'})
# describe 'update(context, name, should)' do
# it 'updates the resource' do
# expect(context).to receive(:notice).with(%r{\AUpdating 'foo'})

provider.update(context, 'foo', name: 'foo', ensure: 'present')
end
end
# provider.update(context, 'foo', name: 'foo', ensure: 'present')
# end
# end

describe 'delete(context, name, should)' do
it 'deletes the resource' do
expect(context).to receive(:notice).with(%r{\ADeleting 'foo'})
# describe 'delete(context, name, should)' do
# it 'deletes the resource' do
# expect(context).to receive(:notice).with(%r{\ADeleting 'foo'})

provider.delete(context, 'foo')
end
end
# provider.delete(context, 'foo')
# end
# end
end