rbeapi 0.2.0 → 0.3.0

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

@@ -34,6 +34,20 @@ describe Rbeapi::Api::Mlag do
       expect(subject.set_domain_id(value: 'foo')).to be_truthy
       expect(subject.get[:global][:domain_id]).to eq('foo')
     end
+
+    it 'negates the mlag domain_id' do
+      expect(subject.set_domain_id(value: 'foo')).to be_truthy
+      expect(subject.get[:global][:domain_id]).to eq('foo')
+      expect(subject.set_domain_id(enable: false)).to be_truthy
+      expect(subject.get[:global][:domain_id]).to be_empty
+    end
+
+    it 'defaults the mlag domain_id' do
+      expect(subject.set_domain_id(value: 'foo')).to be_truthy
+      expect(subject.get[:global][:domain_id]).to eq('foo')
+      expect(subject.set_domain_id(default: true)).to be_truthy
+      expect(subject.get[:global][:domain_id]).to be_empty
+    end
   end
 
   describe '#set_local_interface' do
@@ -44,6 +58,20 @@ describe Rbeapi::Api::Mlag do
       expect(subject.set_local_interface(value: 'Vlan4094')).to be_truthy
       expect(subject.get[:global][:local_interface]).to eq('Vlan4094')
     end
+
+    it 'negates the local interface' do
+      expect(subject.set_local_interface(value: 'Vlan4094')).to be_truthy
+      expect(subject.get[:global][:local_interface]).to eq('Vlan4094')
+      expect(subject.set_local_interface(enable: false)).to be_truthy
+      expect(subject.get[:global][:local_interface]).to be_empty
+    end
+
+    it 'defaults the local interface' do
+      expect(subject.set_local_interface(value: 'Vlan4094')).to be_truthy
+      expect(subject.get[:global][:local_interface]).to eq('Vlan4094')
+      expect(subject.set_local_interface(default: true)).to be_truthy
+      expect(subject.get[:global][:local_interface]).to be_empty
+    end
   end
 
   describe '#set_peer_link' do
@@ -57,6 +85,20 @@ describe Rbeapi::Api::Mlag do
       expect(subject.set_peer_link(value: 'Ethernet1')).to be_truthy
       expect(subject.get[:global][:peer_link]).to eq('Ethernet1')
     end
+
+    it 'negates the mlag peer link' do
+      expect(subject.set_peer_link(value: 'Ethernet1')).to be_truthy
+      expect(subject.get[:global][:peer_link]).to eq('Ethernet1')
+      expect(subject.set_peer_link(enable: false)).to be_truthy
+      expect(subject.get[:global][:peer_link]).to be_empty
+    end
+
+    it 'defaults the mlag peer link' do
+      expect(subject.set_peer_link(value: 'Ethernet1')).to be_truthy
+      expect(subject.get[:global][:peer_link]).to eq('Ethernet1')
+      expect(subject.set_peer_link(default: true)).to be_truthy
+      expect(subject.get[:global][:peer_link]).to be_empty
+    end
   end
 
   describe '#set_peer_address' do
@@ -70,21 +112,71 @@ describe Rbeapi::Api::Mlag do
       expect(subject.set_peer_address(value: '1.1.1.1')).to be_truthy
       expect(subject.get[:global][:peer_address]).to eq('1.1.1.1')
     end
+
+    it 'negates the mlag peer address' do
+      expect(subject.set_peer_address(value: '1.1.1.1')).to be_truthy
+      expect(subject.get[:global][:peer_address]).to eq('1.1.1.1')
+      expect(subject.set_peer_address(enable: false)).to be_truthy
+      expect(subject.get[:global][:peer_address]).to be_empty
+    end
+
+    it 'defaults the mlag peer address' do
+      expect(subject.set_peer_address(value: '1.1.1.1')).to be_truthy
+      expect(subject.get[:global][:peer_address]).to eq('1.1.1.1')
+      expect(subject.set_peer_address(default: true)).to be_truthy
+      expect(subject.get[:global][:peer_address]).to be_empty
+    end
   end
 
   describe '#set_shutdown' do
     it 'configures mlag to be enabled' do
       node.config(['mlag configuration', 'shutdown'])
       expect(subject.get[:global][:shutdown]).to be_truthy
