I believe this is working as intended, the docs for Write says:

Call Close to convert the remaining bytes.

and if you do call that and count the bytes it writes out nothing is missing

I believe this is working as intended, the docs for Write says:

Call Close to convert the remaining bytes.

and if you do call that and count the bytes it writes out nothing is missing

@seankhliao

The example test above does call Close() to flush anything remaining. If for example, I have it write out to bytes.Buffer, at the end, 5 bytes are missing.

The last bytes are written when you call Close(), which you call after your check. Change your discard writer to something that can count how much is written to it and call Close before doing the check

The last bytes are written when you call Close(), which you call after your check. Change your discard writer to something that can count how much is written to it and call Close before doing the check

@seankhliao Yes, to reiterate, I can do that, and it will come up 5 bytes short of what's expected. This problem is not related to the final call to Close().

@seankhliao I have written a reproducer as described in your comment, and bytes did get dropped. Here is the reproducer: https://play.golang.org/p/Zyi0VOMfXop. I think there is a bug in the transform.Writer. When the number of total bytes (excluding the bytes consumed by the first write, which is 5) is greater than the default buf size of the transform.Writer (which is 4096), the extra bytes are dropped.

Here is the unit test based on @saracen's code:

func TestBytesDropped(t *testing.T) {
	var buf bytes.Buffer
	w := transform.NewWriter(&buf, problemTransform{})

	write := func(p []byte) {
		_, err := w.Write(p)
		if err != nil {
			t.Error(err)
		}
	}

	b := bytes.Repeat([]byte("0123456789"), 206)
	// default buf size used by transform.Writer
	defaultBufSize := 4096
	// it fails when the number of total bytes (excluding the bytes consumed by
	// the first write, which is 5) is greater than the default buf size
	total := defaultBufSize + 5 + 1
	firstWrite := 2058
	write(b[:firstWrite])
	// some bytes are dropped in the second write
	write(b[:total-firstWrite])

	if err := w.Close(); err != nil {
		t.Error(err)
	}

	if buf.Len() != total {
		t.Errorf("want number of bytes: %d, got: %d", total, buf.Len())
	}
}

The last bytes are written when you call Close(), which you call after your check. Change your discard writer to something that can count how much is written to it and call Close before doing the check

Docs say Write must return a non-nil error if it returns n < len(p).
So it really is a bug.