rbeapi 0.1.0 → 0.2.0

data/lib/rbeapi/netdev/snmp.rb

@@ -33,9 +33,7 @@
 require 'rbeapi/api'
 
 module Rbeapi
-
   module Netdev
-
     ##
     # The Netdev class is a straight port of the original PuppetX netdev
     # code that existed prior to rbeapi.  This should be considered a legacy
@@ -65,8 +63,6 @@ module Rbeapi
       # parse_snmp_hosts parses the raw text from the `show snmp host`
       # command and returns an Array of resource hashes.
       #
-      # rubocop:disable Metrics/MethodLength
-      #
       # @param [String] text The text of the `show snmp host` output, e.g.
       #   for three hosts:
       #
@@ -237,9 +233,6 @@ module Rbeapi
       # Group          : sysops
       # ```
       #
-      # rubocop:disable Metrics/CyclomaticComplexity
-      # rubocop:disable Metrics/MethodLength
-      #
       # @param [String] text The text to parse
       #
       # @api private
@@ -250,12 +243,12 @@ module Rbeapi
           user_s.scan(/^(\w+).*?: (.*)/).each_with_object({}) do |(h, v), m|
             key = SNMP_USER_PARAM[h.downcase.intern] || h.downcase.intern
             m[key] = case key
-                      when :privacy  then /AES/.match(v) ? :aes128 : :des
-                      when :version  then v.sub('v2c', 'v2').intern
-                      when :auth     then v.downcase.intern
-                      when :roles    then v.sub(/ \(.*?\)/, '')
-                      else v.downcase
-                      end
+                     when :privacy  then /AES/.match(v) ? :aes128 : :des
+                     when :version  then v.sub('v2c', 'v2').intern
+                     when :auth     then v.downcase.intern
+                     when :roles    then v.sub(/ \(.*?\)/, '')
+                     else v.downcase
+                     end
           end
         end
       end
@@ -275,8 +268,6 @@ module Rbeapi
       # snmp_user_set creates or updates an SNMP user account on the target
       # device.
       #
-      # rubocop:disable Metrics/MethodLength
-      #
       # @option opts [String] :name ('johndoe') The username
       #
       # @option opts [Array] :roles (['developers']) The group, as an Array,
@@ -305,7 +296,7 @@ module Rbeapi
         if opts[:password] && version == 'v3'
           privacy = opts[:privacy].to_s.scan(/aes|des/).first
           fail ArgumentError,
-                'privacy is required when managing passwords' unless privacy
+               'privacy is required when managing passwords' unless privacy
           cmd += " auth #{opts[:auth] || 'sha'} #{opts[:password]} "\
             "priv #{privacy} #{opts[:password]}"
         end

data/lib/rbeapi/utils.rb

@@ -36,7 +36,6 @@ module Rbeapi
   ##
   # Utils module
   module Utils
-
     ##
     # Iterates through a hash structure and converts all of the keys
     # to symbols.
@@ -46,9 +45,9 @@ module Rbeapi
     # @return [Hash] An updated hash structure with all keys converted to
     #   symboles
     def self.transform_keys_to_symbols(value)
-      return value if not value.kind_of?(Hash)
-      hash = value.inject({}) do |hsh, (k,v)|
-        hsh[k.to_sym] = self.transform_keys_to_symbols(v)
+      return value unless value.is_a?(Hash)
+      hash = value.each_with_object({}) do |(k, v), hsh|
+        hsh[k.to_sym] = transform_keys_to_symbols(v)
         hsh
       end
       hash
@@ -67,4 +66,3 @@ module Rbeapi
     end
   end
 end
-

data/lib/rbeapi/version.rb

@@ -33,5 +33,5 @@
 # #
 # Rbeapi toplevel namespace
 module Rbeapi
-  VERSION = '0.1.0'
+  VERSION = '0.2.0'
 end

data/rbeapi.gemspec