-      expect(subject.set_shutdown(value: false)).to be_truthy
+      expect(subject.set_shutdown(enable: true)).to be_truthy
       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[:global][:shutdown]).to be_falsy
-      expect(subject.set_shutdown(value: true)).to be_truthy
+      expect(subject.set_shutdown(enable: false)).to be_truthy
       expect(subject.get[:global][:shutdown]).to be_truthy
     end
+
+    it 'defaults the shutdown value' do
+      expect(node).to receive(:config).with(['mlag configuration',
+                                             'default shutdown'])
+      expect(subject.set_shutdown(default: true)).to be_truthy
+    end
+  end
+
+  describe '#set_mlag_id' do
+    before do
+      node.config(['default mlag configuration',
+                   'default interface Ethernet1',
+                   'default interface Port-Channel20',
+                   'interface Ethernet1',
+                   'channel-group 20 mode active',
+                   'interface port-channel 20',
+                   'mlag 20'])
+    end
+
+    it 'configure the mlag id' do
+      expect(subject.get[:interfaces]['Port-Channel20'][:mlag_id]).to eq(20)
+      expect(subject.set_mlag_id('Port-Channel20', value: '1000')).to be_truthy
+      expect(subject.get[:interfaces]['Port-Channel20'][:mlag_id]).to eq(1000)
+    end
+
+    it 'negate the mlag id' do
+      expect(subject.get[:interfaces]['Port-Channel20'][:mlag_id]).to eq(20)
+      expect(subject.set_mlag_id('Port-Channel20', enable: false)).to be_truthy
+      expect(subject.get[:interfaces]['Port-Channel20']).to eq(nil)
+    end
+
+    it 'default the mlag id' do
+      expect(subject.get[:interfaces]['Port-Channel20'][:mlag_id]).to eq(20)
+      expect(subject.set_mlag_id('Port-Channel20', default: true)).to be_truthy
+      expect(subject.get[:interfaces]['Port-Channel20']).to eq(nil)
+    end
   end
 end

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

@@ -35,6 +35,20 @@ describe Rbeapi::Api::Ntp do
       expect(subject.set_source_interface(value: 'Loopback0')).to be_truthy
       expect(subject.get[:source_interface]).to eq('Loopback0')
     end
+
+    it 'negates the ntp source interface' do
+      expect(subject.set_source_interface(value: 'Loopback0')).to be_truthy
+      expect(subject.get[:source_interface]).to eq('Loopback0')
+      expect(subject.set_source_interface(enable: false)).to be_truthy
+      expect(subject.get[:source_interface]).to be_empty
+    end
+
+    it 'defaults the ntp source interface' do
+      expect(subject.set_source_interface(value: 'Loopback0')).to be_truthy
+      expect(subject.get[:source_interface]).to eq('Loopback0')
+      expect(subject.set_source_interface(default: true)).to be_truthy
+      expect(subject.get[:source_interface]).to be_empty
+    end
   end
 
   describe '#add_server' do

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

@@ -31,6 +31,20 @@ describe Rbeapi::Api::Snmp do
       expect(subject.set_location(value: 'foo')).to be_truthy
       expect(subject.get[:location]).to eq('foo')
     end
+
+    it 'negates the snmp location' do
+      expect(subject.set_location(value: 'foo')).to be_truthy
+      expect(subject.get[:location]).to eq('foo')
+      expect(subject.set_location(enable: false)).to be_truthy
+      expect(subject.get[:location]).to be_empty
+    end
+
+    it 'defaults the snmp location' do
+      expect(subject.set_location(value: 'foo')).to be_truthy
+      expect(subject.get[:location]).to eq('foo')
+      expect(subject.set_location(default: true)).to be_truthy
+      expect(subject.get[:location]).to be_empty
+    end
   end
 
   describe '#set_contact' do
