cisco_node_utils 1.0.1 → 1.1.0

data/tests/test_snmpgroup.rb

@@ -12,39 +12,37 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-require File.expand_path("../ciscotest", __FILE__)
-require File.expand_path("../../lib/cisco_node_utils/snmpgroup", __FILE__)
+require_relative 'ciscotest'
+require_relative '../lib/cisco_node_utils/snmpgroup'
 
+# TestSnmpGroup - Minitest for SnmpGroup node utility.
 class TestSnmpGroup < CiscoTestCase
   # NXOS snmp groups will not be empty
   def test_snmpgroup_collection_not_empty
     snmpgroups = SnmpGroup.groups
-    assert_equal(false, snmpgroups.empty?(),
-                 "SnmpGroup collection is empty")
-    snmpgroups = nil
+    refute_empty(snmpgroups)
   end
 
   def test_snmpgroup_collection_valid
     snmpgroups = SnmpGroup.groups
-    s = @device.cmd("show snmp group | include Role | no-more")
-    snmpgroups.each do |name, snmpgroup|
+    s = @device.cmd('show snmp group | include Role | no-more')
+    snmpgroups.each_value do |snmpgroup|
       line = /Role:\s#{snmpgroup.name}/.match(s)
       # puts "line: #{line}"
       assert_equal(false, line.nil?)
     end
-    snmpgroups = nil
   end
 
   def test_snmpgroup_exists_with_name_empty
     assert_raises(ArgumentError) do
-      SnmpGroup.exists?("")
+      SnmpGroup.exists?('')
     end
   end
 
   def test_snmpgroup_exists_with_name_invalid
-    name = "group-dummy"
+    name = 'group-dummy'
     exist = SnmpGroup.exists?(name)
-    s = @device.cmd("show snmp group | in Role | no-more")
+    s = @device.cmd('show snmp group | in Role | no-more')
     line = /Role:\s#{name}/.match(s)
     assert_equal(exist, !line.nil?)
   end
@@ -56,15 +54,15 @@ class TestSnmpGroup < CiscoTestCase
   end
 
   def test_snmpgroup_exists_with_name_valid
-    name = "network-admin"
+    name = 'network-admin'
     exist = SnmpGroup.exists?(name)
-    s = @device.cmd("show snmp group | in Role | no-more")
+    s = @device.cmd('show snmp group | in Role | no-more')
     line = /Role:\s#{name}/.match(s)
     assert_equal(exist, !line.nil?)
   end
 
   def test_snmpgroup_get_name
-    name = "network-operator"
+    name = 'network-operator'
     snmpgroup = SnmpGroup.new(name)
     assert_equal(snmpgroup.name, name)
   end

data/tests/test_snmpserver.rb

@@ -12,43 +12,57 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-require File.expand_path("../ciscotest", __FILE__)
-require File.expand_path("../../lib/cisco_node_utils/snmpserver", __FILE__)
+require_relative 'ciscotest'
+require_relative '../lib/cisco_node_utils/snmpserver'
 
+# TestSnmpServer - Minitest for SnmpServer node utility
 class TestSnmpServer < CiscoTestCase
   DEFAULT_SNMP_SERVER_AAA_USER_CACHE_TIMEOUT = 3600
-  DEFAULT_SNMP_SERVER_LOCATION = ""
-  DEFAULT_SNMP_SERVER_CONTACT = ""
+  DEFAULT_SNMP_SERVER_LOCATION = ''
+  DEFAULT_SNMP_SERVER_CONTACT = ''
   DEFAULT_SNMP_SERVER_PACKET_SIZE = 1500
   DEFAULT_SNMP_SERVER_GLOBAL_ENFORCE_PRIVACY = false
   DEFAULT_SNMP_SERVER_PROTOCOL_ENABLE = true
   DEFAULT_SNMP_SERVER_TCP_SESSION_AUTH = true
 