@@ -19,8 +19,10 @@ Gem::Specification.new do |spec|
   spec.test_files    = spec.files.grep(/^(test|spec|features)\//)
   spec.require_paths = ['lib']
 
-  spec.add_dependency 'net_http_unix'
-  spec.add_dependency 'inifile'
+  spec.add_runtime_dependency 'inifile'
+  spec.add_runtime_dependency 'json'
+  spec.add_runtime_dependency 'net_http_unix'
+  spec.add_runtime_dependency 'netaddr'
 
   spec.add_development_dependency 'bundler', '~> 1.3'
   spec.add_development_dependency 'yard'

data/rbeapi.spec.tmpl

@@ -0,0 +1,72 @@
+# -*- rpm-spec -*-
+%global gem_name rbeapi
+
+Name:		%{?enterprise:pe-}rubygem-%{gem_name}
+Version:	REPLACEME
+Release:	1.eos4
+Summary:	Arista eAPI Ruby Library for the EOS command API
+
+Group:		Development/Languages
+License:    New BSD
+URL:		https://github.com/arista-eosplus/rbeapi
+Source0:	%{gem_name}-%{version}.gem
+
+%if 0%{?enterprise:1} == 1
+# Use these settings for Puppet Enterprise
+%global gem /opt/puppet/bin/gem
+Requires: pe-rubygems
+Requires: pe-ruby
+Requires: pe-rubygem(net-http-unix)
+Requires: pe-rubygem(inifile)
+Provides: pe-rubygem(%{gem_name}) = %{version}
+Provides: pe-rubygem-%{gem_name} = %{version}
+%else
+# Use these settings for all other installs
+%global gem gem
+Requires: ruby(abi) = %{rubyabi}
+Provides: ruby(%{gem_name}) = %{version}-%{release}
+%endif
+
+
+BuildArch: noarch
+
+%description
+The Ruby Cliet for eAPI provides a native Ruby implementation for programming
+Arista EOS network devices using Ruby. The Ruby client provides the ability
+to build native applications in Ruby that can communicate with EOS either
+locally via Unix domain sockets (on-box) or remotely over a HTTP/S transport
+(off-box). It uses a standard INI-style configuration file to specify one or
+more connection profiles.
+
+The rbeapi implementation also provides an API layer for building native Ruby
+objects that allow for configuration and state extraction of EOS nodes. The API
+layer provides a consistent implementation for working with EOS configuration
+resources. The implementation of the API layer is highly extensible and can be
+used as a foundation for building custom data models.
+
+The library is freely provided to the open source community for building robust
+applications using Arista EOS eAPI. Support is provided as best effort through
+Github iusses.
+
+%prep
+%setup -q -D -T -n  .
+
+%install
+mkdir -p %{buildroot}
+install %{SOURCE0} %{buildroot}/
+
+%files
+/%{gem_name}-%{version}.gem
+
+%post
+%{gem} install --local /%{gem_name}-%{version}.gem > /dev/null 2>&1
+
+%preun
+%{gem} uninstall %{gem_name} --version '= %{version}' > /dev/null 2>&1
+
+%changelog
+* Tue May 21 2015 Jere Julian - 0.1.0-2
+- Ubuntu requires we manually create the buildroot
+
+* Tue Mar 17 2015 Jere Julian - 0.1.0-1
+- Initial package

data/spec/support/fixtures.rb

@@ -52,8 +52,7 @@ module FixtureHelpers
   # `spec/fixtures/fixture_foo.json` will be loaded and returned.  This method
   # is memoized across the life of the process.
   #
-  # rubocop:disable Metrics/CyclomaticComplexity, Metrics/MethodLength,
-  # rubocop:disable Metrics/PerceivedComplexity
+  # rubocop:disable Metrics/MethodLength,
   #
   # @param [Symbol] key The fixture name without the `fixture_` prefix or
   #   `.json` suffix.
@@ -73,11 +72,13 @@ module FixtureHelpers
 
     yaml = Pathname.new(File.join(dir, "fixture_#{key}.yaml"))
     json = Pathname.new(File.join(dir, "fixture_#{key}.json"))
+    text = Pathname.new(File.join(dir, "fixture_#{key}.text"))
 
     data = if yaml.exist?; then YAML.load(File.read(yaml))
            elsif json.exist?; then JSON.load(File.read(json))
-           else fail "could not load YAML or JSON fixture #{key} "\
-             "tried:\n  #{yaml}\n  #{json}"
+           elsif text.exist?; then File.read(text)
+           else fail "could not load YAML, JSON or TEXT fixture #{key} "\
+             "tried:\n  #{yaml}\n  #{json} #{text}"
            end
 
     Fixtures[key] = data
@@ -86,6 +87,7 @@ module FixtureHelpers
     when :ruby then data
     when :json then JSON.pretty_generate(data)
     when :yaml then YAML.dump(data)
+    when :text then data
     else fail ArgumentError, "unknown format #{opts[:format].inspect}"
     end
   end

data/spec/support/shared_examples_for_api_modules.rb

@@ -1,11 +1,10 @@
 RSpec.shared_examples 'a configurable entity' do |opts|
-
   before(:each) do
     allow(subject.node).to receive(:config)
   end
 
   applies_to = opts.fetch(:applies_to, [])
-  if applies_to.include?(opts[:args[0,2]]) or applies_to.empty?
+  if applies_to.include?(opts[:args[0, 2]]) || applies_to.empty?
     context 'with node#config' do
       [:create, :delete, :default].each do |action|
         it "calls #{action} using #{opts[:args]}" do
@@ -18,13 +17,12 @@ RSpec.shared_examples 'a configurable entity' do |opts|
 end
 
 RSpec.shared_examples 'a tristate attr' do |opts|
-
   before(:each) do
     allow(subject.node).to receive(:config)
   end
 
   applies_to = opts.fetch(:applies_to, [])
-  if applies_to.include?(opts[:args[0,2]]) or applies_to.empty?
+  if applies_to.include?(opts[:args[0, 2]]) || applies_to.empty?
     context 'with node#config' do
       it "configures with #{opts[:name]}" do
         expect(subject.node).to receive(:config).with(opts[:config])
@@ -36,7 +34,7 @@ RSpec.shared_examples 'a tristate attr' do |opts|
         subject.send(opts[:name], *opts[:args])
       end
 
-      it "defaults" do
+      it 'defaults' do
         expect(subject.node).to receive(:config).with(opts[:default])
         subject.send(opts[:name], *opts[:args], default: true)
       end
@@ -45,7 +43,6 @@ RSpec.shared_examples 'a tristate attr' do |opts|
 end
 
 RSpec.shared_examples 'a settable attr' do |opts|
-
   before(:each) do
     allow(subject.node).to receive(:config)
   end
@@ -59,7 +56,6 @@ RSpec.shared_examples 'a settable attr' do |opts|
 end
 
 RSpec.shared_examples 'a creatable entity' do |args, setup, block|
-
   before(:each) do
     subject.node.config(setup)
   end
@@ -74,7 +70,6 @@ RSpec.shared_examples 'a creatable entity' do |args, setup, block|
 end
 
 RSpec.shared_examples 'a deletable entity' do |args, setup, block|
-
   before(:each) do
     subject.node.config(setup)
   end
@@ -89,7 +84,6 @@ RSpec.shared_examples 'a deletable entity' do |args, setup, block|
 end
 
 RSpec.shared_examples 'a configurable attr' do |opts|
-
   before(:each) do
     subject.node.config(opts[:setup])
   end
@@ -104,21 +98,12 @@ RSpec.shared_examples 'a configurable attr' do |opts|
 end
 
 RSpec.shared_examples 'single entity' do |opts|
-
   describe 'match entity attributes from node' do
     let(:entity) { subject.get(opts[:args]) }
     opts[:entity].each do |key, value|
       it "has #{key} with #{value}" do
-        binding.pry
         expect(entity[key]).to eq(value)
       end
     end
   end
 end
-
-
-
-
-
-
-

data/spec/system/api_acl_spec.rb

@@ -0,0 +1,128 @@
+require 'spec_helper'
+
+require 'rbeapi/client'
+require 'rbeapi/api/acl'
+
+describe Rbeapi::Api::Acl do
+  subject { described_class.new(node) }
+
+  let(:config) { Rbeapi::Client::Config.new(filename: get_fixture('dut.conf')) }
+  let(:node) { Rbeapi::Client.connect_to('dut') }
+
+  before do
+    node.config(['no ip access-list standard test1',
+                 'ip access-list standard test1',
+                 'permit host 1.2.3.4 log',
+                 'permit 1.2.3.4 255.255.0.0 log',
+                 'deny any',
+                 'permit 5.6.7.0/24',
+                 'permit 9.10.11.0 255.255.255.0 log',
+                 'exit'])
+    node.config(['no ip access-list standard test2',
+                 'ip access-list standard test2',
+                 'deny 16.0.0.0/8',
+                 'exit'])
+  end
+
+  let(:test1_entries) do
+    { '10' => { seqno: '10', action: 'permit', srcaddr: '1.2.3.4',
+                srcprefixlen: '255.255.255.255', log: 'log' },
+      '20' => { seqno: '20', action: 'permit', srcaddr: '1.2.3.4',
+                srcprefixlen: '255.255.0.0', log: 'log' },
+      '30' => { seqno: '30', action: 'deny', srcaddr: '0.0.0.0',
+                srcprefixlen: '255.255.255.255', log: nil },
+      '40' => { seqno: '40', action: 'permit', srcaddr: '5.6.7.0',
+                srcprefixlen: '24', log: nil },
+      '50' => { seqno: '50', action: 'permit', srcaddr: '9.10.11.0',
+                srcprefixlen: '255.255.255.0', log: 'log' }
+    }
+  end
+
+  let(:test2_entries) do
+    { '10' => { seqno: '10', action: 'deny', srcaddr: '16.0.0.0',
+                srcprefixlen: '8', log: nil }
+    }
+  end
+
+  describe '#get' do
+    it 'returns the test ACL entries' do
+      expect(subject.get('test1')).to eq(test1_entries)
+      expect(subject.get('test2')).to eq(test2_entries)
+    end
+  end
+
+  describe '#getall' do
+    let(:collection) { subject.getall }
+
+    it 'includes test1 and test2 ACLs' do
+      expect(collection).to include('test1')
+      expect(collection).to include('test2')
+    end
+
+    it 'is a kind of hash' do
+      expect(collection).to be_a_kind_of(Hash)
+    end
+  end
+
+  describe '#create' do
+    before { node.config('no ip access-list standard abc') }
+
+    it 'creates a new ACL resource' do
+      expect(subject.get('abc')).to be_nil
+      expect(subject.create('abc')).to be_truthy
+      expect(subject.get('abc')).not_to be_nil
+    end
+  end
+
+  describe '#delete' do
+    it 'deletes the abc ACL resource' do
+      expect(subject.get('abc')).not_to be_nil
+      expect(subject.delete('abc')).to be_truthy
+      expect(subject.get('abc')).to be_nil
+    end
+  end
+
+  describe '#default' do
+    before { node.config('ip access-list standard xyz') }
+
+    it 'sets ACL xyz to default value' do
+      expect(subject.get('xyz')).to be_truthy
+      expect(subject.default('xyz')).to be_truthy
+      expect(subject.get('xyz')).to be_nil
+    end
+  end
+
+  describe '#update_entry' do
+    let(:update_entry) do
+      { seqno: '50', action: 'deny', srcaddr: '100.0.0.0',
+        srcprefixlen: '8', log: nil }
+    end
+
+    it 'Change entry 50 to values in update_entry' do
+      expect(subject.get('test1')['50'][:action]).to eq('permit')
+      expect(subject.update_entry('test1', update_entry)).to be_truthy
+      expect(subject.get('test1')['50']).to eq(update_entry)
+    end
+  end
+
+  describe '#add_entry' do
+    let(:new_entry) do
+      { seqno: '60', action: 'deny', srcaddr: '1.2.3.0',
+        srcprefixlen: '24', log: 'log' }
+    end
+
+    it 'Add entry 60 to the test1 ACL' do
+      expect(subject.get('test1')['60']).to be_nil
+      expect(subject.add_entry('test1', new_entry)).to be_truthy
+      expect(subject.get('test1')['60']).to eq(new_entry)
+    end
+  end
+
+  describe '#remove_entry' do
+    it 'Remove entry 30 from the test1 ACL' do
+      expect(subject.get('test1')['30']).to be_truthy
+      expect(subject.remove_entry('test1', '30')).to be_truthy
+      expect(subject.get('test1')['30']).to be_nil
+    end
+  end
+end

data/spec/system/api_ospf_interfaces_spec.rb

@@ -7,13 +7,14 @@ describe Rbeapi::Api::OspfInterfaces do
   subject { described_class.new(node) }
 
   let(:config) { Rbeapi::Client::Config.new(filename: get_fixture('dut.conf')) }
-  let(:node) { Rbeapi::Client.connect_to('veos02') }
+  let(:node) { Rbeapi::Client.connect_to('dut') }
 
   describe '#get' do
-
-    before { node.config(['default interface Ethernet1', 'interface Ethernet1',
-                          'no switchport', 'ip address 99.99.99.99/24',
-                          'default interface Ethernet2']) }
+    before do
+      node.config(['default interface Ethernet1', 'interface Ethernet1',
+                   'no switchport', 'ip address 88.99.99.99/24', 'exit',
+                   'default interface Ethernet2'])
+    end
 
     it 'returns an ospf interface resource instance' do
       expect(subject.get('Ethernet1')).not_to be_nil
@@ -25,10 +26,11 @@ describe Rbeapi::Api::OspfInterfaces do
   end
 
   describe '#getall' do
-
-    before { node.config(['default interface Ethernet1', 'interface Ethernet1',
-                          'no switchport', 'ip address 99.99.99.99/24',
-                          'default interface Ethernet2']) }
+    before do
+      node.config(['default interface Ethernet1', 'interface Ethernet1',
+                   'no switchport', 'ip address 88.99.99.99/24', 'exit',
+                   'default interface Ethernet2'])
+    end
 
     it 'returns the ospf resource collection' do
       expect(subject.getall).to be_a_kind_of(Hash)
@@ -41,17 +43,18 @@ describe Rbeapi::Api::OspfInterfaces do
     it 'does not include an instance of Ethernet2' do
       expect(subject.getall).not_to include('Ethernet2')
     end
-
   end
 
   describe '#set_network_type' 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 88.99.99.99/24', 'exit'])
+    end
 
     it 'configures the ospf interface type to point-to-point' do
       expect(subject.get('Ethernet1')['network_type']).to eq('broadcast')
-      expect(subject.set_network_type('Ethernet1', value: 'point-to-point')).to be_truthy
+      expect(subject.set_network_type('Ethernet1', value: 'point-to-point'))
+        .to be_truthy
       expect(subject.get('Ethernet1')['network_type']).to eq('point-to-point')
     end
   end

data/spec/system/api_ospf_spec.rb

@@ -7,10 +7,9 @@ describe Rbeapi::Api::Ospf do
   subject { described_class.new(node) }
 
   let(:config) { Rbeapi::Client::Config.new(filename: get_fixture('dut.conf')) }
-  let(:node) { Rbeapi::Client.connect_to('veos02') }
+  let(:node) { Rbeapi::Client.connect_to('dut') }
 
   describe '#get' do
-
     before { node.config(['no router ospf 1', 'router ospf 1']) }
 
     let(:entity) do
@@ -23,7 +22,6 @@ describe Rbeapi::Api::Ospf do
   end
 
   describe '#getall' do
-
     before { node.config(['no router ospf 1', 'router ospf 1']) }
 
     let(:collection) { subject.getall }
@@ -39,7 +37,6 @@ describe Rbeapi::Api::Ospf do
     it 'is a kind of hash' do
       expect(collection).to be_a_kind_of(Hash)
     end
-
   end
 
   describe '#interfaces' do
@@ -83,17 +80,21 @@ describe Rbeapi::Api::Ospf do
 
     it 'adds the network with area to the ospf process' do
       expect(subject.get('1')['areas']).to be_empty
-      expect(subject.add_network('1', '192.168.10.0/24', '0.0.0.0')).to be_truthy
+      expect(subject.add_network('1', '192.168.10.0/24', '0.0.0.0'))
+        .to be_truthy
       expect(subject.get('1')['areas']['0.0.0.0']).to include('192.168.10.0/24')
     end
   end
 
   describe '#remove_network' do
-    before { node.config(['router ospf 1', 'network 192.168.10.10/24 area 0.0.0.0']) }
+    before do
+      node.config(['router ospf 1', 'network 192.168.10.10/24 area 0.0.0.0'])
+    end
 
     it 'removes the network with area to the ospf process' do
       expect(subject.get('1')['areas']['0.0.0.0']).to include('192.168.10.0/24')
-      expect(subject.remove_network('1', '192.168.10.0/24', '0.0.0.0')).to be_truthy
+      expect(subject.remove_network('1', '192.168.10.0/24', '0.0.0.0'))
+        .to be_truthy
       expect(subject.get('1')['areas']).to be_empty
     end
   end
@@ -107,5 +108,4 @@ describe Rbeapi::Api::Ospf do
       expect(subject.get('1')['redistribute']).to include('static')
     end
   end
-
 end

data/spec/system/api_varp_interfaces_spec.rb

@@ -7,12 +7,15 @@ 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('veos02') }
+  let(:node) { Rbeapi::Client.connect_to('dut') }
 
   describe '#get' do
