rbeapi 0.1.0 → 0.2.0

data/spec/system/rbeapi/api/interfaces_ethernet_spec.rb

@@ -12,10 +12,10 @@ describe Rbeapi::Api::Interfaces do
   end
 
   describe '#get' do
-
     let(:entity) do
       { name: 'Ethernet1', type: 'ethernet', description: '', shutdown: false,
-        sflow: true, flowcontrol_send: 'off', flowcontrol_receive: 'off' }
+        speed: 'auto', forced: false, sflow: true, flowcontrol_send: 'off',
+        flowcontrol_receive: 'off' }
     end
 
     before { node.config(['default interface Ethernet1']) }
@@ -35,7 +35,7 @@ describe Rbeapi::Api::Interfaces do
     it 'returns a hash collection' do
       expect(subject.getall).to be_a_kind_of(Hash)
     end
- end
+  end
 
   describe '#create' do
     it 'raises an error on create' do
@@ -63,7 +63,8 @@ describe Rbeapi::Api::Interfaces do
     it 'sets the description value on the interface' do
       node.config(['interface Ethernet1', 'no description'])
       expect(subject.get('Ethernet1')[:description]).to be_empty
-      expect(subject.set_description('Ethernet1', value: 'foo bar')).to be_truthy
+      expect(subject.set_description('Ethernet1', value: 'foo bar'))
+        .to be_truthy
       expect(subject.get('Ethernet1')[:description]).to eq('foo bar')
     end
   end
@@ -104,14 +105,16 @@ describe Rbeapi::Api::Interfaces do
     it 'sets the flowcontrol send value to on' do
       node.config(['interface Ethernet1', 'flowcontrol send off'])
       expect(subject.get('Ethernet1')[:flowcontrol_send]).to eq('off')
-      expect(subject.set_flowcontrol_send('Ethernet1', value: 'on')).to be_truthy
+      expect(subject.set_flowcontrol_send('Ethernet1', value: 'on'))
+        .to be_truthy
       expect(subject.get('Ethernet1')[:flowcontrol_send]).to eq('on')
     end
 
     it 'sets the flowcontrol send value to off' do
       node.config(['interface Ethernet1', 'flowcontrol send on'])
       expect(subject.get('Ethernet1')[:flowcontrol_send]).to eq('on')
-      expect(subject.set_flowcontrol_send('Ethernet1', value: 'off')).to be_truthy
+      expect(subject.set_flowcontrol_send('Ethernet1', value: 'off'))
+        .to be_truthy
       expect(subject.get('Ethernet1')[:flowcontrol_send]).to eq('off')
     end
   end
@@ -120,16 +123,17 @@ describe Rbeapi::Api::Interfaces do
     it 'sets the flowcontrol receive value to on' do
       node.config(['interface Ethernet1', 'flowcontrol receive off '])
       expect(subject.get('Ethernet1')[:flowcontrol_receive]).to eq('off')
-      expect(subject.set_flowcontrol_receive('Ethernet1', value: 'on')).to be_truthy
+      expect(subject.set_flowcontrol_receive('Ethernet1', value: 'on'))
+        .to be_truthy
       expect(subject.get('Ethernet1')[:flowcontrol_receive]).to eq('on')
     end
 
     it 'sets the flowcontrol receive value to off' do
       node.config(['interface Ethernet1', 'flowcontrol receive on'])
       expect(subject.get('Ethernet1')[:flowcontrol_receive]).to eq('on')
-      expect(subject.set_flowcontrol_receive('Ethernet1', value: 'off')).to be_truthy
+      expect(subject.set_flowcontrol_receive('Ethernet1', value: 'off'))
+        .to be_truthy
       expect(subject.get('Ethernet1')[:flowcontrol_receive]).to eq('off')
     end
   end
 end
-

data/spec/system/rbeapi/api/interfaces_portchannel_spec.rb

@@ -6,21 +6,21 @@ require 'rbeapi/api/interfaces'
 describe Rbeapi::Api::Interfaces do
   subject { described_class.new(node) }
 
