rbeapi 0.2.0 → 0.3.0

data/spec/unit/rbeapi/api/bgp/bgp_neighbors_spec.rb

@@ -0,0 +1,289 @@
+#
+# Copyright (c) 2015, Arista Networks, Inc.
+# All rights reserved.
+#
+# Redistribution and use in source and binary forms, with or without
+# modification, are permitted provided that the following conditions are
+# met:
+#
+#   Redistributions of source code must retain the above copyright notice,
+#   this list of conditions and the following disclaimer.
+#
+#   Redistributions in binary form must reproduce the above copyright
+#   notice, this list of conditions and the following disclaimer in the
+#   documentation and/or other materials provided with the distribution.
+#
+#   Neither the name of Arista Networks nor the names of its
+#   contributors may be used to endorse or promote products derived from
+#   this software without specific prior written permission.
+#
+# THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+# "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+# LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+# A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL ARISTA NETWORKS
+# BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
+# CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
+# SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR
+# BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY,
+# WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE
+# OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN
+# IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+#
+require 'spec_helper'
+
+require 'rbeapi/api/bgp'
+
+include FixtureHelpers
+
+describe Rbeapi::Api::BgpNeighbors do
+  subject { described_class.new(node) }
+
+  let(:node) { double('node') }
+
+  let(:test) do
+    { bgp_as: '64600',
+      router_id: '192.168.254.1',
+      shutdown: false,
+      networks: [
+        { prefix: '192.168.254.1', masklen: 32, route_map: nil },
+        { prefix: '192.168.254.2', masklen: 32, route_map: 'rmap' },
+        { prefix: '192.168.254.3', masklen: 32, route_map: nil }
+      ],
+      neighbors: {
+        'eBGP_GROUP' => {
+          peer_group: nil, remote_as: nil, send_community: false,
+          shutdown: false, description: nil, next_hop_self: false,
+          route_map_in: nil, route_map_out: nil
+        },
+        '192.168.255.1' => {
+          peer_group: 'eBGP_GROUP', remote_as: '65000', send_community: true,
+          shutdown: true, description: nil, next_hop_self: true,
+          route_map_in: nil, route_map_out: nil
+        },
+        '192.168.255.3' => {
+          peer_group: 'eBGP_GROUP', remote_as: '65001', send_community: true,
+          shutdown: true, description: nil, next_hop_self: true,
+          route_map_in: nil, route_map_out: nil
+        }
+      }
+    }
+  end
+  let(:bgp_as) { test[:bgp_as] }
+
+  def bgp
+    bgp = Fixtures[:bgp]
+    return bgp if bgp
+    fixture('bgp', format: :text, dir: File.dirname(__FILE__))
+  end
+
+  before :each do
+    allow(subject.node).to receive(:running_config).and_return(bgp)
+  end
+
+  describe '#get' do
+    it 'returns the BGP neighbor resource' do
+      expect(subject.get('eBGP_GROUP')).to eq(test[:neighbors]['eBGP_GROUP'])
+    end
+  end
+
+  describe '#getall' do
+    it 'returns all the neighbors' do
+      expect(subject.getall).to eq(test[:neighbors])
+    end
+
+    it 'returns a hash collection' do
+      expect(subject.getall).to be_a_kind_of(Hash)
+    end
+
+    it 'has three entries' do
+      expect(subject.getall.size).to eq(3)
+    end
+  end
+
+  describe '#create' do
+    it 'create a new BGP neighbor' do
+      expect(node).to receive(:config).with(['router bgp 64600',
+                                             'neighbor edge shutdown'])
+      expect(subject.create('edge')).to be_truthy
+    end
+  end
+
+  describe '#delete' do
+    it 'delete a BGP resource' do
+      expect(node).to receive(:config).with(['router bgp 64600',
+                                             'neighbor edge shutdown'])
+      expect(subject.create('edge')).to be_truthy
+      expect(node).to receive(:config).with(['router bgp 64600',
+                                             'no neighbor edge'])
+      expect(subject.delete('edge')).to be_truthy
+    end
+  end
+
+  describe '#set_peer_group' do
+    it 'set the peer group' do
+      expect(node).to receive(:config).with(['router bgp 64600',
+                                             'neighbor eng peer-group 1.2.3.4'])
+      expect(subject.set_peer_group('eng', value: '1.2.3.4')).to be_truthy
+    end
+
+    it 'remove the peer group value' do
+      expect(node).to receive(:config).with(['router bgp 64600',
+                                             'no neighbor eng peer-group'])
+      expect(subject.set_peer_group('eng', enable: false))
+        .to be_truthy
+    end
+
+    it 'defaults the peer group value' do
+      expect(node).to receive(:config).with(['router bgp 64600',
+                                             'default neighbor eng peer-group'])
+      expect(subject.set_peer_group('eng', default: true))
+        .to be_truthy
+    end
+  end
+
+  describe '#set_remote_as' do
+    it 'set the remote AS value' do
+      expect(node).to receive(:config).with(['router bgp 64600',
+                                             'neighbor eng remote-as 10'])
+      expect(subject.set_remote_as('eng', value: '10')).to be_truthy
+    end
+
+    it 'remove the remote AS value' do
+      expect(node).to receive(:config).with(['router bgp 64600',
+                                             'no neighbor eng remote-as'])
+      expect(subject.set_remote_as('eng', enable: false))
+        .to be_truthy
+    end
+
+    it 'defaults the remote AS value' do
+      expect(node).to receive(:config).with(['router bgp 64600',
+                                             'default neighbor eng remote-as'])
+      expect(subject.set_remote_as('eng', default: true))
+        .to be_truthy
+    end
+  end
+
+  describe '#set_shutdown' do
+    it 'shutdown neighbor' do
+      expect(node).to receive(:config).with(['router bgp 64600',
+                                             'neighbor eng shutdown'])
+      expect(subject.set_shutdown('eng')).to be_truthy
+    end
+
+    it 'negate shutdown neighbor' do
+      expect(node).to receive(:config).with(['router bgp 64600',
+                                             'neighbor eng shutdown'])
+      expect(subject.set_shutdown('eng', enable: false)).to be_truthy
+    end
+
+    it 'default shutdown neighbor' do
+      expect(node).to receive(:config).with(['router bgp 64600',
+                                             'default neighbor eng shutdown'])
+      expect(subject.set_shutdown('eng', default: true)).to be_truthy
+    end
+  end
+
+  describe '#set_send_community' do
+    it 'enable neighbor send community' do
+      expect(node).to receive(:config).with(['router bgp 64600',
+                                             'neighbor eng send-community'])
+      expect(subject.set_send_community('eng')).to be_truthy
+    end
+
+    it 'negate neighbor send community' do
+      expect(node).to receive(:config).with(['router bgp 64600',
+                                             'no neighbor eng send-community'])
+      expect(subject.set_send_community('eng', enable: false)).to be_truthy
+    end
+
+    it 'default neighbor send community' do
+      expect(node).to receive(:config)
+        .with(['router bgp 64600', 'default neighbor eng send-community'])
+      expect(subject.set_send_community('eng', default: true)).to be_truthy
+    end
+  end
+
+  describe '#set_next_hop_self' do
+    it 'enable neighbor next hop self' do
+      expect(node).to receive(:config).with(['router bgp 64600',
+                                             'neighbor eng next-hop-self'])
+      expect(subject.set_next_hop_self('eng')).to be_truthy
+    end
+
+    it 'negate neighbor next hop self' do
+      expect(node).to receive(:config).with(['router bgp 64600',
+                                             'no neighbor eng next-hop-self'])
+      expect(subject.set_next_hop_self('eng', enable: false)).to be_truthy
+    end
+
+    it 'default neighbor next hop self' do
+      expect(node).to receive(:config)
+        .with(['router bgp 64600', 'default neighbor eng next-hop-self'])
+      expect(subject.set_next_hop_self('eng', default: true)).to be_truthy
+    end
+  end
+
+  describe '#set_route_map_in' do
+    it 'set route map in value' do
+      expect(node).to receive(:config).with(['router bgp 64600',
+                                             'neighbor eng route-map edge in'])
+      expect(subject.set_route_map_in('eng', value: 'edge')).to be_truthy
+    end
+
+    it 'negate route map in value' do
+      expect(node).to receive(:config)
+        .with(['router bgp 64600', 'no neighbor eng route-map edge in'])
+      expect(subject.set_route_map_in('eng', value: 'edge', enable: false))
+        .to be_truthy
+    end
+
+    it 'default route map in value' do
+      expect(node).to receive(:config)
+        .with(['router bgp 64600', 'default neighbor eng route-map edge in'])
+      expect(subject.set_route_map_in('eng', value: 'edge', default: true))
+        .to be_truthy
+    end
+  end
+
+  describe '#set_route_map_out' do
+    it 'set route map out value' do
+      expect(node).to receive(:config).with(['router bgp 64600',
+                                             'neighbor eng route-map edge out'])
+      expect(subject.set_route_map_out('eng', value: 'edge')).to be_truthy
+    end
+
+    it 'negate route map out value' do
+      expect(node).to receive(:config)
+        .with(['router bgp 64600', 'no neighbor eng route-map edge out'])
+      expect(subject.set_route_map_out('eng', value: 'edge', enable: false))
+        .to be_truthy
+    end
+
+    it 'default route map out value' do
+      expect(node).to receive(:config)
+        .with(['router bgp 64600', 'default neighbor eng route-map edge out'])
+      expect(subject.set_route_map_out('eng', value: 'edge', default: true))
+        .to be_truthy
+    end
+  end
+
+  describe '#set_description' do
+    it 'set the description value' do
+      expect(node).to receive(:config).with(['router bgp 64600',
+                                             'neighbor eng description text'])
+      expect(subject.set_description('eng', value: 'text')).to be_truthy
+    end
+
+    it 'negate the description value' do
+      expect(node).to receive(:config)
+        .with(['router bgp 64600', 'no neighbor eng description'])
+      expect(subject.set_description('eng', enable: false)).to be_truthy
+    end
+
+    it 'defaults the description value' do
+      expect(node).to receive(:config)
+        .with(['router bgp 64600', 'default neighbor eng description'])
+      expect(subject.set_description('eng', default: true)).to be_truthy
+    end
+  end
+end

