From 616249a7e26b8bf7fba1f875fdee347ab2b391df Mon Sep 17 00:00:00 2001
From: Richard Warburton <richard.warburton@gmail.com>
Date: Thu, 4 Feb 2021 18:36:14 +0000
Subject: [PATCH] Bring in custom acceptance tests for the resend request chunk
 size, in order to avoid copying a quickfix bug where the newseqno field of a
 gapfill message was being treated incorrectly.

---
 ...2ResendRequestChunkSizeAcceptanceTest.java |  4 +-
 .../SequenceGapFollowedByMessageResent.def    | 37 ++++++++++++++++
 ...eGapFollowedBySequenceResetWithGapFill.def | 43 +++++++++++++++++++
 ...4ResendRequestChunkSizeAcceptanceTest.java |  4 +-
 .../SequenceGapFollowedByMessageResent.def    | 37 ++++++++++++++++
 ...eGapFollowedBySequenceResetWithGapFill.def | 43 +++++++++++++++++++
 ...0ResendRequestChunkSizeAcceptanceTest.java |  5 +--
 .../SequenceGapFollowedByMessageResent.def    | 37 ++++++++++++++++
 ...eGapFollowedBySequenceResetWithGapFill.def | 43 +++++++++++++++++++
 .../AbstractFixSpecAcceptanceTest.java        |  9 +++-
 10 files changed, 251 insertions(+), 11 deletions(-)
 create mode 100644 test-fix-42/src/test/resources/custom_definitions/fix42/SequenceGapFollowedByMessageResent.def
 create mode 100644 test-fix-42/src/test/resources/custom_definitions/fix42/SequenceGapFollowedBySequenceResetWithGapFill.def
 create mode 100644 test-fix-44/src/test/resources/custom_definitions/fix44/SequenceGapFollowedByMessageResent.def
 create mode 100644 test-fix-44/src/test/resources/custom_definitions/fix44/SequenceGapFollowedBySequenceResetWithGapFill.def
 create mode 100644 test-fix-50/src/test/resources/custom_definitions/fix50/SequenceGapFollowedByMessageResent.def
 create mode 100644 test-fix-50/src/test/resources/custom_definitions/fix50/SequenceGapFollowedBySequenceResetWithGapFill.def

