Skip to content

Commit de285fb

Browse files
committed
Update add/remove appids test
Signed-off-by: Dmitrii Petukhov <[email protected]>
1 parent 54cf79e commit de285fb

File tree

1 file changed

+17
-5
lines changed

1 file changed

+17
-5
lines changed

src/integration-tests/test_restart_between_modes.py

Lines changed: 17 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -742,7 +742,11 @@ def test_restart_between_legacy_and_fsm_add_remove_app(
742742
- "baz" gets 3 messages
743743
- "quux" gets the third message
744744
745-
4. POST MORE MESSAGES
745+
4. Add/Remove more appIds
746+
Add "corge" appId
747+
Post to fanout queue
748+
Remove "foo" appId
749+
Post more messages to priority and fanout queues
746750
5. SWITCH BACK
747751
Switch back to the corresponding backup mode
748752
6. VERIFY AGAIN
@@ -794,18 +798,26 @@ def test_restart_between_legacy_and_fsm_add_remove_app(
794798
assert re.match(r"msg3", quux_messages[0].payload)
795799

796800
# 4. POST MORE MESSAGES
801+
current_app_ids.append("corge")
802+
cluster.set_app_ids(current_app_ids, du)
803+
post_few_messages(producer, fanout_queue, ["msg4"])
804+
805+
current_app_ids.remove("foo")
806+
cluster.set_app_ids(current_app_ids, du)
807+
797808
for queue in [priority_queue, fanout_queue]:
798-
post_few_messages(producer, queue, ["msg4"])
809+
post_few_messages(producer, queue, ["msg5"])
799810

800811
# 5. SWITCH BACK
801812
switch_cluster_mode[1](cluster, producer)
802813

803814
# 6. VERIFY AGAIN
804815
check_if_queue_has_n_messages(consumer, priority_queue, 1 + 1)
805-
check_if_queue_has_n_messages(consumer, fanout_queue + "?id=foo", 2 + 1)
816+
check_if_queue_has_n_messages(consumer, fanout_queue + "?id=foo", 0)
806817
check_if_queue_has_n_messages(consumer, fanout_queue + "?id=bar", 0)
807-
check_if_queue_has_n_messages(consumer, fanout_queue + "?id=baz", 3 + 1)
808-
check_if_queue_has_n_messages(consumer, fanout_queue + "?id=quux", 1 + 1)
818+
check_if_queue_has_n_messages(consumer, fanout_queue + "?id=baz", 3 + 2)
819+
check_if_queue_has_n_messages(consumer, fanout_queue + "?id=quux", 1 + 2)
820+
check_if_queue_has_n_messages(consumer, fanout_queue + "?id=corge", 0 + 2)
809821

810822

811823
def test_restart_between_legacy_and_fsm_purge_queue_app(

0 commit comments

Comments
 (0)