summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDanilo M. <danix@danix.xyz>2026-06-30 10:40:20 +0200
committerDanilo M. <danix@danix.xyz>2026-06-30 10:40:20 +0200
commitb357d987b3a149924dbc4eb584ab36529aaf195d (patch)
tree9c7e045180f5bd057ed6c1a1f0c2715be07bf89e
parenta305c9a0c024e912548631e464e5f08ac24b1562 (diff)
downloadfirefly-cli-b357d987b3a149924dbc4eb584ab36529aaf195d.tar.gz
firefly-cli-b357d987b3a149924dbc4eb584ab36529aaf195d.zip
feat: error types
-rw-r--r--firefly_cli/errors.py18
-rw-r--r--tests/__init__.py0
-rw-r--r--tests/unit/__init__.py0
-rw-r--r--tests/unit/test_errors.py16
4 files changed, 34 insertions, 0 deletions
diff --git a/firefly_cli/errors.py b/firefly_cli/errors.py
new file mode 100644
index 0000000..0c0ec56
--- /dev/null
+++ b/firefly_cli/errors.py
@@ -0,0 +1,18 @@
+# Copyright (C) 2026 Danilo M. <danix@danix.xyz> GPL-2.0-only
+
+class FireflyError(Exception):
+ """Base for all firefly-cli errors."""
+
+class ConfigError(FireflyError):
+ """Missing or invalid configuration."""
+
+class ResolutionError(FireflyError):
+ """A name could not be resolved to a single id."""
+
+class ApiError(FireflyError):
+ """Firefly returned a non-2xx response."""
+ def __init__(self, status, body):
+ self.status = status
+ self.body = body
+ msg = body.get("message") if isinstance(body, dict) else body
+ super().__init__(f"API error {status}: {msg}")
diff --git a/tests/__init__.py b/tests/__init__.py
new file mode 100644
index 0000000..e69de29
--- /dev/null
+++ b/tests/__init__.py
diff --git a/tests/unit/__init__.py b/tests/unit/__init__.py
new file mode 100644
index 0000000..e69de29
--- /dev/null
+++ b/tests/unit/__init__.py
diff --git a/tests/unit/test_errors.py b/tests/unit/test_errors.py
new file mode 100644
index 0000000..63731c5
--- /dev/null
+++ b/tests/unit/test_errors.py
@@ -0,0 +1,16 @@
+import unittest
+from firefly_cli.errors import FireflyError, ConfigError, ApiError, ResolutionError
+
+class TestErrors(unittest.TestCase):
+ def test_subclassing(self):
+ for cls in (ConfigError, ApiError, ResolutionError):
+ self.assertTrue(issubclass(cls, FireflyError))
+
+ def test_api_error_carries_status_and_body(self):
+ e = ApiError(422, {"message": "bad"})
+ self.assertEqual(e.status, 422)
+ self.assertEqual(e.body, {"message": "bad"})
+ self.assertIn("422", str(e))
+
+if __name__ == "__main__":
+ unittest.main()