Add unit test for the case of the group presence

Author: martinfantini

tests/fast_test_coding_case_v2.hpp

@@ -18,8 +18,7 @@ class fast_test_coding_case_v2
             decoder_v2_(DESC::instance())
         {}
 
-        bool
-        encoding(const mfast::message_cref& msg_ref, const byte_stream& result, bool reset=false)
+        bool encoding(const mfast::message_cref& msg_ref, const byte_stream& result, bool reset=false)
         {
             const int buffer_size = 128;
             char buffer[buffer_size];
@@ -32,8 +31,7 @@ class fast_test_coding_case_v2
             return false;
         }
 
-        bool
-        decoding(const byte_stream& bytes, const mfast::message_cref& result, bool reset=false)
+        bool decoding(const byte_stream& bytes, const mfast::message_cref& result, bool reset=false)
         {
           const char* first = bytes.data();
           mfast::message_cref msg = decoder_v2_.decode(first, first+bytes.size(), reset);

tests/sequence_encoder_decoder.cpp

@@ -237,3 +237,19 @@ TEST_CASE("group sequence inside sequence encoder/decoder","[group_sequence_insi
         REQUIRE(test_case.decoding("\xC0\x86\x81\xD0\xB2\x82\xB2",test_6.cref(),true));
     }
 }
+
+TEST_CASE("sequence with optional group encoder/decoder","[sequence_optional_group_encoder_decoder]")
+{
+    fast_test_coding_case<simple14::templates_description> test_case;
+    simple14::Test_7 test_7;
+    simple14::Test_7_mref test_7_mref = test_7.mref();
+
+    auto sequence_7_mref = test_7_mref.set_sequence_7();
+    sequence_7_mref.resize(1);
+
+    auto element_sequence = sequence_7_mref.front();
+    element_sequence.set_field_7_3().as(50);
+
+    REQUIRE(test_case.encoding(test_7.cref(),"\xC0\x87\x81\xC0\xB2",true));
+    REQUIRE(test_case.decoding("\xC0\x87\x81\xC0\xB2",test_7.cref(),true));
+}

tests/sequence_encoder_decoder_v2.cpp

@@ -239,3 +239,19 @@ TEST_CASE("group sequence inside sequence encoder_V2/decoder_v2","[group_sequenc
         REQUIRE(test_case.decoding("\xC0\x86\x81\xD0\xB2\x82\xB2",test_6.cref(),true));
     }
 }
+
+TEST_CASE("sequence with optional group encoder_V2/decoder_v2","[sequence_optional_group_encoder_v2_decoder_v2]")
+{
+    fast_test_coding_case_v2<simple14::templates_description> test_case;
+    simple14::Test_7 test_7;
+    simple14::Test_7_mref test_7_mref = test_7.mref();
+
+    auto sequence_7_mref = test_7_mref.set_sequence_7();
+    sequence_7_mref.resize(1);
+
+    auto element_sequence = sequence_7_mref.front();
+    element_sequence.set_field_7_3().as(50);
+
+    REQUIRE(test_case.encoding(test_7.cref(),"\xC0\x87\x81\xC0\xB2",true));
+    REQUIRE(test_case.decoding("\xC0\x87\x81\xC0\xB2",test_7.cref(),true));
+}

tests/simple14.xml

@@ -54,4 +54,13 @@
             </sequence>
         </group>
     </template>
+    <template name="Test_7" id="7">
+        <sequence name="sequence_7">
+            <length name="field_7_2" id="72"></length>
+            <uInt32 name="field_7_3" id="73"><copy/></uInt32>
+            <group name="group_7" presence="optional">
+                <uInt32 name="field_7_4" id="74"/>
+            </group>
+        </sequence>
+    </template>
 </templates>