-
   let(:node) do
     Rbeapi::Client.config.read(fixture_file('dut.conf'))
     Rbeapi::Client.connect_to('dut')
   end
 
   describe '#get' do
-
     let(:entity) do
       { name: 'Port-Channel1', type: 'portchannel', description: '',
         shutdown: false, members: [], lacp_mode: 'on', minimum_links: '0',
         lacp_timeout: '90', lacp_fallback: 'disabled' }
     end
 
-    before { node.config(['no interface Port-Channel1', 'interface Port-Channel1']) }
+    before do
+      node.config(['no interface Port-Channel1', 'interface Port-Channel1'])
+    end
 
     it 'returns the interface resource' do
       expect(subject.get('Port-Channel1')).to eq(entity)
@@ -28,7 +28,9 @@ describe Rbeapi::Api::Interfaces do
   end
 
   describe '#getall' do
-    before { node.config(['no interface Port-Channel1', 'interface Port-Channel1']) }
+    before do
+      node.config(['no interface Port-Channel1', 'interface Port-Channel1'])
+    end
 
     it 'returns the interface collection' do
       expect(subject.getall).to include('Port-Channel1')
@@ -37,7 +39,7 @@ describe Rbeapi::Api::Interfaces do
     it 'returns a hash collection' do
       expect(subject.getall).to be_a_kind_of(Hash)
     end
- end
+  end
 
   describe '#create' do
     before { node.config('no interface Port-Channel1') }
@@ -73,7 +75,8 @@ describe Rbeapi::Api::Interfaces do
     it 'sets the description value on the interface' do
       node.config(['interface Port-Channel1', 'no description'])
       expect(subject.get('Port-Channel1')[:description]).to be_empty
-      expect(subject.set_description('Port-Channel1', value: 'foo bar')).to be_truthy
+      expect(subject.set_description('Port-Channel1', value: 'foo bar'))
+        .to be_truthy
       expect(subject.get('Port-Channel1')[:description]).to eq('foo bar')
     end
   end
@@ -95,19 +98,24 @@ describe Rbeapi::Api::Interfaces do
   end
 
   describe '#set_minimum_links' do
-    before { node.config(['interface Port-Channel1',
-                          'port-channel min-links 0']) }
+    before do
+      node.config(['interface Port-Channel1',
+                   'port-channel min-links 0'])
+    end
 
     it 'sets the minimum links value on the interface' do
       expect(subject.get('Port-Channel1')[:minimum_links]).to eq('0')
-      expect(subject.set_minimum_links('Port-Channel1', value: '2')).to be_truthy
+      expect(subject.set_minimum_links('Port-Channel1', value: '2'))
+        .to be_truthy
       expect(subject.get('Port-Channel1')[:minimum_links]).to eq('2')
     end
   end
 
   describe '#set_members' do
-    before { node.config(['no interface Port-Channel1',
-                          'interface Port-Channel1']) }
+    before do
+      node.config(['no interface Port-Channel1',
+                   'interface Port-Channel1'])
+    end
 
     it 'adds new members to the port-channel interface' do
       node.config(['no interface Port-Channel1', 'interface Port-Channel1'])
