rbeapi 0.3.0 → 0.4.0

data/spec/system/{api_varp_interfaces_spec.rb → rbeapi/api/varp_interfaces_spec.rb}

@@ -6,18 +6,21 @@ require 'rbeapi/api/varp'
 describe Rbeapi::Api::VarpInterfaces do
   subject { described_class.new(node) }
 
-  let(:config) { Rbeapi::Client::Config.new(filename: get_fixture('dut.conf')) }
-  let(:node) { Rbeapi::Client.connect_to('dut') }
+  let(:node) do
+    Rbeapi::Client.config.read(fixture_file('dut.conf'))
+    Rbeapi::Client.connect_to('dut')
+  end
 
   describe '#get' do
     before do
       node.config(['ip virtual-router mac-address aabb.ccdd.eeff',
-                   'default interface vlan 100', 'interface vlan 100',
+                   'no interface Vlan99', 'no interface Vlan100',
+                   'default interface Vlan100', 'interface Vlan100',
                    'ip address 99.99.99.99/24',
                    'ip virtual-router address 99.99.99.98', 'exit'])
     end
 
-    it 'returns an instance for vlan 100' do
+    it 'returns an instance for Vlan100' do
       expect(subject.get('Vlan100')).not_to be_nil
     end
 
@@ -29,12 +32,13 @@ describe Rbeapi::Api::VarpInterfaces do
   describe '#getall' do
     before do
       node.config(['ip virtual-router mac-address aabb.ccdd.eeff',
-                   'default interface vlan 100', 'interface vlan 100',
+                   'no interface Vlan99', 'no interface Vlan100',
+                   'default interface Vlan100', 'interface Vlan100',
                    'ip address 99.99.99.99/24',
                    'ip virtual-router address 99.99.99.98', 'exit'])
     end
 
-    it 'returns a collection that includes vlan 100' do
+    it 'returns a collection that includes Vlan100' do
       expect(subject.getall).to include('Vlan100')
     end
 
@@ -46,40 +50,45 @@ describe Rbeapi::Api::VarpInterfaces do
   describe '#set_addresses' do
     before do
       node.config(['ip virtual-router mac-address aabb.ccdd.eeff',
-                   'default interface vlan 100', 'interface vlan 100',
+                   'no interface Vlan99', 'no interface Vlan100',
+                   'default interface Vlan100', 'interface Vlan100',
                    'ip address 99.99.99.99/24', 'exit'])
     end
 
     it 'adds new address to the list of addresses' do
-      expect(subject.get('Vlan100')['addresses']).not_to include('99.99.99.98')
+      expect(subject.get('Vlan100')[:addresses]).not_to include('99.99.99.98')
       expect(subject.set_addresses('Vlan100', value: ['99.99.99.98']))
         .to be_truthy
-      expect(subject.get('Vlan100')['addresses']).to include('99.99.99.98')
+      expect(subject.get('Vlan100')[:addresses]).to include('99.99.99.98')
     end
 
-    it 'removes address to the list of addresses' do
+    it 'removes address from the list of addresses' do
       node.config(['interface vlan 100', 'ip address 99.99.99.99/24',
                    'ip virtual-router address 99.99.99.98'])
-      expect(subject.get('Vlan100')['addresses']).to include('99.99.99.98')
+      expect(subject.get('Vlan100')[:addresses]).to include('99.99.99.98')
       expect(subject.set_addresses('Vlan100', value: ['99.99.99.97']))
         .to be_truthy
-      expect(subject.get('Vlan100')['addresses']).not_to include('99.99.99.98')
+      expect(subject.get('Vlan100')[:addresses]).not_to include('99.99.99.98')
     end
 
     it 'negate the list of addresses' do
       expect(subject.set_addresses('Vlan100', value: ['99.99.99.98']))
         .to be_truthy
-      expect(subject.get('Vlan100')['addresses']).to include('99.99.99.98')
+      expect(subject.get('Vlan100')[:addresses]).to include('99.99.99.98')
       expect(subject.set_addresses('Vlan100', enable: false)).to be_truthy