@@ -41,6 +55,20 @@ describe Rbeapi::Api::Snmp do
       expect(subject.set_contact(value: 'foo')).to be_truthy
       expect(subject.get[:contact]).to eq('foo')
     end
+
+    it 'negates the snmp contact' do
+      expect(subject.set_contact(value: 'foo')).to be_truthy
+      expect(subject.get[:contact]).to eq('foo')
+      expect(subject.set_contact(enable: false)).to be_truthy
+      expect(subject.get[:contact]).to be_empty
+    end
+
+    it 'defaults the snmp contact' do
+      expect(subject.set_contact(value: 'foo')).to be_truthy
+      expect(subject.get[:contact]).to eq('foo')
+      expect(subject.set_contact(default: true)).to be_truthy
+      expect(subject.get[:contact]).to be_empty
+    end
   end
 
   describe '#set_chassis_id' do
@@ -51,6 +79,20 @@ describe Rbeapi::Api::Snmp do
       expect(subject.set_chassis_id(value: 'foo')).to be_truthy
       expect(subject.get[:chassis_id]).to eq('foo')
     end
+
+    it 'negates the chassis id' do
+      expect(subject.set_chassis_id(value: 'foo')).to be_truthy
+      expect(subject.get[:chassis_id]).to eq('foo')
+      expect(subject.set_chassis_id(enable: false)).to be_truthy
+      expect(subject.get[:chassis_id]).to be_empty
+    end
+
+    it 'defaults the chassis id' do
+      expect(subject.set_chassis_id(value: 'foo')).to be_truthy
+      expect(subject.get[:chassis_id]).to eq('foo')
+      expect(subject.set_chassis_id(default: true)).to be_truthy
+      expect(subject.get[:chassis_id]).to be_empty
+    end
   end
 
   describe '#set_source_interface' do
@@ -61,5 +103,68 @@ describe Rbeapi::Api::Snmp do
       expect(subject.set_source_interface(value: 'Loopback0')).to be_truthy
       expect(subject.get[:source_interface]).to eq('Loopback0')
     end
