Display event IDs after Nostr publish

src/tests/test_manager_workflow.py

@@ -22,6 +22,7 @@ class FakeNostrClient:
 
     def publish_snapshot(self, data: bytes):
         self.published.append(data)
+        return None, "abcd"
 
 
 def test_manager_workflow(monkeypatch):

src/tests/test_nostr_backup.py

@@ -25,7 +25,7 @@ def test_backup_and_publish_to_nostr():
 
         with patch(
             "nostr.client.NostrClient.publish_snapshot",
-            AsyncMock(return_value=None),
+            AsyncMock(return_value=(None, "abcd")),
         ) as mock_publish, patch("nostr.client.ClientBuilder"), patch(
             "nostr.client.KeyManager"
         ), patch.object(
@@ -38,4 +38,4 @@ def test_backup_and_publish_to_nostr():
             result = asyncio.run(nostr_client.publish_snapshot(encrypted_index))
 
         mock_publish.assert_awaited_with(encrypted_index)
-        assert result is None
+        assert result == (None, "abcd")

src/tests/test_nostr_dummy_client.py

@@ -29,7 +29,7 @@ def test_retrieve_multi_chunk_snapshot(dummy_nostr_client):
 
     client, relay = dummy_nostr_client
     data = os.urandom(120000)
-    manifest = asyncio.run(client.publish_snapshot(data, limit=50000))
+    manifest, _ = asyncio.run(client.publish_snapshot(data, limit=50000))
     assert len(manifest.chunks) > 1
     fetched_manifest, chunk_bytes = asyncio.run(client.fetch_latest_snapshot())
     assert len(chunk_bytes) == len(manifest.chunks)
@@ -40,7 +40,7 @@ def test_retrieve_multi_chunk_snapshot(dummy_nostr_client):
 def test_publish_and_fetch_deltas(dummy_nostr_client):
     client, relay = dummy_nostr_client
     base = b"base"
-    manifest = asyncio.run(client.publish_snapshot(base))
+    manifest, _ = asyncio.run(client.publish_snapshot(base))
     manifest_id = relay.manifests[-1].id
     d1 = b"d1"
     d2 = b"d2"

src/tests/test_password_change.py

@@ -43,7 +43,7 @@ def test_change_password_triggers_nostr_backup(monkeypatch):
 
         with patch("password_manager.manager.NostrClient") as MockClient:
             mock_instance = MockClient.return_value
-            mock_instance.publish_snapshot = AsyncMock(return_value=None)
+            mock_instance.publish_snapshot = AsyncMock(return_value=(None, "abcd"))
             pm.nostr_client = mock_instance
             pm.change_password()
             mock_instance.publish_snapshot.assert_called_once()

src/tests/test_password_unlock_after_change.py

@@ -54,7 +54,9 @@ def test_password_change_and_unlock(monkeypatch):
         pm.fingerprint_dir = fp
         pm.current_fingerprint = "fp"
         pm.parent_seed = SEED
-        pm.nostr_client = SimpleNamespace(publish_snapshot=lambda *a, **k: None)
+        pm.nostr_client = SimpleNamespace(
+            publish_snapshot=lambda *a, **k: (None, "abcd")
+        )
 
         monkeypatch.setattr(
             "password_manager.manager.prompt_existing_password", lambda *_: old_pw
@@ -64,7 +66,9 @@ def test_password_change_and_unlock(monkeypatch):
         )
         monkeypatch.setattr(
             "password_manager.manager.NostrClient",
-            lambda *a, **kw: SimpleNamespace(publish_snapshot=lambda *a, **k: None),
+            lambda *a, **kw: SimpleNamespace(
+                publish_snapshot=lambda *a, **k: (None, "abcd")
+            ),
         )
 
         pm.change_password()

src/tests/test_post_sync_messages.py

@@ -9,16 +9,17 @@ import main
 
 def test_handle_post_success(capsys):
     pm = SimpleNamespace(
-        sync_vault=lambda alt_summary=None: True,
+        sync_vault=lambda alt_summary=None: "abcd",
     )
     main.handle_post_to_nostr(pm)
     out = capsys.readouterr().out
     assert "✅ Sync complete." in out
+    assert "abcd" in out
 
 
 def test_handle_post_failure(capsys):
     pm = SimpleNamespace(
-        sync_vault=lambda alt_summary=None: False,
+        sync_vault=lambda alt_summary=None: None,
     )
     main.handle_post_to_nostr(pm)
     out = capsys.readouterr().out

src/tests/test_profile_management.py

@@ -62,7 +62,10 @@ def test_add_and_delete_entry(monkeypatch):
 
         published = []
         pm.nostr_client = SimpleNamespace(
-            publish_snapshot=lambda data, alt_summary=None: published.append(data)
+            publish_snapshot=lambda data, alt_summary=None: (
+                published.append(data),
+                (None, "abcd"),
+            )[1]
         )
 
         inputs = iter([str(index)])

src/tests/test_publish_json_result.py

@@ -81,8 +81,9 @@ def test_publish_snapshot_success():
         with patch.object(
             client.client, "send_event", side_effect=fake_send
         ) as mock_send:
-            manifest = asyncio.run(client.publish_snapshot(b"data"))
+            manifest, event_id = asyncio.run(client.publish_snapshot(b"data"))
             assert isinstance(manifest, Manifest)
+            assert event_id == "abcd"
             assert mock_send.await_count >= 1
 
 

src/tests/test_settings_menu.py

@@ -33,7 +33,7 @@ def setup_pm(tmp_path, monkeypatch):
         relays=list(DEFAULT_RELAYS),
         close_client_pool=lambda: None,
         initialize_client_pool=lambda: None,
-        publish_snapshot=lambda data, alt_summary=None: None,
+        publish_snapshot=lambda data, alt_summary=None: (None, "abcd"),
         key_manager=SimpleNamespace(get_npub=lambda: "npub"),
     )