cisco_node_utils 1.2.0 → 1.3.0

data/tests/test_syslog_server.rb

@@ -34,9 +34,17 @@ class TestSyslogServer < CiscoTestCase
 
   def no_syslogserver
     # Turn the feature off for a clean test.
-    config('no logging server 1.2.3.4',
-           'no logging server 2003::2',
-           'no vrf context red')
+    if platform == :ios_xr
+      config('no logging 1.2.3.4',
+             'no logging 1.2.3.4 vrf red',
+             'no logging 2003::2',
+             'no logging 2003::2 vrf red',
+             'no vrf red')
+    else
+      config('no logging server 1.2.3.4',
+             'no logging server 2003::2',
+             'no vrf context red')
+    end
   end
 
   # TESTS
@@ -48,6 +56,7 @@ class TestSyslogServer < CiscoTestCase
     server = Cisco::SyslogServer.new(id, 2, 'default', true)
     assert_includes(Cisco::SyslogServer.syslogservers, id)
     assert_equal(server, Cisco::SyslogServer.syslogservers[id])
+    assert_equal(2, Cisco::SyslogServer.syslogservers[id].level)
 
     server.destroy
     refute_includes(Cisco::SyslogServer.syslogservers, id)
@@ -60,6 +69,7 @@ class TestSyslogServer < CiscoTestCase
     server = Cisco::SyslogServer.new(id, 2, 'default', true)
     assert_includes(Cisco::SyslogServer.syslogservers, id)
     assert_equal(server, Cisco::SyslogServer.syslogservers[id])
+    assert_equal(2, Cisco::SyslogServer.syslogservers[id].level)
 
     server.destroy
     refute_includes(Cisco::SyslogServer.syslogservers, id)
@@ -72,11 +82,13 @@ class TestSyslogServer < CiscoTestCase
     refute_includes(Cisco::SyslogServer.syslogservers, id2)
 
     server = Cisco::SyslogServer.new(id, 2, 'default', true)
-    server2 = Cisco::SyslogServer.new(id2, 2, 'default', true)
+    server2 = Cisco::SyslogServer.new(id2, 3, 'default', true)
     assert_includes(Cisco::SyslogServer.syslogservers, id)
     assert_equal(server, Cisco::SyslogServer.syslogservers[id])
+    assert_equal(2, Cisco::SyslogServer.syslogservers[id].level)
     assert_includes(Cisco::SyslogServer.syslogservers, id2)
     assert_equal(server2, Cisco::SyslogServer.syslogservers[id2])
+    assert_equal(3, Cisco::SyslogServer.syslogservers[id2].level)
 
     server.destroy
     server2.destroy
@@ -85,28 +97,40 @@ class TestSyslogServer < CiscoTestCase
   end
 
   def test_create_destroy_single_vrf_ipv4
-    config('vrf context red')
+    if platform == :ios_xr
+      config('vrf red')
+    else
+      config('vrf context red')
+    end
+
     id = '1.2.3.4'
 
     refute_includes(Cisco::SyslogServer.syslogservers, id)
 
-    server = Cisco::SyslogServer.new(id, 2, 'red', true)
+    server = Cisco::SyslogServer.new(id, 4, 'red', true)
     assert_includes(Cisco::SyslogServer.syslogservers, id)
     assert_equal(server, Cisco::SyslogServer.syslogservers[id])
+    assert_equal(4, Cisco::SyslogServer.syslogservers[id].level)
 
     server.destroy
     refute_includes(Cisco::SyslogServer.syslogservers, id)
   end
 
   def test_create_destroy_single_vrf_ipv6
-    config('vrf context red')
+    if platform == :ios_xr
+      config('vrf red')
+    else
+      config('vrf context red')
+    end
+
     id = '2003::2'
 
     refute_includes(Cisco::SyslogServer.syslogservers, id)
 