@@ -118,17 +126,20 @@ describe Rbeapi::Api::Interfaces do
 
     it 'updates the member interfaces on existing interface' do
       node.config(['no interface Port-Channel1', 'interface Ethernet1-2',
-                  'channel-group 1 mode on'])
-      expect(subject.get('Port-Channel1')[:members]).to eq(['Ethernet1', 'Ethernet2'])
-      expect(subject.set_members('Port-Channel1', ['Ethernet1', 'Ethernet3'])).to be_truthy
-      expect(subject.get('Port-Channel1')[:members]).to eq(['Ethernet1', 'Ethernet3'])
+                   'channel-group 1 mode on'])
+      expect(subject.get('Port-Channel1')[:members]).to eq(%w(Ethernet1
+                                                              Ethernet2))
+      expect(subject.set_members('Port-Channel1',
+                                 %w(Ethernet1 Ethernet3))).to be_truthy
+      expect(subject.get('Port-Channel1')[:members]).to eq(%w(Ethernet1
+                                                              Ethernet3))
     end
   end
 
   describe '#set_lacp_mode' do
     it 'sets the lacp mode on the port-channel to active' do
       node.config(['no interface Port-Channel1', 'interface Ethernet1-3',
-                  'channel-group 1 mode on'])
+                   'channel-group 1 mode on'])
       expect(subject.get('Port-Channel1')[:lacp_mode]).to eq('on')
       expect(subject.set_lacp_mode('Port-Channel1', 'active')).to be_truthy
       expect(subject.get('Port-Channel1')[:lacp_mode]).to eq('active')
@@ -136,7 +147,7 @@ describe Rbeapi::Api::Interfaces do
 
     it 'sets the lacp mode on the port-channel to passive' do
       node.config(['no interface Port-Channel1', 'interface Ethernet1-3',
-                  'channel-group 1 mode on'])
+                   'channel-group 1 mode on'])
       expect(subject.get('Port-Channel1')[:lacp_mode]).to eq('on')
       expect(subject.set_lacp_mode('Port-Channel1', 'passive')).to be_truthy
       expect(subject.get('Port-Channel1')[:lacp_mode]).to eq('passive')
@@ -144,7 +155,7 @@ describe Rbeapi::Api::Interfaces do
 
     it 'sets the lacp mode on the port-channel to on' do
       node.config(['no interface Port-Channel1', 'interface Ethernet1-3',
-                  'channel-group 1 mode active'])
+                   'channel-group 1 mode active'])
       expect(subject.get('Port-Channel1')[:lacp_mode]).to eq('active')
       expect(subject.set_lacp_mode('Port-Channel1', 'on')).to be_truthy
       expect(subject.get('Port-Channel1')[:lacp_mode]).to eq('on')
@@ -155,34 +166,40 @@ describe Rbeapi::Api::Interfaces do
     it 'sets the lacp fallback on the port-channel to static' do
       node.config(['interface Port-Channel1', 'no port-channel lacp fallback'])
       expect(subject.get('Port-Channel1')[:lacp_fallback]).to eq('disabled')
-      expect(subject.set_lacp_fallback('Port-Channel1', value: 'static')).to be_truthy
+      expect(subject.set_lacp_fallback('Port-Channel1', value: 'static'))
+        .to be_truthy
       expect(subject.get('Port-Channel1')[:lacp_fallback]).to eq('static')
     end
 
     it 'sets the lacp fallback on the port-channel to individual' do
       node.config(['interface Port-Channel1', 'no port-channel lacp fallback'])
       expect(subject.get('Port-Channel1')[:lacp_fallback]).to eq('disabled')
-      expect(subject.set_lacp_fallback('Port-Channel1', value: 'individual')).to be_truthy
+      expect(subject.set_lacp_fallback('Port-Channel1', value: 'individual'))
+        .to be_truthy
       expect(subject.get('Port-Channel1')[:lacp_fallback]).to eq('individual')
     end
 
     it 'sets the lacp fallback on the port-channel to disabled' do
-      node.config(['interface Port-Channel1', 'port-channel lacp fallback static'])
+      node.config(['interface Port-Channel1',
+                   'port-channel lacp fallback static'])
       expect(subject.get('Port-Channel1')[:lacp_fallback]).to eq('static')
-      expect(subject.set_lacp_fallback('Port-Channel1', value: 'disabled')).to be_truthy
+      expect(subject.set_lacp_fallback('Port-Channel1', value: 'disabled'))
+        .to be_truthy
       expect(subject.get('Port-Channel1')[:lacp_fallback]).to eq('disabled')
     end
   end
 
   describe '#set_lacp_timeout' do