-      expect(subject.get('Vlan100')['addresses']).to be_empty
+      expect(subject.get('Vlan100')[:addresses]).to be_empty
     end
 
     it 'default the list of addresses' do
       expect(subject.set_addresses('Vlan100', value: ['99.99.99.98']))
         .to be_truthy
-      expect(subject.get('Vlan100')['addresses']).to include('99.99.99.98')
+      expect(subject.get('Vlan100')[:addresses]).to include('99.99.99.98')
       expect(subject.set_addresses('Vlan100', default: true)).to be_truthy
-      expect(subject.get('Vlan100')['addresses']).to be_empty
+      expect(subject.get('Vlan100')[:addresses]).to be_empty
+    end
+
+    it 'can not evaluate without addresses' do
+      expect { subject.set_addresses('Vlan100') }.to raise_error ArgumentError
     end
   end
 end

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

@@ -0,0 +1,76 @@
+require 'spec_helper'
+
+require 'rbeapi/client'
+require 'rbeapi/api/varp'
+
+describe Rbeapi::Api::Varp 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(:resource) { subject.get }
+
+    before do
+      node.config(['no ip virtual-router mac-address',
+                   'no interface Vlan99', 'no interface Vlan100',
+                   'ip virtual-router mac-address aa:bb:cc:dd:ee:ff',
+                   'interface Vlan99', 'interface Vlan100'])
+    end
+
+    it 'returns a varp resource instance' do
+      expect(subject.get).to be_a_kind_of(Hash)
+    end
+
+    it 'has a key for mac_address' do
+      expect(subject.get).to include(:mac_address)
+    end
+
+    it 'has a key for interfaces' do
+      expect(subject.get).to include(:interfaces)
+    end
+  end
+
+  describe '#interfaces' do
+    it 'is a kind of VarpInterfaces' do
+      expect(subject.interfaces).to be_a_kind_of(Rbeapi::Api::VarpInterfaces)
+    end
+  end
+
+  describe '#set_mac_address' do
+    before { node.config('no ip virtual-router mac-address') }
+
+    it 'set mac-address to aa:bb:cc:dd:ee:ff' do
+      expect(subject.get[:mac_address]).to be_empty
+      expect(subject.set_mac_address(value: 'aa:bb:cc:dd:ee:ff')).to be_truthy
+      expect(subject.get[:mac_address]).to eq('aa:bb:cc:dd:ee:ff')
+    end
+
+    it 'set mac-address to ff-ff-ff-ff-ff-ff' do
+      expect(subject.get[:mac_address]).to be_empty
+      expect(subject.set_mac_address(value: 'ff-ff-ff-ff-ff-ff')).to be_truthy
+      expect(subject.get[:mac_address]).to eq('ff:ff:ff:ff:ff:ff')
+    end
+
+    it 'set mac-address to ffff.ffff.ffff' do
+      expect(subject.get[:mac_address]).to be_empty
+      expect(subject.set_mac_address(value: 'ffff.ffff.ffff')).to be_truthy
+      expect(subject.get[:mac_address]).to eq('ff:ff:ff:ff:ff:ff')
+    end
+
+    it 'set mac-address to ffff:ffff:ffff fails' do
+      expect(subject.get[:mac_address]).to be_empty
+      expect(subject.set_mac_address(value: 'ffff:ffff:ffff')).to be_falsey
+      expect(subject.get[:mac_address]).to eq('')
+    end
+
+    it 'set mac-address to ff.ff.ff.ff.ff.ff fails' do
+      expect(subject.get[:mac_address]).to be_empty
+      expect(subject.set_mac_address(value: 'ff.ff.ff.ff.ff.ff')).to be_falsey
+      expect(subject.get[:mac_address]).to eq('')
+    end
+  end
+end

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

