GNU bug report logs - #21985
25.0.50; PATCH] Extend the test suite for json.el

Previous Next

Package: emacs;

Reported by: Simen Heggestøyl <simenheg <at> gmail.com>

Date: Sun, 22 Nov 2015 21:31:02 UTC

Severity: wishlist

Tags: patch

Found in version 25.0.50

Done: Simen Heggestøyl <simenheg <at> gmail.com>

Bug is archived. No further changes may be made.

Full log


Message #8 received at 21985 <at> debbugs.gnu.org (full text, mbox):

From: Dmitry Gutov <dgutov <at> yandex.ru>
To: Simen Heggestøyl <simenheg <at> gmail.com>,
 21985 <at> debbugs.gnu.org
Subject: Re: bug#21985: 25.0.50; PATCH] Extend the test suite for json.el
Date: Mon, 23 Nov 2015 01:15:07 +0200
On 11/22/2015 11:30 PM, Simen Heggestøyl wrote:

> I was unsure how to best test `json-encode' without duplicating what
> the other tests (`test-json-encode-string',
> `test-json-encode-keyword', and so on) already do test.

IMO, there's no too much need to exhaustively test `json-encode', aside 
from one encode test (with any value), one decode test, and maybe one 
try-to-encode-something-unusual-and-end-up-with-an-error test.

Testing all contents of json-readtable would seem excessive to me. But 
hey, if we miscalculate, we can add more tests as we fix regressions, later.




This bug report was last modified 9 years and 182 days ago.

Previous Next


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