-    before { node.config(['interface Port-Channel1',
-                          'default port-channel lacp fallback timeout']) }
+    before do
+      node.config(['interface Port-Channel1',
+                   'default port-channel lacp fallback timeout'])
+    end
 
     it 'sets the lacp fallback timeout value on the interface' do
       expect(subject.get('Port-Channel1')[:lacp_timeout]).to eq('90')
-      expect(subject.set_lacp_timeout('Port-Channel1', value: '100')).to be_truthy
+      expect(subject.set_lacp_timeout('Port-Channel1', value: '100'))
+        .to be_truthy
       expect(subject.get('Port-Channel1')[:lacp_timeout]).to eq('100')
     end
   end
 end
-

data/spec/system/rbeapi/api/interfaces_vxlan_spec.rb

@@ -12,10 +12,10 @@ describe Rbeapi::Api::Interfaces do
   end
 
   describe '#get' do
-
     let(:entity) do
       { name: 'Vxlan1', type: 'vxlan', description: '', shutdown: false,
-        source_interface: '', multicast_group: ''}
+        source_interface: '', multicast_group: '', udp_port: 4789,
+        flood_list: [], vlans: {} }
     end
 
     before { node.config(['no interface Vxlan1', 'interface Vxlan1']) }
@@ -35,7 +35,7 @@ describe Rbeapi::Api::Interfaces do
     it 'returns a hash collection' do
       expect(subject.getall).to be_a_kind_of(Hash)
     end
- end
+  end
 
   describe '#create' do
     before { node.config('no interface Vxlan1') }
@@ -97,7 +97,8 @@ describe Rbeapi::Api::Interfaces do
 
     it 'sets the source interface value on the interface' do
       expect(subject.get('Vxlan1')[:source_interface]).to be_empty
-      expect(subject.set_source_interface('Vxlan1', value: 'Loopback0')).to be_truthy
+      expect(subject.set_source_interface('Vxlan1', value: 'Loopback0'))
+        .to be_truthy
       expect(subject.get('Vxlan1')[:source_interface]).to eq('Loopback0')
     end
   end
@@ -107,9 +108,9 @@ describe Rbeapi::Api::Interfaces do
 
     it 'sets the multicast group value on the interface' do
       expect(subject.get('Vxlan1')[:multicast_group]).to be_empty
-      expect(subject.set_multicast_group('Vxlan1', value: '239.10.10.10')).to be_truthy
+      expect(subject.set_multicast_group('Vxlan1', value: '239.10.10.10'))
+        .to be_truthy
       expect(subject.get('Vxlan1')[:multicast_group]).to eq('239.10.10.10')
     end
   end
 end
-

data/spec/system/rbeapi/api/ipinterfaces_spec.rb

@@ -12,13 +12,14 @@ describe Rbeapi::Api::Ipinterfaces do
   end
 
   describe '#get' do
-
     let(:entity) do
-      { address: '99.99.99.99/24', mtu: '1500', helper_addresses: [] }
+      { address: '77.99.99.99/24', mtu: '1500', helper_addresses: [] }
     end
 
-    before { node.config(['default interface Ethernet1', 'interface Ethernet1',
-                         'no switchport', 'ip address 99.99.99.99/24']) }
+    before do
+      node.config(['default interface Ethernet1', 'interface Ethernet1',
+                   'no switchport', 'ip address 77.99.99.99/24'])
+    end
 
     it 'returns the ipinterface resource' do
       expect(subject.get('Ethernet1')).to eq(entity)
@@ -26,8 +27,10 @@ describe Rbeapi::Api::Ipinterfaces do
   end
 
   describe '#getall' do
-    before { node.config(['default interface Ethernet1', 'interface Ethernet1',
-                         'no switchport', 'ip address 99.99.99.99/24']) }
+    before do
+      node.config(['default interface Ethernet1', 'interface Ethernet1',
+                   'no switchport', 'ip address 77.99.99.99/24'])
+    end
 
     it 'returns the ipinterface collection' do
       expect(subject.getall).to include('Ethernet1')
