refactor: remove dead wrappers and commented-out blocks

Drop cj/apply-browser-choice (browser-config) and cj/load-fallback-theme (ui-theme), orphaned wrappers with no caller that just duplicated logic the live paths already inline, plus their tests. Delete commented-out blocks: a duplicate contact capture template (org-contacts-config), a disabled personal-info-dir :init (help-config), a stale TODO setq (org-config), and an old commented regex (test-runner).

1 files changed, 0 insertions, 23 deletions

diff --git a/tests/test-browser-config.el b/tests/test-browser-config.el
index 7faecbfc8..9fe5b02e4 100644
--- a/tests/test-browser-config.el
+++ b/tests/test-browser-config.el
@@ -273,29 +273,6 @@
       (should (string= (plist-get loaded :name) "Second"))))
   (test-browser-teardown))
 
-;;; Public wrappers (message side-effects mocked)
-
-(ert-deftest test-browser-apply-wrapper-success-messages-name ()
-  "Normal: =cj/apply-browser-choice= reports the chosen name on success."
-  (test-browser-setup)
-  (let ((browser (test-browser-make-plist "Wrapper Test"))
-        (received nil))
-    (cl-letf (((symbol-function 'message)
-               (lambda (fmt &rest args) (setq received (apply #'format fmt args)))))
-      (cj/apply-browser-choice browser))
-    (should (string-match-p "Wrapper Test" received))
-    (should (string-match-p "Default browser set" received)))
-  (test-browser-teardown))
-
-(ert-deftest test-browser-apply-wrapper-invalid-plist-messages-error ()
-  "Error: =cj/apply-browser-choice= surfaces an error message for a bad plist."
-  (test-browser-setup)
-  (let ((received nil))
-    (cl-letf (((symbol-function 'message)
-               (lambda (fmt &rest args) (setq received (apply #'format fmt args)))))
-      (cj/apply-browser-choice nil))
-    (should (string-match-p "Invalid" received)))
-  (test-browser-teardown))
 
 (ert-deftest test-browser-initialize-wrapper-loaded-branch-applies ()
   "Normal: =cj/initialize-browser= applies the saved browser when one is loaded."