-    server = Cisco::SyslogServer.new(id, 2, 'red', true)
+    server = Cisco::SyslogServer.new(id, 5, 'red', true)
     assert_includes(Cisco::SyslogServer.syslogservers, id)
     assert_equal(server, Cisco::SyslogServer.syslogservers[id])
+    assert_equal(5, Cisco::SyslogServer.syslogservers[id].level)
 
     server.destroy
     refute_includes(Cisco::SyslogServer.syslogservers, id)

data/tests/test_syslog_settings.rb

@@ -21,12 +21,14 @@ require_relative '../lib/cisco_node_utils/syslog_settings'
 # TestSyslogSetting - Minitest for SyslogSetting node utility.
 class TestSyslogSettings < CiscoTestCase
   def setup
+    return if platform != :nexus
     # setup runs at the beginning of each test
     super
     no_syslogsettings
   end
 
   def teardown
+    return if platform != :nexus
     # teardown runs at the end of each test
     no_syslogsettings
     super
@@ -41,14 +43,25 @@ class TestSyslogSettings < CiscoTestCase
 
   def test_syslogsettings_create
     syslog_setting = Cisco::SyslogSettings.new('default')
-    assert_includes(Cisco::SyslogSettings.syslogsettings, 'default')
-    assert_equal(Cisco::SyslogSettings.syslogsettings['default'],
-                 syslog_setting)
-
-    syslog_setting.timestamp = 'milliseconds'
-    assert_equal(Cisco::SyslogSettings.syslogsettings['default'].timestamp,
-                 'milliseconds')
-    assert_equal(syslog_setting.timestamp,
-                 'milliseconds')
+
+    if platform == :ios_xr
+      assert_nil(syslog_setting.timestamp)
+      assert_raises(Cisco::UnsupportedError) do
+        syslog_setting.timestamp = 'milliseconds'
+      end
+    else
+      assert_includes(Cisco::SyslogSettings.syslogsettings, 'default')
+      assert_equal(syslog_setting,
+                   Cisco::SyslogSettings.syslogsettings['default'],
+                  )
+
+      syslog_setting.timestamp = 'milliseconds'
+      assert_equal('milliseconds',
+                   Cisco::SyslogSettings.syslogsettings['default'].timestamp,
+                  )
+      assert_equal('milliseconds',
+                   syslog_setting.timestamp,
+                  )
+    end
   end
 end

data/tests/test_tacacs_server.rb

@@ -17,6 +17,8 @@ require_relative '../lib/cisco_node_utils/tacacs_server'
 
 # TestTacacsServer - Minitest for TacacsServer node utility
 class TestTacacsServer < CiscoTestCase
+  @skip_unless_supported = 'tacacs_server'
+
   def assert_tacacsserver_feature
     assert_show_match(command: 'show run all | no-more',
                       pattern: /feature tacacs\+/)
@@ -33,16 +35,22 @@ class TestTacacsServer < CiscoTestCase
     # 'directed-request' command is under 'show run aaa all'
     @default_show_command = 'show run tacacs all | no-more ; ' \
                             'show run aaa all | no-more'
+    config_no_warn('no feature tacacs+')
   end
 
-  def test_tacacsserver_create_valid
+  def teardown
+    config_no_warn('no feature tacacs+')
+    super
+  end
+
+  def test_create_valid
     tacacs = TacacsServer.new
     assert_tacacsserver_feature
     tacacs.destroy
   end
 
-  def test_tacacsserver_get_encryption_type
-    config('no feature tacacs+', 'feature tacacs+')
+  def test_get_encryption_type
+    config_no_warn('feature tacacs+')
     encryption_type = TACACS_SERVER_ENC_UNKNOWN
     # Get encryption password when not configured
     tacacs = TacacsServer.new
@@ -72,16 +80,14 @@ class TestTacacsServer < CiscoTestCase
     tacacs.destroy
   end
 
-  def test_tacacsserver_get_default_encryption
+  def test_get_default_encryption
     # Ruby can use defines, but only they're not initialized from an enum
     assert_equal(TACACS_SERVER_ENC_NONE,
                  TacacsServer.default_encryption_type,
                  'Error: Tacacs Server, default encryption incorrect')
   end
 
