Fix assert() condition in MessageBufferOutput subclasses

[komamitsu]

The assert() in ChannelBufferOutput/OutputStreamBufferOutput#flush() looks to have a wrong condition. With -ea option on IntelliJ, org.msgpack.core.MessagePackTest("pack/unpack binary") failed.

For sbt, I tried to reproduce but I didn't find a proper way. The following change didn't work...

diff --git a/project/Build.scala b/project/Build.scala
index 21d221b..ce1aba9 100644
--- a/project/Build.scala
+++ b/project/Build.scala
@@ -51,6 +51,7 @@ object Build extends Build {
       //resolvers += Resolver.mavenLocal,
       scalacOptions ++= Seq("-encoding", "UTF-8", "-deprecation", "-unchecked", "-target:jvm-1.6", "-feature"),
       javacOptions in (Compile, compile) ++= Seq("-encoding", "UTF-8", "-Xlint:unchecked", "-Xlint:deprecation", "-source", "1.6", "-target", "1.6"),
+      javaOptions ++= Seq("-ea"),
       pomExtra := {
         <url>http://msgpack.org/</url>
           <licenses>

[xerial]

Good catch!

Regarding sbt, all test cases are running in Scala, so scalaOptions in Test should be modified.

[xerial]

Sorry. There is no scalaOptions setting in sbt. And also simply setting javaOptions in Test doesn't work either, because sbt reuses the JVM running the sbt itself.

A bettter approach would be launching the sbt with -ea option.