Following results of CI build.

Results are inconsistent: Windows 7 maintained its previous behaviour, but Windows 8.1 and Nano Server behaved differently, failing with the same inputs.
It seems like we're hitting undefined behaviour in Windows' underlying ber_scanf, when processing two consecutive zero-length octet strings with the "v" and "V" specifiers. This isn't the point of the test though - changed the test inputs to supply well-formed BER data so that we can move on to actually testing the format specifier.

Author: edwardneal

src/libraries/System.DirectoryServices.Protocols/tests/BerConverterTests.cs

@@ -159,10 +159,7 @@ public static IEnumerable<object[]> Decode_TestData()
 
                 // Content: sequence containing three octet strings
                 // Parsed as two sequences of octet strings
-                // This is parsed differently between Windows 7 and later versions: the first element returned by Windows 7 will be an array of length 2,
-                // while this will be an array of length 3 in later versions.
-                yield return new object[] { "vv", new byte[] { 48, 132, 0, 0, 0, 9, 4, 3, 97, 98, 99, 4, 0, 4, 0 },
-                    Environment.OSVersion.Version <= new Version(6, 1) ? new object[] { new string[] { "abc", "" }, null } : new object[] { new string[] { "abc", "", "" }, null } };
+                yield return new object[] { "vv", new byte[] { 48, 132, 0, 0, 0, 12, 4, 3, 97, 98, 99, 4, 2, 100, 101, 4, 1, 102 }, new object[] { new string[] { "abc", "de", "f" }, null } };
 
                 // Content: sequence containing two sequences of octet strings
                 // Parsed as such
@@ -182,10 +179,7 @@ public static IEnumerable<object[]> Decode_TestData()
 
                 // Content: sequence of octet strings
                 // Parsed as two sequences of octet strings (returned as bytes)
-                // This is parsed differently between Windows 7 and later versions: the first element returned by Windows 7 will be an array of length 2,
-                // while this will be an array of length 3 in later versions.
-                yield return new object[] { "VV", new byte[] { 48, 132, 0, 0, 0, 9, 4, 3, 97, 98, 99, 4, 0, 4, 0 },
-                    Environment.OSVersion.Version <= new Version(6, 1) ? new object[] { new byte[][] { [97, 98, 99], [] }, null } : new object[]{ new byte[][] { [97, 98, 99], [], [] }, null } };
+                yield return new object[] { "VV", new byte[] { 48, 132, 0, 0, 0, 12, 4, 3, 97, 98, 99, 4, 2, 100, 101, 4, 1, 102 },new object[]{ new byte[][] { [97, 98, 99], [100, 101], [102] }, null } };
 
                 // Content: sequence containing two booleans
                 // Parsed as a sequence containing two sequences of octet strings (returned as bytes)