Skip to content

Commit de11d51

Browse files
DifferentialOrangeDifferentialOrange
committed
test: do not rely on replication in more read cases
Some read cases were missing in #404, #406 and #408 test stabilization commits. This patch updates missing read cases. 1. https://github.com/tarantool/crud/actions/runs/7322196516/job/19943422319 Follows #404 Part of #412 Part of #415
1 parent 5b05d38 commit de11d51

File tree

8 files changed

+140
-79
lines changed

8 files changed

+140
-79
lines changed

‎test/doc/playground_test.lua‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ end
6161

6262
local cases = {
6363
test_select_customers = {
64-
request = "crud.select('customers', {{'<=', 'age', 35}}, {first = 10})",
64+
request = "crud.select('customers', {{'<=', 'age', 35}}, {first = 10, mode = 'write'})",
6565
retval_1 = {
6666
metadata = {
6767
{name = 'id', type = 'unsigned'},
@@ -79,7 +79,7 @@ local cases = {
7979
}
8080
},
8181
test_select_developers = {
82-
request = "crud.select('developers', nil, {first = 6})",
82+
request = "crud.select('developers', nil, {first = 6, mode = 'write'})",
8383
retval_1 = {
8484
metadata = {
8585
{name = 'id', type = 'unsigned'},
@@ -117,7 +117,7 @@ local cases = {
117117
test_error = {
118118
request = [[
119119
do
120-
local res, err = crud.select('non_existent', nil, {first = 10})
120+
local res, err = crud.select('non_existent', nil, {first = 10, mode = 'write'})
121121
return res, err and err.err or nil
122122
end
123123
]],

‎test/integration/borders_test.lua‎

Lines changed: 50 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -26,83 +26,103 @@ end)
2626

2727
pgroup.test_non_existent_space = function(g)
2828
-- min
29-
local result, err = g.cluster.main_server.net_box:call(
30-
'crud.min', {'non_existent_space'}
31-
)
29+
local result, err = g.cluster.main_server.net_box:call('crud.min', {
30+
'non_existent_space',
31+
nil,
32+
{mode = 'write'},
33+
})
3234

3335
t.assert_equals(result, nil)
3436
t.assert_str_contains(err.err, "Space \"non_existent_space\" doesn't exist")
3537

3638
-- max
37-
local result, err = g.cluster.main_server.net_box:call(
38-
'crud.max', {'non_existent_space'}
39-
)
39+
local result, err = g.cluster.main_server.net_box:call('crud.max', {
40+
'non_existent_space',
41+
nil,
42+
{mode = 'write'},
43+
})
4044

4145
t.assert_equals(result, nil)
4246
t.assert_str_contains(err.err, "Space \"non_existent_space\" doesn't exist")
4347
end
4448

4549
pgroup.test_non_existent_index = function(g)
4650
-- min
47-
local result, err = g.cluster.main_server.net_box:call(
48-
'crud.min', {'customers', 'non_existent_index'}
49-
)
51+
local result, err = g.cluster.main_server.net_box:call('crud.min', {
52+
'customers',
53+
'non_existent_index',
54+
{mode = 'write'},
55+
})
5056

5157
t.assert_equals(result, nil)
5258
t.assert_str_contains(err.err, "Index \"non_existent_index\" of space \"customers\" doesn't exist")
5359

54-
local result, err = g.cluster.main_server.net_box:call(
55-
'crud.min', {'customers', 13}
56-
)
60+
local result, err = g.cluster.main_server.net_box:call('crud.min', {
61+
'customers',
62+
13,
63+
{mode = 'write'},
64+
})
5765

5866
t.assert_equals(result, nil)
5967
t.assert_str_contains(err.err, "Index \"13\" of space \"customers\" doesn't exist")
6068

6169
-- max
62-
local result, err = g.cluster.main_server.net_box:call(
63-
'crud.max', {'customers', 'non_existent_index'}
64-
)
70+
local result, err = g.cluster.main_server.net_box:call('crud.max', {
71+
'customers',
72+
'non_existent_index',
73+
{mode = 'write'},
74+
})
6575

6676
t.assert_equals(result, nil)
6777
t.assert_str_contains(err.err, "Index \"non_existent_index\" of space \"customers\" doesn't exist")
6878

69-
local result, err = g.cluster.main_server.net_box:call(
70-
'crud.max', {'customers', 13}
71-
)
79+
local result, err = g.cluster.main_server.net_box:call('crud.max', {
80+
'customers',
81+
13,
82+
{mode = 'write'},
83+
})
7284

7385
t.assert_equals(result, nil)
7486
t.assert_str_contains(err.err, "Index \"13\" of space \"customers\" doesn't exist")
7587
end
7688

7789
pgroup.test_empty_space = function(g)
7890
-- min
79-
local result, err = g.cluster.main_server.net_box:call(
80-
'crud.min', {'customers'}
81-
)
91+
local result, err = g.cluster.main_server.net_box:call('crud.min', {
92+
'customers',
93+
nil,
94+
{mode = 'write'},
95+
})
8296

8397
t.assert_equals(err, nil)
8498
t.assert_equals(#result.rows, 0)
8599

86100
-- min by age index with fields
87-
local result, err = g.cluster.main_server.net_box:call(
88-
'crud.min', {'customers', 'age_index', {fields = {'name'}}}
89-
)
101+
local result, err = g.cluster.main_server.net_box:call('crud.min', {
102+
'customers',
103+
'age_index',
104+
{fields = {'name'}, mode = 'write'},
105+
})
90106

91107
t.assert_equals(err, nil)
92108
t.assert_equals(#result.rows, 0)
93109

94110
-- max
95-
local result, err = g.cluster.main_server.net_box:call(
96-
'crud.max', {'customers'}
97-
)
111+
local result, err = g.cluster.main_server.net_box:call('crud.max', {
112+
'customers',
113+
nil,
114+
{mode = 'write'},
115+
})
98116

99117
t.assert_equals(err, nil)
100118
t.assert_equals(#result.rows, 0)
101119

102120
-- max by age index with fields
103-
local result, err = g.cluster.main_server.net_box:call(
104-
'crud.max', {'customers', 'age_index', {fields = {'name'}}}
105-
)
121+
local result, err = g.cluster.main_server.net_box:call('crud.max', {
122+
'customers',
123+
'age_index',
124+
{fields = {'name'}, mode = 'write'},
125+
})
106126

107127
t.assert_equals(err, nil)
108128
t.assert_equals(#result.rows, 0)

‎test/integration/count_test.lua‎

Lines changed: 25 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -33,14 +33,22 @@ pgroup.before_each(function(g)
3333
end)
3434

3535
pgroup.test_count_non_existent_space = function(g)
36-
local result, err = g.cluster.main_server.net_box:call('crud.count', {'non_existent_space', nil, {fullscan = true}})
36+
local result, err = g.cluster.main_server.net_box:call('crud.count', {
37+
'non_existent_space',
38+
nil,
39+
{fullscan = true, mode = 'write'},
40+
})
3741

3842
t.assert_equals(result, nil)
3943
t.assert_str_contains(err.err, "Space \"non_existent_space\" doesn't exist")
4044
end
4145

4246
pgroup.test_count_empty_space = function(g)
43-
local result, err = g.cluster.main_server.net_box:call('crud.count', {'customers', nil, {fullscan = true}})
47+
local result, err = g.cluster.main_server.net_box:call('crud.count', {
48+
'customers',
49+
nil,
50+
{fullscan = true, mode = 'write'},
51+
})
4452

4553
t.assert_equals(err, nil)
4654
t.assert_equals(result, 0)
@@ -51,9 +59,11 @@ pgroup.test_not_valid_value_type = function(g)
5159
{'==', 'id', 'not_number'}
5260
}
5361

54-
local result, err = g.cluster.main_server.net_box:call('crud.count',
55-
{'customers', conditions}
56-
)
62+
local result, err = g.cluster.main_server.net_box:call('crud.count', {
63+
'customers',
64+
conditions,
65+
{mode = 'write'},
66+
})
5767

5868
t.assert_equals(result, nil)
5969
t.assert_str_contains(err.err, "Supplied key type of part 0 does not match index part type: expected unsigned")
@@ -64,9 +74,11 @@ pgroup.test_not_valid_operation = function(g)
6474
{{}, 'id', 5}
6575
}
6676

67-
local result, err = g.cluster.main_server.net_box:call('crud.count',
68-
{'customers', conditions, {fullscan = true}}
69-
)
77+
local result, err = g.cluster.main_server.net_box:call('crud.count', {
78+
'customers',
79+
conditions,
80+
{fullscan = true, mode = 'write'},
81+
})
7082

7183
t.assert_equals(result, nil)
7284
t.assert_str_contains(err.err, "Failed to parse conditions")
@@ -77,9 +89,11 @@ pgroup.test_conditions_with_non_existed_field = function(g)
7789
{'==', 'non_existed_field', 'value'}
7890
}
7991

80-
local result, err = g.cluster.main_server.net_box:call('crud.count',
81-
{'customers', conditions}
82-
)
92+
local result, err = g.cluster.main_server.net_box:call('crud.count', {
93+
'customers',
94+
conditions,
95+
{mode = 'write'},
96+
})
8397

8498
t.assert_equals(err, nil)
8599
t.assert_equals(result, 0)

‎test/integration/pairs_test.lua‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -419,7 +419,7 @@ pgroup.test_pairs_partial_result = function(g)
419419
local conditions, fields = ...
420420
421421
local tuples = {}
422-
for _, tuple in crud.pairs('customers', conditions, {fields = fields}) do
422+
for _, tuple in crud.pairs('customers', conditions, {fields = fields, mode = 'write'}) do
423423
table.insert(tuples, tuple)
424424
end
425425

‎test/integration/select_test.lua‎

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,9 @@ end)
4040
pgroup.test_non_existent_space = function(g)
4141
-- insert
4242
local obj, err = g.cluster.main_server.net_box:call('crud.select', {
43-
'non_existent_space', nil, {fullscan = true},
43+
'non_existent_space',
44+
nil,
45+
{fullscan = true, mode = 'write'},
4446
})
4547

4648
t.assert_equals(obj, nil)
@@ -49,7 +51,9 @@ end
4951

5052
pgroup.test_select_no_index = function(g)
5153
local obj, err = g.cluster.main_server.net_box:call('crud.select', {
52-
'no_index_space', nil, {fullscan = true},
54+
'no_index_space',
55+
nil,
56+
{fullscan = true, mode = 'write'},
5357
})
5458

5559
t.assert_equals(obj, nil)

0 commit comments

Comments
 (0)