annotate src/lib-index/test-mail-index-sync-ext.c @ 22576:707ae9de3812

lib: istream-multiplex - Minor code cleanup Avoid propagating the error twice, and avoid any confusion about what "got" actually contains.
author Timo Sirainen <timo.sirainen@dovecot.fi>
date Thu, 05 Oct 2017 20:24:11 +0300
parents 2e2563132d5f
children cb108f786fb4
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
21390
2e2563132d5f Updated copyright notices to include the year 2017.
Stephan Bosch <stephan.bosch@dovecot.fi>
parents: 21389
diff changeset
1 /* Copyright (c) 2010-2017 Dovecot authors, see the included COPYING file */
11221
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
2
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
3 #include "lib.h"
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
4 #include "array.h"
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
5 #include "test-common.h"
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
6 #include "mail-index-sync-private.h"
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
7 #include "mail-index-modseq.h"
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
8
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
9
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
10 void mail_index_sync_set_corrupted(struct mail_index_sync_map_ctx *ctx ATTR_UNUSED,
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
11 const char *fmt ATTR_UNUSED, ...) {}
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
12 struct mail_index_map *
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
13 mail_index_sync_get_atomic_map(struct mail_index_sync_map_ctx *ctx) { return ctx->view->map; }
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
14 uint32_t
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
15 mail_index_map_register_ext(struct mail_index_map *map ATTR_UNUSED,
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
16 const char *name ATTR_UNUSED, uint32_t ext_offset ATTR_UNUSED,
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
17 const struct mail_index_ext_header *ext_hdr ATTR_UNUSED) { return 0; }
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
18 bool mail_index_ext_lookup(struct mail_index *index ATTR_UNUSED,
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
19 const char *name ATTR_UNUSED,
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
20 uint32_t *ext_id_r ATTR_UNUSED) { return FALSE; }
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
21 bool mail_index_map_lookup_ext(struct mail_index_map *map ATTR_UNUSED,
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
22 const char *name ATTR_UNUSED,
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
23 uint32_t *idx_r ATTR_UNUSED) { return FALSE; }
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
24 int mail_index_map_ext_hdr_check(const struct mail_index_header *hdr ATTR_UNUSED,
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
25 const struct mail_index_ext_header *ext_hdr ATTR_UNUSED,
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
26 const char *name ATTR_UNUSED,
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
27 const char **error_r ATTR_UNUSED) { return -1; }
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
28 void mail_index_modseq_hdr_update(struct mail_index_modseq_sync *ctx ATTR_UNUSED) {}
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
29 bool mail_index_lookup_seq(struct mail_index_view *view ATTR_UNUSED,
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
30 uint32_t uid, uint32_t *seq_r) {
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
31 *seq_r = uid;
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
32 return TRUE;
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
33 }
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
34
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
35 static void test_mail_index_sync_ext_atomic_inc(void)
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
36 {
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
37 struct mail_index_sync_map_ctx ctx;
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
38 struct mail_transaction_ext_atomic_inc u;
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
39 struct mail_index_ext *ext;
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
40 void *ptr;
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
41
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
42 test_begin("mail index sync ext atomic inc");
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
43
21389
59437f8764c6 global: Replaced all instances of memset(p, 0, sizeof(*p)) with the new i_zero() macro.
Stephan Bosch <stephan.bosch@dovecot.fi>
parents: 20324
diff changeset
44 i_zero(&ctx);
11221
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
45 ctx.view = t_new(struct mail_index_view, 1);
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
46 ctx.view->map = t_new(struct mail_index_map, 1);
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
47 ctx.view->map->hdr.next_uid = 2;
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
48 ctx.view->map->hdr.record_size = sizeof(struct mail_index_record) + 16;
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
49 ctx.view->map->rec_map = t_new(struct mail_index_record_map, 1);
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
50 ctx.view->map->rec_map->records =
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
51 t_malloc(ctx.view->map->hdr.record_size);
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
52 t_array_init(&ctx.view->map->extensions, 4);
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
53 ext = array_append_space(&ctx.view->map->extensions);
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
54 ext->record_offset = sizeof(struct mail_index_record);
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
55 ptr = PTR_OFFSET(ctx.view->map->rec_map->records, ext->record_offset);
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
56
21389
59437f8764c6 global: Replaced all instances of memset(p, 0, sizeof(*p)) with the new i_zero() macro.
Stephan Bosch <stephan.bosch@dovecot.fi>
parents: 20324
diff changeset
57 i_zero(&u);
11221
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
58 test_assert(mail_index_sync_ext_atomic_inc(&ctx, &u) == -1);
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
59
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
60 u.uid = 2;
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
61 test_assert(mail_index_sync_ext_atomic_inc(&ctx, &u) == -1);
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
62
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
63 u.uid = 1;
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
64 #define TEST_ATOMIC(_type, _value, _diff, _ret) \
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
65 { _type *n = ptr; *n = _value; } \
20324
97fa8fcdffd4 lib-index: Fixed test-mail-index-sync-ext
Timo Sirainen <timo.sirainen@dovecot.fi>
parents: 19552
diff changeset
66 ctx.cur_ext_record_size = sizeof(_type); \
11221
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
67 u.diff = _diff; \
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
68 test_assert(mail_index_sync_ext_atomic_inc(&ctx, &u) == _ret);
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
69
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
70 #define TEST_ATOMIC_BLOCK(_type, _max) \
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
71 TEST_ATOMIC(_type, 1, -1, 1); \
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
72 TEST_ATOMIC(_type, 1, -2, -1); \
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
73 TEST_ATOMIC(_type, 0, -1, -1); \
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
74 TEST_ATOMIC(_type, 0, _max, 1); \
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
75 TEST_ATOMIC(_type, 1, _max, -1); \
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
76 TEST_ATOMIC(_type, 0, (_max+1), -1); \
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
77 TEST_ATOMIC(_type, _max, 1, -1); \
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
78 TEST_ATOMIC(_type, _max, -_max, 1); \
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
79 TEST_ATOMIC(_type, _max, -(_max+1), -1);
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
80
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
81 TEST_ATOMIC_BLOCK(uint8_t, 255);
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
82 TEST_ATOMIC_BLOCK(uint16_t, 65535);
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
83
20324
97fa8fcdffd4 lib-index: Fixed test-mail-index-sync-ext
Timo Sirainen <timo.sirainen@dovecot.fi>
parents: 19552
diff changeset
84 ctx.cur_ext_record_size = 5;
11221
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
85 u.diff = 0;
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
86 test_assert(mail_index_sync_ext_atomic_inc(&ctx, &u) == -1);
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
87
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
88 test_end();
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
89 }
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
90
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
91 int main(void)
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
92 {
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
93 static void (*test_functions[])(void) = {
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
94 test_mail_index_sync_ext_atomic_inc,
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
95 NULL
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
96 };
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
97 return test_run(test_functions);
204bb387aeed lib-index: mail_index_sync_ext_atomic_inc() does now better error checking.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
98 }