Avoid class mutation

[kibertoad]

In several places we have code like this:

    const byteCounter = new AssertByteCountStream(end - start)
    byteCounter.destroy = function () {
      <...>
     }

This should be a permanent method instead, and instance fields should be used to pass the closure variables.

This change should only be done after tests are working.