From 49c9af4c213d5f420a405d860c0454fade26c297 Mon Sep 17 00:00:00 2001
From: "Danilo M." <danix@danix.xyz>
Date: Tue, 30 Jun 2026 11:02:43 +0200
Subject: feat: name-to-id resolver with loud ambiguity errors

---
 tests/unit/test_resolver.py | 43 +++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 43 insertions(+)
 create mode 100644 tests/unit/test_resolver.py

(limited to 'tests')

diff --git a/tests/unit/test_resolver.py b/tests/unit/test_resolver.py
new file mode 100644
index 0000000..2e00e83
--- /dev/null
+++ b/tests/unit/test_resolver.py
@@ -0,0 +1,43 @@
+import unittest
+from unittest.mock import MagicMock
+from firefly_cli.resolver import Resolver
+from firefly_cli.errors import ResolutionError
+
+def client_returning(items):
+    c = MagicMock()
+    c.request.return_value = {
+        "data": [
+            {"id": i["id"], "type": "accounts", "attributes": i["attrs"]}
+            for i in items
+        ]
+    }
+    return c
+
+class TestResolver(unittest.TestCase):
+    def test_resolves_unique_account_name(self):
+        c = client_returning([
+            {"id": "3", "attrs": {"name": "Checking", "type": "asset"}},
+            {"id": "4", "attrs": {"name": "Savings", "type": "asset"}},
+        ])
+        r = Resolver(c)
+        acc = r.account("checking")  # case-insensitive
+        self.assertEqual(acc["id"], "3")
+        self.assertEqual(acc["type"], "asset")
+
+    def test_no_match_raises_with_candidates(self):
+        c = client_returning([{"id": "3", "attrs": {"name": "Checking", "type": "asset"}}])
+        r = Resolver(c)
+        with self.assertRaises(ResolutionError) as ctx:
+            r.account("Nope")
+        self.assertIn("Checking", str(ctx.exception))
+
+    def test_ambiguous_match_raises(self):
+        c = client_returning([
+            {"id": "3", "attrs": {"name": "Cash", "type": "asset"}},
+            {"id": "9", "attrs": {"name": "Cash", "type": "asset"}},
+        ])
+        r = Resolver(c)
+        with self.assertRaises(ResolutionError) as ctx:
+            r.account("Cash")
+        self.assertIn("3", str(ctx.exception))
+        self.assertIn("9", str(ctx.exception))
-- 
cgit v1.2.3