@@ -44,6 +44,8 @@ describe Rbeapi::Api::BgpNeighbors do
     { bgp_as: '64600',
       router_id: '192.168.254.1',
       shutdown: false,
+      maximum_paths: 32,
+      maximum_ecmp_paths: 32,
       networks: [
         { prefix: '192.168.254.1', masklen: 32, route_map: nil },
         { prefix: '192.168.254.2', masklen: 32, route_map: 'rmap' },

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

@@ -44,6 +44,8 @@ describe Rbeapi::Api::Bgp do
     { bgp_as: '64600',
       router_id: '192.168.254.1',
       shutdown: false,
+      maximum_paths: 32,
+      maximum_ecmp_paths: 32,
       networks: [
         { prefix: '192.168.254.1', masklen: 32, route_map: nil },
         { prefix: '192.168.254.2', masklen: 32, route_map: 'rmap' },
@@ -88,9 +90,40 @@ describe Rbeapi::Api::Bgp do
 
   describe '#create' do
     it 'create a new BGP resource' do
-      expect(node).to receive(:config).with('router bgp 1000')
+      expect(node).to receive(:config).with(['router bgp 1000'])
       expect(subject.create('1000')).to be_truthy
     end
+    it 'create with enable' do
+      expect(node).to receive(:config).with(['router bgp 1000', 'no shutdown'])
+      expect(subject.create('1000', enable: true)).to be_truthy
+    end
+    it 'create with router_id' do
+      expect(node).to receive(:config).with(['router bgp 1000', 'router-id 1'])
+      expect(subject.create('1000', router_id: 1)).to be_truthy
+    end
+    it 'create with maximum paths' do
+      expect(node).to receive(:config).with(['router bgp 1000',
+                                             'maximum-paths 1'])
+      expect(subject.create('1000', maximum_paths: 1)).to be_truthy
+    end
+    it 'create with maximum paths and ecmp paths' do
+      expect(node).to receive(:config).with(['router bgp 1000',
+                                             'maximum-paths 13 ecmp 13'])
+      expect(subject.create('1000', maximum_paths: 13,
+                                    maximum_ecmp_paths: 13)).to be_truthy
+    end
+    it 'raises ArgumentError for create with ecmp paths only' do
+      expect { subject.create('1000', maximum_ecmp_paths: 13) }.to \
+        raise_error ArgumentError
+    end
+    it 'create with all options set' do
+      expect(node).to receive(:config).with(['router bgp 1000', 'no shutdown',
+                                             'router-id 1',
+                                             'maximum-paths 13 ecmp 13'])
+      expect(subject.create('1000', enable: true, router_id: 1,
+                                    maximum_paths: 13,
+                                    maximum_ecmp_paths: 13)).to be_truthy
+    end
   end
 
   describe '#delete' do
@@ -162,6 +195,26 @@ describe Rbeapi::Api::Bgp do
     end
   end
 
+  describe '#set_maximum_paths' do
+    it 'set the maximum paths and ecmp paths' do
+      expect(node).to receive(:config).with(['router bgp 64600',
+                                             'maximum-paths 13 ecmp 200'])
+      expect(subject.set_maximum_paths(13, 200)).to be_truthy
+    end
+
+    it 'remove the maximum paths' do
+      expect(node).to receive(:config).with(['router bgp 64600',
+                                             'no maximum-paths'])
+      expect(subject.set_maximum_paths(0, 0, enable: false)).to be_truthy
+    end
+
+    it 'defaults the maximum paths' do
+      expect(node).to receive(:config).with(['router bgp 64600',
+                                             'default maximum-paths'])
+      expect(subject.set_maximum_paths(0, 0, 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)

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

@@ -22,7 +22,7 @@ describe Rbeapi::Api::PortchannelInterface do
   describe '#get' do
     before :each do
       allow(subject.node).to receive(:enable)
-        .with(include('show port-channel'), format: 'text')
+        .with(include('show port-channel'), encoding: 'text')
         .and_return([{ result:
                        { 'output' => "Port Channel Port-Channel1:\n  Active " \
                                      'Ports: Ethernet1 PeerEthernet1 ' \

data/spec/unit/rbeapi/api/routemaps/default_spec.rb

@@ -0,0 +1,370 @@
+#
+# 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/routemaps'
+
+include FixtureHelpers
+
+describe Rbeapi::Api::Routemaps do
+  subject { described_class.new(node) }
+
+  let(:node) { double('node') }
+
+  let(:test) do
+    {
+      'permit' => {
+        10 => {
+          match: ['interface Loopback0',
+                  'ip address prefix-list MYLOOPBACK'],
+          set: ['community internet 5555:5555'],
+          description: 'description',
+          continue: 99
+        }
+      }
+    }
+  end
+  let(:name) { 'test1' }
+
+  def routemaps
+    routemaps = Fixtures[:routemaps]
+    return routemaps if routemaps
+    fixture('routemaps', format: :text, dir: File.dirname(__FILE__))
+  end
+
+  before :each do
+    allow(subject.node).to receive(:running_config).and_return(routemaps)
+  end
+
+  describe '#getall' do
+    let(:test1_entries) do
+      {
+        'test1' => {
+          'permit' => {
+            10 => {
+              match: ['interface Loopback0',
+                      'ip address prefix-list MYLOOPBACK'],
+              set: ['community internet 5555:5555'],
+              description: 'description',
+              continue: 99
+            }
+          }
+        },
+        'test' => {
+          'permit' => {
+            10 => {
+              match: ['interface Vlan100'],
+              description: 'description',
+              continue: 99
+            },
+            20 => {
+              continue: 99,
+              description: 'description',
+              set: ['community internet 5555:5555']
+            }
+          },
+          'deny' => {
+            10 => {
+              match: ['interface Vlan100'],
+              description: 'description',
+              continue: 99
+            },
+            20 => {
+              continue: 99,
+              description: 'description',
+              set: ['community internet 5555:5555']
+            }
+          }
+        }
+      }
+    end
+
+    it 'returns the routemap collection' do
+      expect(subject.getall).to include(test1_entries)
+    end
+
+    it 'returns a hash collection' do
+      expect(subject.getall).to be_a_kind_of(Hash)
+    end
+
+    it 'has two entries' do
+      expect(subject.getall.size).to eq(2)
+    end
+  end
+
+  describe '#get' do
+    it 'returns the routemap resource for given name' do
+      expect(subject.get(name)).to eq(test)
+    end
+
+    it 'returns a hash' do
+      expect(subject.get(name)).to be_a_kind_of(Hash)
+    end
+
+    it 'has two entries' do
+      expect(subject.get(name).size).to eq(1)
+    end
+  end
+
+  describe '#create' do
+    it 'create an existing routemap test1 permit 10' do
+      expect(node).to receive(:config).with(['route-map test1 permit 10'])
+      expect(subject.create('test1', 'permit', 10)).to be_truthy
+    end
+
+    it 'create an existing routemap test deny 20' do
+      expect(node).to receive(:config).with(['route-map test deny 20'])
+      expect(subject.create('test', 'deny', 20)).to be_truthy
+    end
+
+    it 'create a new routemap test4 permit 20' do
+      expect(node).to receive(:config).with(['route-map test4 permit 20'])
+      expect(subject.create('test4', 'permit', 20)).to be_truthy
+    end
+
+    it 'create a new routemap test4 permit 20 with enable false' do
+      expect(node).to receive(:config).with(['no route-map test4 permit 20'])
+      expect(subject.create('test4', 'permit', 20, enable: false)).to be_truthy
+    end
+
+    it 'create a new routemap test4 permit 20 with enable true' do
+      expect(node).to receive(:config).with(['route-map test4 permit 20'])
+      expect(subject.create('test4', 'permit', 20, enable: true)).to be_truthy
+    end
+
+    it 'add description to routemap test1 permit 10 with create' do
+      expect(node).to receive(:config)
+        .with(['route-map test1 permit 10', 'no description',
+               'description description'])
+      expect(subject.create('test1', 'permit', 10,
+                            description: 'description')).to be_truthy
+    end
+
+    it 'add description to routemap test deny 20 with create' do
+      expect(node).to receive(:config)
+        .with(['route-map test deny 20', 'no description',
+               'description description'])
+      expect(subject.create('test', 'deny', 20,
+                            description: 'description')).to be_truthy
+    end
+
+    it 'add match statements to routemap test1 permit 10 with create' do
+      expect(node).to receive(:config)
+        .with(['route-map test1 permit 10',
+               'no match interface Loopback0',
+               'no match ip address prefix-list MYLOOPBACK',
+               'match ip address prefix-list MYLOOPBACK',
+               'match interface Loopback0'])
+      expect(subject.create('test1', 'permit', 10,
+                            match: ['ip address prefix-list MYLOOPBACK',
+                                    'interface Loopback0'])).to be_truthy
+    end
+
+    it 'add match statements to routemap test deny 20 with create' do
+      expect(node).to receive(:config)
+        .with(['route-map test deny 20',
+               'match ip address prefix-list MYLOOPBACK',
+               'match interface Loopback0'])
+      expect(subject.create('test', 'deny', 20,
+                            match: ['ip address prefix-list MYLOOPBACK',
+                                    'interface Loopback0'])).to be_truthy
+    end
+
+    it 'add set statements to routemap test1 permit 10 with create' do
+      expect(node).to receive(:config)
+        .with(['route-map test1 permit 10',
+               'no set community internet 5555:5555',
+               'set community internet 5555:5555'])
+      expect(subject.create('test1', 'permit', 10,
+                            set: ['community internet 5555:5555'])).to be_truthy
+    end
+
+    it 'add set statements to routemap test deny 20 with create' do
+      expect(node).to receive(:config)
+        .with(['route-map test deny 20',
+               'no set community internet 5555:5555',
+               'set community internet 5555:5555'])
+      expect(subject.create('test', 'deny', 20,
+                            set: ['community internet 5555:5555'])).to be_truthy
+    end
+
+    it 'add continue to routemap test1 permit 10 with create' do
+      expect(node).to receive(:config)
+        .with(['route-map test1 permit 10', 'no continue', 'continue 99'])
+      expect(subject.create('test1', 'permit', 10,
+                            continue: 99)).to be_truthy
+    end
+
+    it 'add continue to routemap test deny 20 with create' do
+      expect(node).to receive(:config)
+        .with(['route-map test deny 20', 'no continue', 'continue 99'])
+      expect(subject.create('test', 'deny', 20,
+                            continue: 99)).to be_truthy
+    end
+
+    it 'default routemap test permit 10 with create' do
+      expect(node).to receive(:config)
+        .with(['default route-map test1 permit 10'])
+      expect(subject.create('test1', 'permit', 10,
+                            default: true)).to be_truthy
+    end
+
+    it 'default routemap test deny 20 with create' do
+      expect(node).to receive(:config)
+        .with(['default route-map test deny 20'])
+      expect(subject.create('test', 'deny', 20,
+                            default: true)).to be_truthy
+    end
+  end
+
+  describe '#delete' do
+    it 'delete test1 permit 10 routemap resource' do
+      expect(node).to receive(:config).with(['no route-map test1 permit 10'])
+      expect(subject.delete('test1', 'permit', 10)).to be_truthy
+    end
+
+    it 'delete test deny 20 routemap resource' do
+      expect(node).to receive(:config).with(['no route-map test deny 20'])
+      expect(subject.delete('test', 'deny', 20)).to be_truthy
+    end
+
+    it 'delete non existent routemap' do
+      expect(node).to receive(:config).with(['no route-map blah deny 30'])
+      expect(subject.delete('blah', 'deny', 30)).to be_truthy
+    end
+  end
+
+  describe '#default' do
+    it 'default test1 permit 10 routemap resource' do
+      expect(node).to receive(:config)
+        .with(['default route-map test1 permit 10'])
+      expect(subject.default('test1', 'permit', 10)).to be_truthy
+    end
+  end
+
+  describe '#set_match_statements' do
+    it 'set the match statements on exsiting routemap' do
+      expect(node).to receive(:config)
+        .with(['route-map test1 permit 10',
+               'no match interface Loopback0',
+               'no match ip address prefix-list MYLOOPBACK',
+               'match ip address prefix-list MYLOOPBACK',
+               'match interface Loopback0'])
+      expect(
+        subject
+          .set_match_statements('test1', 'permit', 10,
+                                ['ip address prefix-list MYLOOPBACK',
+                                 'interface Loopback0'])
+      ).to be_truthy
+      expect(subject.get('test1').assoc('permit')[1].assoc(10)[1][:match])
+        .to include('ip address prefix-list MYLOOPBACK',
+                    'interface Loopback0')
+    end
+
+    it 'set the match statements on a new routemap' do
+      expect(node).to receive(:config)
+        .with(['route-map test4 permit 20',
+               'match ip address prefix-list MYLOOPBACK',
+               'match interface Loopback0'])
+      expect(
+        subject
+          .set_match_statements('test4', 'permit', 20,
+                                ['ip address prefix-list MYLOOPBACK',
+                                 'interface Loopback0'])
+      ).to be_truthy
+    end
+  end
+
+  describe '#set_set_statements' do
+    it 'set the set statements on existing routemap' do
+      expect(node).to receive(:config)
+        .with(['route-map test1 permit 10',
+               'no set community internet 5555:5555',
+               'set community internet 5555:5555'])
+      expect(
+        subject.set_set_statements('test1', 'permit', 10,
+                                   ['community internet 5555:5555'])
+      ).to be_truthy
+      expect(subject.get('test1').assoc('permit')[1].assoc(10)[1][:set])
+        .to include('community internet 5555:5555')
+    end
+
+    it 'set the set statements on new routemap' do
+      expect(node).to receive(:config)
+        .with(['route-map test4 permit 20',
+               'set community internet 5555:5555'])
+      expect(
+        subject.set_set_statements('test4', 'permit', 20,
+                                   ['community internet 5555:5555'])
+      ).to be_truthy
+    end
+  end
+
+  describe '#set_continue' do
+    it 'set the continue statement on existing routemap' do
+      expect(node).to receive(:config).with(['route-map test1 permit 10',
+                                             'no continue',
+                                             'continue 99'])
+      expect(subject.set_continue('test1', 'permit', 10, 99)).to be_truthy
+      expect(subject.get('test1').assoc('permit')[1]
+        .assoc(10)[1][:continue]).to eq(99)
+    end
+
+    it 'set the continue statement on new routemap' do
+      expect(node).to receive(:config).with(['route-map test4 permit 10',
+                                             'no continue',
+                                             'continue 99'])
+      expect(subject.set_continue('test4', 'permit', 10, 99)).to be_truthy
+    end
+  end
+
+  describe '#set_description' do
+    it 'set the description statement on existing routemap' do
+      expect(node).to receive(:config).with(['route-map test1 permit 10',
+                                             'no description',
+                                             'description description'])
+      expect(subject.set_description('test1', 'permit', 10,
+                                     'description')).to be_truthy
+      expect(subject.get('test1')
+        .assoc('permit')[1].assoc(10)[1][:description])
+        .to eq('description')
+    end
+
+    it 'set the description statement on new routemap' do
+      expect(node).to receive(:config).with(['route-map test4 permit 20',
+                                             'no description',
+                                             'description description'])
+      expect(subject.set_description('test4', 'permit', 20,
+                                     'description')).to be_truthy
+    end
+  end
+end

data/spec/unit/rbeapi/api/routemaps/fixture_routemaps.text

@@ -0,0 +1,27 @@
+route-map test1 permit 10
+   description description
+   match interface Loopback0
+   match ip address prefix-list MYLOOPBACK
+   continue 99
+   set community internet 5555:5555
+!
+route-map test permit 10
+   description description
+   match interface Vlan100
+   continue 99
+!
+route-map test permit 20
+   description description
+   continue 99
+   set community internet 5555:5555
+!
+route-map test deny 10
+   description description
+   match interface Vlan100
+   continue 99
+!
+route-map test deny 20
+   description description
+   continue 99
+   set community internet 5555:5555
+!