aboutsummaryrefslogtreecommitdiff
path: root/tests/test-music-config--tidy-host.el
diff options
context:
space:
mode:
authorCraig Jennings <c@cjennings.net>2026-07-06 16:56:40 -0500
committerCraig Jennings <c@cjennings.net>2026-07-06 16:56:40 -0500
commit0a7afa70fa9c2c20771696b303c9cae4aa7c58a2 (patch)
tree8d97a9f3b8cba2ad97d808114e9a00edce903711 /tests/test-music-config--tidy-host.el
parentea8bbce911650515b88fd3a61fdb88186e550347 (diff)
downloaddotemacs-0a7afa70fa9c2c20771696b303c9cae4aa7c58a2.tar.gz
dotemacs-0a7afa70fa9c2c20771696b303c9cae4aa7c58a2.zip
feat(music): show station and track names, not stream URLs (fancy UI phase 1)
The EMMS playlist showed the raw stream URL for every radio track, so the buffer read like a debug log. This is phase 1 of the fancy player: real names, still plain text. It ships on its own and becomes the fallback for the image and GUI phases. I split the old cj/music--track-description into a pure, name-only cj/music--display-name that the header's Current line and the row renderer both call, so the two never drift. A url track resolves to its #EXTINF label (read once from all the playlist .m3u files into a cached map) or a tidied host. A file shows Artist - Title or its filename. The row renderer adds a dim nerd-icon glyph and the duration as right-aligned meta with an :align-to space, so it survives a window resize. The header line stays clean: just the name plus an on-air or duration suffix. The pure helpers carry the tests: display-name, tidy-host, m3u-labels, bar-fill, format-meta, Normal/Boundary/Error each (32 tests across four files). The glyph, the aligned meta, and the disk-backed name map are the thin composition, verified live against 44 real stations. The progress bar's visual and its live elapsed source come in phase 3. The bar-fill helper is the tested pure core it will feed. The full suite is green.
Diffstat (limited to 'tests/test-music-config--tidy-host.el')
-rw-r--r--tests/test-music-config--tidy-host.el47
1 files changed, 47 insertions, 0 deletions
diff --git a/tests/test-music-config--tidy-host.el b/tests/test-music-config--tidy-host.el
new file mode 100644
index 00000000..92b104a6
--- /dev/null
+++ b/tests/test-music-config--tidy-host.el
@@ -0,0 +1,47 @@
+;;; test-music-config--tidy-host.el --- Tests for stream-URL host tidying -*- coding: utf-8; lexical-binding: t; -*-
+;;
+;; Author: Craig Jennings <c@cjennings.net>
+;;
+;;; Commentary:
+;; Unit tests for `cj/music--tidy-host': reduce a stream URL to a readable host
+;; label (scheme dropped, a leading "www." removed), used as the last-resort
+;; display name for a url track with no #EXTINF label.
+;;
+;;; Code:
+
+(require 'ert)
+
+(defvar-keymap cj/custom-keymap :doc "Stub keymap for testing")
+
+(let ((emms-dir (car (file-expand-wildcards
+ (expand-file-name "elpa/emms-*" user-emacs-directory)))))
+ (when emms-dir (add-to-list 'load-path emms-dir)))
+
+(require 'emms)
+(require 'music-config)
+
+(ert-deftest test-music-config--tidy-host-normal ()
+ "Normal: scheme and path are dropped, host kept."
+ (should (string= (cj/music--tidy-host "https://ice6.somafm.com/groovesalad-256-mp3")
+ "somafm.com")))
+
+(ert-deftest test-music-config--tidy-host-normal-http ()
+ "Normal: plain http host with a port keeps the host, drops the port."
+ (should (string= (cj/music--tidy-host "http://stream.example.org:8000/live")
+ "example.org")))
+
+(ert-deftest test-music-config--tidy-host-boundary-strip-www ()
+ "Boundary: a leading www. is stripped."
+ (should (string= (cj/music--tidy-host "https://www.radioparadise.com/m3u/mp3-128.m3u")
+ "radioparadise.com")))
+
+(ert-deftest test-music-config--tidy-host-boundary-bare-domain ()
+ "Boundary: a two-label domain is returned unchanged."
+ (should (string= (cj/music--tidy-host "http://somafm.com/") "somafm.com")))
+
+(ert-deftest test-music-config--tidy-host-error-not-a-url ()
+ "Error: a non-URL string is returned as-is rather than erroring."
+ (should (string= (cj/music--tidy-host "not a url") "not a url")))
+
+(provide 'test-music-config--tidy-host)
+;;; test-music-config--tidy-host.el ends here