-    before { node.config(['ip virtual-router mac-address aabb.ccdd.eeff',
-                          'interface vlan 100', 'ip address 99.99.99.99/24',
-                          'ip virtual-router address 99.99.99.98']) }
+    before do
+      node.config(['ip virtual-router mac-address aabb.ccdd.eeff',
+                   'default interface vlan 100', 'interface vlan 100',
+                   '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
       expect(subject.get('Vlan100')).not_to be_nil
@@ -24,9 +27,12 @@ describe Rbeapi::Api::VarpInterfaces do
   end
 
   describe '#getall' do
-    before { node.config(['ip virtual-router mac-address aabb.ccdd.eeff',
-                          'interface vlan 100', 'ip address 99.99.99.99/24',
-                          'ip virtual-router address 99.99.99.98']) }
+    before do
+      node.config(['ip virtual-router mac-address aabb.ccdd.eeff',
+                   'default interface vlan 100', 'interface vlan 100',
+                   '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
       expect(subject.getall).to include('Vlan100')
@@ -38,13 +44,16 @@ describe Rbeapi::Api::VarpInterfaces do
   end
 
   describe '#set_addresses' do
-    before { node.config(['ip virtual-router mac-address aabb.ccdd.eeff',
-                          'no interface vlan 100', 'interface vlan 100',
-                          'ip address 99.99.99.99/24']) }
+    before do
+      node.config(['ip virtual-router mac-address aabb.ccdd.eeff',
+                   'default interface vlan 100', 'interface vlan 100',
+                   '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.set_addresses('Vlan100', value: ['99.99.99.98'])).to be_truthy
+      expect(subject.set_addresses('Vlan100', value: ['99.99.99.98']))
+        .to be_truthy
       expect(subject.get('Vlan100')['addresses']).to include('99.99.99.98')
     end
 
@@ -52,9 +61,9 @@ describe Rbeapi::Api::VarpInterfaces 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.set_addresses('Vlan100', value: ['99.99.99.97'])).to be_truthy
+      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')
     end
-
   end
 end

data/spec/system/api_varp_spec.rb

@@ -7,10 +7,9 @@ describe Rbeapi::Api::Varp do
   subject { described_class.new(node) }
 
   let(:config) { Rbeapi::Client::Config.new(filename: get_fixture('dut.conf')) }
-  let(:node) { Rbeapi::Client.connect_to('veos02') }
+  let(:node) { Rbeapi::Client.connect_to('dut') }
 
   describe '#get' do
-
     it 'returns a varp resource instance' do
       expect(subject.get).to be_a_kind_of(Hash)
     end
@@ -38,7 +37,5 @@ describe Rbeapi::Api::Varp do
       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
-
   end
-
 end

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

@@ -12,7 +12,6 @@ describe Rbeapi::Api::Interfaces do
   end
 
   describe '#get' do
-
     let(:entity) do
       { name: 'Loopback0', type: 'generic', description: '', shutdown: false }
     end
@@ -34,7 +33,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 Loopback0') }
@@ -70,7 +69,8 @@ describe Rbeapi::Api::Interfaces do
     it 'sets the description value on the interface' do
       node.config(['interface Loopback0', 'no description'])
       expect(subject.get('Loopback0')[:description]).to be_empty
-      expect(subject.set_description('Loopback0', value: 'foo bar')).to be_truthy
+      expect(subject.set_description('Loopback0', value: 'foo bar'))
+        .to be_truthy
       expect(subject.get('Loopback0')[:description]).to eq('foo bar')
     end
   end
@@ -91,4 +91,3 @@ describe Rbeapi::Api::Interfaces do
     end
   end
 end
-