+
+    it 'negates the snmp source-interface' do
+      expect(subject.set_source_interface(value: 'Loopback0')).to be_truthy
+      expect(subject.get[:source_interface]).to eq('Loopback0')
+      expect(subject.set_source_interface(enable: false)).to be_truthy
+      expect(subject.get[:source_interface]).to be_empty
+    end
+
+    it 'defaults the snmp source-interface' do
+      expect(subject.set_source_interface(value: 'Loopback0')).to be_truthy
+      expect(subject.get[:source_interface]).to eq('Loopback0')
+      expect(subject.set_source_interface(default: true)).to be_truthy
+      expect(subject.get[:source_interface]).to be_empty
+    end
+  end
+
+  describe '#set_community_acl' do
+    before do
+      node.config(['no snmp-server community foo',
+                   'no snmp-server community bar'])
+    end
+
+    it 'configures nil acl for snmp community foo and bar' do
+      expect(subject.get[:communities]).to be_empty
+      expect(subject.set_community_acl('foo')).to be_truthy
+      expect(subject.get[:communities]['foo']).to eq(access: 'ro', acl: nil)
+      expect(subject.set_community_acl('bar')).to be_truthy
+      expect(subject.get[:communities]['bar']).to eq(access: 'ro', acl: nil)
+    end
+
+    it 'configures IPv4 acl for snmp community foo and bar' do
+      expect(subject.get[:communities]).to be_empty
+      expect(subject.set_community_acl('foo', value: 'eng')).to be_truthy
+      expect(subject.get[:communities]['foo']).to eq(access: 'ro', acl: 'eng')
+      expect(subject.set_community_acl('bar', value: 'eng')).to be_truthy
+      expect(subject.get[:communities]['bar']).to eq(access: 'ro', acl: 'eng')
+    end
+
+    it 'negates the snmp community ACL for bar' do
+      expect(subject.get[:communities]).to be_empty
+      expect(subject.set_community_acl('foo', value: 'eng')).to be_truthy
+      expect(subject.get[:communities]['foo']).to eq(access: 'ro', acl: 'eng')
+      expect(subject.set_community_acl('bar', value: 'eng')).to be_truthy
+      expect(subject.get[:communities]['bar']).to eq(access: 'ro', acl: 'eng')
+      # Remove bar
+      expect(subject.set_community_acl('bar', enable: false)).to be_truthy
+      expect(subject.get[:communities]['bar']).to be_falsy
+      # Make sure foo is still there
+      expect(subject.get[:communities]['foo']).to eq(access: 'ro', acl: 'eng')
+    end
+
+    it 'defaults the snmp community ACL for bar' do
+      expect(subject.get[:communities]).to be_empty
+      expect(subject.set_community_acl('foo', value: 'eng')).to be_truthy
+      expect(subject.get[:communities]['foo']).to eq(access: 'ro', acl: 'eng')
+      expect(subject.set_community_acl('bar', value: 'eng')).to be_truthy
+      expect(subject.get[:communities]['bar']).to eq(access: 'ro', acl: 'eng')
+      # Default bar
+      expect(subject.set_community_acl('bar', default: true)).to be_truthy
+      expect(subject.get[:communities]['bar']).to be_falsy
+      # Make sure foo is still there
+      expect(subject.get[:communities]['foo']).to eq(access: 'ro', acl: 'eng')
+    end
   end
 end

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

@@ -36,33 +36,70 @@ describe Rbeapi::Api::StpInterfaces do
   end
 
   describe '#set_portfast' do
-    it 'sets the portfast value to true' do
+    it 'enables portfast' do
       node.config(['interface Ethernet1', 'no spanning-tree portfast'])
       expect(subject.get('Ethernet1')[:portfast]).to be_falsy
-      expect(subject.set_portfast('Ethernet1', value: true)).to be_truthy
+      expect(subject.set_portfast('Ethernet1', enable: true)).to be_truthy
       expect(subject.get('Ethernet1')[:portfast]).to be_truthy
     end
 
-    it 'sets the portfast value to false' do
+    it 'disable portfast' do
       node.config(['interface Ethernet1', 'spanning-tree portfast'])
       expect(subject.get('Ethernet1')[:portfast]).to be_truthy
-      expect(subject.set_portfast('Ethernet1', value: false)).to be_truthy
+      expect(subject.set_portfast('Ethernet1', enable: false)).to be_truthy
       expect(subject.get('Ethernet1')[:portfast]).to be_falsy
     end
