diff options
Diffstat (limited to 'packages/meshbay-hub/tests/test_app_settings_plugin.py')
| -rw-r--r-- | packages/meshbay-hub/tests/test_app_settings_plugin.py | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/packages/meshbay-hub/tests/test_app_settings_plugin.py b/packages/meshbay-hub/tests/test_app_settings_plugin.py index 892fd09..af86f12 100644 --- a/packages/meshbay-hub/tests/test_app_settings_plugin.py +++ b/packages/meshbay-hub/tests/test_app_settings_plugin.py @@ -254,17 +254,19 @@ def test_each_app_takes_a_list_of_directories(app, prop): f"{app} still reads {singular} — one shape per idea") -def test_an_older_node_still_fills_the_lists(): +def test_the_lists_are_read_under_one_name_each(): """ - A node speaking MNP 1.0 sends `video_root`, not `video_directories`. - Reading the missing plural as "nothing configured" would empty a working - Videos tab on every group hosted by a node that has not been upgraded. + One name per app on the ack — `<app>_directories`, always a list. A second + name for the same answer means a page that reads whichever it thinks of + first, and a node that fills only the other one empties a working tab. """ page = GROUP_PAGE.read_text(encoding="utf-8") block = page[page.index("setAppDirectories({"):] block = block[:block.index("setChatDirectory")] - assert "ack.video_root" in block and "ack.audio_root" in block - assert "ack.photo_roots" in block + assert "ack.video_directories" in block and "ack.music_directories" in block + assert "ack.photo_directories" in block + for gone in ("ack.video_root", "ack.audio_root", "ack.photo_roots"): + assert gone not in block, f"{gone} is a second name for the same answer" def test_the_search_cache_reads_both_shapes(): |