@@ -36,7 +39,7 @@ describe Rbeapi::Api::Ipinterfaces do
     it 'returns a hash collection' do
       expect(subject.getall).to be_a_kind_of(Hash)
     end
- end
+  end
 
   describe '#create' do
     before { node.config(['interface Ethernet1', 'switchport']) }
@@ -49,8 +52,10 @@ describe Rbeapi::Api::Ipinterfaces do
   end
 
   describe '#delete' do
-    before { node.config(['interface Ethernet1', 'no switchport',
-                          'ip address 99.99.99.99/24']) }
+    before do
+      node.config(['interface Ethernet1', 'no switchport',
+                   'ip address 77.99.99.99/24'])
+    end
 
     it 'deletes a ipinterface resource' do
       expect(subject.get('Ethernet1')).not_to be_nil
@@ -60,19 +65,24 @@ describe Rbeapi::Api::Ipinterfaces do
   end
 
   describe '#set_address' do
-    before { node.config(['default interface Ethernet1', 'interface Ethernet1',
-                          'no switchport'])  }
+    before do
+      node.config(['default interface Ethernet1', 'interface Ethernet1',
+                   'no switchport'])
+    end
 
     it 'sets the address value' do
       expect(subject.get('Ethernet1')[:address]).to be_empty
-      expect(subject.set_address('Ethernet1', value: '99.99.99.99/24')).to be_truthy
-      expect(subject.get('Ethernet1')[:address]).to eq('99.99.99.99/24')
+      expect(subject.set_address('Ethernet1', value: '77.99.99.99/24'))
+        .to be_truthy
+      expect(subject.get('Ethernet1')[:address]).to eq('77.99.99.99/24')
     end
   end
 
   describe '#set_mtu' do
-    before { node.config(['default interface Ethernet1', 'interface Ethernet1',
-                          'no switchport'])  }
+    before do
+      node.config(['default interface Ethernet1', 'interface Ethernet1',
+                   'no switchport'])
+    end
 
     it 'sets the mtu value on the interface' do
       expect(subject.get('Ethernet1')[:mtu]).to eq('1500')
@@ -82,16 +92,19 @@ describe Rbeapi::Api::Ipinterfaces do
   end
 
   describe '#set_helper_addresses' do
-    before { node.config(['default interface Ethernet1', 'interface Ethernet1',
-                          'no switchport', 'ip address 99.99.99.99/24'])  }
+    before do
+      node.config(['default interface Ethernet1', 'interface Ethernet1',
+                   'no switchport', 'ip address 77.99.99.99/24'])
+    end
 
-    let(:helpers) { %w(99.99.99.98 99.99.99.97) }
+    let(:helpers) { %w(77.99.99.98 77.99.99.97) }
 
     it 'sets the helper addresses on the interface' do
       expect(subject.get('Ethernet1')[:helper_addresses]).to be_empty
-      expect(subject.set_helper_addresses('Ethernet1', value: helpers)).to be_truthy
-      expect(subject.get('Ethernet1')[:helper_addresses].sort).to eq(helpers.sort)
+      expect(subject.set_helper_addresses('Ethernet1', value: helpers))
+        .to be_truthy
+      expect(subject.get('Ethernet1')[:helper_addresses].sort)
+        .to eq(helpers.sort)
     end
   end
 end
-

data/spec/system/rbeapi/api/mlag_interfaces_spec.rb

@@ -3,7 +3,7 @@ require 'spec_helper'
 require 'rbeapi/client'
 require 'rbeapi/api/mlag'
 
-describe Rbeapi::Api::MlagInterfaces do
+describe Rbeapi::Api::Mlag do
   subject { described_class.new(node) }
 
   let(:node) do
@@ -12,15 +12,14 @@ describe Rbeapi::Api::MlagInterfaces do
   end
 
   describe '#get' do
-
     let(:entity) do
