bgp: T6106: fix test and verify()

(cherry picked from commit 2ba435f)

smoketest/scripts/cli/test_protocols_bgp.py

@@ -1236,13 +1236,11 @@ def test_bgp_26_ipv6_labeled_unicast_peer_group(self):
         self.assertIn(f'  neighbor {pg_ipv6} activate', afiv6_config)
         self.assertIn(f'  neighbor {pg_ipv6} maximum-prefix {ipv6_max_prefix}', afiv6_config)
 
-    def test_bgp_26_commit_error(self):
+    def test_bgp_27_route_reflector_client(self):
         self.cli_set(base_path + ['peer-group', 'peer1', 'address-family', 'l2vpn-evpn', 'route-reflector-client'])
         with self.assertRaises(ConfigSessionError) as e:
             self.cli_commit()
 
-        self.assertTrue("% Invalid command. Not an internal neighbor" in str(e.exception))
-
     def test_bgp_99_bmp(self):
         target_name = 'instance-bmp'
         target_address = '127.0.0.1'

src/conf_mode/protocols_bgp.py

@@ -457,6 +457,8 @@ def verify(bgp):
                             peer_group_as = dict_search(f'peer_group.{peer_group}.remote_as', bgp)
                             if peer_group_as != None and peer_group_as != 'internal' and peer_group_as != bgp['system_as']:
                                 raise ConfigError('route-reflector-client only supported for iBGP peers')
+                        else:
+                            raise ConfigError('route-reflector-client only supported for iBGP peers')
 
     # Throw an error if a peer group is not configured for allow range
     for prefix in dict_search('listen.range', bgp) or []: