cisco_node_utils 1.7.0 → 1.8.0

data/tests/test_feature.rb

@@ -103,7 +103,7 @@ class TestFeature < CiscoTestCase
   end
 
   def test_fabric_forwarding
-    if node.product_id[/N(3)/]
+    if node.product_id[/N(3)/] && !node.product_id.include?('-F')
       assert_nil(Feature.fabric_forwarding_enabled?)
       assert_raises(Cisco::UnsupportedError) do
         Feature.fabric_forwarding_enable
@@ -126,7 +126,7 @@ class TestFeature < CiscoTestCase
   end
 
   def test_nv_overlay_evpn
-    if node.product_id[/N(3)/]
+    if node.product_id[/N(3)/] && !node.product_id.include?('-F')
       assert_nil(Feature.nv_overlay_evpn_enabled?)
       assert_raises(Cisco::UnsupportedError) { Feature.nv_overlay_evpn_enable }
       return

data/tests/test_interface.rb

@@ -1,4 +1,4 @@
-# Copyright (c) 2013-2016 Cisco and/or its affiliates.
+# Copyright (c) 2013-2017 Cisco and/or its affiliates.
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
@@ -629,7 +629,7 @@ class TestInterface < CiscoTestCase
   end
 
   def test_speed
-    skip_legacy_defect?('7.0.3.I5.2', 'CSCvd41419')
+    skip_legacy_defect?('7.0.3.I5|7.0.3.F3', 'CSCvd41419')
     interface = Interface.new(interfaces[0])
     if validate_property_excluded?('interface', 'speed')
       assert_nil(interface.speed)
@@ -665,7 +665,7 @@ class TestInterface < CiscoTestCase
   end
 
   def test_duplex
-    skip_legacy_defect?('7.0.3.I5.2', 'CSCvd41419')
+    skip_legacy_defect?('7.0.3.I5|7.0.3.F3', 'CSCvd41419')
     interface = Interface.new(interfaces[0])
     if validate_property_excluded?('interface', 'duplex')
       assert_nil(interface.duplex)
@@ -820,7 +820,11 @@ class TestInterface < CiscoTestCase
   end
 
   def test_negotiate_auto_ethernet
-    skip_legacy_defect?('7.0.3.I5.2', 'CSCvd41419')
+    # negotiate auto is LC dependent configuration.
+    # On some line cards it could fail with error for ex:
+    # Cisco::CliError: [interface ethernet8/1] The command 'no negotiate auto' was rejected with error:
+    # ERROR: Ethernet8/1: requested config change not allowed
+    skip_legacy_defect?('7.0.3.I5|7.0.3.F3', 'CSCvd41419')
     inf_name = interfaces[0]
     interface = Interface.new(inf_name)
 
@@ -1837,6 +1841,28 @@ class TestInterface < CiscoTestCase
     lb.destroy
   end
 
+  def test_default_physical
+    name = interfaces[0]
+    int = Interface.new(name)
+    int.switchport_mode = :disabled
+
+    # Verify l3 -> default
+    int.description = 'default_pysical'
+    int.ipv4_addr_mask_set('192.168.0.1', '24')
+    refute(int.default?)
+
+    int.destroy
+    assert(int.default?)
+
+    # Verify l2 trunk -> default
+    int.switchport_mode = :access
+    int.switchport_autostate_exclude = true
+    refute(int.default?)
+
+    int.destroy
+    assert(int.default?)
+  end
+
   def test_purge_config
     name = interfaces[0]
     int = Interface.new(name)

data/tests/test_node_ext.rb

@@ -1,4 +1,4 @@
-# Copyright (c) 2013-2016 Cisco and/or its affiliates.
+# Copyright (c) 2013-2017 Cisco and/or its affiliates.
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
@@ -141,7 +141,7 @@ class TestNodeExt < CiscoTestCase
   end
 
   def test_get_product_id
-    # N9K Fretta product_id gets a '-F' appended so remove it for this check
+    # N3|9K Fretta product_id gets a '-F' appended so remove it for this check
     if Utils.image_version?(/7.0.3.F/)
       chassis = node.product_id.sub('-F', '')
     else

data/tests/test_object_group.rb

@@ -0,0 +1,122 @@
+# Copyright (c) 2017 Cisco and/or its affiliates.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+require_relative 'ciscotest'
+require_relative '../lib/cisco_node_utils/object_group'
+require_relative '../lib/cisco_node_utils/object_group_entry'
+
+# TestObject_group - Minitest for Object_group
+# node utility class
+class TestObjectGroup < CiscoTestCase
+  @skip_unless_supported = 'object_group'
+  @@pre_clean_needed = true # rubocop:disable Style/ClassVars
+
+  def setup
+    super
+    @og_name_v4 = 'test-my-v4'
+    @og_name_v6 = 'test-my-v6'
+    @og_name_port = 'test-my-port'
+    remove_all_object_groups if @@pre_clean_needed
+    @@pre_clean_needed = false # rubocop:disable Style/ClassVars
+  end
+
+  def teardown
+    remove_all_object_groups
+    super
+  end
+
+  def remove_all_object_groups
+    ObjectGroup.object_groups.each do |_afis, types|
+      types.each do |_type, grps|
+        grps.values.each(&:destroy)
+      end
+    end
+  end
+
+  def test_create_object_group
+    %w(ipv4 ipv6).each do |afi|
+      og_name = afi[/ipv6/] ? @og_name_v6 : @og_name_v4
+      og = ObjectGroup.new(afi, 'address', og_name)
+      assert(ObjectGroup.object_groups[afi]['address'].key?(og_name))
+      og.destroy
+      assert_nil(ObjectGroup.object_groups[afi])
+    end
+    og_name = @og_name_port
+    og = ObjectGroup.new('ipv4', 'port', og_name)
+    assert(ObjectGroup.object_groups['ipv4']['port'].key?(og_name))
+    og.destroy
+    assert_nil(ObjectGroup.object_groups['ipv4'])
+  end
+
+  def entry_helper(afi, type, name, props=nil)
+    test_hash = {
+      address: 'host 1.1.1.1',
+      port:    'eq 200',
+    }
+    test_hash.merge!(props) unless props.nil?
+
+    ObjectGroup.new(afi, type, name)
+    e = ObjectGroupEntry.new(afi, type, name, 10)
+    e.entry_set(test_hash)
+    e
+  end
+
+  def test_ipv4_address_host
+    e = entry_helper('ipv4', 'address', @og_name_v4, port: nil)
+    assert_equal('host 1.1.1.1', e.address)
+    e = entry_helper('ipv4', 'address', @og_name_v4, address: 'host 2.2.2.2', port: nil)
+    assert_equal('host 2.2.2.2', e.address)
+  end
+
+  def test_ipv4_address_prefix
+    e = entry_helper('ipv4', 'address', @og_name_v4, address: '2.2.2.2/17', port: nil)
+    assert_equal('2.2.2.2/17', e.address)
+    e = entry_helper('ipv4', 'address', @og_name_v4, address: '3.3.3.3/31', port: nil)
+    assert_equal('3.3.3.3/31', e.address)
+  end
+
+  def test_ipv4_address_netmask
+    e = entry_helper('ipv4', 'address', @og_name_v4, address: '2.2.2.2 255.255.255.0', port: nil)
+    assert_equal('2.2.2.2 255.255.255.0', e.address)
+    e = entry_helper('ipv4', 'address', @og_name_v4, address: '3.3.3.3 10.11.12.13', port: nil)
+    assert_equal('3.3.3.3 10.11.12.13', e.address)
+  end
+
+  def test_ipv6_address_host
+    e = entry_helper('ipv6', 'address', @og_name_v6, address: 'host 2000::1', port: nil)
+    assert_equal('host 2000::1', e.address)
+    e = entry_helper('ipv6', 'address', @og_name_v6, address: 'host 2001::2', port: nil)
+    assert_equal('host 2001::2', e.address)
+  end
+
+  def test_ipv6_address_prefix
+    e = entry_helper('ipv6', 'address', @og_name_v6, address: '2000::1/127', port: nil)
+    assert_equal('2000::1/127', e.address)
+    e = entry_helper('ipv6', 'address', @og_name_v6, address: '2001::10/64', port: nil)
+    assert_equal('2001::10/64', e.address)
+  end
+
+  def test_ipv4_port
+    e = entry_helper('ipv4', 'port', @og_name_port, address: nil)
+    assert_equal('eq 200', e.port)
+    e = entry_helper('ipv4', 'port', @og_name_port, port: 'lt 100', address: nil)
+    assert_equal('lt 100', e.port)
+    e = entry_helper('ipv4', 'port', @og_name_port, port: 'neq 150', address: nil)
+    assert_equal('neq 150', e.port)
+    e = entry_helper('ipv4', 'port', @og_name_port, port: 'gt 350', address: nil)
+    assert_equal('gt 350', e.port)
+    e = entry_helper('ipv4', 'port', @og_name_port, port: 'range 400 1000', address: nil)
+    assert_equal('range 400 1000', e.port)
+  end
+end

