msgpack#129: Using smaller test repetition number

xerial · xerial · commit 40cb2b3ddf47 · 2014-08-11T15:18:19.000+09:00
diff --git a/msgpack-core/src/test/scala/org/msgpack/core/MessageUnpackerTest.scala b/msgpack-core/src/test/scala/org/msgpack/core/MessageUnpackerTest.scala
@@ -540,7 +540,7 @@ class MessageUnpackerTest extends MessagePackSpec {
       val mb = MessageBuffer.wrap(arr)
 
       val N = 1000
-      val t = time("unpacker", repeat = 500) {
+      val t = time("unpacker", repeat = 10) {
         block("no-buffer-reset") {
           IOUtil.withResource(MessagePackFactory.newDefaultUnpacker(arr)) { unpacker =>
             for (i <- 0 until N) {
@@ -552,30 +552,29 @@ class MessageUnpackerTest extends MessagePackSpec {
           }
         }
 
-        block("reuse-message-buffer") {
+        block("reuse-array-input") {
           IOUtil.withResource(MessagePackFactory.newDefaultUnpacker(arr)) { unpacker =>
             val buf = new ArrayBufferInput(arr)
             for (i <- 0 until N) {
-              buf.reset(mb)
+              buf.reset(arr)
               unpacker.reset(buf)
               unpacker.unpackInt
               unpacker.close
             }
           }
         }
 
-        block("reuse-array-input") {
+        block("reuse-message-buffer") {
           IOUtil.withResource(MessagePackFactory.newDefaultUnpacker(arr)) { unpacker =>
             val buf = new ArrayBufferInput(arr)
             for (i <- 0 until N) {
-              buf.reset(arr)
+              buf.reset(mb)
               unpacker.reset(buf)
               unpacker.unpackInt
               unpacker.close
             }
           }
         }
-
       }
 
       t("reuse-message-buffer").averageWithoutMinMax should be <= t("no-buffer-reset").averageWithoutMinMax

Original file line number	Diff line number	Diff line change
`@@ -540,7 +540,7 @@ class MessageUnpackerTest extends MessagePackSpec {`
`540`	`540`	`val mb = MessageBuffer.wrap(arr)`
`541`	`541`
`542`	`542`	`val N = 1000`
`543`		`- val t = time("unpacker", repeat = 500) {`
	`543`	`+ val t = time("unpacker", repeat = 10) {`
`544`	`544`	`block("no-buffer-reset") {`
`545`	`545`	`IOUtil.withResource(MessagePackFactory.newDefaultUnpacker(arr)) { unpacker =>`
`546`	`546`	`for (i <- 0 until N) {`
`@@ -552,30 +552,29 @@ class MessageUnpackerTest extends MessagePackSpec {`
`552`	`552`	`}`
`553`	`553`	`}`
`554`	`554`
`555`		`- block("reuse-message-buffer") {`
	`555`	`+ block("reuse-array-input") {`
`556`	`556`	`IOUtil.withResource(MessagePackFactory.newDefaultUnpacker(arr)) { unpacker =>`
`557`	`557`	`val buf = new ArrayBufferInput(arr)`
`558`	`558`	`for (i <- 0 until N) {`
`559`		`- buf.reset(mb)`
	`559`	`+ buf.reset(arr)`
`560`	`560`	`unpacker.reset(buf)`
`561`	`561`	`unpacker.unpackInt`
`562`	`562`	`unpacker.close`
`563`	`563`	`}`
`564`	`564`	`}`
`565`	`565`	`}`
`566`	`566`
`567`		`- block("reuse-array-input") {`
	`567`	`+ block("reuse-message-buffer") {`
`568`	`568`	`IOUtil.withResource(MessagePackFactory.newDefaultUnpacker(arr)) { unpacker =>`
`569`	`569`	`val buf = new ArrayBufferInput(arr)`
`570`	`570`	`for (i <- 0 until N) {`
`571`		`- buf.reset(arr)`
	`571`	`+ buf.reset(mb)`
`572`	`572`	`unpacker.reset(buf)`
`573`	`573`	`unpacker.unpackInt`
`574`	`574`	`unpacker.close`
`575`	`575`	`}`
`576`	`576`	`}`
`577`	`577`	`}`
`578`		`-`
`579`	`578`	`}`
`580`	`579`
`581`	`580`	`t("reuse-message-buffer").averageWithoutMinMax should be <= t("no-buffer-reset").averageWithoutMinMax`