+
+    it 'sets portfast to default value' do
+      node.config(['interface Ethernet1', 'spanning-tree portfast'])
+      expect(subject.get('Ethernet1')[:portfast]).to be_truthy
+      expect(subject.set_portfast('Ethernet1', default: true)).to be_truthy
+      expect(subject.get('Ethernet1')[:portfast]).to be_falsy
+    end
+  end
+
+  describe '#set_portfast_type' do
+    it 'raises an ArgumentError if value is not set' do
+      expect { subject.set_portfast_type('Ethernet1') }
+        .to raise_error(ArgumentError)
+    end
+
+    it 'enables portfast type' do
+      node.config(['interface Ethernet1', 'no spanning-tree portfast'])
+      expect(subject.get('Ethernet1')[:portfast_type]).to eq('normal')
+      expect(subject.set_portfast_type('Ethernet1', value: 'edge')).to be_truthy
+      expect(subject.get('Ethernet1')[:portfast_type]).to eq('edge')
+    end
+
+    it 'disable portfast' do
+      node.config(['interface Ethernet1', 'spanning-tree portfast edge'])
+      expect(subject.get('Ethernet1')[:portfast_type]).to eq('edge')
+      expect(subject.set_portfast_type('Ethernet1', value: 'edge',
+                                                    enable: false)).to be_truthy
+      expect(subject.get('Ethernet1')[:portfast_type]).to eq('normal')
+    end
+
+    it 'sets portfast to default value' do
+      node.config(['interface Ethernet1', 'spanning-tree portfast network'])
+      expect(subject.get('Ethernet1')[:portfast_type]).to eq('network')
+      expect(subject.set_portfast_type('Ethernet1', value: 'network',
+                                                    default: true)).to be_truthy
+      expect(subject.get('Ethernet1')[:portfast_type]).to eq('normal')
+    end
   end
 
   describe '#set_bpduguard' do
     it 'sets the bpduguard value to true' do
       node.config(['interface Ethernet1', 'no spanning-tree bpduguard'])
       expect(subject.get('Ethernet1')[:bpduguard]).to be_falsy
-      expect(subject.set_bpduguard('Ethernet1', value: true)).to be_truthy
+      expect(subject.set_bpduguard('Ethernet1', enable: true)).to be_truthy
       expect(subject.get('Ethernet1')[:bpduguard]).to be_truthy
     end
 
     it 'sets the bpduguard value to false' do
       node.config(['interface Ethernet1', 'spanning-tree bpduguard enable'])
       expect(subject.get('Ethernet1')[:bpduguard]).to be_truthy
-      expect(subject.set_bpduguard('Ethernet1', value: false)).to be_truthy
+      expect(subject.set_bpduguard('Ethernet1', enable: false)).to be_truthy
       expect(subject.get('Ethernet1')[:bpduguard]).to be_falsy
     end
   end

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

@@ -46,12 +46,24 @@ describe Rbeapi::Api::Stp do
     end
 
     it 'sets the stp mode to none' do
-      node.config('spanning-tree mode mstp') do
-        node.config('spanning-tree mode mstp')
-        expect(subject.get[:mode]).to eq('mstp')
-        expect(subject.set_mode(value: 'none')).to be_truthy
-        expect(subject.get[:mode]).to eq('none')
-      end
+      node.config('spanning-tree mode mstp')
+      expect(subject.get[:mode]).to eq('mstp')
+      expect(subject.set_mode(value: 'none')).to be_truthy
+      expect(subject.get[:mode]).to eq('none')
+    end
+
+    it 'negates the stp mode' do
+      node.config('spanning-tree mode none')
+      expect(subject.get[:mode]).to eq('none')
+      expect(subject.set_mode(enable: false)).to be_truthy
+      expect(subject.get[:mode]).to eq('mstp')
+    end
+
+    it 'defaults the stp mode' do
+      node.config('spanning-tree mode none')
+      expect(subject.get[:mode]).to eq('none')
+      expect(subject.set_mode(default: true)).to be_truthy
+      expect(subject.get[:mode]).to eq('mstp')
     end
   end
 end

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

@@ -95,6 +95,20 @@ describe Rbeapi::Api::Switchports do
       expect(subject.set_mode('Ethernet1', value: 'trunk')).to be_truthy
       expect(subject.get('Ethernet1')[:mode]).to eq('trunk')
     end
+
+    it 'negate the mode value' do
+      node.config(['interface Ethernet1', 'switchport mode trunk'])
+      expect(subject.get('Ethernet1')[:mode]).to eq('trunk')
+      expect(subject.set_mode('Ethernet1', enable: false)).to be_truthy
+      expect(subject.get('Ethernet1')[:mode]).to eq('access')
+    end
+
+    it 'default the mode value' do
+      node.config(['interface Ethernet1', 'switchport mode trunk'])
+      expect(subject.get('Ethernet1')[:mode]).to eq('trunk')
+      expect(subject.set_mode('Ethernet1', default: true)).to be_truthy
+      expect(subject.get('Ethernet1')[:mode]).to eq('access')
+    end
   end
 
   describe '#set_access_vlan' do