+  def setup
+    super
+    @default_show_command = 'show run snmp all | no-more'
+    # set all commands to defaults
+    s = SnmpServer.new
+    s.aaa_user_cache_timeout = s.default_aaa_user_cache_timeout
+    s.contact = s.default_contact
+    s.global_enforce_priv = s.default_global_enforce_priv
+    s.location = s.default_location
+    s.packet_size = 0
+    s.protocol = s.default_protocol
+    s.tcp_session_auth = s.default_tcp_session_auth
+  end
+
   def test_snmpserver_aaa_user_cache_timeout_set_invalid_upper_range
     snmpserver = SnmpServer.new
     assert_raises(Cisco::CliError) do
-      snmpserver.aaa_user_cache_timeout=86401
+      snmpserver.aaa_user_cache_timeout = 86_401
     end
   end
 
   def test_snmpserver_aaa_user_cache_timeout_set_invalid_lower_range
     snmpserver = SnmpServer.new
     assert_raises(Cisco::CliError) do
-      snmpserver.aaa_user_cache_timeout=0
+      snmpserver.aaa_user_cache_timeout = 0
     end
   end
 
   def test_snmpserver_aaa_user_cache_timeout_set_valid
     snmpserver = SnmpServer.new
     newtimeout = 1400
-    snmpserver.aaa_user_cache_timeout=newtimeout
-    s = @device.cmd("show run snmp all | no-more")
-    cmd='snmp-server aaa-user cache-timeout'
-    line = /#{cmd} (\d+)/.match(s)
-    timeout = line.to_s.split(" ").last.to_i
+    snmpserver.aaa_user_cache_timeout = newtimeout
+    line = assert_show_match(
+      pattern: /snmp-server aaa-user cache-timeout (\d+)/)
+    timeout = line.to_s.split(' ').last.to_i
     assert_equal(timeout, newtimeout)
     # set to default
-    snmpserver.aaa_user_cache_timeout=
+    snmpserver.aaa_user_cache_timeout =
       snmpserver.default_aaa_user_cache_timeout
   end
 
@@ -56,161 +70,157 @@ class TestSnmpServer < CiscoTestCase
     snmpserver = SnmpServer.new
     snmpserver.aaa_user_cache_timeout =
       snmpserver.default_aaa_user_cache_timeout
-    s = @device.cmd("show run snmp all | no-more")
-    cmd='snmp-server aaa-user cache-timeout'
-    line = /#{cmd} (\d+)/.match(s)
-    timeout = line.to_s.split(" ").last.to_i
+    line = assert_show_match(
+      pattern: /snmp-server aaa-user cache-timeout (\d+)/)
+    timeout = line.to_s.split(' ').last.to_i
     assert_equal(timeout, snmpserver.aaa_user_cache_timeout)
   end
 
   def test_snmpserver_aaa_user_cache_timeout_get
     snmpserver = SnmpServer.new
-    s = @device.cmd("show run snmp all | no-more")
-    cmd='snmp-server aaa-user cache-timeout'
-    line = /#{cmd} (\d+)/.match(s)
-    timeout = line.to_s.split(" ").last.to_i
+    line = assert_show_match(
+      pattern: /snmp-server aaa-user cache-timeout (\d+)/)
+    timeout = line.to_s.split(' ').last.to_i
     assert_equal(timeout, snmpserver.aaa_user_cache_timeout)
   end
 
   def test_snmpserver_sys_contact_get
     snmpserver = SnmpServer.new
-    snmpserver.contact = "test-contact"
-    s = @device.cmd("show snmp | no-more")
-    line = /sys contact: [^\n\r]+/.match(s)
-    contact = ""
-    contact = line.to_s.gsub("sys contact:", "").strip unless line.nil?
+    snmpserver.contact = 'test-contact'
+    line = assert_show_match(command: 'show snmp | no-more',
+                             pattern: /sys contact: [^\n\r]+/)
+    contact = ''
+    contact = line.to_s.gsub('sys contact:', '').strip unless line.nil?
     # puts "contact : #{line}, #{snmpserver.contact}"
     assert_equal(contact, snmpserver.contact)
     # set to default
-    snmpserver.contact=snmpserver.default_contact
+    snmpserver.contact = snmpserver.default_contact
   end
 
   def test_snmpserver_sys_contact_set_invalid
     snmpserver = SnmpServer.new
     assert_raises(TypeError) do
-      snmpserver.contact=123
+      snmpserver.contact = 123
     end
   end
 
   def test_snmpserver_sys_contact_set_zero_length
     snmpserver = SnmpServer.new