data/spec/unit/rbeapi/api/bgp/bgp_spec.rb

@@ -0,0 +1,192 @@
+#
+# Copyright (c) 2015, Arista Networks, Inc.
+# All rights reserved.
+#
+# Redistribution and use in source and binary forms, with or without
+# modification, are permitted provided that the following conditions are
+# met:
+#
+#   Redistributions of source code must retain the above copyright notice,
+#   this list of conditions and the following disclaimer.
+#
+#   Redistributions in binary form must reproduce the above copyright
+#   notice, this list of conditions and the following disclaimer in the
+#   documentation and/or other materials provided with the distribution.
+#
+#   Neither the name of Arista Networks nor the names of its
+#   contributors may be used to endorse or promote products derived from
+#   this software without specific prior written permission.
+#
+# THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+# "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+# LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+# A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL ARISTA NETWORKS
+# BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
+# CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
+# SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR
+# BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY,
+# WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE
+# OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN
+# IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+#
+require 'spec_helper'
+
+require 'rbeapi/api/bgp'
+
+include FixtureHelpers
+
+describe Rbeapi::Api::Bgp do
+  subject { described_class.new(node) }
+
+  let(:node) { double('node') }
+
+  let(:test) do
+    { bgp_as: '64600',
+      router_id: '192.168.254.1',
+      shutdown: false,
+      networks: [
+        { prefix: '192.168.254.1', masklen: 32, route_map: nil },
+        { prefix: '192.168.254.2', masklen: 32, route_map: 'rmap' },
+        { prefix: '192.168.254.3', masklen: 32, route_map: nil }
+      ],
+      neighbors: {
+        'eBGP_GROUP' => {
+          peer_group: nil, remote_as: nil, send_community: false,
+          shutdown: false, description: nil, next_hop_self: false,
+          route_map_in: nil, route_map_out: nil
+        },
+        '192.168.255.1' => {
+          peer_group: 'eBGP_GROUP', remote_as: '65000', send_community: true,
+          shutdown: true, description: nil, next_hop_self: true,
+          route_map_in: nil, route_map_out: nil
+        },
+        '192.168.255.3' => {
+          peer_group: 'eBGP_GROUP', remote_as: '65001', send_community: true,
+          shutdown: true, description: nil, next_hop_self: true,
+          route_map_in: nil, route_map_out: nil
+        }
+      }
+    }
+  end
+  let(:bgp_as) { test[:bgp_as] }
+
+  def bgp
+    bgp = Fixtures[:bgp]
+    return bgp if bgp
+    fixture('bgp', format: :text, dir: File.dirname(__FILE__))
+  end
+
+  before :each do
+    allow(subject.node).to receive(:running_config).and_return(bgp)
+  end
+
+  describe '#get' do
+    it 'returns the BGP resource' do
+      expect(subject.get).to eq(test)
+    end
+  end
+
+  describe '#create' do
+    it 'create a new BGP resource' do
+      expect(node).to receive(:config).with('router bgp 1000')
+      expect(subject.create('1000')).to be_truthy
+    end
+  end
+
+  describe '#delete' do
+    it 'delete a BGP resource' do
+      expect(node).to receive(:config).with("no router bgp #{bgp_as}")
+      expect(subject.delete).to be_truthy
+    end
+  end
+
+  describe '#default' do
+    it 'sets router to default value' do
+      expect(node).to receive(:config)
+        .with('default router bgp 64600')
+      expect(subject.default).to be_truthy
+    end
+  end
+
+  describe '#set_router_id' do
+    it 'set the router id' do
+      expect(node).to receive(:config).with(['router bgp 64600',
+                                             'router-id 1.2.3.4'])
+      expect(subject.set_router_id(value: '1.2.3.4')).to be_truthy
+    end
+
+    it 'remove the router-id without a value' do
+      expect(node).to receive(:config).with(['router bgp 64600',
+                                             'no router-id'])
+      expect(subject.set_router_id(enable: false)).to be_truthy
+    end
+
+    it 'remove the router-id with a value' do
+      expect(node).to receive(:config).with(['router bgp 64600',
+                                             'no router-id 1.2.3.4'])
+      expect(subject.set_router_id(value: '1.2.3.4', enable: false))
+        .to be_truthy
+    end
+
+    it 'defaults the router-id without a value' do
+      expect(node).to receive(:config).with(['router bgp 64600',
+                                             'default router-id'])
+      expect(subject.set_router_id(default: true)).to be_truthy
+    end
+
+    it 'defaults the router-id with a value' do
+      expect(node).to receive(:config).with(['router bgp 64600',
+                                             'default router-id 1.2.3.4'])
+      expect(subject.set_router_id(value: '1.2.3.4', default: true))
+        .to be_truthy
+    end
+  end
+
+  describe '#set_shutdown' do
+    it 'enable BGP routing process' do
+      expect(node).to receive(:config).with(['router bgp 64600',
+                                             'no shutdown'])
+      expect(subject.set_shutdown(enable: true)).to be_truthy
+    end
+
+    it 'disable BGP routing process' do
+      expect(node).to receive(:config).with(['router bgp 64600',
+                                             'shutdown'])
+      expect(subject.set_shutdown(enable: false)).to be_truthy
+    end
+
+    it 'default BGP routing process state' do
+      expect(node).to receive(:config).with(['router bgp 64600',
+                                             'default shutdown'])
+      expect(subject.set_shutdown(default: true)).to be_truthy
+    end
+  end
+
+  describe '#add_network' do
+    it 'add a BGP network with a route map' do
+      expect(node).to receive(:config)
+        .with(['router bgp 64600', 'network 1.2.3.0/24 route-map eng'])
+      expect(subject.add_network('1.2.3.0', 24, 'eng')).to be_truthy
+    end
+
+    it 'add a BGP network without a route map' do
+      expect(node).to receive(:config).with(['router bgp 64600',
+                                             'network 1.2.3.0/24'])
+      expect(subject.add_network('1.2.3.0', 24)).to be_truthy
+    end
+  end
+
+  describe '#remove_network' do
+    it 'remove a BGP network with a route map' do
+      expect(node).to receive(:config)
+        .with(['router bgp 64600', 'no network 1.2.3.0/24 route-map eng'])
+      expect(subject.remove_network('1.2.3.0', 24, 'eng')).to be_truthy
+    end
+
+    it 'remove a BGP network without a route map' do
+      expect(node).to receive(:config).with(['router bgp 64600',
+                                             'no network 1.2.3.0/24'])
+      expect(subject.remove_network('1.2.3.0', 24)).to be_truthy
+    end
+  end
+end