@@ -105,6 +119,22 @@ describe Rbeapi::Api::Switchports do
       expect(subject.set_access_vlan('Ethernet1', value: '100')).to be_truthy
       expect(subject.get('Ethernet1')[:access_vlan]).to eq('100')
     end
+
+    it 'negates the access vlan value' do
+      expect(subject.get('Ethernet1')[:access_vlan]).to eq('1')
+      expect(subject.set_access_vlan('Ethernet1', value: '100')).to be_truthy
+      expect(subject.get('Ethernet1')[:access_vlan]).to eq('100')
+      expect(subject.set_access_vlan('Ethernet1', enable: false)).to be_truthy
+      expect(subject.get('Ethernet1')[:access_vlan]).to eq('1')
+    end
+
+    it 'defaults the access vlan value' do
+      expect(subject.get('Ethernet1')[:access_vlan]).to eq('1')
+      expect(subject.set_access_vlan('Ethernet1', value: '100')).to be_truthy
+      expect(subject.get('Ethernet1')[:access_vlan]).to eq('100')
+      expect(subject.set_access_vlan('Ethernet1', default: true)).to be_truthy
+      expect(subject.get('Ethernet1')[:access_vlan]).to eq('1')
+    end
   end
 
   describe '#set_trunk_native_vlan' do
@@ -116,6 +146,26 @@ describe Rbeapi::Api::Switchports do
         .to be_truthy
       expect(subject.get('Ethernet1')[:trunk_native_vlan]).to eq('100')
     end
+
+    it 'negates the trunk native vlan' do
+      expect(subject.get('Ethernet1')[:trunk_native_vlan]).to eq('1')
+      expect(subject.set_trunk_native_vlan('Ethernet1', value: '100'))
+        .to be_truthy
+      expect(subject.get('Ethernet1')[:trunk_native_vlan]).to eq('100')
+      expect(subject.set_trunk_native_vlan('Ethernet1', enable: false))
+        .to be_truthy
+      expect(subject.get('Ethernet1')[:trunk_native_vlan]).to eq('1')
+    end
+
+    it 'defaults the trunk native vlan' do
+      expect(subject.get('Ethernet1')[:trunk_native_vlan]).to eq('1')
+      expect(subject.set_trunk_native_vlan('Ethernet1', value: '100'))
+        .to be_truthy
+      expect(subject.get('Ethernet1')[:trunk_native_vlan]).to eq('100')
+      expect(subject.set_trunk_native_vlan('Ethernet1', default: true))
+        .to be_truthy
+      expect(subject.get('Ethernet1')[:trunk_native_vlan]).to eq('1')
+    end
   end
 
   describe '#set_trunk_allowed_vlans' do
@@ -126,12 +176,34 @@ describe Rbeapi::Api::Switchports do
         .to raise_error(ArgumentError)
     end
 
-    it 'sets vlan 100 to the trunk allowed vlans' do
+    it 'sets vlan 8 and 9 to the trunk allowed vlans' do
       node.config(['interface Ethernet1', 'switchport trunk allowed vlan none'])
       expect(subject.get('Ethernet1')[:trunk_allowed_vlans]).to be_empty
