From: Paul Jakma <paul.jakma@hpe.com>
Date: Wed, 25 Nov 2015 17:14:36 +0000 (+0000)
Subject: tests: add more AS4 capability tests + little fixes for couple of GR test cases.
X-Git-Tag: frr-2.0-rc1~697
X-Git-Url: https://git.puffer.fish/?a=commitdiff_plain;h=3c88dab5b8c47f9b4dc155f43a2ee3282cbaed78;p=matthieu%2Ffrr.git

tests: add more AS4 capability tests + little fixes for couple of GR test cases.

Reviewed-by: Donald Sharp <sharpd@cumulusnetworks.com>
(cherry picked from commit 321d4130a615445d0f49f41c909c92d5401fd5ff)
---

diff --git a/tests/bgp_capability_test.c b/tests/bgp_capability_test.c
index 12221d3b55..ecabc2f19f 100644
--- a/tests/bgp_capability_test.c
+++ b/tests/bgp_capability_test.c
@@ -312,7 +312,16 @@ static struct test_segment misc_segments[] =
     { 0x41, 0x4, 0xab, 0xcd, 0xef, 0x12 }, /* AS: 2882400018 */
     6, SHOULD_PARSE, 2882400018,
   },
-  /* 20 */
+  { "AS4",
+    "AS4 capability: short",
+    { 0x41, 0x4, 0xab, 0xcd, 0xef }, /* AS: 2882400018 */
+    5, SHOULD_ERR,
+  },
+  { "AS4",
+    "AS4 capability: long",
+    { 0x41, 0x4, 0xab, 0xcd, 0xef, 0x12, 0x12 },
+    7, SHOULD_ERR, 2882400018,
+  },
   { "GR",
     "GR capability",
     { /* hdr */		CAPABILITY_CODE_RESTART, 0xe,
@@ -329,7 +338,6 @@ static struct test_segment misc_segments[] =
     },
     16, SHOULD_PARSE,
   },
-  /* 21 */
   { "GR-short",
     "GR capability, but header length too short",
     { /* hdr */		0x40, 0xa,
@@ -344,9 +352,8 @@ static struct test_segment misc_segments[] =
       /* safi */	0x2,
       /* flags */	0x1,
     },
-    16, SHOULD_PARSE,
+    15 /* array is 16 though */, SHOULD_ERR,
   },
-  /* 22 */
   { "GR-long",
     "GR capability, but header length too long",
     { /* hdr */		0x40, 0xf,
@@ -359,6 +366,7 @@ static struct test_segment misc_segments[] =
       /* flags */	0x0,
       /* afi */		0x0, 0x2,
       /* safi */	0x2,
+      /* flags */	0x01,
     },
     16, SHOULD_ERR,
   },