data/tests/test_platform.rb

@@ -110,7 +110,7 @@ class TestPlatform < CiscoTestCase
   end
 
   def test_uptime
-    s = @device.cmd('sh ver').scan(/uptime is (.*)/).flatten.first
+    s = @device.cmd('sh system uptime').scan(/Kernel uptime:\s+(.*)/).flatten.first
     # compare without seconds
     assert_equal(s.gsub(/\d+ sec/, ''), Platform.uptime.gsub(/\d+ sec/, ''))
   end

data/tests/test_portchannel_global.rb

@@ -45,6 +45,7 @@ class TestPortchannelGlobal < CiscoTestCase
 
   def n3k_in_n3k_mode?
     return unless /N3/ =~ node.product_id
+    return if node.product_id.include?('-F')
     mode = config('show system switch-mode')
     # note: an n3k in n9k mode displays: 'system switch-mode n9k'
     patterns = ['system switch-mode n3k',

data/tests/test_radius_global.rb

@@ -70,6 +70,12 @@ class TestRadiusGlobal < CiscoTestCase
       global.key_set(key, nil)
       assert_match(/#{key}/, global.key)
       assert_match(/#{key}/, Cisco::RadiusGlobal.radius_global[id].key)
+      assert_equal(7, global.key_format)
+      # Change to type 6
+      key = 'JDYkqyIFWeBvzpljSfWmRZrmRSRE8'
+      global.key_set(key, 6)
+      assert_match(/#{key}/, global.key)
+      assert_equal(6, global.key_format)
     elsif platform == :ios_xr
       global.key_set('QsEfThUkO', nil)
       assert(!global.key.nil?)
@@ -77,9 +83,23 @@ class TestRadiusGlobal < CiscoTestCase
     end
 
     # Setting back to default and re-checking
-    global.timeout = global.default_timeout
-    global.retransmit_count = global.default_retransmit_count
+    global.timeout = nil
+    global.retransmit_count = nil
+    global.key_set(nil, nil)
     assert_equal(global.timeout, global.default_timeout)
     assert_equal(global.retransmit_count, global.default_retransmit_count)
+    assert_nil(global.key)
+
+    # Default source interface
+    assert_nil(global.source_interface)
+
+    # Set source interface
+    interface = 'loopback0'
+    global.source_interface = interface
+    assert_equal(interface, global.source_interface)
+
+    # Remove source interface
+    global.source_interface = nil
+    assert_nil(global.source_interface)
   end
 end

data/tests/test_route_map.rb

@@ -22,7 +22,7 @@ end
 
 def dplus_n9k?
   return true if Utils.image_version?(/7.0.3.I4/) &&
-                 node.product_id[/N9K/]
+                 product_tag[/n9k/]
   false
 end
 
@@ -154,7 +154,9 @@ class TestRouteMap < CiscoTestCase
     assert_equal(rm.default_match_src_proto, rm.match_src_proto)
     array = %w(tcp udp igmp)
     rm.match_src_proto = array
-    assert_equal(array, rm.match_src_proto)
+    # Protocol order not maintained in running config starting Greensboro.
+    # Sorting arrays to check equality.
+    assert_equal(array.sort, rm.match_src_proto.sort)
     rm.match_src_proto = rm.default_match_src_proto
     assert_equal(rm.default_match_src_proto, rm.match_src_proto)
   end
@@ -962,7 +964,7 @@ class TestRouteMap < CiscoTestCase
   end
 
   def test_set_ipv4_default_next_hop
-    skip('platform not supported for this test') if node.product_id[/(N5|N6|N9|N9.*-F)/]
+    skip('platform not supported for this test') if product_tag[/(n35|n5k|n6k|n9k|n9k-f|n3k-f)/]
     arr = %w(1.1.1.1 2.2.2.2 3.3.3.3)
     rm = lset_ip_next_hop_helper(v4dnh: arr)
     assert_equal(arr, rm.set_ipv4_default_next_hop)
@@ -995,7 +997,7 @@ class TestRouteMap < CiscoTestCase
 
   def test_set_ipv4_next_hop_load_share
     # bug on fretta
-    skip('platform not supported for this test') if node.product_id[/(N5|N6)/]
+    skip('platform not supported for this test') if product_tag[/(n5k|n6k)/]
     skip_incompat_version?('route_map', 'set_ipv4_next_hop_load_share')
     arr = %w(1.1.1.1 2.2.2.2 3.3.3.3)
     rm = lset_ip_next_hop_helper(v4nh: arr)
@@ -1062,7 +1064,7 @@ class TestRouteMap < CiscoTestCase
   end
 
   def test_set_ipv6_default_next_hop
-    skip('platform not supported for this test') if node.product_id[/(N5|N6|N9|N9.*-F)/]
+    skip('platform not supported for this test') if product_tag[/(n35|n5k|n6k|n9k|n9k-f|n3k-f)/]
     arr = %w(2000::1 2000::11 2000::22)
     rm = lset_ip_next_hop_helper(v6dnh: arr)
     assert_equal(arr, rm.set_ipv6_default_next_hop)
@@ -1095,7 +1097,7 @@ class TestRouteMap < CiscoTestCase
 
   def test_set_ipv6_next_hop_load_share
     # bug on fretta
-    skip('platform not supported for this test') if node.product_id[/(N5|N6)/]
+    skip('platform not supported for this test') if product_tag[/(n5k|n6k)/]
     skip_incompat_version?('route_map', 'set_ipv6_next_hop_load_share')
     arr = %w(2000::1 2000::11 2000::22)
     rm = lset_ip_next_hop_helper(v6nh: arr)
@@ -1119,7 +1121,7 @@ class TestRouteMap < CiscoTestCase
 
   def test_set_community_no_asn
     # bug on n5/6k
-    skip('platform not supported for this test') if node.product_id[/(N5|N6)/]
+    skip('platform not supported for this test') if product_tag[/(n5k|n6k)/]
     skip_incompat_version?('route_map', 'set_community')
     rm = create_route_map
     assert_equal(rm.default_set_community_additive,
@@ -1336,7 +1338,8 @@ class TestRouteMap < CiscoTestCase
 
   def test_extcommunity_rt
     # bug CSCvc92395 on fretta and n9k
-    skip('platform not supported for this test') if node.product_id[/N9/]
+    skip('platform not supported for this test') if
+      product_tag[/n9k/] || product_tag[/n3k-f/]
     rm = create_route_map
     assert_equal(rm.default_set_extcommunity_rt_additive,
                  rm.set_extcommunity_rt_additive)

data/tests/test_router_bgp.rb

@@ -3,7 +3,7 @@
 #
 # Mike Wiebe, June, 2015
 #
-# Copyright (c) 2015-2016 Cisco and/or its affiliates.
+# Copyright (c) 2015-2017 Cisco and/or its affiliates.
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
@@ -60,14 +60,6 @@ def setup_vrf
   create_bgp_vrf(@asnum, @vrf)
 end
 
-def newer_image_version?
-  new = true
-  new = false if Utils.image_version?(/7.0.3.I2|I3|I4/) ||
-                 node.product_id[/(N5|N6|N7|N9.*-F)/]
-  new = true if Utils.image_version?(/8.0|8.1/)
-  new
-end
-
 # TestRouterBgp - Minitest for RouterBgp class
 class TestRouterBgp < CiscoTestCase
   @@pre_clean_needed = true # rubocop:disable Style/ClassVars
@@ -595,31 +587,12 @@ class TestRouterBgp < CiscoTestCase
     if validate_property_excluded?('bgp', 'event_history_cli')
       assert_nil(bgp.event_history_cli)
       assert_raises(Cisco::UnsupportedError) do
-        bgp.event_history_cli = 'true'
+        bgp.event_history_cli = 'size_large'
       end
       return
     end
-    assert_equal(bgp.default_event_history_cli, bgp.event_history_cli)
-    bgp.event_history_cli = 'true'
-    assert_equal(bgp.default_event_history_cli, bgp.event_history_cli)
-    bgp.event_history_cli = 'false'
-    assert_equal('false', bgp.event_history_cli)
-    bgp.event_history_cli = 'size_small'
-    assert_equal('size_small', bgp.event_history_cli)
     bgp.event_history_cli = 'size_large'
     assert_equal('size_large', bgp.event_history_cli)
-    bgp.event_history_cli = 'size_medium'
-    assert_equal('size_medium', bgp.event_history_cli)
-    bgp.event_history_cli = 'size_disable'
-    if newer_image_version?
-      assert_equal('false', bgp.event_history_cli)
-    else
-      assert_equal('size_disable', bgp.event_history_cli)
-    end
-    bgp.event_history_cli = '100000'
-    assert_equal('100000', bgp.event_history_cli)
-    bgp.event_history_cli = bgp.default_event_history_cli
-    assert_equal(bgp.default_event_history_cli, bgp.event_history_cli)
   end
 
   def test_event_history_detail
@@ -627,31 +600,12 @@ class TestRouterBgp < CiscoTestCase
     if validate_property_excluded?('bgp', 'event_history_detail')
       assert_nil(bgp.event_history_detail)
       assert_raises(Cisco::UnsupportedError) do
-        bgp.event_history_detail = 'true'
+        bgp.event_history_detail = 'size_large'
       end
       return
     end
-    assert_equal(bgp.default_event_history_detail, bgp.event_history_detail)
-    bgp.event_history_detail = 'true'
-    assert_equal('true', bgp.event_history_detail)
-    bgp.event_history_detail = 'false'
-    assert_equal(bgp.default_event_history_detail, bgp.event_history_detail)
-    bgp.event_history_detail = 'size_small'
-    assert_equal('size_small', bgp.event_history_detail)
     bgp.event_history_detail = 'size_large'
     assert_equal('size_large', bgp.event_history_detail)
-    bgp.event_history_detail = 'size_medium'
-    assert_equal('size_medium', bgp.event_history_detail)
-    bgp.event_history_detail = 'size_disable'
-    if newer_image_version?
-      assert_equal('false', bgp.event_history_detail)
-    else
-      assert_equal('size_disable', bgp.event_history_detail)
-    end
-    bgp.event_history_detail = '100000'
-    assert_equal('100000', bgp.event_history_detail)
-    bgp.event_history_detail = bgp.default_event_history_detail
-    assert_equal(bgp.default_event_history_detail, bgp.event_history_detail)
   end
 
   def test_event_history_errors
@@ -663,25 +617,9 @@ class TestRouterBgp < CiscoTestCase
       end
       return
     end
-    skip('platform not supported for this test') unless newer_image_version?
-    assert_equal(bgp.default_event_history_errors, bgp.event_history_errors)
-    bgp.event_history_errors = 'true'
-    assert_equal(bgp.default_event_history_errors, bgp.event_history_errors)
-    bgp.event_history_errors = 'false'
-    assert_equal('false', bgp.event_history_errors)
-    bgp.event_history_errors = 'size_small'
-    assert_equal('size_small', bgp.event_history_errors) unless
-      Utils.image_version?(/8.0|8.1/)
+    skip_incompat_version?('bgp', 'event_history_errors')
     bgp.event_history_errors = 'size_large'
     assert_equal('size_large', bgp.event_history_errors)
-    bgp.event_history_errors = 'size_medium'
-    assert_equal('size_medium', bgp.event_history_errors)
-    bgp.event_history_errors = 'size_disable'
-    assert_equal('false', bgp.event_history_errors)
-    bgp.event_history_errors = '100000'
-    assert_equal('100000', bgp.event_history_errors)
-    bgp.event_history_errors = bgp.default_event_history_errors
-    assert_equal(bgp.default_event_history_errors, bgp.event_history_errors)
   end
 
   def test_event_history_events
@@ -689,31 +627,12 @@ class TestRouterBgp < CiscoTestCase
     if validate_property_excluded?('bgp', 'event_history_events')
       assert_nil(bgp.event_history_events)
       assert_raises(Cisco::UnsupportedError) do
-        bgp.event_history_events = 'true'
+        bgp.event_history_events = 'size_medium'
       end
       return
     end
-    assert_equal(bgp.default_event_history_events, bgp.event_history_events)
-    bgp.event_history_events = 'true'
-    assert_equal(bgp.default_event_history_events, bgp.event_history_events)
-    bgp.event_history_events = 'false'
-    assert_equal('false', bgp.event_history_events)
-    bgp.event_history_events = 'size_small'
-    assert_equal('size_small', bgp.event_history_events)
-    bgp.event_history_events = 'size_large'
-    assert_equal('size_large', bgp.event_history_events)
     bgp.event_history_events = 'size_medium'
     assert_equal('size_medium', bgp.event_history_events)
-    bgp.event_history_events = 'size_disable'
-    if newer_image_version?
-      assert_equal('false', bgp.event_history_events)
-    else
-      assert_equal('size_disable', bgp.event_history_events)
-    end
-    bgp.event_history_events = '100000'
-    assert_equal('100000', bgp.event_history_events)
-    bgp.event_history_events = bgp.default_event_history_events
-    assert_equal(bgp.default_event_history_events, bgp.event_history_events)
   end
 
   def test_event_history_objstore
@@ -725,24 +644,9 @@ class TestRouterBgp < CiscoTestCase
       end
       return
     end
-    skip('platform not supported for this test') unless newer_image_version?
-    assert_equal(bgp.default_event_history_objstore, bgp.event_history_objstore)
-    bgp.event_history_objstore = 'true'
-    assert_equal('true', bgp.event_history_objstore)
-    bgp.event_history_objstore = 'false'
-    assert_equal(bgp.default_event_history_objstore, bgp.event_history_objstore)
-    bgp.event_history_objstore = 'size_small'
-    assert_equal('size_small', bgp.event_history_objstore)
-    bgp.event_history_objstore = 'size_large'
-    assert_equal('size_large', bgp.event_history_objstore)
+    skip_incompat_version?('bgp', 'event_history_objstore')
     bgp.event_history_objstore = 'size_medium'
     assert_equal('size_medium', bgp.event_history_objstore)
-    bgp.event_history_objstore = 'size_disable'
-    assert_equal('false', bgp.event_history_objstore)
-    bgp.event_history_objstore = '100000'
-    assert_equal('100000', bgp.event_history_objstore)
-    bgp.event_history_objstore = bgp.default_event_history_objstore
-    assert_equal(bgp.default_event_history_objstore, bgp.event_history_objstore)
   end
 
   def test_event_history_periodic
@@ -750,41 +654,12 @@ class TestRouterBgp < CiscoTestCase
     if validate_property_excluded?('bgp', 'event_history_periodic')
       assert_nil(bgp.event_history_periodic)
       assert_raises(Cisco::UnsupportedError) do
-        bgp.event_history_periodic = 'true'
+        bgp.event_history_periodic = 'size_large'
       end
       return
     end
-    assert_equal(bgp.default_event_history_periodic,
-                 bgp.event_history_periodic)
-    bgp.event_history_periodic = 'false'
-    assert_equal('false', bgp.event_history_periodic) unless
-      Utils.image_version?(/8.0|8.1/)
-    bgp.event_history_periodic = 'size_small'
-    assert_equal('size_small', bgp.event_history_periodic)
     bgp.event_history_periodic = 'size_large'
     assert_equal('size_large', bgp.event_history_periodic)
-    bgp.event_history_periodic = 'size_medium'
-    assert_equal('size_medium', bgp.event_history_periodic)
-    bgp.event_history_periodic = '100000'
-    assert_equal('100000', bgp.event_history_periodic)
-    bgp.event_history_periodic = 'size_disable'
-    if newer_image_version?
-      assert_equal(bgp.default_event_history_periodic,
-                   bgp.event_history_periodic)
-    else
-      assert_equal('size_disable', bgp.event_history_periodic)
-    end
-    bgp.event_history_periodic = 'true'
-    if newer_image_version?
-      assert_equal('true', bgp.event_history_periodic) unless
-        Utils.image_version?(/8.0|8.1/)
-    else
-      assert_equal(bgp.default_event_history_periodic,
-                   bgp.event_history_periodic)
-    end
-    bgp.event_history_periodic = bgp.default_event_history_periodic
-    assert_equal(bgp.default_event_history_periodic,
-                 bgp.event_history_periodic)
   end
 
   def test_fast_external_fallover