data/spec/unit/rbeapi/api/bgp/fixture_bgp.text

@@ -0,0 +1,101 @@
+router bgp 64600
+   no shutdown
+   router-id 192.168.254.1
+   bgp convergence time 300
+   bgp convergence slow-peer time 90
+   no bgp confederation identifier
+   no update wait-for-convergence
+   no update wait-install
+   bgp log-neighbor-changes
+   bgp default ipv4-unicast
+   no bgp default ipv6-unicast
+   timers bgp 60 180
+   distance bgp 20 200 200
+   graceful-restart stalepath-time 300
+   no bgp cluster-id
+   bgp client-to-client reflection
+   graceful-restart-helper
+   bgp peer-mac-resolution-timeout 0
+   bgp enforce-first-as
+   no default-metric
+   no bgp always-compare-med
+   no bgp bestpath med missing-as-worst
+   no bgp bestpath med confed
+   no bgp host-routes fib direct-install
+   maximum-paths 32 ecmp 32
+   bgp listen limit 1000
+   no bgp aspath-cmp-include-nexthop
+   no bgp tie-break-on-age
+   no bgp tie-break-on-router-id
+   no bgp tie-break-on-originator-id
+   no bgp tie-break-on-cluster-list-length
+   neighbor eBGP_GROUP peer-group
+   no neighbor eBGP_GROUP remote-as
+   no neighbor eBGP_GROUP import-localpref
+   no neighbor eBGP_GROUP export-localpref
+   no neighbor eBGP_GROUP description
+   no neighbor eBGP_GROUP next-hop-self
+   no neighbor eBGP_GROUP next-hop-peer
+   no neighbor eBGP_GROUP allowas-in
+   no neighbor eBGP_GROUP send-community
+   no neighbor eBGP_GROUP shutdown
+   no neighbor eBGP_GROUP remove-private-as
+   no neighbor eBGP_GROUP out-delay
+   no neighbor eBGP_GROUP local-as
+   no neighbor eBGP_GROUP weight
+   no neighbor eBGP_GROUP transport connection-mode passive
+   no neighbor eBGP_GROUP update-source
+   no neighbor eBGP_GROUP dont-capability-negotiate
+   neighbor eBGP_GROUP fall-over bfd
+   no neighbor eBGP_GROUP local-v6-addr
+   neighbor eBGP_GROUP soft-reconfiguration inbound
+   no neighbor eBGP_GROUP ebgp-multihop
+   no neighbor eBGP_GROUP route-reflector-client
+   no neighbor eBGP_GROUP timers
+   no neighbor eBGP_GROUP route-map in
+   neighbor eBGP_GROUP graceful-restart-helper
+   no neighbor eBGP_GROUP route-map out
+   no neighbor eBGP_GROUP prefix-list in
+   no neighbor eBGP_GROUP prefix-list out
+   neighbor eBGP_GROUP password 7 6C/HkrwJ53+dsHumQf49Wg==
+   no neighbor eBGP_GROUP default-originate
+   neighbor eBGP_GROUP enforce-first-as
+   no neighbor eBGP_GROUP metric-out
+   neighbor eBGP_GROUP maximum-routes 12000 
+   neighbor 192.168.255.1 peer-group eBGP_GROUP
+   neighbor 192.168.255.1 remote-as 65000
+   neighbor 192.168.255.3 peer-group eBGP_GROUP
+   neighbor 192.168.255.3 remote-as 65001
+   network 192.168.254.1/32
+   network 192.168.254.2/32 route-map rmap
+   network 192.168.254.3/32 rmap
+   bgp redistribute-internal
+   aggregate-address 192.168.255.0/28 summary-only
+   no redistribute connected
+   no redistribute ospf match internal
+   no redistribute ospf match external
+   no redistribute ospf match nssa-external
+   no redistribute ospf3 match internal
+   no redistribute ospf3 match external
+   no redistribute static
+   no redistribute rip
+   no redistribute aggregate
+   address-family ipv4
+      default neighbor eBGP_GROUP activate
+      no neighbor eBGP_GROUP route-map in
+      no neighbor eBGP_GROUP route-map out
+      no neighbor eBGP_GROUP default-originate
+      default neighbor 192.168.255.1 activate
+      default neighbor 192.168.255.3 activate
+   !
+   address-family ipv6
+      default neighbor eBGP_GROUP activate
+      no neighbor eBGP_GROUP route-map in
+      no neighbor eBGP_GROUP route-map out
+      no neighbor eBGP_GROUP prefix-list in
+      no neighbor eBGP_GROUP prefix-list out
+      no neighbor eBGP_GROUP default-originate
+      default neighbor 192.168.255.1 activate
+      default neighbor 192.168.255.3 activate
+   !
+!

