claude-multiacc 2.0.57 → 2.0.59

package/tests/test_packaged_commands.py

@@ -61,6 +61,13 @@ class PackagedCommandsTests(unittest.TestCase):
                 self.assertRegex(result.stdout, r"(?i)usage")
         result = run([str(self.package / "install.sh"), "--help"], self.env, self.work)
         self.assertEqual(result.returncode, 0, result.stderr)
+        # The catalog verb refreshes pools when it runs; asked for help it must write nothing.
+        for prefix in (["claude-accounts"], ["codex-accounts"], ["claude-multiacc"], ["claude-multiacc", "codex"]):
+            for args in (["model-catalog", "--help"], ["model-catalog", "refresh", "--help"]):
+                with self.subTest(prefix=prefix, args=args):
+                    result = run(prefix + args, self.env, self.work)
+                    self.assertEqual(result.returncode, 0, result.stderr)
+                    self.assertIn("model-catalog [refresh|show]", result.stdout)
         self.assertEqual(before, sorted(str(path) for path in self.home.rglob("*")))
 
     def test_verify_and_json_reports_use_the_correct_pool(self):
@@ -140,6 +147,10 @@ class PackagedCommandsTests(unittest.TestCase):
                 self.assertEqual(text.count("_claude_multiacc_path_guard() {"), 1, rc)
                 self.assertTrue(text.rstrip().endswith("# <<< claude-multiacc <<<"), rc)
             self.assertFalse((self.home / ".bash_profile").exists())
+            # Every install refreshes the model catalog into both pools (the stubbed CLIs exit 1
+            # and are never needed: the refresh reads files only).
+            for pool in (".claude-accounts", ".codex-accounts"):
+                self.assertEqual(json.loads((self.home / pool / "model-catalog.json").read_text())["schema"], 1)
             # ...and fresh login shells of every flavour now resolve the package's shims,
             # the real binaries in ~/.local/bin notwithstanding (lib/shim_path.py is the
             # same probe `claude-accounts status` and `health` print).
@@ -185,6 +196,36 @@ class PackagedCommandsTests(unittest.TestCase):
             uname.unlink()
             identity.unlink()
 
+    def test_installer_refreshes_the_model_catalog_and_fails_open(self):
+        pools = [self.home / ".claude-accounts", self.home / ".codex-accounts"]
+        install = ["claude-multiacc", "install", "--no-server", "--no-schedule"]
+        for pool in pools:
+            if (pool / "model-catalog.json").exists():
+                (pool / "model-catalog.json").unlink()
+        result = run(install, self.env, self.work)
+        self.assertEqual(result.returncode, 0, result.stderr)
+        self.assertIn("model catalog: refreshed", result.stdout)
+        shipped = json.loads((self.package / "lib/model_catalog.json").read_text())
+        for pool in pools:
+            doc = json.loads((pool / "model-catalog.json").read_text())
+            self.assertEqual(doc["schema"], 1)
+            for provider in ("claude", "codex"):  # no CLI cache in these pools: the shipped lineup
+                self.assertEqual(doc[provider]["models"], shipped[provider]["models"])
+        # A refresh that cannot run never fails the install: one note, nothing written.
+        module = self.package / "lib/model_catalog.py"
+        original = module.read_text()
+        for pool in pools:
+            (pool / "model-catalog.json").unlink()
+        module.write_text("raise SystemExit(1)\n")
+        try:
+            result = run(install, self.env, self.work)
+        finally:
+            module.write_text(original)
+        self.assertEqual(result.returncode, 0, result.stderr)
+        self.assertIn("model catalog could not be refreshed", result.stderr)
+        self.assertIn("install complete", result.stdout)
+        self.assertEqual([pool for pool in pools if (pool / "model-catalog.json").exists()], [])
+
     def test_selector_transport_and_version_match_across_entrypoints(self):
         expected = None
         for prefix in (["multiacc-select"], ["claude-multiacc", "select"]):