From: Aki Tuomi <aki.tuomi@open-xchange.com>
Date: Mon, 9 Nov 2020 07:29:26 +0000 (+0200)
Subject: lib-mail: test-* - Add missing test_assert
X-Git-Tag: 2.3.14.rc1~354
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=3c137ff3ad487ea80e02449e4b47eb2f55bef398;p=thirdparty%2Fdovecot%2Fcore.git

lib-mail: test-* - Add missing test_assert

Need to ensure stream did not fail during parsing.
---

diff --git a/src/lib-mail/test-message-decoder.c b/src/lib-mail/test-message-decoder.c
index 3007283cad..f1b0901673 100644
--- a/src/lib-mail/test-message-decoder.c
+++ b/src/lib-mail/test-message-decoder.c
@@ -141,6 +141,7 @@ static void test_message_decoder_multipart(void)
 
 	message_decoder_deinit(&decoder);
 	message_parser_deinit(&parser, &parts);
+	test_assert(istream->stream_errno == 0);
 	i_stream_unref(&istream);
 	test_end();
 }
diff --git a/src/lib-mail/test-message-parser.c b/src/lib-mail/test-message-parser.c
index 32226febc7..43d6ed66d3 100644
--- a/src/lib-mail/test-message-parser.c
+++ b/src/lib-mail/test-message-parser.c
@@ -57,6 +57,7 @@ static int message_parse_stream(pool_t pool, struct istream *input,
 			message_part_data_parse_from_header(pool, block.part,
 							    block.hdr);
 	message_parser_deinit(&parser, parts_r);
+	test_assert(input->stream_errno == 0);
 	return ret;
 }
 
@@ -118,6 +119,7 @@ static void test_message_parser_small_blocks(void)
 
 	test_assert(ret < 0);
 	message_parser_deinit(&parser, &parts);
+	test_assert(input->stream_errno == 0);
 	test_assert(strcmp(test_msg, str_c(output)) == 0);
 
 	/* parsing in small blocks */
@@ -135,6 +137,7 @@ static void test_message_parser_small_blocks(void)
 			    (ret < 0 && i > TEST_MSG_LEN*2));
 	}
 	message_parser_deinit(&parser, &parts2);
+	test_assert(input->stream_errno == 0);
 	test_assert(message_part_is_equal(parts, parts2));
 
 	/* parsing in small blocks from preparsed parts */
@@ -747,6 +750,7 @@ static void test_message_parser_no_eoh(void)
 		    block.hdr == NULL && block.size == 0);
 	test_assert(message_parser_parse_next_block(parser, &block) < 0);
 	message_parser_deinit(&parser, &parts);
+	test_assert(input->stream_errno == 0);
 
 	test_parsed_parts(input, parts);
 	i_stream_unref(&input);
diff --git a/src/lib-mail/test-message-part.c b/src/lib-mail/test-message-part.c
index 4a51a39f99..49cfd65a21 100644
--- a/src/lib-mail/test-message-part.c
+++ b/src/lib-mail/test-message-part.c
@@ -86,6 +86,7 @@ static void test_message_part_idx(void)
 	}
 	test_assert(ret < 0);
 	message_parser_deinit(&parser, &parts);
+	test_assert(input->stream_errno == 0);
 
 	part = message_part_by_idx(parts, 0);
 	test_assert(part == parts);