data/spec/unit/rbeapi/api/interfaces/base_spec.rb

@@ -71,7 +71,7 @@ describe Rbeapi::Api::BaseInterface do
     it 'negates the interface description' do
       expect(node).to receive(:config).with(['interface Loopback0',
                                              'no description'])
-      expect(subject.set_description('Loopback0')).to be_truthy
+      expect(subject.set_description('Loopback0', enable: false)).to be_truthy
     end
 
     it 'defaults the interface description' do
@@ -80,10 +80,10 @@ describe Rbeapi::Api::BaseInterface do
       expect(subject.set_description('Loopback0', default: true)).to be_truthy
     end
 
-    it 'default is preferred over value' do
+    it 'default is preferred over enable' do
       expect(node).to receive(:config).with(['interface Loopback0',
                                              'default description'])
-      expect(subject.set_description('Loopback0', value: 'test',
+      expect(subject.set_description('Loopback0', enable: false,
                                                   default: true)).to be_truthy
     end
   end
@@ -92,19 +92,13 @@ describe Rbeapi::Api::BaseInterface do
     it 'enables the interface' do
       expect(node).to receive(:config).with(['interface Loopback0',
                                              'no shutdown'])
-      expect(subject.set_shutdown('Loopback0', value: false)).to be_truthy
+      expect(subject.set_shutdown('Loopback0', enable: true)).to be_truthy
     end
 
     it 'disables the interface' do
       expect(node).to receive(:config).with(['interface Loopback0',
                                              'shutdown'])
-      expect(subject.set_shutdown('Loopback0', value: true)).to be_truthy
-    end
-
-    it 'negates the interface description' do
-      expect(node).to receive(:config).with(['interface Loopback0',
-                                             'no shutdown'])
-      expect(subject.set_shutdown('Loopback0')).to be_truthy
+      expect(subject.set_shutdown('Loopback0', enable: false)).to be_truthy
     end
 
     it 'defaults the interface state' do
@@ -113,10 +107,10 @@ describe Rbeapi::Api::BaseInterface do
       expect(subject.set_shutdown('Loopback0', default: true)).to be_truthy
     end
 
-    it 'default is preferred over value' do
+    it 'default is preferred over enable' do
       expect(node).to receive(:config).with(['interface Loopback0',
                                              'default shutdown'])
-      expect(subject.set_shutdown('Loopback0', value: 'test',
+      expect(subject.set_shutdown('Loopback0', enable: false,
                                                default: true)).to be_truthy
     end
   end

data/spec/unit/rbeapi/api/interfaces/ethernet_spec.rb

@@ -70,7 +70,7 @@ describe Rbeapi::Api::EthernetInterface do
     it 'negates the interface description' do
       expect(node).to receive(:config).with(['interface Ethernet1',
                                              'no description'])
-      expect(subject.set_description('Ethernet1')).to be_truthy
+      expect(subject.set_description('Ethernet1', enable: false)).to be_truthy
     end
 
     it 'defaults the interface description' do
@@ -79,10 +79,10 @@ describe Rbeapi::Api::EthernetInterface do
       expect(subject.set_description('Ethernet1', default: true)).to be_truthy
     end
 
-    it 'default is preferred over value' do
+    it 'default is preferred over enable' do
       expect(node).to receive(:config).with(['interface Ethernet1',
                                              'default description'])
-      expect(subject.set_description('Ethernet1', value: 'test',
+      expect(subject.set_description('Ethernet1', enable: false,
                                                   default: true)).to be_truthy
     end
   end