-  def test_tacacsserver_get_encryption_password
-    # Get encryption password when not configured
-    config('no feature tacacs+')
+  def test_get_encryption_password
     tacacs = TacacsServer.new
     assert_equal(node.config_get_default('tacacs_server',
                                          'encryption_password'),
@@ -102,14 +108,14 @@ class TestTacacsServer < CiscoTestCase
     tacacs.destroy
   end
 
-  def test_tacacsserver_get_default_encryption_password
+  def test_get_default_encryption_password
     assert_equal(node.config_get_default('tacacs_server',
                                          'encryption_password'),
                  TacacsServer.default_encryption_password,
                  'Error: Tacacs Server, default encryption password incorrect')
   end
 
-  def test_tacacsserver_key_set
+  def test_key_set
     enc_type = TACACS_SERVER_ENC_NONE
     # This one is needed since the 'sh run' will always display the type
     # differently than the used encryption config type.
@@ -135,8 +141,7 @@ class TestTacacsServer < CiscoTestCase
     tacacs.destroy
   end
 
-  def test_tacacsserver_key_unconfigure
-    config('no feature tacacs+')
+  def test_key_unconfigure
     enc_type = TACACS_SERVER_ENC_NONE
     # This one is needed since the 'sh run' will always display the type
     # differently than the used encryption config type.
@@ -158,7 +163,7 @@ class TestTacacsServer < CiscoTestCase
     tacacs.destroy
   end
 
-  def test_tacacsserver_get_timeout
+  def test_get_timeout
     tacacs = TacacsServer.new
     timeout = node.config_get_default('tacacs_server', 'timeout')
     assert_equal(timeout, tacacs.timeout,
@@ -171,13 +176,13 @@ class TestTacacsServer < CiscoTestCase
     tacacs.destroy
   end
 
-  def test_tacacsserver_get_default_timeout
+  def test_get_default_timeout
     assert_equal(node.config_get_default('tacacs_server', 'timeout'),
                  TacacsServer.default_timeout,
                  'Error: Tacacs Server, default timeout incorrect')
   end
 
-  def test_tacacsserver_set_timeout
+  def test_set_timeout
     timeout = 45
 
     tacacs = TacacsServer.new
@@ -199,7 +204,7 @@ class TestTacacsServer < CiscoTestCase
     tacacs.destroy
   end
 
-  def test_tacacsserver_get_deadtime
+  def test_get_deadtime
     tacacs = TacacsServer.new
     deadtime = node.config_get_default('tacacs_server', 'deadtime')
     assert_equal(deadtime, tacacs.deadtime,
@@ -212,13 +217,13 @@ class TestTacacsServer < CiscoTestCase
     tacacs.destroy
   end
 
-  def test_tacacsserver_get_default_deadtime
+  def test_get_default_deadtime
     assert_equal(node.config_get_default('tacacs_server', 'deadtime'),
                  TacacsServer.default_deadtime,
                  'Error: Tacacs Server, default deadtime incorrect')
   end
 
-  def test_tacacsserver_set_deadtime
+  def test_set_deadtime
     deadtime = 1250
 
     tacacs = TacacsServer.new
@@ -238,7 +243,7 @@ class TestTacacsServer < CiscoTestCase
     tacacs.destroy
   end
 
-  def test_tacacsserver_get_directed_request
+  def test_get_directed_request
     config('feature tacacs', 'tacacs-server directed-request')
     tacacs = TacacsServer.new
     assert(tacacs.directed_request?,
@@ -250,13 +255,13 @@ class TestTacacsServer < CiscoTestCase
     tacacs.destroy
   end
 
-  def test_tacacsserver_get_default_directed_request
+  def test_get_default_directed_request
     assert_equal(node.config_get_default('tacacs_server', 'directed_request'),
                  TacacsServer.default_directed_request,
                  'Error: Tacacs Server, default directed-request incorrect')
   end
 
-  def test_tacacsserver_set_directed_request
+  def test_set_directed_request
     config('feature tacacs', 'tacacs-server directed-request')
     state = true
     tacacs = TacacsServer.new
@@ -294,34 +299,34 @@ class TestTacacsServer < CiscoTestCase
     tacacs.destroy
   end
 
-  def test_tacacsserver_get_source_interface
-    config('no ip tacacs source-interface')
+  def test_get_source_interface
+    config_no_warn('no ip tacacs source-interface')
     tacacs = TacacsServer.new
     intf = node.config_get_default('tacacs_server', 'source_interface')
     assert_equal(intf, tacacs.source_interface,
                  'Error: Tacacs Server, source-interface set')
 
-    intf = 'Ethernet1/1'
+    intf = 'loopback41'
     config("ip tacacs source-interface #{intf}")
     assert_equal(intf, tacacs.source_interface,
                  'Error: Tacacs Server, source-interface not correct')
     tacacs.destroy
   end
 
-  def test_tacacsserver_get_default_source_interface
+  def test_get_default_source_interface
     assert_equal(node.config_get_default('tacacs_server', 'source_interface'),
                  TacacsServer.default_source_interface,
                  'Error: Tacacs Server, default source-interface incorrect')
   end
 
-  def test_tacacsserver_set_source_interface
+  def test_set_source_interface
     config('feature tacacs+', 'no ip tacacs source-int')
     intf = node.config_get_default('tacacs_server', 'source_interface')
     tacacs = TacacsServer.new
     assert_equal(intf, tacacs.source_interface,
                  'Error: Tacacs Server, source-interface set')
 
-    intf = 'Ethernet1/1'
+    intf = 'loopback41'
     tacacs.source_interface = intf
     line = assert_show_match(pattern: /ip tacacs source-interface #{intf}/,
                              msg:     'source-interface not configured')
@@ -344,7 +349,7 @@ class TestTacacsServer < CiscoTestCase
     tacacs.destroy
   end
 
-  def test_tacacsserver_destroy
+  def test_destroy
     tacacs = TacacsServer.new
     assert_tacacsserver_feature
     tacacs.destroy

data/tests/test_tacacs_server_group.rb

@@ -18,17 +18,34 @@ require_relative '../lib/cisco_node_utils/tacacs_server_host'
 
 # Test class for Tacacs Server Group
 class TestTacacsServerGroup < CiscoTestCase
+  @skip_unless_supported = 'tacacs_server_group'
+
+  def setup
+    super
+    # TBD: Remove once CSCuz44696 is resolved.
+    skip('This test is not currently supported on 7.0(3)I3 images') if
+      node.os_version[/7.0\(3\)I3\(/]
+  end
+
   def clean_tacacs_config
     config('no feature tacacs',
            'feature tacacs')
   end
 
   def create_tacacsserverhost(name='defaulttest')
-    TacacsServerHost.new(name)
+    config("tacacs-server host #{name}")
+  end
+
+  def detach_tacacsserverhost(name)
+    config("no tacacs-server host #{name}")
   end
 
-  def detach_tacacsserverhost(host)
-    host.destroy
+  def config_command
+    if platform == :ios_xr
+      'show running-config aaa'
+    else
+      'show run tacacs+ all | no-more'
+    end
   end
 
   def detach_aaaservergroup(aaa_server_group)
@@ -65,7 +82,7 @@ class TestTacacsServerGroup < CiscoTestCase
   def test_create_valid_tacacs
     group_name = 'Group1'
     aaa_group = TacacsServerGroup.new(group_name)
-    assert_show_match(command: 'show run tacacs+ all | no-more',
+    assert_show_match(command: config_command,
                       pattern: /#{group_name}/)
 
     detach_aaaservergroup(aaa_group)
@@ -77,9 +94,9 @@ class TestTacacsServerGroup < CiscoTestCase
     aaa_group1 = TacacsServerGroup.new(group_name1)
     aaa_group2 = TacacsServerGroup.new(group_name2)
 
-    assert_show_match(command: 'show run tacacs+ all | no-more',
+    assert_show_match(command: config_command,
                       pattern: /#{group_name1}/)
-    assert_show_match(command: 'show run tacacs+ all | no-more',
+    assert_show_match(command: config_command,
                       pattern: /#{group_name2}/)
 
     detach_aaaservergroup(aaa_group1)
@@ -87,6 +104,8 @@ class TestTacacsServerGroup < CiscoTestCase
   end
 
   def test_collection_empty_tacacs
+    return if platform == :ios_xr
+
     clean_tacacs_config
     aaa_group_list = TacacsServerGroup.groups
     assert_empty(aaa_group_list,
@@ -124,10 +143,10 @@ class TestTacacsServerGroup < CiscoTestCase
 
   def test_servers_tacacs
     clean_tacacs_config
-    server_name1 = 'server1'
-    server_name2 = 'server2'
-    server1 = create_tacacsserverhost(server_name1)
-    server2 = create_tacacsserverhost(server_name2)
+    server_name1 = '1.1.1.1'
+    server_name2 = '2.2.2.2'
+    create_tacacsserverhost(server_name1)
+    create_tacacsserverhost(server_name2)
 
     aaa_group = TacacsServerGroup.new('Group1')
 
@@ -141,41 +160,41 @@ class TestTacacsServerGroup < CiscoTestCase
     servers = aaa_group.servers
     assert_equal(2, servers.size,
                  'Error: Collection is not two servers')
-    assert(servers.include?('server1'),
+    assert(servers.include?(server_name1),
            "Error: Collection does not contain #{server_name1}")
-    assert(servers.include?('server2'),
+    assert(servers.include?(server_name2),
            "Error: Collection does not contain #{server_name2}")
 
     detach_aaaservergroup(aaa_group)
-    detach_tacacsserverhost(server1)
-    detach_tacacsserverhost(server2)
+    detach_tacacsserverhost(server_name1)
+    detach_tacacsserverhost(server_name2)
   end
 
   def test_add_server_tacacs
-    server_name1 = 'server1'
-    server_name2 = 'server2'
-    server1 = create_tacacsserverhost(server_name1)
-    server2 = create_tacacsserverhost(server_name2)
+    server_name1 = '1.1.1.1'
+    server_name2 = '2.2.2.2'
+    create_tacacsserverhost(server_name1)
+    create_tacacsserverhost(server_name2)
 
     aaa_group = TacacsServerGroup.new('Group1')
     aaa_group.servers = [server_name1, server_name2]
 
-    assert_show_match(command: 'show run tacacs+ all | no-more',
+    assert_show_match(command: config_command,
                       pattern: /server #{server_name1}/)
-    assert_show_match(command: 'show run tacacs+ all | no-more',
+    assert_show_match(command: config_command,
                       pattern: /server #{server_name2}/)
 
     detach_aaaservergroup(aaa_group)
-    detach_tacacsserverhost(server1)
-    detach_tacacsserverhost(server2)
+    detach_tacacsserverhost(server_name1)
+    detach_tacacsserverhost(server_name2)
   end
 
   def test_remove_server_tacacs
     clean_tacacs_config
-    server_name1 = 'server1'
-    server_name2 = 'server2'
-    server1 = create_tacacsserverhost(server_name1)
-    server2 = create_tacacsserverhost(server_name2)
+    server_name1 = '1.1.1.1'
+    server_name2 = '2.2.2.2'
+    create_tacacsserverhost(server_name1)
+    create_tacacsserverhost(server_name2)
 
     aaa_group = TacacsServerGroup.new('Group1')
     aaa_group.servers = [server_name1, server_name2]
@@ -187,24 +206,24 @@ class TestTacacsServerGroup < CiscoTestCase
 
     # Now remove them and then check again
     aaa_group.servers = [server_name2]
-    refute_show_match(command: 'show run tacacs+ all | no-more',
+    refute_show_match(command: config_command,
                       pattern: /server #{server_name1}/)
 
     aaa_group.servers = []
-    refute_show_match(command: 'show run tacacs+ all | no-more',
+    refute_show_match(command: config_command,
                       pattern: /server #{server_name2}/)
 
     detach_aaaservergroup(aaa_group)
-    detach_tacacsserverhost(server1)
-    detach_tacacsserverhost(server2)
+    detach_tacacsserverhost(server_name1)
+    detach_tacacsserverhost(server_name2)
   end
 
   def test_remove_server_twice_tacacs
     clean_tacacs_config
-    server_name1 = 'server1'
-    server_name2 = 'server2'
-    server1 = create_tacacsserverhost(server_name1)
-    server2 = create_tacacsserverhost(server_name2)
+    server_name1 = '1.1.1.1'
+    server_name2 = '2.2.2.2'
+    create_tacacsserverhost(server_name1)
+    create_tacacsserverhost(server_name2)
 
     aaa_group = TacacsServerGroup.new('Group1')
     aaa_group.servers = [server_name1, server_name2]
@@ -216,12 +235,12 @@ class TestTacacsServerGroup < CiscoTestCase
 
     # Remove server 1
     aaa_group.servers = [server_name2]
-    refute_show_match(command: 'show run tacacs+ all | no-more',
+    refute_show_match(command: config_command,
                       pattern: /server #{server_name1}/)
 
     # Now remove server 2
     aaa_group.servers = []
-    refute_show_match(command: 'show run tacacs+ all | no-more',
+    refute_show_match(command: config_command,
                       pattern: /server #{server_name2}/)
 
     # Check collection size
@@ -229,11 +248,15 @@ class TestTacacsServerGroup < CiscoTestCase
     assert_empty(servers, 'Error: Collection not empty')
 
     detach_aaaservergroup(aaa_group)
-    detach_tacacsserverhost(server1)
-    detach_tacacsserverhost(server2)
+    detach_tacacsserverhost(server_name1)
+    detach_tacacsserverhost(server_name2)
   end
 
   def test_get_vrf_tacacs
+    # TODO
+    return if validate_property_excluded?('tacacs_server_group',
+                                          'vrf')
+
     group_name1 = 'Group1'
     aaa_group = TacacsServerGroup.new(group_name1)
 
@@ -255,6 +278,10 @@ class TestTacacsServerGroup < CiscoTestCase
   end
 
   def test_get_default_vrf_tacacs
+    # TODO
+    return if validate_property_excluded?('tacacs_server_group',
+                                          'vrf')
+
     aaa_group = TacacsServerGroup.new('Group1')
     assert_equal(cmd_ref.lookup('tacacs_server_group', 'vrf').default_value,
                  aaa_group.default_vrf,
@@ -263,10 +290,14 @@ class TestTacacsServerGroup < CiscoTestCase
   end
 
   def test_set_vrf_tacacs
+    # TODO
+    return if validate_property_excluded?('tacacs_server_group',
+                                          'vrf')
+
     vrf = 'management-123'
     aaa_group = TacacsServerGroup.new('Group1')
     aaa_group.vrf = vrf
-    assert_show_match(command: 'show run tacacs+ all | no-more',
+    assert_show_match(command: config_command,
                       pattern: /use-vrf #{vrf}/)
 
     # Invalid case
@@ -277,6 +308,10 @@ class TestTacacsServerGroup < CiscoTestCase
   end
 
   def test_get_deadtime_tacacs
+    # TODO
+    return if validate_property_excluded?('tacacs_server_group',
+                                          'deadtime')
+
     group_name = 'Group1'
     aaa_group = TacacsServerGroup.new(group_name)
 
@@ -298,6 +333,10 @@ class TestTacacsServerGroup < CiscoTestCase
   end
 
   def test_get_default_deadtime_tacacs
+    # TODO
+    return if validate_property_excluded?('tacacs_server_group',
+                                          'deadtime')
+
     aaa_group = TacacsServerGroup.new('Group1')
     assert_equal(
       cmd_ref.lookup('tacacs_server_group', 'deadtime').default_value,
@@ -307,10 +346,14 @@ class TestTacacsServerGroup < CiscoTestCase
   end
 
   def test_set_deadtime_tacacs
+    # TODO
+    return if validate_property_excluded?('tacacs_server_group',
+                                          'deadtime')
+
     deadtime = 1250
     aaa_group = TacacsServerGroup.new('Group1')
     aaa_group.deadtime = deadtime
-    assert_show_match(command: 'show run tacacs+ all | no-more',
+    assert_show_match(command: config_command,
                       pattern: /deadtime #{deadtime}/,
                       msg:     'Error: deadtime not configured')
     # Invalid case
@@ -322,6 +365,10 @@ class TestTacacsServerGroup < CiscoTestCase
   end
 
   def test_get_source_interface_tacacs
+    # TODO
+    return if validate_property_excluded?('tacacs_server_group',
+                                          'source_interface')
+
     group_name = 'Group1'
     aaa_group = TacacsServerGroup.new(group_name)
     intf =
@@ -329,12 +376,12 @@ class TestTacacsServerGroup < CiscoTestCase
     assert_equal(intf, aaa_group.source_interface,
                  'Error: TacacsServerGroup, source-interface set')
 
-    intf = 'Ethernet1/1'
+    intf = interfaces[0]
     create_source_interface(group_name, 'tacacs+', intf)
     assert_equal(intf, aaa_group.source_interface,
                  'Error: TacacsServerGroup, source-interface not correct')
 
-    intf = 'Ethernet1/32'
+    intf = interfaces[1]
     create_source_interface(group_name, 'tacacs+', intf)
     assert_equal(intf, aaa_group.source_interface,
                  'Error: TacacsServerGroup, source-interface not correct')
@@ -343,6 +390,10 @@ class TestTacacsServerGroup < CiscoTestCase
   end
 
   def test_get_default_source_interface_tacacs
+    # TODO
+    return if validate_property_excluded?('tacacs_server_group',
+                                          'source_interface')
+
     aaa_group = TacacsServerGroup.new('Group1')
     assert_equal(
       cmd_ref.lookup('tacacs_server_group', 'source_interface').default_value,
@@ -352,6 +403,10 @@ class TestTacacsServerGroup < CiscoTestCase
   end
 
   def test_set_source_interface_tacacs
+    # TODO
+    return if validate_property_excluded?('tacacs_server_group',
+                                          'source_interface')
+
     intf =
       cmd_ref.lookup('tacacs_server_group', 'source_interface').default_value
     aaa_group = TacacsServerGroup.new('Group1')
@@ -359,13 +414,13 @@ class TestTacacsServerGroup < CiscoTestCase
                  'Error: Aaa_Group Server, source-interface not default')
 
     aaa_group.source_interface = 'loopback1'
-    assert_show_match(command: 'show run tacacs+ all | no-more',
+    assert_show_match(command: config_command,
                       pattern: /source-interface loopback1/,
                       msg:     'Error: source-interface not correct')
 
     aaa_group.source_interface =
       cmd_ref.lookup('tacacs_server_group', 'source_interface').default_value
-    refute_show_match(command: 'show run tacacs+ all | no-more',
+    refute_show_match(command: config_command,
                       pattern: /source-interface loopback1/)
 
     # Invalid case
@@ -399,7 +454,7 @@ class TestTacacsServerGroup < CiscoTestCase
     aaa_group = TacacsServerGroup.new(group_name)
 
     detach_aaaservergroup(aaa_group)
-    refute_show_match(command: 'show run tacacs+ all | no-more',
+    refute_show_match(command: config_command,
                       pattern: /#{group_name}/)
   end
 end