aboutsummaryrefslogtreecommitdiff
path: root/tests
diff options
context:
space:
mode:
authorCraig Jennings <c@cjennings.net>2026-06-29 19:20:19 -0400
committerCraig Jennings <c@cjennings.net>2026-06-29 19:20:19 -0400
commit5aaca33fdd57a9987f26476dd44510831ad033ca (patch)
tree90c4326d13959b4d7b90dad9ab85a544543e6583 /tests
parentfae3ad57b80e23b4633e357af357d95c4a76145e (diff)
downloaddotemacs-5aaca33fdd57a9987f26476dd44510831ad033ca.tar.gz
dotemacs-5aaca33fdd57a9987f26476dd44510831ad033ca.zip
refactor(icons): drop all-the-icons, nerd-icons drives everything
nerd-icons already rendered every icon in the config (dashboard, dirvish, ibuffer, completion). all-the-icons survived only as scaffolding: a font-install helper, the all-the-icons-nerd-fonts bridge, and a terminal-blanking advice block the nerd-icons one beside it already duplicated. I removed all of it and pointed the font-install helper at nerd-icons (Symbols Nerd Font Mono), keeping the auto-install-on-first-GUI-frame convenience. I updated the font-config tests to the renamed helper.
Diffstat (limited to 'tests')
-rw-r--r--tests/test-font-config--frame-lifecycle.el10
-rw-r--r--tests/test-font-config.el12
2 files changed, 11 insertions, 11 deletions
diff --git a/tests/test-font-config--frame-lifecycle.el b/tests/test-font-config--frame-lifecycle.el
index 826edbd69..8f338b996 100644
--- a/tests/test-font-config--frame-lifecycle.el
+++ b/tests/test-font-config--frame-lifecycle.el
@@ -2,7 +2,7 @@
;;; Commentary:
;; cj/apply-font-settings-to-frame, cj/cleanup-frame-list, and
-;; cj/maybe-install-all-the-icons-fonts were defined inside use-package
+;; cj/maybe-install-nerd-icons-fonts were defined inside use-package
;; :config / with-eval-after-load (unreachable under `make test'). Lifting
;; them to top level makes their branching unit-testable; env-gui-p and the
;; package side-effect calls are mocked at the boundary.
@@ -57,9 +57,9 @@
(let ((installed nil))
(cl-letf (((symbol-function 'env-gui-p) (lambda () t))
((symbol-function 'cj/font-installed-p) (lambda (_n) nil))
- ((symbol-function 'all-the-icons-install-fonts) (lambda (&rest _) (setq installed t)))
+ ((symbol-function 'nerd-icons-install-fonts) (lambda (&rest _) (setq installed t)))
((symbol-function 'remove-hook) #'ignore))
- (cj/maybe-install-all-the-icons-fonts))
+ (cj/maybe-install-nerd-icons-fonts))
(should installed)))
(ert-deftest test-font-maybe-install-icons-already-present-skips ()
@@ -67,8 +67,8 @@
(let ((installed nil))
(cl-letf (((symbol-function 'env-gui-p) (lambda () t))
((symbol-function 'cj/font-installed-p) (lambda (_n) t))
- ((symbol-function 'all-the-icons-install-fonts) (lambda (&rest _) (setq installed t))))
- (cj/maybe-install-all-the-icons-fonts))
+ ((symbol-function 'nerd-icons-install-fonts) (lambda (&rest _) (setq installed t))))
+ (cj/maybe-install-nerd-icons-fonts))
(should-not installed)))
(provide 'test-font-config--frame-lifecycle)
diff --git a/tests/test-font-config.el b/tests/test-font-config.el
index 8fada25e2..393a77584 100644
--- a/tests/test-font-config.el
+++ b/tests/test-font-config.el
@@ -5,9 +5,10 @@
;; font-config.el is mostly top-level font/package setup. These smoke tests
;; cover the logic that should stay correct regardless of which fonts are
;; installed: the install check, and the daemon-frame font applier (env-gui-p
-;; guard plus idempotency). The module :demand's fontaine and all-the-icons,
-;; so the tests skip when those packages are absent rather than failing on a
-;; bare checkout. GUI and font lookups are stubbed so the run stays headless.
+;; guard plus idempotency). The module :demand's fontaine and references
+;; nerd-icons, so the tests skip when those packages are absent rather than
+;; failing on a bare checkout. GUI and font lookups are stubbed so the run
+;; stays headless.
;;; Code:
@@ -21,9 +22,8 @@
(defconst test-font-config--available
(and (locate-library "fontaine")
- (locate-library "all-the-icons")
- (locate-library "all-the-icons-nerd-fonts"))
- "Non-nil when the packages font-config :demand's are loadable.")
+ (locate-library "nerd-icons"))
+ "Non-nil when the packages font-config needs are loadable.")
;;; cj/font-installed-p