From 5fe8bbd9da944d2a0960ead8cc86ce1f5677c8a4 Mon Sep 17 00:00:00 2001 From: Steve Howell Date: Sat, 3 Apr 2021 16:23:49 +0000 Subject: [PATCH] node tests: Clear message_user_ids. This defends against future intra-test leaks. --- frontend_tests/node_tests/message_store.js | 1 + static/js/message_user_ids.js | 4 ++++ 2 files changed, 5 insertions(+) diff --git a/frontend_tests/node_tests/message_store.js b/frontend_tests/node_tests/message_store.js index 7c9b7bd8df..1436cf59c9 100644 --- a/frontend_tests/node_tests/message_store.js +++ b/frontend_tests/node_tests/message_store.js @@ -90,6 +90,7 @@ function convert_recipients(people) { function test(label, f) { run_test(label, (override) => { message_store.clear_for_testing(); + message_user_ids.clear_for_testing(); f(override); }); } diff --git a/static/js/message_user_ids.js b/static/js/message_user_ids.js index e4684fd99c..d7b2ded28f 100644 --- a/static/js/message_user_ids.js +++ b/static/js/message_user_ids.js @@ -13,6 +13,10 @@ */ const user_set = new Set(); +export function clear_for_testing() { + user_set.clear(); +} + export function user_ids() { return Array.from(user_set); }