-      { mlag_id: '1' }
+      { mlag_id: 1 }
     end
 
     before { node.config(['interface Port-Channel1', 'mlag 1']) }
 
     it 'returns the mlag interface resource' do
-      expect(subject.get('Port-Channel1')).to eq(entity)
+      expect(subject.get[:interfaces]['Port-Channel1']).to eq(entity)
     end
   end
 
@@ -28,53 +27,31 @@ describe Rbeapi::Api::MlagInterfaces do
     before { node.config(['interface Port-Channel1', 'mlag 1']) }
 
     it 'returns the interface collection' do
-      expect(subject.getall).to include('Port-Channel1')
+      expect(subject.get[:interfaces]).to include('Port-Channel1')
     end
 
     it 'returns a hash collection' do
-      expect(subject.getall).to be_a_kind_of(Hash)
-    end
- end
-
-  describe '#create' do
-    before { node.config(['interface Port-Channel1', 'no mlag']) }
-
-    it 'creates a new mlag interface resource' do
-      expect(subject.get('Port-Channel1')).to be_nil
-      expect(subject.create('Port-Channel1', '1')).to be_truthy
-      expect(subject.get('Port-Channel1')).not_to be_nil
+      expect(subject.get[:interfaces]).to be_a_kind_of(Hash)
     end
   end
 
-  describe '#delete' do
-    before { node.config(['interface Port-Channel1', 'mlag 1']) }
+  describe '#set_mlag_id' do
+    before { node.config(['interface Port-Channel1', 'no mlag']) }
 
-    it 'deletes a switchport resource' do
-      expect(subject.get('Port-Channel1')).not_to be_nil
-      expect(subject.delete('Port-Channel1')).to be_truthy
-      expect(subject.get('Port-Channel1')).to be_nil
+    it 'sets mlag_id on Port-Channel1' do
+      expect(subject.get[:interfaces]['Port-Channel1']).to be_nil
+      expect(subject.set_mlag_id('Port-Channel1', value: '1')).to be_truthy
+      expect(subject.get[:interfaces]['Port-Channel1'][:mlag_id]).to eq(1)
     end
   end
 
-  describe '#default' do
+  describe '#set_mlag_id default' do
     before { node.config(['interface Port-Channel1', 'mlag 1']) }
 
     it 'sets Port-Channel1 to default' do
-      expect(subject.get('Port-Channel1')).not_to be_nil
-      expect(subject.default('Port-Channel1')).to be_truthy
-      expect(subject.get('Port-Channel1')).to be_nil
+      expect(subject.get[:interfaces]['Port-Channel1']).not_to be_nil
+      expect(subject.set_mlag_id('Port-Channel1', default: true)).to be_truthy
+      expect(subject.get[:interfaces]['Port-Channel1']).to be_nil
     end
   end
-
-  describe '#set_mlag_id' do
-    before { node.config(['interface Port-Channel1', 'no mlag']) }
-
-    it 'sets Port-Channel1 to default' do
-      expect(subject.get('Port-Channel1')).to be_nil
-      expect(subject.set_mlag_id('Port-Channel1', value: '1')).to be_truthy
-      expect(subject.get('Port-Channel1')[:mlag_id]).to eq('1')
-    end
-  end
-
 end
-

data/spec/system/rbeapi/api/mlag_spec.rb

@@ -12,18 +12,17 @@ describe Rbeapi::Api::Mlag do
   end
 
   describe '#get' do
-
-    let(:keys) do
-      [ :domain_id, :local_interface, :peer_address, :peer_link, :shutdown,
-        :interfaces ]
+    let(:global_keys) do
+      [:domain_id, :local_interface, :peer_address, :peer_link, :shutdown]
     end
 
     before { node.config('default mlag configuration') }
 
-    it 'contains all required keys' do
-      keys.each do |key|
-        expect(subject.get).to include(key)
+    it 'contains all required global keys' do
+      global_keys.each do |key|
+        expect(subject.get[:global]).to include(key)
       end
+      expect(subject.get).to include(:interfaces)
     end
   end
 