-    newcontact = ""
-    snmpserver.contact=newcontact
+    newcontact = ''
+    snmpserver.contact = newcontact
     assert_equal(newcontact, snmpserver.contact)
     # set to default
-    snmpserver.contact=snmpserver.default_contact
+    snmpserver.contact = snmpserver.default_contact
   end
 
   def test_snmpserver_sys_contact_set_valid
     snmpserver = SnmpServer.new
-    newcontact = "mvenkata_test# contact"
-    snmpserver.contact=newcontact
-    s = @device.cmd("show snmp | no-more")
-    line = /sys contact: [^\n\r]+/.match(s)
+    newcontact = 'mvenkata_test# contact'
+    snmpserver.contact = newcontact
+    line = assert_show_match(command: 'show snmp | no-more',
+                             pattern: /sys contact: [^\n\r]+/)
     # puts "line: #{line}"
-    contact = line.to_s.gsub("sys contact:", "").strip
+    contact = line.to_s.gsub('sys contact:', '').strip
     assert_equal(contact, newcontact)
     # set to default
-    snmpserver.contact=snmpserver.default_contact
+    snmpserver.contact = snmpserver.default_contact
   end
 
   def test_snmpserver_sys_contact_set_special_chars
     snmpserver = SnmpServer.new
     # newcontact = "Test{}(%tuvy@_cisco contact$#!@1234^&*()_+"
-    newcontact = "user@example.com @$%&}test ]|[#_@test contact"
-    snmpserver.contact=newcontact
-    s = @device.cmd("show snmp | no-more")
-    line = /sys contact: [^\n\r]+/.match(s)
+    newcontact = 'user@example.com @$%&}test ]|[#_@test contact'
+    snmpserver.contact = newcontact
+    line = assert_show_match(command: 'show snmp | no-more',
+                             pattern: /sys contact: [^\n\r]+/)
     # puts "line: #{line}"
-    contact = line.to_s.gsub("sys contact:", "").strip
+    contact = line.to_s.gsub('sys contact:', '').strip
     assert_equal(contact, newcontact)
     # set to default
-    snmpserver.contact=snmpserver.default_contact
+    snmpserver.contact = snmpserver.default_contact
   end
 
   def test_snmpserver_sys_contact_set_default
     snmpserver = SnmpServer.new
     snmpserver.contact = snmpserver.default_contact
-    s = @device.cmd("show snmp | no-more")
-    line = /sys contact: [^\n\r]+/.match(s)
-    contact = ""
-    contact = line.to_s.gsub("sys contact:", "").strip unless line.nil?
+    line = assert_show_match(command: 'show snmp | no-more',
+                             pattern: /sys contact: [^\n\r]*/)
+    contact = ''
+    contact = line.to_s.gsub('sys contact:', '').strip unless line.nil?
     assert_equal(contact, snmpserver.default_contact)
   end
 
   def test_snmpserver_sys_location_get
     snmpserver = SnmpServer.new
     # set location
-    snmpserver.location = "test-location"
-    s = @device.cmd("show snmp | no-more")
-    line = /sys location: [^\n\r]+/.match(s)
-    location=""
-    location = line.to_s.gsub("sys location:", "").strip unless line.nil?
+    snmpserver.location = 'test-location'
+    line = assert_show_match(command: 'show snmp | no-more',
+                             pattern: /sys location: [^\n\r]+/)
+    location = ''
+    location = line.to_s.gsub('sys location:', '').strip unless line.nil?
     # puts "location : #{location}, #{snmpserver.location}"
     assert_equal(location, snmpserver.location)
     # set to default
-    snmpserver.location=snmpserver.default_location
+    snmpserver.location = snmpserver.default_location
   end
 
   def test_snmpserver_sys_location_set_invalid
     snmpserver = SnmpServer.new
     assert_raises(TypeError) do
-      snmpserver.location=123
+      snmpserver.location = 123
     end
   end
 
   def test_snmpserver_sys_location_set_zero_length
     snmpserver = SnmpServer.new
