GNU bug report logs - #51450
Bugs in gnus-short-group-name

Previous Next

Package: emacs;

Reported by: Morgan.J.Smith <at> outlook.com

Date: Wed, 27 Oct 2021 22:06:01 UTC

Severity: normal

Fixed in version 29.1

Done: Lars Ingebrigtsen <larsi <at> gnus.org>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Morgan.J.Smith <at> outlook.com
To: 51450 <at> debbugs.gnu.org
Cc: Morgan Smith <Morgan.J.Smith <at> outlook.com>
Subject: bug#51450: Bugs in gnus-short-group-name
Date: Wed, 27 Oct 2021 17:59:04 -0400
From: Morgan Smith <Morgan.J.Smith <at> outlook.com>

---
 test/lisp/gnus/gnus-tests.el | 39 ++++++++++++++++++++++++++++--------
 1 file changed, 31 insertions(+), 8 deletions(-)

diff --git a/test/lisp/gnus/gnus-tests.el b/test/lisp/gnus/gnus-tests.el
index 6602e67a34..f96b584064 100644
--- a/test/lisp/gnus/gnus-tests.el
+++ b/test/lisp/gnus/gnus-tests.el
@@ -1,4 +1,4 @@
-;;; gnus-tests.el --- Wrapper for the Gnus tests  -*- lexical-binding:t -*-
+;;; gnus-tests.el --- Gnus tests  -*- lexical-binding:t -*-
 
 ;; Copyright (C) 2011-2021 Free Software Foundation, Inc.
 
@@ -21,13 +21,36 @@
 
 ;;; Commentary:
 
-;; This file should contain nothing but requires for all the Gnus
-;; tests that are not standalone.
-
 ;;; Code:
-;; registry.el is required by gnus-registry.el but this way we're explicit.
-(require 'registry)
-(require 'gnus-registry)
+(require 'gnus)
+
+;; TODO: test with optional LEVELS argument
+(ert-deftest gnus-short-group-name ()
+  (map-apply
+   (lambda (input expected)
+     (should (string-equal (gnus-short-group-name input) expected)))
+   '(
+     ;; I think these are correct and nice and good
+     ("nnimap+email <at> example.com:archives/2020/03" . "email <at> example:a/2/03")
+     ("nndiary+diary:birthdays" . "diary:birthdays")
+     ("nnimap+email <at> example.com:test" . "email <at> example:test")
+     ("nnimap+email <at> example.com:234" . "email <at> example:234")
+
+     ;; This is a very aggressive shortening of the left hand side. I
+     ;; don't like this
+     ("nnimap+email <at> banana.salesman.example.com:234" . "email <at> banana:234")
+     ("nntp+some.where.edu:soc.motss" . "some:s.motss")
+     ("nntp+news.gmane.org:gmane.emacs.gnus.general" . "news:g.e.g.general")
+     ("nntp+news.gnus.org:gmane.text.docbook.apps" . "news:g.t.d.apps")
+
+     ;; I think we should shorten using [[:alnum:]] characters if we
+     ;; can so this would be "I/Bananas" instead of "[/Bananas"
+     ("nnimap+email <at> example.com:[Invoices]/Bananas" . "email <at> example:[/Bananas")
+
+     ;; Both my previous woes in one
+     ("nnimap+email <at> banana.salesman.example.com:[Invoices]/Bananas" . "email <at> banana:[/Bananas")
+
+     ;; What is going on here? Is the "n" from "nnspool" somehow?
+     ("nnspool+alt.binaries.pictures.furniture" . "n.b.p.furniture"))))
 
-(provide 'gnus-tests)
 ;;; gnus-tests.el ends here
-- 
2.33.1





This bug report was last modified 3 years and 263 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.