diff options
| author | Renato Westphal <renato@opensourcerouting.org> | 2021-09-06 19:50:27 -0300 |
|---|---|---|
| committer | Renato Westphal <renato@opensourcerouting.org> | 2021-09-21 17:43:45 -0300 |
| commit | 242a9767f7bf44b1ec8944ca880962c62d5264b2 (patch) | |
| tree | a5ac6d74aad64962a33026f33cdbcf04aa7cf4b6 /tests/topotests/ospf6_topo2/test_ospf6_topo2.py | |
| parent | e81192ad7c08d85f008c8a400157b69e5a42c2e4 (diff) | |
ospf6d: fix metric type of NSSA Type-7 LSAs
Fix wrong comparison since route->path.metric_type is always set
to either 1 or 2. The OSPF6_PATH_TYPE_EXTERNAL2 constant, whose
value is 4, refers to a route type so its usage was incorrect here.
Signed-off-by: Renato Westphal <renato@opensourcerouting.org>
Diffstat (limited to 'tests/topotests/ospf6_topo2/test_ospf6_topo2.py')
| -rw-r--r-- | tests/topotests/ospf6_topo2/test_ospf6_topo2.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/tests/topotests/ospf6_topo2/test_ospf6_topo2.py b/tests/topotests/ospf6_topo2/test_ospf6_topo2.py index acaa3b8a9b..8be461aeac 100644 --- a/tests/topotests/ospf6_topo2/test_ospf6_topo2.py +++ b/tests/topotests/ospf6_topo2/test_ospf6_topo2.py @@ -217,8 +217,8 @@ def test_ospfv3_expected_route_types(): { "numberOfIntraAreaRoutes": 1, "numberOfInterAreaRoutes": 2, - "numberOfExternal1Routes": 4, - "numberOfExternal2Routes": 0, + "numberOfExternal1Routes": 0, + "numberOfExternal2Routes": 4, }, ) @@ -330,7 +330,7 @@ def test_nssa_lsa_type7(): ] route = { "2001:db8:100::/64": { - "pathType": "E1", + "pathType": "E2", "nextHops": [{"nextHop": "::", "interfaceName": "r4-eth0"}], } } |