-    newlocation = ""
-    snmpserver.location=newlocation
+    newlocation = ''
+    snmpserver.location = newlocation
     assert_equal(newlocation, snmpserver.location)
   end
 
   def test_snmpserver_sys_location_set_valid
     snmpserver = SnmpServer.new
-    newlocation = "bxb-300-2-1 test location"
-    snmpserver.location=newlocation
-    s = @device.cmd("show snmp | no-more")
-    line = /sys location: [^\n\r]+/.match(s)
-    location = line.to_s.gsub("sys location:", "").strip
+    newlocation = 'bxb-300-2-1 test location'
+    snmpserver.location = newlocation
+    line = assert_show_match(command: 'show snmp | no-more',
+                             pattern: /sys location: [^\n\r]+/)
+    location = line.to_s.gsub('sys location:', '').strip
     assert_equal(location, newlocation)
     # set to default
-    snmpserver.location=snmpserver.default_location
+    snmpserver.location = snmpserver.default_location
   end
 
   def test_snmpserver_sys_location_set_special_chars
     snmpserver = SnmpServer.new
-    newlocation = "bxb-300 2nd floor test !$%^33&&*) location"
-    snmpserver.location=newlocation
-    s = @device.cmd("show snmp | no-more")
-    line = /sys location: [^\n\r]+/.match(s)
-    location = line.to_s.gsub("sys location:", "").strip
+    newlocation = 'bxb-300 2nd floor test !$%^33&&*) location'
+    snmpserver.location = newlocation
+    line = assert_show_match(command: 'show snmp | no-more',
+                             pattern: /sys location: [^\n\r]+/)
+    location = line.to_s.gsub('sys location:', '').strip
     assert_equal(location, newlocation)
     # set to default
-    snmpserver.location=snmpserver.default_location
+    snmpserver.location = snmpserver.default_location
   end
 
   def test_snmpserver_sys_location_set_default
     snmpserver = SnmpServer.new
     #    snmpserver.location = snmpserver.default_location
-    snmpserver.location = "FOO"
-    s = @device.cmd("show snmp | no-more")
-    line = /sys location: [^\n\r]+/.match(s)
-    location = ""
-    location = line.to_s.gsub("sys location:", "").strip unless line.nil?
+    snmpserver.location = 'FOO'
+    line = assert_show_match(command: 'show snmp | no-more',
+                             pattern: /sys location: [^\n\r]+/)
+    location = ''
+    location = line.to_s.gsub('sys location:', '').strip unless line.nil?
     assert_equal(location, snmpserver.location)
     # set to default
-    snmpserver.location=snmpserver.default_location
+    snmpserver.location = snmpserver.default_location
   end
 
   def test_snmpserver_packetsize_get
     snmpserver = SnmpServer.new
-    snmpserver.packet_size=2000
-    s = @device.cmd("show run snmp all | no-more")
-    cmd='snmp-server packetsize'
-    line = /#{cmd} (\d+)/.match(s)
+    snmpserver.packet_size = 2000
+    line = assert_show_match(pattern: /snmp-server packetsize (\d+)/)
     packetsize = 0
-    packetsize = line.to_s.split(" ").last.to_i unless line.nil?
+    packetsize = line.to_s.split(' ').last.to_i unless line.nil?
     assert_equal(packetsize, snmpserver.packet_size)
     # unset to default
     snmpserver.packet_size = 0
@@ -218,9 +228,9 @@ class TestSnmpServer < CiscoTestCase
 
   def test_snmpserver_packetsize_set_invalid_upper_range
     snmpserver = SnmpServer.new
-    newpacketsize = 17383
+    newpacketsize = 17_383
     assert_raises(Cisco::CliError) do
-      snmpserver.packet_size=newpacketsize
+      snmpserver.packet_size = newpacketsize
     end
   end
 
@@ -228,18 +238,16 @@ class TestSnmpServer < CiscoTestCase
     snmpserver = SnmpServer.new
     newpacketsize = 480
     assert_raises(Cisco::CliError) do
-      snmpserver.packet_size=newpacketsize
+      snmpserver.packet_size = newpacketsize
     end
   end
 
   def test_snmpserver_packetsize_set_valid
     snmpserver = SnmpServer.new
     newpacketsize = 1600
-    snmpserver.packet_size=newpacketsize
-    s = @device.cmd("show run snmp all | no-more")
-    cmd='snmp-server packetsize'
-    line = /#{cmd} (\d+)/.match(s)
-    packetsize = line.to_s.split(" ").last.to_i
+    snmpserver.packet_size = newpacketsize
+    line = assert_show_match(pattern: /snmp-server packetsize (\d+)/)
+    packetsize = line.to_s.split(' ').last.to_i
     assert_equal(packetsize, newpacketsize)
     # unset to default
     snmpserver.packet_size = 0
@@ -247,30 +255,22 @@ class TestSnmpServer < CiscoTestCase
 
   def test_snmpserver_packetsize_set_default
     snmpserver = SnmpServer.new
-
-    packetsize = DEFAULT_SNMP_SERVER_PACKET_SIZE
-    s = @device.cmd("show run snmp all | no-more")
-    cmd='snmp-server packetsize'
-    line = /#{cmd} (\d+)/.match(s)
-    packetsize = line.to_s.split(" ").last.to_i
-    assert_equal(packetsize, snmpserver.packet_size,
-                 "Error: Snmp Server, packet size not default")
+    refute_show_match(pattern: /snmp-server packetsize (\d+)/)
+    assert_equal(0, snmpserver.packet_size,
+                 'Error: Snmp Server, packet size not default')
 
     snmpserver.packet_size = 850
-    s = @device.cmd("show run snmp all | no-more")
-    cmd='snmp-server packetsize'
-    line = /#{cmd} (\d+)/.match(s)
-    packetsize = line.to_s.split(" ").last.to_i
+    line = assert_show_match(pattern: /snmp-server packetsize (\d+)/)
+    packetsize = line.to_s.split(' ').last.to_i
     assert_equal(packetsize, snmpserver.packet_size,
-                 "Error: Snmp Server, packet size not default")
+                 'Error: Snmp Server, packet size not default')
 
     snmpserver.packet_size = snmpserver.default_packet_size
-    s = @device.cmd("show run snmp all | no-more")
-    cmd='snmp-server packetsize'
-    line = /#{cmd} (\d+)/.match(s)
-    packetsize = line.to_s.split(" ").last.to_i
+    # TODO: this seems weird, why is default_packet_size not 0 as above?
+    line = assert_show_match(pattern: /snmp-server packetsize (\d+)/)
+    packetsize = line.to_s.split(' ').last.to_i
     assert_equal(packetsize, snmpserver.packet_size,
-                 "Error: Snmp Server, packet size not default")
+                 'Error: Snmp Server, packet size not default')
     # set to default
     snmpserver.packet_size = 0
   end
@@ -281,17 +281,14 @@ class TestSnmpServer < CiscoTestCase
     # Get orginal packet size
     org_packet_size = snmpserver.packet_size
     snmpserver.packet_size = 0
-    s = @device.cmd("show run snmp all | no-more")
-    cmd='snmp-server packetsize'
-    line = /#{cmd} (\d+)/.match(s)
-    packetsize = line.to_s.split(" ").last.to_i
-    assert_equal(packetsize, snmpserver.packet_size,
-                 "Error: Snmp Server, packet size not unset")
+    refute_show_match(pattern: /snmp-server packetsize (\d+)/)
+    assert_equal(0, snmpserver.packet_size,
+                 'Error: Snmp Server, packet size not unset')
 
     # Restore packet size
     snmpserver.packet_size = org_packet_size
     assert_equal(org_packet_size, snmpserver.packet_size,
-                 "Error: Snmp Server, packet size not restored")
+                 'Error: Snmp Server, packet size not restored')
     # set to default
     snmpserver.packet_size = 0
   end
@@ -299,145 +296,119 @@ class TestSnmpServer < CiscoTestCase
   def test_snmpserver_global_enforce_priv_get_default
     snmpserver = SnmpServer.new
     # default is false
-    snmpserver.global_enforce_priv=false
+    snmpserver.global_enforce_priv = false
     device_enabled = snmpserver.global_enforce_priv?