@@ -31,9 +30,9 @@ describe Rbeapi::Api::Mlag do
     before { node.config('default mlag configuration') }
 
     it 'configures the mlag domain-id value' do
-      expect(subject.get[:domain_id]).to be_empty
+      expect(subject.get[:global][:domain_id]).to be_empty
       expect(subject.set_domain_id(value: 'foo')).to be_truthy
-      expect(subject.get[:domain_id]).to eq('foo')
+      expect(subject.get[:global][:domain_id]).to eq('foo')
     end
   end
 
@@ -41,54 +40,51 @@ describe Rbeapi::Api::Mlag do
     before { node.config(['default mlag configuration', 'interface vlan4094']) }
 
     it 'configures the mlag local interface value' do
-      expect(subject.get[:local_interface]).to be_empty
+      expect(subject.get[:global][:local_interface]).to be_empty
       expect(subject.set_local_interface(value: 'Vlan4094')).to be_truthy
-      expect(subject.get[:local_interface]).to eq('Vlan4094')
+      expect(subject.get[:global][:local_interface]).to eq('Vlan4094')
     end
   end
 
   describe '#set_peer_link' do
-    before { node.config(['default mlag configuration',
-                          'default interface Ethernet1']) }
+    before do
+      node.config(['default mlag configuration',
+                   'default interface Ethernet1'])
+    end
 
     it 'configures the mlag peer link value' do
-      expect(subject.get[:peer_link]).to be_empty
+      expect(subject.get[:global][:peer_link]).to be_empty
       expect(subject.set_peer_link(value: 'Ethernet1')).to be_truthy
-      expect(subject.get[:peer_link]).to eq('Ethernet1')
+      expect(subject.get[:global][:peer_link]).to eq('Ethernet1')
     end
   end
 
   describe '#set_peer_address' do
-    before { node.config(['default mlag configuration',
-                          'default interface Ethernet1']) }
+    before do
+      node.config(['default mlag configuration',
+                   'default interface Ethernet1'])
+    end
 
     it 'configures the mlag peer address value' do
-      expect(subject.get[:peer_address]).to be_empty
+      expect(subject.get[:global][:peer_address]).to be_empty
       expect(subject.set_peer_address(value: '1.1.1.1')).to be_truthy
-      expect(subject.get[:peer_address]).to eq('1.1.1.1')
+      expect(subject.get[:global][:peer_address]).to eq('1.1.1.1')
     end
   end
 
   describe '#set_shutdown' do
     it 'configures mlag to be enabled' do
       node.config(['mlag configuration', 'shutdown'])
-      expect(subject.get[:shutdown]).to be_truthy
+      expect(subject.get[:global][:shutdown]).to be_truthy
       expect(subject.set_shutdown(value: false)).to be_truthy
-      expect(subject.get[:shutdown]).to be_falsy
+      expect(subject.get[:global][:shutdown]).to be_falsy
     end
 
     it 'configures mlag to be disabled' do
       node.config(['mlag configuration', 'no shutdown'])
-      expect(subject.get[:shutdown]).to be_falsy
+      expect(subject.get[:global][:shutdown]).to be_falsy
       expect(subject.set_shutdown(value: true)).to be_truthy
-      expect(subject.get[:shutdown]).to be_truthy
+      expect(subject.get[:global][:shutdown]).to be_truthy
     end
   end
-
-  describe '#interfaces' do
-    it 'is a kind of MlagInterfaces' do
-      expect(subject.interfaces).to be_a_kind_of(Rbeapi::Api::MlagInterfaces)
-    end
-  end
-
 end

data/spec/system/rbeapi/api/snmp_spec.rb

@@ -12,7 +12,6 @@ describe Rbeapi::Api::Snmp do
   end
 
   describe '#get' do
-
     let(:keys) do
       [:location, :contact, :chassis_id, :source_interface]
     end
@@ -64,5 +63,3 @@ describe Rbeapi::Api::Snmp do
     end
   end
 end
-
-