From 3da4afd259667ce57b5b41db4a0e589ba861d1a9 Mon Sep 17 00:00:00 2001 From: Milo Yip Date: Sat, 16 Apr 2016 15:19:34 +0800 Subject: [PATCH] Another trial on writer coverage --- test/unittest/simdtest.cpp | 38 ++++++++++++++++++++---------------- test/unittest/writertest.cpp | 6 +++++- 2 files changed, 26 insertions(+), 18 deletions(-) diff --git a/test/unittest/simdtest.cpp b/test/unittest/simdtest.cpp index 1b6fcef..84f8cb0 100644 --- a/test/unittest/simdtest.cpp +++ b/test/unittest/simdtest.cpp @@ -105,24 +105,28 @@ struct ScanCopyUnescapedStringHandler : BaseReaderHandler, ScanCopyUnesca template void TestScanCopyUnescapedString() { - for (size_t step = 0; step < 1024; step++) { - char json[1024 + 5]; - char *p = json; - *p ++= '\"'; - for (size_t i = 0; i < step; i++) - *p++= "ABCD"[i % 4]; - *p++ = '\\'; - *p++ = '\\'; - *p++ = '\"'; - *p++ = '\0'; + char buffer[1024 + 5 + 32]; - StreamType s(json); - Reader reader; - ScanCopyUnescapedStringHandler h; - reader.Parse(s, h); - EXPECT_TRUE(memcmp(h.buffer, json + 1, step) == 0); - EXPECT_EQ('\\', h.buffer[step]); // escaped - EXPECT_EQ('\0', h.buffer[step + 1]); + for (size_t offset = 0; offset < 32; offset++) { + for (size_t step = 0; step < 1024; step++) { + char* json = buffer + offset; + char *p = json; + *p++ = '\"'; + for (size_t i = 0; i < step; i++) + *p++ = "ABCD"[i % 4]; + *p++ = '\\'; + *p++ = '\\'; + *p++ = '\"'; + *p++ = '\0'; + + StreamType s(json); + Reader reader; + ScanCopyUnescapedStringHandler h; + reader.Parse(s, h); + EXPECT_TRUE(memcmp(h.buffer, json + 1, step) == 0); + EXPECT_EQ('\\', h.buffer[step]); // escaped + EXPECT_EQ('\0', h.buffer[step + 1]); + } } } diff --git a/test/unittest/writertest.cpp b/test/unittest/writertest.cpp index af09f8b..9c68c53 100644 --- a/test/unittest/writertest.cpp +++ b/test/unittest/writertest.cpp @@ -448,6 +448,10 @@ TEST(Writer, NaN) { StringBuffer buffer; Writer writer(buffer); EXPECT_FALSE(writer.Double(nan)); + + GenericStringBuffer > buffer2; + Writer > > writer2(buffer2); + EXPECT_FALSE(writer2.Double(nan)); } TEST(Writer, Inf) { @@ -456,7 +460,7 @@ TEST(Writer, Inf) { StringBuffer buffer; { Writer writer(buffer); - EXPECT_FALSE(writer.Double(inf)); + EXPECT_FALSE(writer.Double(inf)); } { Writer writer(buffer);