-    s = @device.cmd("show run snmp all | no-more")
-    cmd='no snmp-server globalEnforcePriv'
-    line = /#{cmd}/.match(s)
-    assert_equal(line.nil?, device_enabled)
+    if device_enabled
+      refute_show_match(pattern: /no snmp-server globalEnforcePriv/)
+    else
+      assert_show_match(pattern: /no snmp-server globalEnforcePriv/)
+    end
     # set to default
-    snmpserver.global_enforce_priv=snmpserver.default_global_enforce_priv
+    snmpserver.global_enforce_priv = snmpserver.default_global_enforce_priv
   end
 
   def test_snmpserver_global_enforce_priv_get_enabled
     snmpserver = SnmpServer.new
-    snmpserver.global_enforce_priv=true
+    snmpserver.global_enforce_priv = true
     device_enabled = snmpserver.global_enforce_priv?
-    s = @device.cmd("show run snmp all | no-more")
-    cmd='snmp-server globalEnforcePriv'
-    line = /#{cmd}/.match(s)
-    assert_equal(!line.nil?, device_enabled)
+    if device_enabled
+      refute_show_match(pattern: /no snmp-server globalEnforcePriv/)
+    else
+      assert_show_match(pattern: /no snmp-server globalEnforcePriv/)
+    end
     # set to default
-    snmpserver.global_enforce_priv=snmpserver.default_global_enforce_priv
+    snmpserver.global_enforce_priv = snmpserver.default_global_enforce_priv
   end
 
   def test_snmpserver_global_enforce_priv_set_enabled
     snmpserver = SnmpServer.new
-    snmpserver.global_enforce_priv=true
-    s = @device.cmd("show run snmp all | no-more")
-    cmd='snmp-server globalEnforcePriv'
-    line = /#{cmd}/.match(s)
-    # puts "line : #{line}"
-    assert_equal(!line.nil?, true)
+    snmpserver.global_enforce_priv = true
+    assert_show_match(pattern: /snmp-server globalEnforcePriv/)
     # set to default
-    snmpserver.global_enforce_priv=snmpserver.default_global_enforce_priv
+    snmpserver.global_enforce_priv = snmpserver.default_global_enforce_priv
   end
 
   def test_snmpserver_global_enforce_priv_set_disabled
     snmpserver = SnmpServer.new
-    snmpserver.global_enforce_priv=false
-    s = @device.cmd("show run snmp all | no-more")
-    cmd='no snmp-server globalEnforcePriv'
-    line = /#{cmd}/.match(s)
-    assert_equal(line.nil?, false)
+    snmpserver.global_enforce_priv = false
+    assert_show_match(pattern: /no snmp-server globalEnforcePriv/)
     # set to default
-    snmpserver.global_enforce_priv=snmpserver.default_global_enforce_priv
+    snmpserver.global_enforce_priv = snmpserver.default_global_enforce_priv
   end
 
   def test_snmpserver_protocol_get_default
     snmpserver = SnmpServer.new
     # set default