diff --git a/test-fix-42/src/test/java/uk/co/real_logic/artio/acceptance_tests/Fix42ResendRequestChunkSizeAcceptanceTest.java b/test-fix-42/src/test/java/uk/co/real_logic/artio/acceptance_tests/Fix42ResendRequestChunkSizeAcceptanceTest.java
index 8f3f89d..5f18949 100644
--- a/test-fix-42/src/test/java/uk/co/real_logic/artio/acceptance_tests/Fix42ResendRequestChunkSizeAcceptanceTest.java
+++ b/test-fix-42/src/test/java/uk/co/real_logic/artio/acceptance_tests/Fix42ResendRequestChunkSizeAcceptanceTest.java
@@ -10,13 +10,11 @@
 @RunWith(Parameterized.class)
 public class Fix42ResendRequestChunkSizeAcceptanceTest extends AbstractFixSpecAcceptanceTest
 {
-    private static final String QUICKFIX_4_2_ROOT_PATH = QUICKFIX_RESEND_REQUEST_CHUNK_SIZE_DEFINITIONS + "/fix42";
-
     @Parameterized.Parameters(name = "Acceptance: {1}")
     public static Collection<Object[]> data()
     {
         return testsFor(
-            QUICKFIX_4_2_ROOT_PATH,
+            CUSTOM_ROOT_PATH + "/fix42",
             QUICKFIX_RESEND_CHUNK_INCLUDE_LIST,
             () -> Environment.fix42(new NewOrderSingleClonerImpl(), RESEND_REQUEST_CHUNK_SIZE));
     }
diff --git a/test-fix-42/src/test/resources/custom_definitions/fix42/SequenceGapFollowedByMessageResent.def b/test-fix-42/src/test/resources/custom_definitions/fix42/SequenceGapFollowedByMessageResent.def
new file mode 100644
index 0000000..fe75df2
--- /dev/null
+++ b/test-fix-42/src/test/resources/custom_definitions/fix42/SequenceGapFollowedByMessageResent.def
@@ -0,0 +1,37 @@
+# If message sequence number is too high, send a resend request for
+# missing messages
+
+iCONNECT
+I8=FIX.4.2
35=A
34=1
49=TW
52=<TIME>
56=ISLD
98=0
108=30

+E8=FIX.4.2
9=57
35=A
34=1
49=ISLD
52=00000000-00:00:00
56=TW
98=0
108=30
10=0

+
+I8=FIX.4.2
35=D
34=2
49=TW
52=<TIME>
56=ISLD
11=ID
21=3
38=100
40=1
54=1
55=INTC
60=<TIME>

+E8=FIX.4.2
9=81
35=D
34=2
49=ISLD
52=00000000-00:00:00
56=TW
11=ID
21=3
38=100
40=1
54=1
55=INTC
10=0

+
+I8=FIX.4.2
35=D
34=3
49=TW
52=<TIME>
56=ISLD
11=ID
21=3
38=100
40=1
54=1
55=INTC
60=<TIME>

+E8=FIX.4.2
9=81
35=D
34=3
49=ISLD
52=00000000-00:00:00
56=TW
11=ID
21=3
38=100
40=1
54=1
55=INTC
10=0

+
+# Sending message 10 when last message sent was 3
+I8=FIX.4.2
35=0
34=10
49=TW
52=<TIME>
56=ISLD

+
+# Receive Resend request for messages 4 to (4+5-1 = 8)
+E8=FIX.4.2
9=54
35=2
34=4
49=ISLD
52=00000000-00:00:00
56=TW
7=4
16=8
10=0

+I8=FIX.4.2
35=D
34=4
49=TW
52=<TIME>
56=ISLD
97=Y
11=ID
21=3
38=100
40=1
54=1
55=INTC
60=<TIME>

+I8=FIX.4.2
35=D
34=5
49=TW
52=<TIME>
56=ISLD
97=Y
11=ID
21=3
38=100
40=1
54=1
55=INTC
60=<TIME>

+I8=FIX.4.2
35=D
34=6
49=TW
52=<TIME>
56=ISLD
97=Y
11=ID
21=3
38=100
40=1
54=1
55=INTC
60=<TIME>

+I8=FIX.4.2
35=D
34=7
49=TW
52=<TIME>
56=ISLD
97=Y
11=ID
21=3
38=100
40=1
54=1
55=INTC
60=<TIME>

+I8=FIX.4.2
35=D
34=8
49=TW
52=<TIME>
56=ISLD
97=Y
11=ID
21=3
38=100
40=1
54=1
55=INTC
60=<TIME>

+
+# Receive Resend request for messages 9 to infinity
+E8=FIX.4.2
9=55
35=2
34=5
49=ISLD
52=00000000-00:00:00
56=TW
7=9
16=0
10=0

+I8=FIX.4.2
35=D
34=9
49=TW
52=<TIME>
56=ISLD
97=Y
11=ID
21=3
38=100
40=1
54=1
55=INTC
60=<TIME>

+
+#Sent test request to validate no seq num issue
+I8=FIX.4.2
35=1
34=11
49=TW
52=<TIME>
56=ISLD
112=TEST

+E8=FIX.4.2
9=54
35=0
34=6
49=ISLD
52=00000000-00:00:00
56=TW
112=TEST
10=0

+
+# logout message and response
+I8=FIX.4.2
35=5
34=12
49=TW
52=<TIME>
56=ISLD

+E8=FIX.4.2
9=45
35=5
34=7
49=ISLD
52=00000000-00:00:00
56=TW
10=0

+
+eDISCONNECT
\ No newline at end of file
diff --git a/test-fix-42/src/test/resources/custom_definitions/fix42/SequenceGapFollowedBySequenceResetWithGapFill.def b/test-fix-42/src/test/resources/custom_definitions/fix42/SequenceGapFollowedBySequenceResetWithGapFill.def
new file mode 100644
index 0000000..378712e
--- /dev/null
+++ b/test-fix-42/src/test/resources/custom_definitions/fix42/SequenceGapFollowedBySequenceResetWithGapFill.def
@@ -0,0 +1,43 @@
+# If message sequence number is too high, send a resend request for
+# missing messages
+
+# Modified from quickfix because the newSeqNo field on a gap fill should refer to the sequence number of the next message
+
+iCONNECT
+I8=FIX.4.2
35=A
34=1
49=TW
52=<TIME>
56=ISLD
98=0
108=30

+E8=FIX.4.2
9=57
35=A
34=1
49=ISLD
52=00000000-00:00:00
56=TW
98=0
108=30
10=0

+
+I8=FIX.4.2
35=D
34=2
49=TW
52=<TIME>
56=ISLD
11=ID
21=3
38=100
40=1
54=1
55=INTC
60=<TIME>

+E8=FIX.4.2
9=81
35=D
34=2
49=ISLD
52=00000000-00:00:00
56=TW
11=ID
21=3
38=100
40=1
54=1
55=INTC
10=0

+
+I8=FIX.4.2
35=D
34=3
49=TW
52=<TIME>
56=ISLD
11=ID
21=3
38=100
40=1
54=1
55=INTC
60=<TIME>

+E8=FIX.4.2
9=81
35=D
34=3
49=ISLD
52=00000000-00:00:00
56=TW
11=ID
21=3
38=100
40=1
54=1
55=INTC
10=0

+
+# Sending message 15 when last message sent was 3
+I8=FIX.4.2
35=0
34=15
49=TW
52=<TIME>
56=ISLD

+
+# Receive Resend request for messages 4 to (4+5-1 = 8)
+E8=FIX.4.2
9=54
35=2
34=4
49=ISLD
52=00000000-00:00:00
56=TW
7=4
16=8
10=0

+# Send a GapFill up to the last message 8
+I8=FIX.4.2
35=4
34=4
49=TW
52=<TIME>
56=ISLD
123=Y
36=9

+
+# Receive Resend request for messages 8 to (8+5-1 = 12)
+E8=FIX.4.2
9=55
35=2
34=5
49=ISLD
52=00000000-00:00:00
56=TW
7=9
16=13
10=0

+# Send a GapFill up to the last message 12
+I8=FIX.4.2
35=4
34=9
49=TW
52=<TIME>
56=ISLD
123=Y
36=14

+
+# Receive Resend request for messages 12 to 15)
+E8=FIX.4.2
9=60
35=2
34=6
49=ISLD
52=00000000-00:00:00
56=TW
7=14
16=0
10=0

+# Send a GapFill up to the last message 15
+I8=FIX.4.2
35=4
34=14
49=TW
52=<TIME>
56=ISLD
123=Y
36=16

+
+#Sent test request to validate no seq num issue
+I8=FIX.4.2
35=1
34=16
49=TW
52=<TIME>
56=ISLD
112=TEST

+E8=FIX.4.2
9=54
35=0
34=7
49=ISLD
52=00000000-00:00:00
56=TW
112=TEST
10=0

+
+
+# logout message and response
+I8=FIX.4.2
35=5
34=17
49=TW
52=<TIME>
56=ISLD

+E8=FIX.4.2
9=45
35=5
34=8
49=ISLD
52=00000000-00:00:00
56=TW
10=0

+
+eDISCONNECT
diff --git a/test-fix-44/src/test/java/uk/co/real_logic/artio/acceptance_tests/Fix44ResendRequestChunkSizeAcceptanceTest.java b/test-fix-44/src/test/java/uk/co/real_logic/artio/acceptance_tests/Fix44ResendRequestChunkSizeAcceptanceTest.java
index cd3bf4d..ac4c0f8 100644
--- a/test-fix-44/src/test/java/uk/co/real_logic/artio/acceptance_tests/Fix44ResendRequestChunkSizeAcceptanceTest.java
+++ b/test-fix-44/src/test/java/uk/co/real_logic/artio/acceptance_tests/Fix44ResendRequestChunkSizeAcceptanceTest.java
@@ -10,13 +10,11 @@
 @RunWith(Parameterized.class)
 public class Fix44ResendRequestChunkSizeAcceptanceTest extends AbstractFixSpecAcceptanceTest
 {
-    private static final String QUICKFIX_4_4_ROOT_PATH = QUICKFIX_RESEND_REQUEST_CHUNK_SIZE_DEFINITIONS + "/fix44";
-
     @Parameterized.Parameters(name = "Acceptance: {1}")
     public static Collection<Object[]> data()
     {
         return testsFor(
-            QUICKFIX_4_4_ROOT_PATH,
+            CUSTOM_ROOT_PATH + "/fix44",
             QUICKFIX_RESEND_CHUNK_INCLUDE_LIST,
             () -> Environment.fix44(new NewOrderSingleClonerImpl(), RESEND_REQUEST_CHUNK_SIZE));
     }
diff --git a/test-fix-44/src/test/resources/custom_definitions/fix44/SequenceGapFollowedByMessageResent.def b/test-fix-44/src/test/resources/custom_definitions/fix44/SequenceGapFollowedByMessageResent.def
new file mode 100644
index 0000000..2cdf7f1
--- /dev/null
+++ b/test-fix-44/src/test/resources/custom_definitions/fix44/SequenceGapFollowedByMessageResent.def
@@ -0,0 +1,37 @@
+# If message sequence number is too high, send a resend request for
+# missing messages
+
+iCONNECT
+I8=FIX.4.4
35=A
34=1
49=TW
52=<TIME>
56=ISLD
98=0
108=30

+E8=FIX.4.4
9=57
35=A
34=1
49=ISLD
52=00000000-00:00:00
56=TW
98=0
108=30
10=0

+
+I8=FIX.4.4
35=D
34=2
49=TW
52=<TIME>
56=ISLD
11=ID
21=3
38=100
40=1
54=1
55=INTC
60=<TIME>

+E8=FIX.4.4
9=81
35=D
34=2
49=ISLD
52=00000000-00:00:00
56=TW
11=ID
21=3
38=100
40=1
54=1
55=INTC
10=0

+
+I8=FIX.4.4
35=D
34=3
49=TW
52=<TIME>
56=ISLD
11=ID
21=3
38=100
40=1
54=1
55=INTC
60=<TIME>

+E8=FIX.4.4
9=81
35=D
34=3
49=ISLD
52=00000000-00:00:00
56=TW
11=ID
21=3
38=100
40=1
54=1
55=INTC
10=0

+
+# Sending message 10 when last message sent was 3
+I8=FIX.4.4
35=0
34=10
49=TW
52=<TIME>
56=ISLD

+
+# Receive Resend request for messages 4 to (4+5-1 = 8)
+E8=FIX.4.4
9=54
35=2
34=4
49=ISLD
52=00000000-00:00:00
56=TW
7=4
16=8
10=0

+I8=FIX.4.4
35=D
34=4
49=TW
52=<TIME>
56=ISLD
97=Y
11=ID
21=3
38=100
40=1
54=1
55=INTC
60=<TIME>

+I8=FIX.4.4
35=D
34=5
49=TW
52=<TIME>
56=ISLD
97=Y
11=ID
21=3
38=100
40=1
54=1
55=INTC
60=<TIME>

+I8=FIX.4.4
35=D
34=6
49=TW
52=<TIME>
56=ISLD
97=Y
11=ID
21=3
38=100
40=1
54=1
55=INTC
60=<TIME>

+I8=FIX.4.4
35=D
34=7
49=TW
52=<TIME>
56=ISLD
97=Y
11=ID
21=3
38=100
40=1
54=1
55=INTC
60=<TIME>

+I8=FIX.4.4
35=D
34=8
49=TW
52=<TIME>
56=ISLD
97=Y
11=ID
21=3
38=100
40=1
54=1
55=INTC
60=<TIME>

+
+# Receive Resend request for messages 9 to infinity
+E8=FIX.4.4
9=55
35=2
34=5
49=ISLD
52=00000000-00:00:00
56=TW
7=9
16=0
10=0

+I8=FIX.4.4
35=D
34=9
49=TW
52=<TIME>
56=ISLD
97=Y
11=ID
21=3
38=100
40=1
54=1
55=INTC
60=<TIME>

+
+#Sent test request to validate no seq num issue
+I8=FIX.4.4
35=1
34=11
49=TW
52=<TIME>
56=ISLD
112=TEST

+E8=FIX.4.4
9=54
35=0
34=6
49=ISLD
52=00000000-00:00:00
56=TW
112=TEST
10=0

+
+# logout message and response
+I8=FIX.4.4
35=5
34=12
49=TW
52=<TIME>
56=ISLD

+E8=FIX.4.4
9=45
35=5
34=7
49=ISLD
52=00000000-00:00:00
56=TW
10=0

+
+eDISCONNECT
\ No newline at end of file
diff --git a/test-fix-44/src/test/resources/custom_definitions/fix44/SequenceGapFollowedBySequenceResetWithGapFill.def b/test-fix-44/src/test/resources/custom_definitions/fix44/SequenceGapFollowedBySequenceResetWithGapFill.def
new file mode 100644
index 0000000..081f0b7
--- /dev/null
+++ b/test-fix-44/src/test/resources/custom_definitions/fix44/SequenceGapFollowedBySequenceResetWithGapFill.def
@@ -0,0 +1,43 @@
+# If message sequence number is too high, send a resend request for
+# missing messages
+
+# Modified from quickfix because the newSeqNo field on a gap fill should refer to the sequence number of the next message
+
+iCONNECT
+I8=FIX.4.4
35=A
34=1
49=TW
52=<TIME>
56=ISLD
98=0
108=30

+E8=FIX.4.4
9=57
35=A
34=1
49=ISLD
52=00000000-00:00:00
56=TW
98=0
108=30
10=0

+
+I8=FIX.4.4
35=D
34=2
49=TW
52=<TIME>
56=ISLD
11=ID
21=3
38=100
40=1
54=1
55=INTC
60=<TIME>

+E8=FIX.4.4
9=81
35=D
34=2
49=ISLD
52=00000000-00:00:00
56=TW
11=ID
21=3
38=100
40=1
54=1
55=INTC
10=0

+
+I8=FIX.4.4
35=D
34=3
49=TW
52=<TIME>
56=ISLD
11=ID
21=3
38=100
40=1
54=1
55=INTC
60=<TIME>

+E8=FIX.4.4
9=81
35=D
34=3
49=ISLD
52=00000000-00:00:00
56=TW
11=ID
21=3
38=100
40=1
54=1
55=INTC
10=0

+
+# Sending message 15 when last message sent was 3
+I8=FIX.4.4
35=0
34=15
49=TW
52=<TIME>
56=ISLD

+
+# Receive Resend request for messages 4 to (4+5-1 = 8)
+E8=FIX.4.4
9=54
35=2
34=4
49=ISLD
52=00000000-00:00:00
56=TW
7=4
16=8
10=0

+# Send a GapFill up to the last message 8
+I8=FIX.4.4
35=4
34=4
49=TW
52=<TIME>
56=ISLD
123=Y
36=9

+
+# Receive Resend request for messages 8 to (8+5-1 = 12)
+E8=FIX.4.4
9=55
35=2
34=5
49=ISLD
52=00000000-00:00:00
56=TW
7=9
16=13
10=0

+# Send a GapFill up to the last message 12
+I8=FIX.4.4
35=4
34=9
49=TW
52=<TIME>
56=ISLD
123=Y
36=14

+
+# Receive Resend request for messages 12 to 15)
+E8=FIX.4.4
9=60
35=2
34=6
49=ISLD
52=00000000-00:00:00
56=TW
7=14
16=0
10=0

+# Send a GapFill up to the last message 15
+I8=FIX.4.4
35=4
34=14
49=TW
52=<TIME>
56=ISLD
123=Y
36=16

+
+#Sent test request to validate no seq num issue
+I8=FIX.4.4
35=1
34=16
49=TW
52=<TIME>
56=ISLD
112=TEST

+E8=FIX.4.4
9=54
35=0
34=7
49=ISLD
52=00000000-00:00:00
56=TW
112=TEST
10=0

+
+
+# logout message and response
+I8=FIX.4.4
35=5
34=17
49=TW
52=<TIME>
56=ISLD

+E8=FIX.4.4
9=45
35=5
34=8
49=ISLD
52=00000000-00:00:00
56=TW
10=0

+
+eDISCONNECT
diff --git a/test-fix-50/src/test/java/uk/co/real_logic/artio/acceptance_tests/Fix50ResendRequestChunkSizeAcceptanceTest.java b/test-fix-50/src/test/java/uk/co/real_logic/artio/acceptance_tests/Fix50ResendRequestChunkSizeAcceptanceTest.java
index 839fbf5..e9a4d13 100644
--- a/test-fix-50/src/test/java/uk/co/real_logic/artio/acceptance_tests/Fix50ResendRequestChunkSizeAcceptanceTest.java
+++ b/test-fix-50/src/test/java/uk/co/real_logic/artio/acceptance_tests/Fix50ResendRequestChunkSizeAcceptanceTest.java
@@ -10,13 +10,12 @@
 @RunWith(Parameterized.class)
 public class Fix50ResendRequestChunkSizeAcceptanceTest extends AbstractFixSpecAcceptanceTest
 {
-    private static final String QUICKFIX_5_0_ROOT_PATH = QUICKFIX_RESEND_REQUEST_CHUNK_SIZE_DEFINITIONS + "/fix50";
-
     @Parameterized.Parameters(name = "Acceptance: {1}")
     public static Collection<Object[]> data()
     {
         return testsFor(
-            QUICKFIX_5_0_ROOT_PATH, QUICKFIX_RESEND_CHUNK_INCLUDE_LIST,
+            CUSTOM_ROOT_PATH + "/fix50",
+            QUICKFIX_RESEND_CHUNK_INCLUDE_LIST,
             () -> Environment.fix50(
                 new Fix50SessionCustomizationStrategy(),
                 new NewOrderSingleClonerImpl(),
diff --git a/test-fix-50/src/test/resources/custom_definitions/fix50/SequenceGapFollowedByMessageResent.def b/test-fix-50/src/test/resources/custom_definitions/fix50/SequenceGapFollowedByMessageResent.def
new file mode 100644
index 0000000..9bafd63
--- /dev/null
+++ b/test-fix-50/src/test/resources/custom_definitions/fix50/SequenceGapFollowedByMessageResent.def
@@ -0,0 +1,37 @@
+# If message sequence number is too high, send a resend request for
+# missing messages
+
+iCONNECT
+I8=FIXT.1.1
35=A
34=1
49=TW
52=<TIME>
56=ISLD
98=0
108=30
1137=7

+E8=FIXT.1.1
9=64
35=A
34=1
49=ISLD
52=00000000-00:00:00
56=TW
98=0
108=30
1137=7
10=0

+
+I8=FIXT.1.1
35=D
34=2
49=TW
52=<TIME>
56=ISLD
11=ID
21=3
38=100
40=1
54=1
55=INTC
60=<TIME>

+E8=FIXT.1.1
9=81
35=D
34=2
49=ISLD
52=00000000-00:00:00
56=TW
11=ID
21=3
38=100
40=1
54=1
55=INTC
10=0

+
+I8=FIXT.1.1
35=D
34=3
49=TW
52=<TIME>
56=ISLD
11=ID
21=3
38=100
40=1
54=1
55=INTC
60=<TIME>

+E8=FIXT.1.1
9=81
35=D
34=3
49=ISLD
52=00000000-00:00:00
56=TW
11=ID
21=3
38=100
40=1
54=1
55=INTC
10=0

+
+# Sending message 10 when last message sent was 3
+I8=FIXT.1.1
35=0
34=10
49=TW
52=<TIME>
56=ISLD

+
+# Receive Resend request for messages 4 to (4+5-1 = 8)
+E8=FIXT.1.1
9=54
35=2
34=4
49=ISLD
52=00000000-00:00:00
56=TW
7=4
16=8
10=0

+I8=FIXT.1.1
35=D
34=4
49=TW
52=<TIME>
56=ISLD
97=Y
11=ID
21=3
38=100
40=1
54=1
55=INTC
60=<TIME>

+I8=FIXT.1.1
35=D
34=5
49=TW
52=<TIME>
56=ISLD
97=Y
11=ID
21=3
38=100
40=1
54=1
55=INTC
60=<TIME>

+I8=FIXT.1.1
35=D
34=6
49=TW
52=<TIME>
56=ISLD
97=Y
11=ID
21=3
38=100
40=1
54=1
55=INTC
60=<TIME>

+I8=FIXT.1.1
35=D
34=7
49=TW
52=<TIME>
56=ISLD
97=Y
11=ID
21=3
38=100
40=1
54=1
55=INTC
60=<TIME>

+I8=FIXT.1.1
35=D
34=8
49=TW
52=<TIME>
56=ISLD
97=Y
11=ID
21=3
38=100
40=1
54=1
55=INTC
60=<TIME>

+
+# Receive Resend request for messages 9 to infinity
+E8=FIXT.1.1
9=55
35=2
34=5
49=ISLD
52=00000000-00:00:00
56=TW
7=9
16=0
10=0

+I8=FIXT.1.1
35=D
34=9
49=TW
52=<TIME>
56=ISLD
97=Y
11=ID
21=3
38=100
40=1
54=1
55=INTC
60=<TIME>

+
+#Sent test request to validate no seq num issue
+I8=FIXT.1.1
35=1
34=11
49=TW
52=<TIME>
56=ISLD
112=TEST

+E8=FIXT.1.1
9=54
35=0
34=6
49=ISLD
52=00000000-00:00:00
56=TW
112=TEST
10=0

+
+# logout message and response
+I8=FIXT.1.1
35=5
34=12
49=TW
52=<TIME>
56=ISLD

+E8=FIXT.1.1
9=45
35=5
34=7
49=ISLD
52=00000000-00:00:00
56=TW
10=0

+
+eDISCONNECT
\ No newline at end of file
diff --git a/test-fix-50/src/test/resources/custom_definitions/fix50/SequenceGapFollowedBySequenceResetWithGapFill.def b/test-fix-50/src/test/resources/custom_definitions/fix50/SequenceGapFollowedBySequenceResetWithGapFill.def
new file mode 100644
index 0000000..50c39fc
--- /dev/null
+++ b/test-fix-50/src/test/resources/custom_definitions/fix50/SequenceGapFollowedBySequenceResetWithGapFill.def
@@ -0,0 +1,43 @@
+# If message sequence number is too high, send a resend request for
+# missing messages
+
+# Modified from quickfix because the newSeqNo field on a gap fill should refer to the sequence number of the next message
+
+iCONNECT
+I8=FIXT.1.1
35=A
34=1
49=TW
52=<TIME>
56=ISLD
98=0
108=2
1137=7

+E8=FIXT.1.1
9=57
35=A
34=1
49=ISLD
52=00000000-00:00:00
56=TW
98=0
108=2
1137=7
10=0

+
+I8=FIXT.1.1
35=D
34=2
49=TW
52=<TIME>
56=ISLD
11=ID
21=3
38=100
40=1
54=1
55=INTC
60=<TIME>

+E8=FIXT.1.1
9=81
35=D
34=2
49=ISLD
52=00000000-00:00:00
56=TW
11=ID
21=3
38=100
40=1
54=1
55=INTC
10=0

+
+I8=FIXT.1.1
35=D
34=3
49=TW
52=<TIME>
56=ISLD
11=ID
21=3
38=100
40=1
54=1
55=INTC
60=<TIME>

+E8=FIXT.1.1
9=81
35=D
34=3
49=ISLD
52=00000000-00:00:00
56=TW
11=ID
21=3
38=100
40=1
54=1
55=INTC
10=0

+
+# Sending message 15 when last message sent was 3
+I8=FIXT.1.1
35=0
34=15
49=TW
52=<TIME>
56=ISLD

+
+# Receive Resend request for messages 4 to (4+5-1 = 8)
+E8=FIXT.1.1
9=54
35=2
34=4
49=ISLD
52=00000000-00:00:00
56=TW
7=4
16=8
10=0

+# Send a GapFill up to the last message 8
+I8=FIXT.1.1
35=4
34=4
49=TW
52=<TIME>
56=ISLD
123=Y
36=9

+
+# Receive Resend request for messages 8 to (8+5-1 = 12)
+E8=FIXT.1.1
9=55
35=2
34=5
49=ISLD
52=00000000-00:00:00
56=TW
7=9
16=13
10=0

+# Send a GapFill up to the last message 12
+I8=FIXT.1.1
35=4
34=9
49=TW
52=<TIME>
56=ISLD
123=Y
36=14

+
+# Receive Resend request for messages 12 to 15)
+E8=FIXT.1.1
9=60
35=2
34=6
49=ISLD
52=00000000-00:00:00
56=TW
7=14
16=0
10=0

+# Send a GapFill up to the last message 15
+I8=FIXT.1.1
35=4
34=14
49=TW
52=<TIME>
56=ISLD
123=Y
36=16

+
+#Sent test request to validate no seq num issue
+I8=FIXT.1.1
35=1
34=16
49=TW
52=<TIME>
56=ISLD
112=TEST

+E8=FIXT.1.1
9=54
35=0
34=7
49=ISLD
52=00000000-00:00:00
56=TW
112=TEST
10=0

+
+
+# logout message and response
+I8=FIXT.1.1
35=5
34=17
49=TW
52=<TIME>
56=ISLD

+E8=FIXT.1.1
9=45
35=5
34=8
49=ISLD
52=00000000-00:00:00
56=TW
10=0

+
+eDISCONNECT
diff --git a/test-framework/src/test/java/uk/co/real_logic/artio/acceptance_tests/AbstractFixSpecAcceptanceTest.java b/test-framework/src/test/java/uk/co/real_logic/artio/acceptance_tests/AbstractFixSpecAcceptanceTest.java
index bf010c6..488ccd0 100644
--- a/test-framework/src/test/java/uk/co/real_logic/artio/acceptance_tests/AbstractFixSpecAcceptanceTest.java
+++ b/test-framework/src/test/java/uk/co/real_logic/artio/acceptance_tests/AbstractFixSpecAcceptanceTest.java
@@ -49,7 +49,7 @@ public abstract class AbstractFixSpecAcceptanceTest
     @Rule
     public Timeout timeout = Timeout.millis(Long.getLong(FIX_TEST_TIMEOUT_PROP, FIX_TEST_TIMEOUT_DEFAULT));
 
-    protected static List<Object[]> testsFor(
+    protected static List<Object[]>  testsFor(
         final String rootDirPath, final List<String> files, final Supplier<Environment> environment)
     {
         final String rootDir = getCorrectDirectory(rootDirPath);
@@ -57,10 +57,15 @@ protected static List<Object[]> testsFor(
         return files
             .stream()
             .map((file) -> Paths.get(rootDir, file))
-            .map((path) -> new Object[]{ path, path.getFileName(), environment })
+            .map((path) -> testFor(path, environment))
             .collect(toList());
     }
 
+    protected static Object[] testFor(final Path path, final Supplier<Environment> environment)
+    {
+        return new Object[]{path, path.getFileName(), environment};
+    }
+
     private static String getCorrectDirectory(final String rootDirPath)
     {
         final File rootDirFile = new File(rootDirPath);