-      expect(subject.set_trunk_allowed_vlans('Ethernet1', value: [100]))
+      expect(subject.set_trunk_allowed_vlans('Ethernet1', value: [8, 9]))
+        .to be_truthy
+      expect(subject.get('Ethernet1')[:trunk_allowed_vlans]).to eq([8, 9])
+    end
+
+    it 'negate switchport trunk allowed vlan' do
+      node.config(['interface Ethernet1', 'switchport trunk allowed vlan none'])
+      expect(subject.get('Ethernet1')[:trunk_allowed_vlans]).to be_empty
+      expect(subject.set_trunk_allowed_vlans('Ethernet1', value: [8, 9]))
+        .to be_truthy
+      expect(subject.get('Ethernet1')[:trunk_allowed_vlans]).to eq([8, 9])
+      expect(subject.set_trunk_allowed_vlans('Ethernet1', enable: false))
+        .to be_truthy
+      expect(subject.get('Ethernet1')[:trunk_allowed_vlans].length).to eq(4094)
+    end
+
+    it 'default switchport trunk allowed vlan' do
+      node.config(['interface Ethernet1', 'switchport trunk allowed vlan none'])
+      expect(subject.get('Ethernet1')[:trunk_allowed_vlans]).to be_empty
+      expect(subject.set_trunk_allowed_vlans('Ethernet1', value: [8, 9]))
+        .to be_truthy
+      expect(subject.get('Ethernet1')[:trunk_allowed_vlans]).to eq([8, 9])
+      expect(subject.set_trunk_allowed_vlans('Ethernet1', default: true))
         .to be_truthy
-      expect(subject.get('Ethernet1')[:trunk_allowed_vlans]).to include(100)
+      expect(subject.get('Ethernet1')[:trunk_allowed_vlans].length).to eq(4094)
     end
   end
 end

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

@@ -31,5 +31,21 @@ describe Rbeapi::Api::System do
       expect(subject.set_hostname(value: 'foo')).to be_truthy
       expect(subject.get[:hostname]).to eq('foo')
     end
+
+    it 'configures the system hostname with a dot value' do
+      expect(subject.get[:hostname]).to eq('localhost')
+      expect(subject.set_hostname(value: 'foo.bar')).to be_truthy
+      expect(subject.get[:hostname]).to eq('foo.bar')
+    end
+
+    it 'negates the hostname' do
+      expect(subject.set_hostname(enable: false)).to be_truthy
+      expect(subject.get[:hostname]).to be_empty
+    end
+
+    it 'defaults the hostname' do
+      expect(subject.set_hostname(default: true)).to be_truthy
+      expect(subject.get[:hostname]).to be_empty
+    end
   end
 end

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

@@ -91,6 +91,20 @@ describe Rbeapi::Api::Vlans do
       expect(subject.set_name('1', value: 'foo')).to be_truthy
       expect(subject.get('1')[:name]).to eq('foo')
     end
+
+    it 'negates the vlan name' do
+      expect(subject.set_name('1', value: 'foo')).to be_truthy
+      expect(subject.get('1')[:name]).to eq('foo')
+      expect(subject.set_name('1', enable: false)).to be_truthy
+      expect(subject.get('1')[:name]).to eq('default')
+    end
+
+    it 'defaults the vlan name' do
+      expect(subject.set_name('1', value: 'foo')).to be_truthy
+      expect(subject.get('1')[:name]).to eq('foo')
+      expect(subject.set_name('1', default: true)).to be_truthy
+      expect(subject.get('1')[:name]).to eq('default')
+    end
   end
 
   describe '#set_state' do
@@ -107,6 +121,20 @@ describe Rbeapi::Api::Vlans do
       expect(subject.set_state('1', value: 'active')).to be_truthy
       expect(subject.get('1')[:state]).to eq('active')
     end
+
+    it 'negate vlan 1 state' do
+      node.config(['vlan 1', 'state suspend'])
+      expect(subject.get('1')[:state]).to eq('suspend')
+      expect(subject.set_state('1', enable: false)).to be_truthy
+      expect(subject.get('1')[:state]).to eq('active')
+    end
+
+    it 'set vlan 1 state to default' do
+      node.config(['vlan 1', 'state suspend'])
+      expect(subject.get('1')[:state]).to eq('suspend')
+      expect(subject.set_state('1', default: true)).to be_truthy
+      expect(subject.get('1')[:state]).to eq('active')
+    end
   end
 
   describe '#add_trunk_group' do