-    snmpserver.protocol=true
-    device_enabled = snmpserver.protocol?
-    cmd='^snmp-server protocol enable'
-    s = @device.cmd("show run snmp all | i '#{cmd}'")
-    assert_match(/#{cmd}/, s)
+    snmpserver.protocol = true
+    assert_show_match(pattern: /^snmp-server protocol enable/)
     # set to default
-    snmpserver.protocol=snmpserver.default_protocol
+    snmpserver.protocol = snmpserver.default_protocol
   end
 
   def test_snmpserver_protocol_get_disabled
     snmpserver = SnmpServer.new
-    snmpserver.protocol=false
+    snmpserver.protocol = false
     device_enabled = snmpserver.protocol?
-    s = @device.cmd("show run snmp all | no-more")
-    cmd='no snmp-server protocol enable'
-    line = /#{cmd}/.match(s)
-    # puts "line #{line}"
-    assert_equal(line.nil?, device_enabled)
+    if device_enabled
+      assert_show_match(pattern: /^snmp-server protocol enable/)
+    else
+      assert_show_match(pattern: /no snmp-server protocol enable/)
+    end
     # set to default
-    snmpserver.protocol=snmpserver.default_protocol
+    snmpserver.protocol = snmpserver.default_protocol
   end
 
   def test_snmpserver_protocol_set_enabled
     snmpserver = SnmpServer.new
-    snmpserver.protocol=true
-    s = @device.cmd("show run snmp all | no-more")
-    cmd='snmp-server protocol enable'
-    line = /#{cmd}/.match(s)
-    # puts "line : #{line}"
-    assert_equal(!line.nil?, true)
+    snmpserver.protocol = true
+    assert_show_match(pattern: /^snmp-server protocol enable/)
     # set to default
-    snmpserver.protocol=snmpserver.default_protocol
+    snmpserver.protocol = snmpserver.default_protocol
   end
 
   def test_snmpserver_protocol_set_disabled
     snmpserver = SnmpServer.new
-    snmpserver.protocol=false
-    s = @device.cmd("show run snmp all | no-more")
-    cmd='no snmp-server protocol enable'
-    line = /#{cmd}/.match(s)
-    # puts "line : #{line}"
-    assert_equal(line.nil?, false)
+    snmpserver.protocol = false
+    assert_show_match(pattern: /no snmp-server protocol enable/)
     # set to default
-    snmpserver.protocol=snmpserver.default_protocol
+    snmpserver.protocol = snmpserver.default_protocol
   end
 
   def test_snmpserver_tcp_session_auth_get_default
     snmpserver = SnmpServer.new
     # default value is false
-    snmpserver.tcp_session_auth=false
+    snmpserver.tcp_session_auth = false
     device_enabled = snmpserver.tcp_session_auth?
-    s = @device.cmd("show run snmp all | no-more")
-    cmd='no snmp-server tcp-session auth'
-    line = /#{cmd}/.match(s)
-    assert_equal(line.nil?, device_enabled)
+    if device_enabled
+      assert_show_match(pattern: /^snmp-server tcp-session auth/)
+    else
+      assert_show_match(pattern: /no snmp-server tcp-session auth/)
+    end
     # set to default
-    snmpserver.tcp_session_auth=snmpserver.default_tcp_session_auth
+    snmpserver.tcp_session_auth = snmpserver.default_tcp_session_auth
   end
 
   def test_snmpserver_tcp_session_auth_get_enabled
     snmpserver = SnmpServer.new
-    snmpserver.tcp_session_auth=true
-    device_enabled = snmpserver.tcp_session_auth?
-    cmd='^snmp-server tcp-session auth'
-    s = @device.cmd("show run snmp all | i '#{cmd}'")
-    assert_match(/#{cmd}/, s)
+    snmpserver.tcp_session_auth = true
+    assert_show_match(pattern: /^snmp-server tcp-session auth/)
     # set to default
-    snmpserver.tcp_session_auth=snmpserver.default_tcp_session_auth
+    snmpserver.tcp_session_auth = snmpserver.default_tcp_session_auth
   end
 
   def test_snmpserver_tcp_session_auth_set_enabled
     snmpserver = SnmpServer.new
-    snmpserver.tcp_session_auth=true
-    s = @device.cmd("show run snmp all | no-more")
-    cmd='snmp-server tcp-session auth'
-    line = /#{cmd}/.match(s)
-    # puts "line : #{line}"
-    assert_equal(!line.nil?, true)
+    snmpserver.tcp_session_auth = true
+    assert_show_match(pattern: /^snmp-server tcp-session auth/)
     # set to default
-    snmpserver.tcp_session_auth=snmpserver.default_tcp_session_auth
+    snmpserver.tcp_session_auth = snmpserver.default_tcp_session_auth
   end
 
   def test_snmpserver_tcp_session_auth_set_default
     snmpserver = SnmpServer.new
-    snmpserver.tcp_session_auth=false
-    s = @device.cmd("show run snmp all | no-more")
-    cmd='no snmp-server tcp-session auth'
-    line = /#{cmd}/.match(s)
-    # puts "line : #{line}"
-    assert_equal(line.nil?, false)
+    snmpserver.tcp_session_auth = false
+    assert_show_match(pattern: /no snmp-server tcp-session auth/)
     # set to default
-    snmpserver.tcp_session_auth=snmpserver.default_tcp_session_auth
+    snmpserver.tcp_session_auth = snmpserver.default_tcp_session_auth
   end
 end