jhillyerd / enmime / 31622775533

87%

push

github

Close the header section when a part has an empty body (#410)

* Close the header section when a part has an empty body

Part.Encode only wrote the empty line separating the header section from
the body when the part had content, so a message built with no body ended
after its last header line. Reading that back with net/textproto's
ReadMIMEHeader returns the headers along with io.EOF, because the header
section was terminated by end of input rather than by the empty line.

Write the separator for parts without children even when the body is
empty. Parts with children are unchanged: they already get that line from
the leading CRLF of the first boundary marker.

Fixes #196

* test: use require.NoError for the ReadMIMEHeader error check

testifylint's require-error rule: an error assertion that gates later
assertions should stop the test (require) rather than continue (assert).

5 of 5 new or added lines in 1 file covered. (100.0%)

3328 of 3844 relevant lines covered (86.58%)

4281.58 hits per line